Change SBThreatType type where Safe Browsing Subresource Filter is triggered.

Change SBThreatType type where Safe Browsing Subresource Filter is triggered.

BUG=609747, 641037
Committed: https://crrev.com/069bcfc58e3505b86889b15f27e2da8539c2339d
Cr-Commit-Position: refs/heads/master@{#414886}

[melandory, 2016-08-26 19:20:57]

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

[commit-bot: I haz the power, 2016-08-26 19:21:54]

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

[melandory, 2016-08-26 19:47:31]

melandory@chromium.org changed reviewers:
+ engedy@chromium.org

[melandory, 2016-08-26 19:47:32]

PTAL, thanks!

[commit-bot: I haz the power, 2016-08-26 20:00:08]

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

[commit-bot: I haz the power, 2016-08-26 20:00:09]

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

[engedy, 2016-08-26 20:19:08]

LGTM % updating one test case. Thanks!

https://codereview.chromium.org/2286883002/diff/1/components/subresource_filt...
File
components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc
(right):

https://codereview.chromium.org/2286883002/diff/1/components/subresource_filt...
components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc:67:
{false, subresource_filter::kActivationListPhishingInterstitial,
According to trybots this should be true now, which actually makes sense.

[engedy, 2016-08-26 20:19:53]

Description was changed from

==========
Change SBThreatType type where Safe Browsing subresource Filter is triggered.

BUG=609747, 641037
==========

to

==========
Change SBThreatType type where Safe Browsing Subresource Filter is triggered.

BUG=609747, 641037
==========

[melandory, 2016-08-26 21:19:00]

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

[commit-bot: I haz the power, 2016-08-26 21:19:59]

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

[melandory, 2016-08-26 22:07:54]

https://codereview.chromium.org/2286883002/diff/1/components/subresource_filt...
File
components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc
(right):

https://codereview.chromium.org/2286883002/diff/1/components/subresource_filt...
components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc:67:
{false, subresource_filter::kActivationListPhishingInterstitial,
On 2016/08/26 20:19:08, engedy wrote:
> According to trybots this should be true now, which actually makes sense.

yep ;)

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

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

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

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

[melandory, 2016-08-26 22:44:44]

The CQ bit was checked by melandory@chromium.org

[melandory, 2016-08-26 22:44:44]

The patchset sent to the CQ was uploaded after l-g-t-m from engedy@chromium.org
Link to the patchset: https://codereview.chromium.org/2286883002/#ps20001
(title: "fix tests")

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

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-26 23:56:46]

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

[commit-bot: I haz the power, 2016-08-26 23:56:47]

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

[melandory, 2016-08-27 02:54:24]

The CQ bit was checked by melandory@chromium.org

[commit-bot: I haz the power, 2016-08-27 02:54:54]

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-27 10:27:49]

Description was changed from

==========
Change SBThreatType type where Safe Browsing Subresource Filter is triggered.

BUG=609747, 641037
==========

to

==========
Change SBThreatType type where Safe Browsing Subresource Filter is triggered.

BUG=609747, 641037
==========

[commit-bot: I haz the power, 2016-08-27 10:27:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-27 10:31:14]

Description was changed from

==========
Change SBThreatType type where Safe Browsing Subresource Filter is triggered.

BUG=609747, 641037
==========

to

==========
Change SBThreatType type where Safe Browsing Subresource Filter is triggered.

BUG=609747, 641037

Committed: https://crrev.com/069bcfc58e3505b86889b15f27e2da8539c2339d
Cr-Commit-Position: refs/heads/master@{#414886}
==========

[commit-bot: I haz the power, 2016-08-27 10:31:15]

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