diff options
author | sdefresne <sdefresne@chromium.org> | 2015-07-16 01:34:35 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-07-16 08:35:10 +0000 |
commit | d967d55cf7a0f78e6ace9ec681d2d082e631810b (patch) | |
tree | 09139d0dae0027b25dd3f06637e663d4cad173eb /components/components_tests.gyp | |
parent | 835f5b61c251e60088256d952188632d2d671cba (diff) | |
download | chromium_src-d967d55cf7a0f78e6ace9ec681d2d082e631810b.zip chromium_src-d967d55cf7a0f78e6ace9ec681d2d082e631810b.tar.gz chromium_src-d967d55cf7a0f78e6ace9ec681d2d082e631810b.tar.bz2 |
Componentize chrome/browser/rlz
Add new class RLZTrackerDelegate that abstract access to embedder
specific singletons and informations and provide an implementation for
Chrome based on the previous implementation.
Split rlz_unittest.cc in tests of the RLZTracker and Chrome
implementation of the RLZTrackerDelegate interface and move the
RLZTracker tests into the component.
Add a new gyp/gn variable "enable_rlz_support". This variable is true on
the platforms that support RLZ (currently Windows, Mac, iOS and
ChromeOS). Use it to build library and unit tests even when the RLZ
support is not enabled in the Chrome binary (this is still controlled by
"enable_rlz" and depends on the branding).
Enable the tests on iOS and convert rlz_tracker_ios.mm to a C++ file.
TEST=Run unit_tests and components_unittests on a platform that supports
RLZ (Windows, Mac, iOS or ChromeOS) and check that they pass. Then build
Chrome with "enable_rlz" and manually checks that RLZ are sent with
searches as expected.
BUG=504841,508148
Review URL: https://codereview.chromium.org/1212163011
Cr-Commit-Position: refs/heads/master@{#339002}
Diffstat (limited to 'components/components_tests.gyp')
-rw-r--r-- | components/components_tests.gyp | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/components/components_tests.gyp b/components/components_tests.gyp index b52326c..066fff8 100644 --- a/components/components_tests.gyp +++ b/components/components_tests.gyp @@ -491,6 +491,9 @@ 'rappor/rappor_utils_unittest.cc', 'rappor/sampler_unittest.cc', ], + 'rlz_unittest_sources': [ + 'rlz/rlz_tracker_unittest.cc', + ], 'safe_json_unittest_sources': [ 'safe_json/json_sanitizer_unittest.cc', ], @@ -870,6 +873,23 @@ 'mime_util/mime_util.gyp:mime_util', ], 'conditions': [ + ['enable_rlz_support==1', { + 'sources': [ + '<@(rlz_unittest_sources)', + ], + 'dependencies': [ + '../net/net.gyp:net_test_support', + '../rlz/rlz.gyp:test_support_rlz', + 'components.gyp:rlz', + ], + 'conditions': [ + ['OS == "ios"', { + 'dependencies': [ + '../ui/base/ui_base.gyp:ui_base', + ], + }], + ], + }], ['toolkit_views == 1', { 'sources': [ 'bookmarks/browser/bookmark_node_data_unittest.cc', |