| commit | 5ad8ffbfd99372e02bde808b264313cf69fa7e4b | [log] [tgz] |
|---|---|---|
| author | horo <horo@chromium.org> | Wed Nov 02 01:05:38 2016 |
| committer | Commit bot <commit-bot@chromium.org> | Wed Nov 02 01:07:36 2016 |
| tree | 73725efe01c9e2ce2a721935f9428a48dbafb041 | |
| parent | c08c51564e5f1c0e1c3adf276fe6074a3f5b6dde [diff] |
Stop using MojoURLLoaderFactoryGetter. https://crrev.com/2410333006/ introduced MojoURLLoaderFactoryGetter which is passed from RenderProcessHostImpl to ServiceWorkerFetchDispatcher. The MojoURLLoaderFactoryGette is created on the UI thread and holds a WeakPtr of the RenderProcessHostImpl. And it is passed to IO thread. But WeakPtr is not thread safe. http://www.chromium.org/developers/design-documents/threading#base_WeakPtr_and_Cancellation So this CL stops using MojoURLLoaderFactoryGette and changes ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload() to get the ResourceMessageFilter from ResourceeRequestInfoImpl. BUG=649558 Review-Url: https://codereview.chromium.org/2465813003 Cr-Commit-Position: refs/heads/master@{#429169}