Reland: Start checking URLs using PVer4. Verdict not returned to client yet.

Reland: Start checking URLs using PVer4. Verdict not returned to client yet.
Reason for failure: The features that are enabled by Finch were also enabled by #DEBUG. This enabled them on the DEBUG build of Android, which is wrong.

BUG=543161
TBR=mmenke

Committed: https://crrev.com/939c27d111805a5c1e40212373bd4cb4864b7dc3
Cr-Commit-Position: refs/heads/master@{#422445}

[vakh (use Gerrit instead), 2016-09-29 22:42:51]

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

[commit-bot: I haz the power, 2016-09-29 22:43:15]

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-30 00:49:37]

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

[commit-bot: I haz the power, 2016-09-30 00:49:38]

Dry run: This issue passed the CQ dry run.

[vakh (use Gerrit instead), 2016-09-30 02:05:45]

Remove #DEBUG -- it enables these features on Android :-(

[vakh (use Gerrit instead), 2016-09-30 02:05:47]

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

[commit-bot: I haz the power, 2016-09-30 02:06:08]

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

[vakh (use Gerrit instead), 2016-09-30 02:10:15]

Description was changed from

==========
Reland: Start checking URLs using PVer4. Verdict not returned to client yet.

BUG=543161
==========

to

==========
Reland: Start checking URLs using PVer4. Verdict not returned to client yet.
Reason for failure: The features that are enabled by Finch were also enabled by
#DEBUG. This enabled them on the DEBUG build of Android, which is wrong.

BUG=543161
==========

[vakh (use Gerrit instead), 2016-09-30 02:10:15]

vakh@chromium.org changed reviewers:
+ mmenke@chromium.org, nparker@chromium.org

[commit-bot: I haz the power, 2016-09-30 02:58:36]

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

[commit-bot: I haz the power, 2016-09-30 02:58:37]

Dry run: This issue passed the CQ dry run.

[Nathan Parker, 2016-09-30 18:59:58]

lgtm

[vakh (use Gerrit instead), 2016-09-30 19:34:24]

mmenke@ -- could you please review the changes in:
chrome/browser/loader/safe_browsing_resource_throttle.*

It is the same as: https://codereview.chromium.org/2371043003/

[vakh (use Gerrit instead), 2016-10-01 07:27:29]

Description was changed from

==========
Reland: Start checking URLs using PVer4. Verdict not returned to client yet.
Reason for failure: The features that are enabled by Finch were also enabled by
#DEBUG. This enabled them on the DEBUG build of Android, which is wrong.

BUG=543161
==========

to

==========
Reland: Start checking URLs using PVer4. Verdict not returned to client yet.
Reason for failure: The features that are enabled by Finch were also enabled by
#DEBUG. This enabled them on the DEBUG build of Android, which is wrong.

BUG=543161
TBR=mmenke
==========

[vakh (use Gerrit instead), 2016-10-01 07:27:39]

The CQ bit was checked by vakh@chromium.org

[commit-bot: I haz the power, 2016-10-01 07:27:43]

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

[vakh (use Gerrit instead), 2016-10-01 07:28:30]

The CQ bit was unchecked by vakh@chromium.org

[mmenke, 2016-10-03 14:34:32]

On 2016/10/01 07:27:43, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

Sorry, took Friday off.  LGTM (And I agree that it was completely fine to TBR
this, stuff I own hasn't changed, and wasn't a major reviewer of this CL)

[vakh (use Gerrit instead), 2016-10-03 16:06:34]

The CQ bit was checked by vakh@chromium.org

[commit-bot: I haz the power, 2016-10-03 16:06: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-10-03 17:20:17]

Description was changed from

==========
Reland: Start checking URLs using PVer4. Verdict not returned to client yet.
Reason for failure: The features that are enabled by Finch were also enabled by
#DEBUG. This enabled them on the DEBUG build of Android, which is wrong.

BUG=543161
TBR=mmenke
==========

to

==========
Reland: Start checking URLs using PVer4. Verdict not returned to client yet.
Reason for failure: The features that are enabled by Finch were also enabled by
#DEBUG. This enabled them on the DEBUG build of Android, which is wrong.

BUG=543161
TBR=mmenke
==========

[commit-bot: I haz the power, 2016-10-03 17:20:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-03 17:22:06]

Description was changed from

==========
Reland: Start checking URLs using PVer4. Verdict not returned to client yet.
Reason for failure: The features that are enabled by Finch were also enabled by
#DEBUG. This enabled them on the DEBUG build of Android, which is wrong.

BUG=543161
TBR=mmenke
==========

to

==========
Reland: Start checking URLs using PVer4. Verdict not returned to client yet.
Reason for failure: The features that are enabled by Finch were also enabled by
#DEBUG. This enabled them on the DEBUG build of Android, which is wrong.

BUG=543161
TBR=mmenke

Committed: https://crrev.com/939c27d111805a5c1e40212373bd4cb4864b7dc3
Cr-Commit-Position: refs/heads/master@{#422445}
==========

[commit-bot: I haz the power, 2016-10-03 17:22:07]

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