diff options
| author | amohammadkhan <amohammadkhan@google.com> | 2015-09-14 10:34:43 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-09-14 17:35:19 +0000 |
| commit | f76ae11f25dc0056cea63e15c16039fdbd5f3470 (patch) | |
| tree | fd1285923858b8b5ed211c405aefd9ec9cb98106 /sync/tools | |
| parent | 7ba3d6825c0bb7ce3eee3b4fae4a3fa7325d5e11 (diff) | |
| download | chromium_src-f76ae11f25dc0056cea63e15c16039fdbd5f3470.zip chromium_src-f76ae11f25dc0056cea63e15c16039fdbd5f3470.tar.gz chromium_src-f76ae11f25dc0056cea63e15c16039fdbd5f3470.tar.bz2 | |
Report data usage UMA for Chrome services
This issue has the same goals as issue 1330443002( It was reverted
because it breaded compile on multiple bots).
data_use_measurement is a component which records the data use of the
services, if a DataUseUserData is attached to their request. Hence to
measure and compare data use of different services, a small change is
needed in the services' code where they create a URLFetcher. After
creating a URLFetcher by a service, a proper tag for that service should
be attached to the request.
Besides the necessary changes in the service, the enum in DataUseUserData
and the function returning the names based on this enum should be updated
for different services too. Also the name of target services should be
added to proper enum and histograms in histograms.xml file.
Lastly the dependency files should be updated accordingly too.
TBR=bartfab@chromium.org,andrewhayden@chromium.org,pkasting@chromium.org,sclittle@chromium.org,thestig@chromium.org,isherman@chromium.org,rouslan@chromium.org,nyquist@chromium.org,bengr@chromium.org,asvltkine@chromium.org,zea@chromium.org,mmenke@chromium.org
BUG=527304
Review URL: https://codereview.chromium.org/1336333003
Cr-Commit-Position: refs/heads/master@{#348645}
Diffstat (limited to 'sync/tools')
| -rw-r--r-- | sync/tools/sync_client.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sync/tools/sync_client.cc b/sync/tools/sync_client.cc index 2ec1f19..5999e11 100644 --- a/sync/tools/sync_client.cc +++ b/sync/tools/sync_client.cc @@ -34,6 +34,7 @@ #include "sync/internal_api/public/base_node.h" #include "sync/internal_api/public/engine/passive_model_worker.h" #include "sync/internal_api/public/http_bridge.h" +#include "sync/internal_api/public/http_post_provider_factory.h" #include "sync/internal_api/public/internal_components_factory_impl.h" #include "sync/internal_api/public/read_node.h" #include "sync/internal_api/public/sync_manager.h" @@ -405,7 +406,7 @@ int SyncClientMain(int argc, char* argv[]) { new HttpBridgeFactory(context_getter.get(), base::Bind(&StubNetworkTimeUpdateCallback), &factory_cancelation_signal)); - post_factory->Init(kUserAgent); + post_factory->Init(kUserAgent, BindToTrackerCallback()); // Used only when committing bookmarks, so it's okay to leave this // as NULL. ExtensionsActivity* extensions_activity = NULL; |