Revert of Add FrameHost mojo service (patchset #23 id:490001 of https://codereview.chromium.org/196…

Revert of Add FrameHost mojo service (patchset #23 id:490001 of https://codereview.chromium.org/1964273002/ )

Reason for revert:
Async request for zoom level doesn't work in all cases https://crbug.com/614348 https://crbug.com/613979.

I thought https://www.chromium.org/developers/design-documents/mojo/chrome-ipc-to-mojo-ipc-cheet-sheet#TOC-Threading-Model implied the ordering would be correct, but it seems that was too hopeful.

Original issue's description:
> Add FrameHost mojo service
>
> Adds new frame-level service with one initial method to handle host zoom
> level.
>
> This moves zoom level supply from async_resource_handler.cc to being a
> request made when render_frame_impl handles a willSendRequest.
>
> The goal of this change is to remove the dependency of
> content/browser/loader on the rest of content/browser in particular
> here, removing the use of c/b/host_zoom_map_impl.h in
> content/browser/loader/async_resource_handler.cc.
>
> BUG=598073, 609607
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/68c6f2ce16d9807b5cb82679099c82c40f39e911
> Cr-Commit-Position: refs/heads/master@{#394547}

TBR=dcheng@chromium.org,ben@chromium.org,jam@chromium.org,nasko@chromium.org,wjmaclean@chromium.org
BUG=598073, 609607, 614348, 613979
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2007203002
Cr-Commit-Position: refs/heads/master@{#395761}
(cherry picked from commit 34ce95dd9c2ca3b90fc87169f1e4ed266e7bb94b)

Committed: https://chromium.googlesource.com/chromium/src/+/b43ea3087da4e351f0bf94481c9810738c3ae938

[scottmg, 2016-05-27 22:43:06]

Description was changed from

==========
Revert of Add FrameHost mojo service (patchset #23 id:490001 of
https://codereview.chromium.org/1964273002/ )

Reason for revert:
Async request for zoom level doesn't work in all cases https://crbug.com/614348
https://crbug.com/613979.

I thought
https://www.chromium.org/developers/design-documents/mojo/chrome-ipc-to-mojo-...
implied the ordering would be correct, but it seems that was too hopeful.

Original issue's description:
> Add FrameHost mojo service
>
> Adds new frame-level service with one initial method to handle host zoom
> level.
>
> This moves zoom level supply from async_resource_handler.cc to being a
> request made when render_frame_impl handles a willSendRequest.
>
> The goal of this change is to remove the dependency of
> content/browser/loader on the rest of content/browser in particular
> here, removing the use of c/b/host_zoom_map_impl.h in
> content/browser/loader/async_resource_handler.cc.
>
> BUG=598073,609607
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/68c6f2ce16d9807b5cb82679099c82c40f39e911
> Cr-Commit-Position: refs/heads/master@{#394547}

TBR=dcheng@chromium.org,ben@chromium.org,jam@chromium.org,nasko@chromium.org,...
BUG=598073,609607,614348,613979
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2007203002
Cr-Commit-Position: refs/heads/master@{#395761}
(cherry picked from commit 34ce95dd9c2ca3b90fc87169f1e4ed266e7bb94b)
==========

to

==========
Revert of Add FrameHost mojo service (patchset #23 id:490001 of
https://codereview.chromium.org/1964273002/ )

Reason for revert:
Async request for zoom level doesn't work in all cases https://crbug.com/614348
https://crbug.com/613979.

I thought
https://www.chromium.org/developers/design-documents/mojo/chrome-ipc-to-mojo-...
implied the ordering would be correct, but it seems that was too hopeful.

Original issue's description:
> Add FrameHost mojo service
>
> Adds new frame-level service with one initial method to handle host zoom
> level.
>
> This moves zoom level supply from async_resource_handler.cc to being a
> request made when render_frame_impl handles a willSendRequest.
>
> The goal of this change is to remove the dependency of
> content/browser/loader on the rest of content/browser in particular
> here, removing the use of c/b/host_zoom_map_impl.h in
> content/browser/loader/async_resource_handler.cc.
>
> BUG=598073,609607
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/68c6f2ce16d9807b5cb82679099c82c40f39e911
> Cr-Commit-Position: refs/heads/master@{#394547}

TBR=dcheng@chromium.org,ben@chromium.org,jam@chromium.org,nasko@chromium.org,...
BUG=598073,609607,614348,613979
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2007203002
Cr-Commit-Position: refs/heads/master@{#395761}
(cherry picked from commit 34ce95dd9c2ca3b90fc87169f1e4ed266e7bb94b)

Committed:
https://chromium.googlesource.com/chromium/src/+/b43ea3087da4e351f0bf94481c98...
==========

[scottmg, 2016-05-27 22:43:08]

Committed patchset #1 (id:1) manually as
b43ea3087da4e351f0bf94481c9810738c3ae938.