Safe Browsing: Only check origins for API checks.

Safe Browsing: Only check origins for API checks.

The SB blacklist will contain information at the origin level so we
shouldn't waste resources checking longer paths.

BUG=561867, 647107, 543161
Committed: https://crrev.com/58011218715aba59b766b5886296b66e9c324824
Cr-Commit-Position: refs/heads/master@{#422986}

[kcarattini, 2016-10-04 06:16:40]

kcarattini@chromium.org changed reviewers:
+ nparker@chromium.org

[vakh (use Gerrit instead), 2016-10-04 06:35:36]

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

[vakh (use Gerrit instead), 2016-10-04 06:35:36]

lgtm

Can you please add bug 543161 also?

[Nathan Parker, 2016-10-04 20:32:49]

lgtm

[kcarattini, 2016-10-04 22:00:05]

Description was changed from

==========
Safe Browsing: Only check origins for API checks.

The SB blacklist will contain information at the origin level so we
shouldn't waste resources checking longer paths.

BUG=561867,647107
==========

to

==========
Safe Browsing: Only check origins for API checks.

The SB blacklist will contain information at the origin level so we
shouldn't waste resources checking longer paths.

BUG=561867,647107,543161
==========

[kcarattini, 2016-10-04 22:00:19]

On 2016/10/04 06:35:36, vakh (Varun Khaneja) wrote:
> lgtm
> 
> Can you please add bug 543161 also?

Done. Thanks!

[kcarattini, 2016-10-04 22:00:29]

The CQ bit was checked by kcarattini@chromium.org

[commit-bot: I haz the power, 2016-10-04 22:01: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-10-04 23:11:19]

Description was changed from

==========
Safe Browsing: Only check origins for API checks.

The SB blacklist will contain information at the origin level so we
shouldn't waste resources checking longer paths.

BUG=561867,647107,543161
==========

to

==========
Safe Browsing: Only check origins for API checks.

The SB blacklist will contain information at the origin level so we
shouldn't waste resources checking longer paths.

BUG=561867,647107,543161
==========

[commit-bot: I haz the power, 2016-10-04 23:11:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-04 23:14:43]

Description was changed from

==========
Safe Browsing: Only check origins for API checks.

The SB blacklist will contain information at the origin level so we
shouldn't waste resources checking longer paths.

BUG=561867,647107,543161
==========

to

==========
Safe Browsing: Only check origins for API checks.

The SB blacklist will contain information at the origin level so we
shouldn't waste resources checking longer paths.

BUG=561867,647107,543161

Committed: https://crrev.com/58011218715aba59b766b5886296b66e9c324824
Cr-Commit-Position: refs/heads/master@{#422986}
==========

[commit-bot: I haz the power, 2016-10-04 23:14:44]

Patchset 1 (id:??) landed as
https://crrev.com/58011218715aba59b766b5886296b66e9c324824
Cr-Commit-Position: refs/heads/master@{#422986}