diff options
author | nparker <nparker@chromium.org> | 2016-02-26 18:12:29 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-02-27 02:14:36 +0000 |
commit | c7614e617a9a7830a6b0da02701b0340699566b3 (patch) | |
tree | f4b4bdf4843f3f216211f191beca0e1157e0c957 /components/components_tests.gyp | |
parent | ef6f6ae5e4c96622286b563658d5cd62a6cf1197 (diff) | |
download | chromium_src-c7614e617a9a7830a6b0da02701b0340699566b3.zip chromium_src-c7614e617a9a7830a6b0da02701b0340699566b3.tar.gz chromium_src-c7614e617a9a7830a6b0da02701b0340699566b3.tar.bz2 |
Switch Safe Browsing's metadata from string to struct.
This is needed so we can add more types to it. This CL plumbs through
the struct but keeps the string in-tact. This is backward compatible with clank/'s current implementation.
I've added parsing methods in the appropriate place in the stack:
- For Pver4: v4_get_hash_protocol_manager.cc
- For XLB: safe_browsing_api_handler_util.cc
- For Pver3: protocol_parser.cc [separate CL]
The next two CLs:
- Switch clank/ to use the struct. https://chrome-internal-review.googlesource.com/#/c/249571/
- Enable parsing for Pver3, switch ui_manager.cc to use the parsed values, and clean up.
BUG=589610
Review URL: https://codereview.chromium.org/1726403006
Cr-Commit-Position: refs/heads/master@{#378082}
Diffstat (limited to 'components/components_tests.gyp')
-rw-r--r-- | components/components_tests.gyp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/components/components_tests.gyp b/components/components_tests.gyp index 1fc03c4..f7204ed 100644 --- a/components/components_tests.gyp +++ b/components/components_tests.gyp @@ -634,6 +634,7 @@ ], 'safe_browsing_db_unittest_sources': [ 'safe_browsing_db/prefix_set_unittest.cc', + 'safe_browsing_db/testing_util.h', 'safe_browsing_db/util_unittest.cc', 'safe_browsing_db/v4_get_hash_protocol_manager_unittest.cc', 'safe_browsing_db/v4_protocol_manager_util_unittest.cc', |