Fix broken threading model in CheckDesktopNotificationPermission

extensions/browser/info_map.cc

 // Copyright 2013 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 "extensions/browser/info_map.h"
 
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_set.h"
 #include "content/public/browser/browser_thread.h"
 #include "extensions/common/constants.h"
@@ skipping 11 matching lines @@
 
 }  // namespace
 
 struct InfoMap::ExtraData {
   // When the extension was installed.
   base::Time install_time;
 
   // True if the user has allowed this extension to run in incognito mode.
   bool incognito_enabled;
 
+  // True if the user has disabled notifications for this extension manually.
+  bool notifications_disabled;
+
   ExtraData();
   ~ExtraData();
 };
 
 InfoMap::ExtraData::ExtraData() : incognito_enabled(false) {}
 
 InfoMap::ExtraData::~ExtraData() {}
 
 InfoMap::InfoMap() : signin_process_id_(-1) {}
 
 const ProcessMap& InfoMap::process_map() const { return process_map_; }
 
 void InfoMap::AddExtension(const Extension* extension,
                            base::Time install_time,
-                           bool incognito_enabled) {
+                           bool incognito_enabled,
+                           bool notifications_disabled) {
   CheckOnValidThread();
   extensions_.Insert(extension);
   disabled_extensions_.Remove(extension->id());
 
   extra_data_[extension->id()].install_time = install_time;
   extra_data_[extension->id()].incognito_enabled = incognito_enabled;
+  extra_data_[extension->id()].notifications_disabled = notifications_disabled;
 }
 
 void InfoMap::RemoveExtension(const std::string& extension_id,
                               const UnloadedExtensionInfo::Reason reason) {
   CheckOnValidThread();
   const Extension* extension = extensions_.GetByID(extension_id);
   extra_data_.erase(extension_id);  // we don't care about disabled extra data
   bool was_uninstalled = (reason != UnloadedExtensionInfo::REASON_DISABLE &&
                           reason != UnloadedExtensionInfo::REASON_TERMINATE);
   if (extension) {
@@ skipping 101 matching lines @@
 }
 
 void InfoMap::SetSigninProcess(int process_id) {
   signin_process_id_ = process_id;
 }
 
 bool InfoMap::IsSigninProcess(int process_id) const {
   return process_id == signin_process_id_;
 }
 
+void InfoMap::SetNotificationsDisabled(
+    const std::string& extension_id,
+    bool notifications_disabled) {
+  ExtraDataMap::iterator iter = extra_data_.find(extension_id);
+  if (iter != extra_data_.end())
+    iter->second.notifications_disabled = notifications_disabled;
+}
+
+bool InfoMap::AreNotificationsDisabled(
+    const std::string& extension_id) const {
+  ExtraDataMap::const_iterator iter = extra_data_.find(extension_id);
+  if (iter != extra_data_.end())
+    return iter->second.notifications_disabled;
+  return false;
+}
+
 InfoMap::~InfoMap() {
   if (quota_service_) {
     BrowserThread::DeleteSoon(
         BrowserThread::IO, FROM_HERE, quota_service_.release());
   }
 }
 
 }  // namespace extensions