Google Git
Sign in
chromium / chromium / src / 6d17e158c4ddc0259e32fa13ad9c5a67876d50fd
commit6d17e158c4ddc0259e32fa13ad9c5a67876d50fd[log] [tgz]
authorHiroki Nakagawa <nhiroki@chromium.org>Wed Dec 13 08:34:20 2017
committerCommit Bot <commit-bot@chromium.org>Wed Dec 13 08:34:20 2017
tree500061d347d505179f657ceb06437eac0a0e66af
parent6f7e8e1721c1d373efce9d187e5b9be20acfe3f2 [diff]
Worker: Remove ThreadedMessagingProxyBase::WorkerThreadCreated() for cleanup

ThreadedMessagingProxyBase::WorkerThreadCreated() is provided as a virtual
function for DedicatedWorkerMessagingProxy to post queued tasks to the worker
after thread creation. However, DedicatedWorkerMessagingProxy itself initiates
the thread creation in StartWorkerGlobalScope() and can know when it's
available, so such the virtual function is not necessary.

For cleanup, this CL removes WorkerThreadCreated() and inlines posting the
queued tasks in StartWorkerGlobalScope().

Bug: 680046
Change-Id: I5248c78d381e0820461e167249ce89bac93e44a5
Reviewed-on: https://chromium-review.googlesource.com/823052
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523719}
4 files changed
tree: 500061d347d505179f657ceb06437eac0a0e66af
  1. android_webview/
  2. apps/
  3. ash/
  4. base/
  5. blink/
  6. build/
  7. build_overrides/
  8. cc/
  9. chrome/
  10. chrome_elf/
  11. chromecast/
  12. chromeos/
  13. cloud_print/
  14. components/
  15. content/
  16. courgette/
  17. crypto/
  18. dbus/
  19. device/
  20. docs/
  21. extensions/
  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 .