Google Git
Sign in
chromium / chromium / src / 4658f3f5b86ab1da34311121251581c42c9ff31f
commit4658f3f5b86ab1da34311121251581c42c9ff31f[log] [tgz]
authordanakj <danakj@chromium.org>Fri Feb 01 15:57:01 2019
committerCommit Bot <commit-bot@chromium.org>Fri Feb 01 15:57:01 2019
treeed9cb2349e033e5c8d4bb2c091bea0e91be71a60
parentfe4c2aff507ce20005141617b07a32e49db25304 [diff]
Remove null-check for |delegate_| in RWHI::GetVisualProperties().

RenderWidgetHostImpl already checks for |delegate_| before calling the
method. The other caller does so in the same stack that it uses the
WebContents, which is the delegate. The delegate is removed only when
the WebContents is destroyed, so we know it will be present in both
cases.

TBR=ajwong@chromium.org

Change-Id: Ia0fe71c00a87bb92b22ebeff545762fbcf72848d
Bug: 912193
Reviewed-on: https://chromium-review.googlesource.com/c/1443933
Commit-Queue: danakj <danakj@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Albert J. Wong <ajwong@chromium.org>
Cr-Commit-Position: refs/heads/master@{#628349}
2 files changed
tree: ed9cb2349e033e5c8d4bb2c091bea0e91be71a60
  1. android_webview/
  2. apps/
  3. ash/
  4. base/
  5. build/
  6. build_overrides/
  7. cc/
  8. chrome/
  9. chrome_elf/
  10. chromecast/
  11. chromeos/
  12. cloud_print/
  13. components/
  14. content/
  15. courgette/
  16. crypto/
  17. dbus/
  18. device/
  19. docs/
  20. extensions/
  21. fuchsia/
  22. gin/
  23. google_apis/
  24. google_update/
  25. gpu/
  26. headless/
  27. infra/
  28. ios/
  29. ipc/
  30. jingle/
  31. mash/
  32. media/
  33. mojo/
  34. native_client_sdk/
  35. net/
  36. pdf/
  37. ppapi/
  38. printing/
  39. remoting/
  40. rlz/
  41. sandbox/
  42. services/
  43. skia/
  44. sql/
  45. storage/
  46. styleguide/
  47. testing/
  48. third_party/
  49. tools/
  50. ui/
  51. url/
  52. .clang-format
  53. .eslintrc.js
  54. .git-blame-ignore-revs
  55. .gitattributes
  56. .gitignore
  57. .gn
  58. .vpython
  59. AUTHORS
  60. BUILD.gn
  61. CODE_OF_CONDUCT.md
  62. codereview.settings
  63. DEPS
  64. ENG_REVIEW_OWNERS
  65. LICENSE
  66. LICENSE.chromium_os
  67. OWNERS
  68. PRESUBMIT.py
  69. PRESUBMIT_test.py
  70. PRESUBMIT_test_mocks.py
  71. README.md
  72. WATCHLISTS
README.md

Logo Chromium

Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.

The project's web site is https://www.chromium.org.

Documentation in the source is rooted in docs/README.md.

Learn how to Get Around the Chromium Source Code Directory Structure .