[HBD] Hardcode SEI threshold to 30 (for testers)

[HBD] Hardcode SEI threshold to 30 (for testers)

Normal users will have it configured to the real value via Finch. This
hardcoding is only for ease of testing in case the user does not have
a Finch config.

BUG=650903
Committed: https://crrev.com/ef97eaf65cfaaa1728bfeebb0fa82701a4d5481c
Cr-Commit-Position: refs/heads/master@{#422966}

[tommycli, 2016-10-04 00:50:50]

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

[commit-bot: I haz the power, 2016-10-04 00:51:25]

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

[tommycli, 2016-10-04 00:51:35]

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

[tommycli, 2016-10-04 00:51:36]

dominickn: PTAL, thanks!

[dominickn, 2016-10-04 00:54:27]

lgtm

[tommycli, 2016-10-04 00:54:59]

The CQ bit was unchecked by tommycli@chromium.org

[tommycli, 2016-10-04 00:54:59]

The CQ bit was checked by tommycli@chromium.org

[tommycli, 2016-10-04 00:55:10]

On 2016/10/04 00:54:27, dominickn wrote:
> lgtm

thank you!

[commit-bot: I haz the power, 2016-10-04 00:55:14]

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-10-04 01:03:24]

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

[commit-bot: I haz the power, 2016-10-04 01:03:24]

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

[tommycli, 2016-10-04 19:24:14]

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

[commit-bot: I haz the power, 2016-10-04 19:24:45]

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-10-04 21:03:34]

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

[commit-bot: I haz the power, 2016-10-04 21:03:34]

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_...)

[tommycli, 2016-10-04 21:11:43]

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

[commit-bot: I haz the power, 2016-10-04 21:12:23]

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

[tommycli, 2016-10-04 22:03:12]

tommycli@chromium.org changed reviewers:
+ thestig@chromium.org

[tommycli, 2016-10-04 22:03:13]

thestig: PTAL chrome/ stamp plz thanks!

[commit-bot: I haz the power, 2016-10-04 22:03:23]

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

[commit-bot: I haz the power, 2016-10-04 22:03:23]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2016-10-04 22:13:36]

rs lgtm

[tommycli, 2016-10-04 22:14:53]

On 2016/10/04 22:13:36, Lei Zhang wrote:
> rs lgtm

thank you sir!

[tommycli, 2016-10-04 22:14:57]

The CQ bit was checked by tommycli@chromium.org

[tommycli, 2016-10-04 22:14:57]

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/2391823003/#ps40001
(title: "fix one more test")

[commit-bot: I haz the power, 2016-10-04 22:15:51]

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-10-04 22:23:48]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-04 22:25:38]

Description was changed from

==========
[HBD] Hardcode SEI threshold to 30 (for testers)

Normal users will have it configured to the real value via Finch. This
hardcoding is only for ease of testing in case the user does not have
a Finch config.

BUG=650903
==========

to

==========
[HBD] Hardcode SEI threshold to 30 (for testers)

Normal users will have it configured to the real value via Finch. This
hardcoding is only for ease of testing in case the user does not have
a Finch config.

BUG=650903

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

[commit-bot: I haz the power, 2016-10-04 22:25:39]

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

[Reid Kleckner, 2016-10-05 17:32:46]

rnk@chromium.org changed reviewers:
+ rnk@chromium.org

[Reid Kleckner, 2016-10-05 17:32:47]

https://codereview.chromium.org/2391823003/diff/40001/chrome/browser/prerende...
File chrome/browser/prerender/prerender_browsertest.cc (right):

https://codereview.chromium.org/2391823003/diff/40001/chrome/browser/prerende...
chrome/browser/prerender/prerender_browsertest.cc:1369: CONTENT_SETTING_ALLOW);
This change broke this test on all the Clang ToT Win64 bots:
https://build.chromium.org/p/chromium.fyi/builders/ClangToTWin64%20tester/bui...

[tommycli, 2016-10-05 17:34:34]

On 2016/10/05 17:32:47, Reid Kleckner wrote:
>
https://codereview.chromium.org/2391823003/diff/40001/chrome/browser/prerende...
> File chrome/browser/prerender/prerender_browsertest.cc (right):
> 
>
https://codereview.chromium.org/2391823003/diff/40001/chrome/browser/prerende...
> chrome/browser/prerender/prerender_browsertest.cc:1369:
CONTENT_SETTING_ALLOW);
> This change broke this test on all the Clang ToT Win64 bots:
>
https://build.chromium.org/p/chromium.fyi/builders/ClangToTWin64%20tester/bui...

No problem I'll take a look.