include safe_browsing_db/util.h directly in files not related to chunking

include safe_browsing_db/util.h directly in files not related to chunking

BUG=543161
Committed: https://crrev.com/fa183faf88675560951d0be353291218c6df48aa
Cr-Commit-Position: refs/heads/master@{#383755}

[vakh (use Gerrit instead), 2016-03-22 01:14:44]

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

[kcarattini, 2016-03-22 01:18:33]

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

[kcarattini, 2016-03-22 01:18:34]

Have you removed safe_browsing_util.h entirely? Or is it still being used for
something?

Kendra

[vakh (use Gerrit instead), 2016-03-22 01:20:12]

On 2016/03/22 at 01:18:34, kcarattini wrote:
> Have you removed safe_browsing_util.h entirely? Or is it still being used for
something?
> 
> Kendra

It is still being used by the chunking code. I'll remove it eventually as part
of v3 cleanup.
No v4 code should depend on it.

[Nathan Parker, 2016-03-24 19:10:01]

lgtm

Just add a little more context in the CL subj+description. thx

[vakh (use Gerrit instead), 2016-03-25 23:29:37]

Description was changed from

==========
include the correct util file

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

to

==========
include only safe_browsing_db/util.h directly in files unrelated to chunking

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

[vakh (use Gerrit instead), 2016-03-25 23:29:55]

On 2016/03/24 19:10:01, Nathan Parker wrote:
> lgtm
> 
> Just add a little more context in the CL subj+description. thx

Done.

[vakh (use Gerrit instead), 2016-03-26 00:44:07]

git fetch && git rebase

[vakh (use Gerrit instead), 2016-03-26 00:49:09]

Description was changed from

==========
include only safe_browsing_db/util.h directly in files unrelated to chunking

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

to

==========
include safe_browsing_db/util.h directly in files not related to chunking

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

[vakh (use Gerrit instead), 2016-03-26 00:50:59]

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

[vakh (use Gerrit instead), 2016-03-26 00:51:00]

mmenke@chromium.org: Please review changes in 
chrome/browser/prerender/prerender_browsertest.cc

benwells@chromium.org: Please review changes in 
chrome/browser/extensions/blacklist.cc
chrome/browser/extensions/fake_safe_browsing_database_manager.cc

[mmenke, 2016-03-28 14:57:23]

On 2016/03/26 00:51:00, vakh wrote:
> mailto:mmenke@chromium.org: Please review changes in 
> chrome/browser/prerender/prerender_browsertest.cc
> 
> mailto:benwells@chromium.org: Please review changes in 
> chrome/browser/extensions/blacklist.cc
> chrome/browser/extensions/fake_safe_browsing_database_manager.cc

prerender/ LGTM

[benwells, 2016-03-29 06:41:16]

c/b/e lgtm

[vakh (use Gerrit instead), 2016-03-29 16:25:22]

The CQ bit was checked by vakh@chromium.org

[vakh (use Gerrit instead), 2016-03-29 16:25:22]

The patchset sent to the CQ was uploaded after l-g-t-m from nparker@chromium.org
Link to the patchset: https://codereview.chromium.org/1824933002/#ps20001
(title: "git fetch && git rebase")

[commit-bot: I haz the power, 2016-03-29 16:25:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1824933002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1824933002/20001

[commit-bot: I haz the power, 2016-03-29 17:33:45]

Description was changed from

==========
include safe_browsing_db/util.h directly in files not related to chunking

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

to

==========
include safe_browsing_db/util.h directly in files not related to chunking

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

[commit-bot: I haz the power, 2016-03-29 17:33:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-29 17:36:13]

Description was changed from

==========
include safe_browsing_db/util.h directly in files not related to chunking

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

to

==========
include safe_browsing_db/util.h directly in files not related to chunking

BUG=543161

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

[commit-bot: I haz the power, 2016-03-29 17:36:15]

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