Remove dependency of c/b/loader on c/b/host_zoom_map_impl.h

Remove dependency of c/b/loader on c/b/host_zoom_map_impl.h

Contrary to previous attempt at:
https://chromium.googlesource.com/chromium/src.git/+/68c6f2ce16d9807b5cb82679099c82c40f39e911

RenderFrameMessageFilter is on the IO thread, and is a BrowserAssociatedInterface, so will maintain the ordering of the resource request and the zoom level request. See also https://codereview.chromium.org/2167513003.

BUG=609607
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/d4423d511f18f4b0d2e94cf17bf6043bee449c55
Cr-Commit-Position: refs/heads/master@{#410417}

[scottmg, 2016-08-04 01:01:45]

Description was changed from

==========
wip on trying to re-remove host zoom level

BUG=609607
==========

to

==========
wip on trying to re-remove host zoom level

BUG=609607
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[scottmg, 2016-08-05 19:15:02]

Description was changed from

==========
wip on trying to re-remove host zoom level

BUG=609607
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove dependency of c/b/loader on c/b/host_zoom_map_impl.h

Contrary to previous attempt at:
https://chromium.googlesource.com/chromium/src.git/+/68c6f2ce16d9807b5cb82679...

RenderFrameMessageFilter is on the IO thread, and is a
BrowserAssociatedInterface, so will maintain the ordering of the resource
request and the zoom level request. See also
https://codereview.chromium.org/2167513003.

BUG=609607
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[scottmg, 2016-08-05 19:15:02]

scottmg@chromium.org changed reviewers:
+ jam@chromium.org

[scottmg, 2016-08-05 19:17:02]

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

[commit-bot: I haz the power, 2016-08-05 19:17:24]

Dry run: 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-08-05 19:20:41]

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

[commit-bot: I haz the power, 2016-08-05 19:20:42]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[scottmg, 2016-08-05 19:25:18]

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

[commit-bot: I haz the power, 2016-08-05 19:25:36]

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

[scottmg, 2016-08-05 19:28:50]

scottmg@chromium.org changed reviewers:
+ tsepez@chromium.org

[scottmg, 2016-08-05 19:28:51]

jam: overall
tsepez: _messages and .mojom

[scottmg, 2016-08-05 19:34:25]

Patchset #1 (id:1) has been deleted

[scottmg, 2016-08-05 19:35:35]

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

[commit-bot: I haz the power, 2016-08-05 19:35:44]

Dry run: 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-08-05 19:48:06]

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

[commit-bot: I haz the power, 2016-08-05 19:48:07]

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

[Tom Sepez, 2016-08-05 20:03:56]

mojom/messages LGTM

[jam, 2016-08-05 20:05:51]

lgtm

https://codereview.chromium.org/2211713002/diff/80001/content/common/render_f...
File content/common/render_frame_message_filter.mojom (right):

https://codereview.chromium.org/2211713002/diff/80001/content/common/render_f...
content/common/render_frame_message_filter.mojom:21: GetHostZoomLevel(int32
render_frame_id, url.mojom.Url url)
nit: document

[scottmg, 2016-08-05 20:06:00]

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

[commit-bot: I haz the power, 2016-08-05 20:06:19]

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

[scottmg, 2016-08-05 20:08:07]

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

[scottmg, 2016-08-05 20:08:11]

https://codereview.chromium.org/2211713002/diff/80001/content/common/render_f...
File content/common/render_frame_message_filter.mojom (right):

https://codereview.chromium.org/2211713002/diff/80001/content/common/render_f...
content/common/render_frame_message_filter.mojom:21: GetHostZoomLevel(int32
render_frame_id, url.mojom.Url url)
On 2016/08/05 20:05:51, jam wrote:
> nit: document

Done.

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

Dry run: 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-08-05 20:46:52]

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

[commit-bot: I haz the power, 2016-08-05 20:46:53]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[scottmg, 2016-08-05 22:05:04]

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

[commit-bot: I haz the power, 2016-08-05 22:05:26]

Dry run: 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-08-05 22:32:34]

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

[commit-bot: I haz the power, 2016-08-05 22:32:35]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[scottmg, 2016-08-05 22:47:36]

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

[commit-bot: I haz the power, 2016-08-05 22:47:56]

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

[scottmg, 2016-08-06 00:01:24]

The CQ bit was unchecked by scottmg@chromium.org

[scottmg, 2016-08-06 00:01:29]

The CQ bit was checked by scottmg@chromium.org

[scottmg, 2016-08-06 00:01:29]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
jam@chromium.org
Link to the patchset: https://codereview.chromium.org/2211713002/#ps160001
(title: ".")

[commit-bot: I haz the power, 2016-08-06 00:02:04]

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-08-06 00:43:39]

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

[commit-bot: I haz the power, 2016-08-06 00:43:41]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[scottmg, 2016-08-08 18:01:22]

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

[commit-bot: I haz the power, 2016-08-08 18:02:34]

Dry run: 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-08-08 19:32:54]

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

[commit-bot: I haz the power, 2016-08-08 19:32:56]

Dry run: This issue passed the CQ dry run.

[scottmg, 2016-08-08 19:49:06]

The CQ bit was checked by scottmg@chromium.org

[scottmg, 2016-08-08 19:49:07]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
jam@chromium.org
Link to the patchset: https://codereview.chromium.org/2211713002/#ps180001
(title: ".")

[commit-bot: I haz the power, 2016-08-08 19:49:55]

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-08-08 20:12:03]

Description was changed from

==========
Remove dependency of c/b/loader on c/b/host_zoom_map_impl.h

Contrary to previous attempt at:
https://chromium.googlesource.com/chromium/src.git/+/68c6f2ce16d9807b5cb82679...

RenderFrameMessageFilter is on the IO thread, and is a
BrowserAssociatedInterface, so will maintain the ordering of the resource
request and the zoom level request. See also
https://codereview.chromium.org/2167513003.

BUG=609607
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove dependency of c/b/loader on c/b/host_zoom_map_impl.h

Contrary to previous attempt at:
https://chromium.googlesource.com/chromium/src.git/+/68c6f2ce16d9807b5cb82679...

RenderFrameMessageFilter is on the IO thread, and is a
BrowserAssociatedInterface, so will maintain the ordering of the resource
request and the zoom level request. See also
https://codereview.chromium.org/2167513003.

BUG=609607
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

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

Committed patchset #9 (id:180001)

[commit-bot: I haz the power, 2016-08-08 20:13:25]

Description was changed from

==========
Remove dependency of c/b/loader on c/b/host_zoom_map_impl.h

Contrary to previous attempt at:
https://chromium.googlesource.com/chromium/src.git/+/68c6f2ce16d9807b5cb82679...

RenderFrameMessageFilter is on the IO thread, and is a
BrowserAssociatedInterface, so will maintain the ordering of the resource
request and the zoom level request. See also
https://codereview.chromium.org/2167513003.

BUG=609607
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove dependency of c/b/loader on c/b/host_zoom_map_impl.h

Contrary to previous attempt at:
https://chromium.googlesource.com/chromium/src.git/+/68c6f2ce16d9807b5cb82679...

RenderFrameMessageFilter is on the IO thread, and is a
BrowserAssociatedInterface, so will maintain the ordering of the resource
request and the zoom level request. See also
https://codereview.chromium.org/2167513003.

BUG=609607
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/d4423d511f18f4b0d2e94cf17bf6043bee449c55
Cr-Commit-Position: refs/heads/master@{#410417}
==========

[commit-bot: I haz the power, 2016-08-08 20:13:27]

Patchset 9 (id:??) landed as
https://crrev.com/d4423d511f18f4b0d2e94cf17bf6043bee449c55
Cr-Commit-Position: refs/heads/master@{#410417}

[scottmg, 2016-08-09 16:43:36]

A revert of this CL (patchset #9 id:180001) has been created in
https://codereview.chromium.org/2222403002/ by scottmg@chromium.org.

The reason for reverting is: Looks like there's a race during initialization on
chromeos https://bugs.chromium.org/p/chromium/issues/detail?id=635832#c3.