Fix crashes in ExtensionService

chrome/browser/ui/browser_init.cc

 // Copyright (c) 2011 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/ui/browser_init.h"
 
 #include <algorithm>   // For max().
 
 #include "base/environment.h"
 #include "base/event_recorder.h"
@@ skipping 439 matching lines @@
 void RecordCmdLineAppHistogram() {
   UMA_HISTOGRAM_ENUMERATION(extension_misc::kAppLaunchHistogram,
                             extension_misc::APP_LAUNCH_CMD_LINE_APP,
                             extension_misc::APP_LAUNCH_BUCKET_BOUNDARY);
 }
 
 void RecordAppLaunches(
     Profile* profile,
     const std::vector<GURL>& cmd_line_urls,
     const std::vector<BrowserInit::LaunchWithProfile::Tab>& autolaunch_tabs) {
+  // TODO: the ExtensionService should never be NULL, but in some cases it is,
+  // see bug 73768. After it is resolved, the explicit test can go away.
   ExtensionService* extension_service = profile->GetExtensionService();
+  if (!extension_service)
+    return;
 
   for (size_t i = 0; i < cmd_line_urls.size(); ++i) {
     if (extension_service->IsInstalledApp(cmd_line_urls.at(i))) {
       UMA_HISTOGRAM_ENUMERATION(extension_misc::kAppLaunchHistogram,
                                 extension_misc::APP_LAUNCH_CMD_LINE_URL,
                                 extension_misc::APP_LAUNCH_BUCKET_BOUNDARY);
     }
   }
 
   for (size_t i = 0; i < autolaunch_tabs.size(); ++i) {
@@ skipping 803 matching lines @@
     return false;
   automation->SetExpectedTabCount(expected_tabs);
 
   AutomationProviderList* list =
       g_browser_process->InitAutomationProviderList();
   DCHECK(list);
   list->AddProvider(automation);
 
   return true;
 }