Revert of Add FrameHost mojo service

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
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=598073, 609607, 614348, 613979
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/34ce95dd9c2ca3b90fc87169f1e4ed266e7bb94b
Cr-Commit-Position: refs/heads/master@{#395761}

[scottmg, 2016-05-24 22:47:25]

Created Revert of Add FrameHost mojo service

[scottmg, 2016-05-24 22:47:55]

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,...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=598073,609607
==========

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,...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=598073,609607,614348,613979
==========

[commit-bot: I haz the power, 2016-05-24 22:48:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2007203002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2007203002/1

[commit-bot: I haz the power, 2016-05-24 22:59:37]

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

[commit-bot: I haz the power, 2016-05-24 22:59:38]

Try jobs failed on following builders:
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)

[scottmg, 2016-05-24 23:07:15]

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,...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=598073,609607,614348,613979
==========

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,...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=598073,609607,614348,613979
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[scottmg, 2016-05-24 23:07:37]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2016-05-24 23:08:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2007203002/310001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2007203002/310001

[commit-bot: I haz the power, 2016-05-25 01:05:21]

Committed patchset #2 (id:310001)

[commit-bot: I haz the power, 2016-05-25 01:08:52]

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,...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=598073,609607,614348,613979
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

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,...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=598073,609607,614348,613979
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/34ce95dd9c2ca3b90fc87169f1e4ed266e7bb94b
Cr-Commit-Position: refs/heads/master@{#395761}
==========

[commit-bot: I haz the power, 2016-05-25 01:08:53]

Patchset 2 (id:??) landed as
https://crrev.com/34ce95dd9c2ca3b90fc87169f1e4ed266e7bb94b
Cr-Commit-Position: refs/heads/master@{#395761}