commit | a03e09410f05b4a9bb6a4fb549a09e886b089964 | [log] [tgz] |
---|---|---|
author | Yi Gu <yigu@chromium.org> | Sat Mar 02 02:18:52 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Sat Mar 02 02:18:52 2019 |
tree | 78d91195fc095602df95932a145ce87398c735a1 | |
parent | 573d10a0391a5adec126093a649718d04d9e6243 [diff] |
Revert "Hook up running extension SW information to ProcessManager" This reverts commit 7466707780d996e09adff361f84c0ae0add4602c. Reason for revert: Findit says this patch fails Linux ChromiumOS MSan Tests. See https://findit-for-me.appspot.com/waterfall/failure?url=https://build.chromium.org/p/chromium.memory/builders/Linux%20ChromiumOS%20MSan%20Tests/builds/11570. Original change's description: > Hook up running extension SW information to ProcessManager > > When service worker context initializes, add the corresponding > entry to ProcessManager and remove it when the context stops. > > Introduce IPC for worker initialization > (DidInitializeServiceWorkerContext) and modify worker > stop IPC (DidStopServiceWorkerContext) as necessary. > > Note that we cannot use loadstop > (DidStartServiceWorkerContext) for PM entry addition because it > can be too late as extension API calls might arrive in browser > process before this. > > Also add SW information cleanup on render process termination as > termination doesn't trigger stop worker messages (where the > regular cleanup in PM happens). > > This CL will enable extension messaging in future CLs. > > Add tests for shutdown and termination behavior. > > Bug: 925918 > Change-Id: I9442bac9a1e9acf01063084a1e6ef2fac3279e09 > Reviewed-on: https://chromium-review.googlesource.com/c/1455267 > Reviewed-by: Dominick Ng <dominickn@chromium.org> > Reviewed-by: Devlin <rdevlin.cronin@chromium.org> > Commit-Queue: Istiaque Ahmed <lazyboy@chromium.org> > Cr-Commit-Position: refs/heads/master@{#636929} TBR=lazyboy@chromium.org,rdevlin.cronin@chromium.org,dominickn@chromium.org Change-Id: I3d0174bf3e8497351c49a8695e62b81749c125b2 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 925918 Reviewed-on: https://chromium-review.googlesource.com/c/1497798 Reviewed-by: Yi Gu <yigu@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#637065}
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 .