PlzNavigate: Fix DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

PlzNavigate: Fix DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org, dominickn@chromium.org
BUG=510836
Committed: https://crrev.com/045680dc6a3ef6a2b55e834443aa171b85a9f534
Cr-Commit-Position: refs/heads/master@{#417373}

[scottmg, 2016-09-07 17:56:53]

Description was changed from

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

R=clamy@chromium.org
BUG=510836
==========

to

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org
BUG=510836
==========

[scottmg, 2016-09-07 17:58:50]

Description was changed from

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org
BUG=510836
==========

to

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org
BUG=510836
==========

[scottmg, 2016-09-07 17:58:50]

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

[scottmg, 2016-09-07 17:59:13]

Description was changed from

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org
BUG=510836
==========

to

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org, dominickn@chromium.org
BUG=510836
==========

[scottmg, 2016-09-07 18:00:09]

clamy: overall review and content OWNERS
dominickn: chrome/browser/download/download_request_limiter_unittest.cc

[scottmg, 2016-09-07 18:00:19]

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

[commit-bot: I haz the power, 2016-09-07 18:00:54]

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-09-07 18:17:42]

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

[commit-bot: I haz the power, 2016-09-07 18:17:42]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  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-09-07 18:30:34]

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

[commit-bot: I haz the power, 2016-09-07 18:31:05]

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-09-07 20:27:46]

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

[commit-bot: I haz the power, 2016-09-07 20:27:46]

Dry run: This issue passed the CQ dry run.

[dominickn, 2016-09-08 01:42:33]

On 2016/09/07 18:00:09, scottmg wrote:
> clamy: overall review and content OWNERS
> dominickn: chrome/browser/download/download_request_limiter_unittest.cc

chrome/browser/download/download_request_limiter_unittest.cc lgtm. Thanks for
helping make this a lot cleaner!

[scottmg, 2016-09-08 01:56:13]

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

[commit-bot: I haz the power, 2016-09-08 01:56:37]

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

[scottmg, 2016-09-08 01:56:47]

Patchset #4 (id:60001) has been deleted

[commit-bot: I haz the power, 2016-09-08 01:58:27]

Dry run: Try jobs failed on following builders:
  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-09-08 02:01:19]

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

[commit-bot: I haz the power, 2016-09-08 02:01:53]

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-09-08 04:32:02]

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

[commit-bot: I haz the power, 2016-09-08 04:32:03]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[clamy, 2016-09-08 13:13:40]

Thanks! Lgtm.

[scottmg, 2016-09-08 14:38:36]

The CQ bit was checked by scottmg@chromium.org

[scottmg, 2016-09-08 14:38:37]

The patchset sent to the CQ was uploaded after l-g-t-m from
dominickn@chromium.org
Link to the patchset: https://codereview.chromium.org/2319733002/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2016-09-08 14:38:57]

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-09-08 16:59:52]

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

[commit-bot: I haz the power, 2016-09-08 16:59:53]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[scottmg, 2016-09-08 17:29:38]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2016-09-08 17:30:19]

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-09-08 19:56:34]

Description was changed from

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org, dominickn@chromium.org
BUG=510836
==========

to

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org, dominickn@chromium.org
BUG=510836
==========

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

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2016-09-08 19:59:05]

Description was changed from

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org, dominickn@chromium.org
BUG=510836
==========

to

==========
PlzNavigate: Fix
DownloadRequestLimiterTest.DownloadRequestLimiter_RendererInitiated

Expose NavigateAndCommitRendererInitiated in TestRendererHost, use it to
fix download request limiter test for renderer-initiated case in
PlzNavigate.

R=clamy@chromium.org, dominickn@chromium.org
BUG=510836

Committed: https://crrev.com/045680dc6a3ef6a2b55e834443aa171b85a9f534
Cr-Commit-Position: refs/heads/master@{#417373}
==========

[commit-bot: I haz the power, 2016-09-08 19:59:06]

Patchset 4 (id:??) landed as
https://crrev.com/045680dc6a3ef6a2b55e834443aa171b85a9f534
Cr-Commit-Position: refs/heads/master@{#417373}