Add module for counting date-last-roll-call and persisting those counts

chrome/browser/component_updater/chrome_component_updater_configurator.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/component_updater/chrome_component_updater_configurator.h"
 
 #include <string>
 #include <vector>
 
+#include "base/files/file_path.h"
+#include "base/path_service.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "base/version.h"
 #if defined(OS_WIN)
 #include "base/win/win_util.h"
 #endif
 #include "chrome/browser/component_updater/component_patcher_operation_out_of_process.h"
 #include "chrome/browser/google/google_brand.h"
 #include "chrome/browser/update_client/chrome_update_query_params_delegate.h"
 #include "chrome/common/channel_info.h"
+#include "chrome/common/chrome_paths.h"
 #if defined(OS_WIN)
 #include "chrome/installer/util/google_update_settings.h"
 #endif
 #include "components/component_updater/configurator_impl.h"
 #include "content/public/browser/browser_thread.h"
 
 namespace component_updater {
 
 namespace {
 
@@ skipping 18 matching lines @@
   std::string ExtraRequestParams() const override;
   std::string GetDownloadPreference() const override;
   net::URLRequestContextGetter* RequestContext() const override;
   scoped_refptr<update_client::OutOfProcessPatcher> CreateOutOfProcessPatcher()
       const override;
   bool DeltasEnabled() const override;
   bool UseBackgroundDownloader() const override;
   bool UseCupSigning() const override;
   scoped_refptr<base::SequencedTaskRunner> GetSequencedTaskRunner()
       const override;
+  base::FilePath GetMetadataPath() const override;
 
  private:
   friend class base::RefCountedThreadSafe<ChromeConfigurator>;
 
   ConfiguratorImpl configurator_impl_;
 
   ~ChromeConfigurator() override {}
 };
 
 // Allows the component updater to use non-encrypted communication with the
@@ skipping 97 matching lines @@
 // imposes certain requirements for the code using the task runner, such as
 // not accessing any global browser state while the code is running.
 scoped_refptr<base::SequencedTaskRunner>
 ChromeConfigurator::GetSequencedTaskRunner() const {
   return content::BrowserThread::GetBlockingPool()
       ->GetSequencedTaskRunnerWithShutdownBehavior(
           base::SequencedWorkerPool::GetSequenceToken(),
           base::SequencedWorkerPool::CONTINUE_ON_SHUTDOWN);
 }
 
+base::FilePath ChromeConfigurator::GetMetadataPath() const {
+  base::FilePath result;
+  CHECK(PathService::Get(chrome::DIR_USER_DATA, &result));

[Sorin Jianu, 2016/04/07 04:03:19]

Do we need to CHECK? Can we be fault tolerant? I don't have an opinion, I am
just asking the question.

[waffles, 2016/04/07 17:04:43]

Done. In practice it won't matter because several component installers do this
same CHECK, but you make a good point.

+  return result.AppendASCII("component_metadata.json");
+}
+
 }  // namespace
 
 scoped_refptr<update_client::Configurator>
 MakeChromeComponentUpdaterConfigurator(
     const base::CommandLine* cmdline,
     net::URLRequestContextGetter* context_getter) {
   return new ChromeConfigurator(cmdline, context_getter);
 }
 
 }  // namespace component_updater