Remove unnecessary copies in CRLSet parsing.

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"
@@ skipping 48 matching lines @@
           base::Bind(
               &CRLSetFetcher::RegisterComponent,
               this,
               sequence_of_loaded_crl))) {
     NOTREACHED();
   }
 }
 
 void CRLSetFetcher::LoadFromDisk(base::FilePath path,
                                  scoped_refptr<net::CRLSet>* out_crl_set) {
+  TRACE_EVENT0("CRLSetFetcher", "LoadFromDisk");
+
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
   std::string crl_set_bytes;
-  if (!base::ReadFileToString(path, &crl_set_bytes))
-    return;
+  {
+    TRACE_EVENT0("CRLSetFetcher", "ReadFileToString");
+    if (!base::ReadFileToString(path, &crl_set_bytes))
+      return;
+  }
 
   if (!net::CRLSet::Parse(crl_set_bytes, out_crl_set)) {
     LOG(WARNING) << "Failed to parse CRL set from " << path.MaybeAsASCII();
     return;
   }
 
   VLOG(1) << "Loaded " << crl_set_bytes.size() << " bytes of CRL set from disk";
 
   if (!BrowserThread::PostTask(
           BrowserThread::IO, FROM_HERE,
@@ skipping 114 matching lines @@
 
   return true;
 }
 
 bool CRLSetFetcher::GetInstalledFile(
     const std::string& file, base::FilePath* installed_file) {
   return false;
 }
 
 CRLSetFetcher::~CRLSetFetcher() {}