[Extensions] Validate Profile extistence in ActivityLog reporting

chrome/browser/extensions/activity_log/activity_log.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/extensions/activity_log/activity_log.h"
 
 #include <stddef.h>
 #include <set>
 #include <utility>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/json/json_string_value_serializer.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/synchronization/lock.h"
 #include "base/threading/thread_checker.h"
+#include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/activity_log/activity_action_constants.h"
 #include "chrome/browser/extensions/activity_log/counting_policy.h"
 #include "chrome/browser/extensions/activity_log/fullstream_ui_policy.h"
 #include "chrome/browser/extensions/api/activity_log_private/activity_log_private_api.h"
 #include "chrome/browser/extensions/extension_tab_util.h"
 #include "chrome/browser/prerender/prerender_manager.h"
 #include "chrome/browser/prerender/prerender_manager_factory.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "components/syncable_prefs/pref_service_syncable.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/web_contents.h"
@@ skipping 350 matching lines @@
     LAZY_INSTANCE_INITIALIZER;
 
 // Calls into the ActivityLog to log an api event or function call.
 // Must be called on the UI thread.
 void LogApiActivityOnUI(content::BrowserContext* browser_context,
                         const std::string& extension_id,
                         const std::string& activity_name,
                         std::unique_ptr<base::ListValue> args,
                         Action::ActionType type) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  // There's a chance that the |browser_context| was deleted some time during
+  // the thread hops.
+  // TODO(devlin): We should probably be doing this more extensively throughout
+  // extensions code.
+  if (g_browser_process->IsShuttingDown() ||
+      !g_browser_process->profile_manager()->IsValidProfile(browser_context)) {
+    return;
+  }
   ActivityLog* activity_log = ActivityLog::GetInstance(browser_context);
   if (!activity_log || !activity_log->ShouldLog(extension_id))
     return;
   scoped_refptr<Action> action =
       new Action(extension_id, base::Time::Now(), type, activity_name);
   action->set_args(std::move(args));
   activity_log->LogAction(action);
 }
 
 // Generic thread-safe handler for API calls and events.
@@ skipping 487 matching lines @@
   }
 }
 
 template <>
 void BrowserContextKeyedAPIFactory<ActivityLog>::DeclareFactoryDependencies() {
   DependsOn(ExtensionsBrowserClient::Get()->GetExtensionSystemFactory());
   DependsOn(ExtensionRegistryFactory::GetInstance());
 }
 
 }  // namespace extensions