Certificate Transparency: EV certificates whitelist support for ChromeOS

chrome/browser/chrome_browser_main.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/chrome_browser_main.h"
 
 #include <set>
 #include <string>
 #include <vector>
 
@@ skipping 413 matching lines @@
   if (PathService::Get(chrome::DIR_USER_DATA, &path)) {
 #if defined(OS_ANDROID)
     // The CRLSet component was enabled for some releases. This code attempts to
     // delete it from the local disk of those how may have downloaded it.
     g_browser_process->crl_set_fetcher()->DeleteFromDisk(path);
 #elif !defined(OS_CHROMEOS)
     // CRLSetFetcher attempts to load a CRL set from either the local disk or
     // network.
     // For Chrome OS this registration is delayed until user login.
     g_browser_process->crl_set_fetcher()->StartInitialLoad(cus, path);
+    // Android does not currently have the EV indicator. No need to get the

[Sorin Jianu, 2014/11/07 17:30:44]

Can you please clarify the comment? 

[Eran Messeri, 2014/11/10 22:54:56]

Done.

+    // EV certs whitelist on Android, then.
+    // For Chrome OS this registration is delayed until user login.
+    RegisterEVWhitelistComponent(cus, path);
 #endif
   }
 
-#if !defined(OS_ANDROID)
-  // Android does not currently have the EV indicator. No need to get the
-  // EV certs whitelist on Android, then.
-  RegisterEVWhitelistComponent(cus);
-#endif
-
 #if defined(OS_WIN)
   RegisterSwReporterComponent(cus, g_browser_process->local_state());
 #endif
 
   cus->Start();
 }
 
 #if !defined(OS_ANDROID)
 bool ProcessSingletonNotificationCallback(
     const CommandLine& command_line,
@@ skipping 1246 matching lines @@
   chromeos::CrosSettings::Shutdown();
 #endif
 #endif
 }
 
 // Public members:
 
 void ChromeBrowserMainParts::AddParts(ChromeBrowserMainExtraParts* parts) {
   chrome_extra_parts_.push_back(parts);
 }