Enabling CRLSet for ChromeOS

chrome/browser/net/crl_set_fetcher.cc

 // Copyright (c) 2012 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/net/crl_set_fetcher.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/file_util.h"
 #include "base/numerics/safe_conversions.h"
 #include "base/path_service.h"
 #include "base/rand_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/time/time.h"
 #include "chrome/browser/component_updater/component_updater_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "content/public/browser/browser_thread.h"
 #include "net/cert/crl_set.h"
 #include "net/cert/crl_set_storage.h"
 #include "net/ssl/ssl_config_service.h"
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/profiles/profile_helper.h"
+#endif
 
 using component_updater::ComponentUpdateService;
 using content::BrowserThread;
 
 CRLSetFetcher::CRLSetFetcher() : cus_(NULL) {}
 
-bool CRLSetFetcher::GetCRLSetFilePath(base::FilePath* path) const {
-  bool ok = PathService::Get(chrome::DIR_USER_DATA, path);
+bool CRLSetFetcher::SetCRLSetFilePath(const std::string* user_id_hash) {
+
+  base::FilePath path;
+  bool ok = PathService::Get(chrome::DIR_USER_DATA, &path);
   if (!ok) {
     NOTREACHED();
     return false;
   }
-  *path = path->Append(chrome::kCRLSetFilename);
+#if defined(OS_CHROMEOS)

[Ryan Sleevi, 2014/07/16 19:39:28]

Every #ifdef makes for a sad panda. We should try to abstract this knowledge
into something more meaningful/platform agnostic.

Ideal world, zero #ifdefs, and any special functionality is gated on compile
time.
Acceptable world, #ifdefs would be limited to the IO thread initializer (or
similarly, singular place)

+  if (user_id_hash)
+    path =
+      path.Append(chromeos::ProfileHelper::GetUserProfileDir(*user_id_hash));

[Jorge Lucangeli Obes, 2014/07/16 16:36:34]

Four-space indentation (double indentation) after newline inside a statement.

+#endif
+  crl_path_ = path.Append(chrome::kCRLSetFilename);
   return true;
 }
 
+base::FilePath CRLSetFetcher::GetCRLSetFilePath() const {
+  return crl_path_;
+}
+
 void CRLSetFetcher::StartInitialLoad(ComponentUpdateService* cus) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
+  if (!SetCRLSetFilePath(NULL))
+    return;
+  PostInitialLoad(cus);
+}
+
+void CRLSetFetcher::StartInitialLoadWithHash(ComponentUpdateService* cus,
+                                             const std::string& user_id_hash) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+
+  if (!SetCRLSetFilePath(&user_id_hash))
+    return;
+  PostInitialLoad(cus);
+}
+
+void CRLSetFetcher::DeleteFromDisk() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+
+  if (!SetCRLSetFilePath(NULL))
+    return;
+  if (!BrowserThread::PostTask(
+          BrowserThread::FILE, FROM_HERE,
+          base::Bind(&CRLSetFetcher::DoDeleteFromDisk, this))) {
+    NOTREACHED();
+  }
+}
+
+void CRLSetFetcher::PostInitialLoad(ComponentUpdateService* cus) {
   cus_ = cus;
 
   if (!BrowserThread::PostTask(
           BrowserThread::FILE, FROM_HERE,
           base::Bind(&CRLSetFetcher::DoInitialLoadFromDisk, this))) {
     NOTREACHED();
   }
 }
-
-void CRLSetFetcher::DeleteFromDisk() {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-
-  if (!BrowserThread::PostTask(
-          BrowserThread::FILE, FROM_HERE,
-          base::Bind(&CRLSetFetcher::DoDeleteFromDisk, this))) {
-    NOTREACHED();
-  }
-}
 
 void CRLSetFetcher::DoInitialLoadFromDisk() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
-  base::FilePath crl_set_file_path;
-  if (!GetCRLSetFilePath(&crl_set_file_path))
-    return;
-
-  LoadFromDisk(crl_set_file_path, &crl_set_);
+  LoadFromDisk(GetCRLSetFilePath(), &crl_set_);
 
   uint32 sequence_of_loaded_crl = 0;
   if (crl_set_.get())
     sequence_of_loaded_crl = crl_set_->sequence();
 
   // Get updates, advertising the sequence number of the CRL set that we just
   // loaded, if any.
   if (!BrowserThread::PostTask(
           BrowserThread::UI, FROM_HERE,
           base::Bind(
@@ skipping 74 matching lines @@
 
   if (cus_->RegisterComponent(component) !=
       ComponentUpdateService::kOk) {
     NOTREACHED() << "RegisterComponent returned error";
   }
 }
 
 void CRLSetFetcher::DoDeleteFromDisk() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
-  base::FilePath crl_set_file_path;
-  if (!GetCRLSetFilePath(&crl_set_file_path))
-    return;
-
-  DeleteFile(crl_set_file_path, false /* not recursive */);
+  DeleteFile(GetCRLSetFilePath(), false /* not recursive */);
 }
 
 void CRLSetFetcher::OnUpdateError(int error) {
   LOG(WARNING) << "CRLSetFetcher got error " << error
                << " from component installer";
 }
 
 bool CRLSetFetcher::Install(const base::DictionaryValue& manifest,
                             const base::FilePath& unpack_path) {
   base::FilePath crl_set_file_path =
       unpack_path.Append(FILE_PATH_LITERAL("crl-set"));
-  base::FilePath save_to;
-  if (!GetCRLSetFilePath(&save_to))
-    return true;
+  base::FilePath save_to = GetCRLSetFilePath();
 
   std::string crl_set_bytes;
   if (!base::ReadFileToString(crl_set_file_path, &crl_set_bytes)) {
     LOG(WARNING) << "Failed to find crl-set file inside CRX";
     return false;
   }
 
   bool is_delta;
   if (!net::CRLSetStorage::GetIsDeltaUpdate(crl_set_bytes, &is_delta)) {
     LOG(WARNING) << "GetIsDeltaUpdate failed on CRL set from update CRX";
@@ skipping 42 matching lines @@
 
   return true;
 }
 
 bool CRLSetFetcher::GetInstalledFile(
     const std::string& file, base::FilePath* installed_file) {
   return false;
 }
 
 CRLSetFetcher::~CRLSetFetcher() {}