Stop using MojoURLLoaderFactoryGetter.

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
Committed: https://crrev.com/5ad8ffbfd99372e02bde808b264313cf69fa7e4b
Cr-Commit-Position: refs/heads/master@{#429169}

[horo, 2016-10-31 15:49:43]

Description was changed from

==========
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_an...

So this CL stops usin  MojoURLLoaderFactoryGette and change
ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload() to get the
ResourceMessageFilter from ResourceeRequestInfoImpl.

BUG=
==========

to

==========
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_an...

So this CL stops using MojoURLLoaderFactoryGette and changes
ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload() to get the
ResourceMessageFilter from ResourceeRequestInfoImpl.

BUG=649558
==========

[horo, 2016-10-31 15:49:49]

The CQ bit was checked by horo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-31 15:50:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[horo, 2016-10-31 15:52:44]

horo@chromium.org changed reviewers:
+ falken@chromium.org

[horo, 2016-10-31 15:52:45]

falken@
Could you please review this?

[commit-bot: I haz the power, 2016-10-31 17:16:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-31 17:16:18]

Dry run: This issue passed the CQ dry run.

[falken, 2016-11-01 01:41:58]

looks a lot simpler too. lgtm

[horo, 2016-11-01 02:48:16]

horo@chromium.org changed reviewers:
+ piman@chromium.org

[horo, 2016-11-01 02:48:16]

piman@
Could you please review
content/browser/renderer_host/render_process_host_impl.*?

[piman, 2016-11-01 19:40:05]

lgtm

[horo, 2016-11-01 23:26:53]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2016-11-01 23:27:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-02 01:06:03]

Description was changed from

==========
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_an...

So this CL stops using MojoURLLoaderFactoryGette and changes
ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload() to get the
ResourceMessageFilter from ResourceeRequestInfoImpl.

BUG=649558
==========

to

==========
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_an...

So this CL stops using MojoURLLoaderFactoryGette and changes
ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload() to get the
ResourceMessageFilter from ResourceeRequestInfoImpl.

BUG=649558
==========

[commit-bot: I haz the power, 2016-11-02 01:06:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-02 01:07:44]

Description was changed from

==========
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_an...

So this CL stops using MojoURLLoaderFactoryGette and changes
ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload() to get the
ResourceMessageFilter from ResourceeRequestInfoImpl.

BUG=649558
==========

to

==========
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_an...

So this CL stops using MojoURLLoaderFactoryGette and changes
ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload() to get the
ResourceMessageFilter from ResourceeRequestInfoImpl.

BUG=649558

Committed: https://crrev.com/5ad8ffbfd99372e02bde808b264313cf69fa7e4b
Cr-Commit-Position: refs/heads/master@{#429169}
==========

[commit-bot: I haz the power, 2016-11-02 01:07:45]

Patchset 1 (id:??) landed as
https://crrev.com/5ad8ffbfd99372e02bde808b264313cf69fa7e4b
Cr-Commit-Position: refs/heads/master@{#429169}