Disable ProfileKeyedServices on import process by default.

chrome/browser/webdata/web_data_service.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/webdata/web_data_service.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/message_loop.h"
 #include "base/stl_util.h"
@@ skipping 72 matching lines @@
       request_manager_(new WebDataRequestManager()),
       app_locale_(AutofillCountry::ApplicationLocale()),
       autocomplete_syncable_service_(NULL),
       autofill_profile_syncable_service_(NULL),
       failed_init_(false),
       should_commit_(false),
       main_loop_(MessageLoop::current()) {
   // WebDataService requires DB thread if instantiated.
   // Set WebDataServiceFactory::GetInstance()->SetTestingFactory(&profile, NULL)
   // if you do not want to instantiate WebDataService in your test.
-  DCHECK(!ProfileManager::IsImportProcess(*CommandLine::ForCurrentProcess()));
+  CHECK(!ProfileManager::IsImportProcess(*CommandLine::ForCurrentProcess()));

[Peter Kasting, 2013/02/22 21:34:55]

CHECK means "Chrome has to crash immediately if this assertion fails, because we
are about to have a security exploit"; or "temporarily changed this to CHECK to
try and track down a bug".  Are those really what you mean?  If so, please add
an appropriate comment.  Otherwise, I don't think we should be using CHECK.

Also please see my comment on your last code review here.

   DCHECK(BrowserThread::IsWellKnownThread(BrowserThread::DB));
 }
 
 // static
 void WebDataService::NotifyOfMultipleAutofillChanges(
     WebDataService* web_data_service) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
 
   if (!web_data_service)
     return;
@@ skipping 844 matching lines @@
 
 void WebDataService::DestroyAutofillCreditCardResult(
       const WDTypedResult* result) {
   DCHECK(result->GetType() == AUTOFILL_CREDITCARDS_RESULT);
   const WDResult<std::vector<CreditCard*> >* r =
       static_cast<const WDResult<std::vector<CreditCard*> >*>(result);
 
   std::vector<CreditCard*> credit_cards = r->GetValue();
   STLDeleteElements(&credit_cards);
 }