Split incognito extension content settings in session-only and persistent.

chrome/browser/extensions/extension_content_settings_store.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/extension_content_settings_store.h"
 
 #include <set>
 
 #include "base/logging.h"
 #include "base/stl_util-inl.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/extension_content_settings_api_constants.h"
 #include "chrome/browser/extensions/extension_content_settings_helpers.h"
 #include "content/browser/browser_thread.h"
 
 namespace helpers = extension_content_settings_helpers;
 namespace keys = extension_content_settings_api_constants;
 
 using content_settings::ResourceIdentifier;
 
 struct ExtensionContentSettingsStore::ExtensionEntry {
   // Installation time of the extension.
   base::Time install_time;
   // Whether extension is enabled in the profile.
   bool enabled;
   // Content settings.
   ContentSettingSpecList settings;
-  // Incognito content settings.
-  ContentSettingSpecList incognito_settings;
+  // Persistent incognito content settings.
+  ContentSettingSpecList incognito_persistent_settings;
+  // Session-only incognito content settings.
+  ContentSettingSpecList incognito_session_only_settings;
 };
 
 ExtensionContentSettingsStore::ExtensionContentSettingsStore() {
   DCHECK(OnCorrectThread());
 }
 
 ExtensionContentSettingsStore::~ExtensionContentSettingsStore() {
   DCHECK(OnCorrectThread());
   NotifyOfDestruction();
   STLDeleteValues(&entries_);
 }
 
 void ExtensionContentSettingsStore::SetExtensionContentSetting(
     const std::string& ext_id,
     const ContentSettingsPattern& embedded_pattern,
     const ContentSettingsPattern& top_level_pattern,
     ContentSettingsType type,
     const content_settings::ResourceIdentifier& identifier,
     ContentSetting setting,
-    bool incognito) {
+    extension_prefs_scope::Scope scope) {
   {
     base::AutoLock lock(lock_);
     ContentSettingSpecList* setting_spec_list =
-        GetContentSettingSpecList(ext_id, incognito);
+        GetContentSettingSpecList(ext_id, scope);
 
     // Find |ContentSettingSpec|.
     ContentSettingSpecList::iterator setting_spec = setting_spec_list->begin();
     while (setting_spec != setting_spec_list->end()) {
       if (setting_spec->embedded_pattern == embedded_pattern &&
           setting_spec->top_level_pattern == top_level_pattern &&
           setting_spec->content_type == type &&
           setting_spec->resource_identifier == identifier) {
         break;
       }
@@ skipping 14 matching lines @@
       setting_spec->setting = setting;
     } else {
       setting_spec_list->erase(setting_spec);
     }
   }
 
   // Send notification that content settings changed.
   // TODO(markusheintz): Notifications should only be sent if the set content
   // setting is effective and not hidden by another setting of another
   // extension installed more recently.
-  NotifyOfContentSettingChanged(ext_id, incognito);
+  NotifyOfContentSettingChanged(ext_id,
+                                scope != extension_prefs_scope::kRegular);
 }
 
 void ExtensionContentSettingsStore::RegisterExtension(
     const std::string& ext_id,
     const base::Time& install_time,
     bool is_enabled) {
   base::AutoLock lock(lock_);
   ExtensionEntryMap::iterator i = entries_.find(ext_id);
   if (i != entries_.end())
     delete i->second;
 
   entries_[ext_id] = new ExtensionEntry;
   entries_[ext_id]->install_time = install_time;
   entries_[ext_id]->enabled = is_enabled;
 }
 
 void ExtensionContentSettingsStore::UnregisterExtension(
     const std::string& ext_id) {
   bool notify = false;
   bool notify_incognito = false;
   {
     base::AutoLock lock(lock_);
     ExtensionEntryMap::iterator i = entries_.find(ext_id);
     if (i == entries_.end())
       return;
     notify = !i->second->settings.empty();
-    notify_incognito = !i->second->incognito_settings.empty();
+    notify_incognito = !i->second->incognito_persistent_settings.empty() ||
+                       !i->second->incognito_session_only_settings.empty();
 
     delete i->second;
     entries_.erase(i);
   }
   if (notify)
     NotifyOfContentSettingChanged(ext_id, false);
   if (notify_incognito)
     NotifyOfContentSettingChanged(ext_id, true);
 }
 
 void ExtensionContentSettingsStore::SetExtensionState(
     const std::string& ext_id, bool is_enabled) {
   bool notify = false;
   bool notify_incognito = false;
   {
     base::AutoLock lock(lock_);
     ExtensionEntryMap::const_iterator i = entries_.find(ext_id);
     if (i == entries_.end())
       return;
     notify = !i->second->settings.empty();
-    notify_incognito = !i->second->incognito_settings.empty();
+    notify_incognito = !i->second->incognito_persistent_settings.empty() ||
+                       !i->second->incognito_session_only_settings.empty();
 
     i->second->enabled = is_enabled;
   }
   if (notify)
     NotifyOfContentSettingChanged(ext_id, false);
   if (notify_incognito)
     NotifyOfContentSettingChanged(ext_id, true);
 }
 
 ExtensionContentSettingsStore::ContentSettingSpecList*
     ExtensionContentSettingsStore::GetContentSettingSpecList(
         const std::string& ext_id,
-        bool incognito) {
+        extension_prefs_scope::Scope scope) {
   ExtensionEntryMap::const_iterator i = entries_.find(ext_id);
   DCHECK(i != entries_.end());
-  return incognito ? &(i->second->incognito_settings)
-                   : &(i->second->settings);
+  switch (scope) {
+    case extension_prefs_scope::kRegular:
+      return &(i->second->settings);
+    case extension_prefs_scope::kIncognitoPersistent:
+      return &(i->second->incognito_persistent_settings);
+    case extension_prefs_scope::kIncognitoSessionOnly:
+      return &(i->second->incognito_session_only_settings);
+  }
+  NOTREACHED();
+  return NULL;
 }
 
 const ExtensionContentSettingsStore::ContentSettingSpecList*
     ExtensionContentSettingsStore::GetContentSettingSpecList(
         const std::string& ext_id,
-        bool incognito) const {
+        extension_prefs_scope::Scope scope) const {
   ExtensionEntryMap::const_iterator i = entries_.find(ext_id);
   DCHECK(i != entries_.end());
-  return incognito ? &(i->second->incognito_settings)
-                   : &(i->second->settings);
+  switch (scope) {
+    case extension_prefs_scope::kRegular:
+      return &(i->second->settings);
+    case extension_prefs_scope::kIncognitoPersistent:
+      return &(i->second->incognito_persistent_settings);
+    case extension_prefs_scope::kIncognitoSessionOnly:
+      return &(i->second->incognito_session_only_settings);
+  }
+  NOTREACHED();
+  return NULL;
 }
 
-
 ContentSetting ExtensionContentSettingsStore::GetContentSettingFromSpecList(
     const GURL& embedded_url,
     const GURL& top_level_url,
     ContentSettingsType type,
     const content_settings::ResourceIdentifier& identifier,
     const ContentSettingSpecList& setting_spec_list) const {
   ContentSettingSpecList::const_iterator winner_spec = setting_spec_list.end();
 
   for (ContentSettingSpecList::const_iterator spec = setting_spec_list.begin();
        spec != setting_spec_list.end(); ++spec) {
     if (!spec->embedded_pattern.Matches(embedded_url) ||
         !spec->top_level_pattern.Matches(top_level_url) ||
         spec->content_type != type ||
         spec->resource_identifier != identifier) {
       continue;
     }
 
-    // TODO(markusheintz): Compare top-level patterns as well?
+    // TODO(markusheintz): Compare embedded patterns as well?
     if (winner_spec == setting_spec_list.end() ||
-        spec->embedded_pattern.Compare(winner_spec->embedded_pattern) ==
+        winner_spec->top_level_pattern.Compare(spec->top_level_pattern) ==
             ContentSettingsPattern::SUCCESSOR) {
       winner_spec = spec;
     }
   }
 
   return (winner_spec != setting_spec_list.end()) ? winner_spec->setting
                                                   : CONTENT_SETTING_DEFAULT;
 }
 
 ContentSetting ExtensionContentSettingsStore::GetEffectiveContentSetting(
@@ skipping 12 matching lines @@
     const base::Time& install_time = i->second->install_time;
     const bool enabled = i->second->enabled;
 
     if (!enabled)
       continue;
     if (install_time < winners_install_time)
       continue;
 
     ContentSetting setting = CONTENT_SETTING_DEFAULT;
     if (incognito) {
-      // Get incognito setting
-      setting = GetContentSettingFromSpecList(embedded_url, top_level_url, type,
-                                              identifier,
-                                              i->second->incognito_settings);
+      // Try session-only incognito setting first.
+      setting = GetContentSettingFromSpecList(
+          embedded_url, top_level_url, type, identifier,
+          i->second->incognito_session_only_settings);
+      if (setting == CONTENT_SETTING_DEFAULT) {
+        // Next, persistent incognito setting.
+        setting = GetContentSettingFromSpecList(
+            embedded_url, top_level_url, type, identifier,
+            i->second->incognito_persistent_settings);
+      }
     }
     if (setting == CONTENT_SETTING_DEFAULT) {
-      // Get non incognito setting
+      // Then, non-incognito setting.
       setting = GetContentSettingFromSpecList(embedded_url, top_level_url, type,
                                               identifier, i->second->settings);
     }
 
     if (setting != CONTENT_SETTING_DEFAULT) {
       winners_install_time = install_time;
       winner_setting = setting;
     }
   }
 
   return winner_setting;
 }
 
+void ExtensionContentSettingsStore::ClearContentSettingsForExtension(
+    const std::string& ext_id,
+    extension_prefs_scope::Scope scope) {
+  bool notify = false;
+  {
+    base::AutoLock lock(lock_);
+    ContentSettingSpecList* setting_spec_list =
+        GetContentSettingSpecList(ext_id, scope);
+    notify = !setting_spec_list->empty();
+    setting_spec_list->clear();
+  }
+  if (notify) {
+    NotifyOfContentSettingChanged(ext_id,
+                                  scope != extension_prefs_scope::kRegular);
+  }
+}
+
+// static
+void ExtensionContentSettingsStore::AddRules(
+    ContentSettingsType type,
+    const content_settings::ResourceIdentifier& identifier,
+    content_settings::ProviderInterface::Rules* rules,
+    const ContentSettingSpecList* setting_spec_list) {
+  ContentSettingSpecList::const_iterator it;
+  for (it = setting_spec_list->begin(); it != setting_spec_list->end(); ++it) {
+    if (it->content_type == type && it->resource_identifier == identifier) {
+      rules->push_back(content_settings::ProviderInterface::Rule(
+          it->embedded_pattern, it->top_level_pattern, it->setting));
+    }
+  }
+}
+
 void ExtensionContentSettingsStore::GetContentSettingsForContentType(
     ContentSettingsType type,
     const content_settings::ResourceIdentifier& identifier,
     bool incognito,
     content_settings::ProviderInterface::Rules* rules) const {
   base::AutoLock lock(lock_);
   ExtensionEntryMap::const_iterator ext_it;
   for (ext_it = entries_.begin(); ext_it != entries_.end(); ++ext_it) {
-    const ContentSettingSpecList* setting_spec_list =
-        GetContentSettingSpecList(ext_it->first, incognito);
-    ContentSettingSpecList::const_iterator it;
-    for (it = setting_spec_list->begin(); it != setting_spec_list->end();
-         ++it) {
-      if (it->content_type == type && it->resource_identifier == identifier) {
-        rules->push_back(content_settings::ProviderInterface::Rule(
-            it->embedded_pattern, it->top_level_pattern, it->setting));
-      }
+    if (incognito) {
+      AddRules(type, identifier, rules,
+               GetContentSettingSpecList(
+                   ext_it->first,
+                   extension_prefs_scope::kIncognitoSessionOnly));
+      AddRules(type, identifier, rules,
+               GetContentSettingSpecList(
+                   ext_it->first,
+                   extension_prefs_scope::kIncognitoPersistent));

[battre, 2011/06/03 18:48:36]

This can create duplicated entires in |rules|. Is this ok?

[Bernhard Bauer, 2011/06/03 20:34:28]

The entries are going to be sorted (and un-duped) by the HostContentSettingsMap
before showing them e.g. in the UI, but we should put session-only settings
after persistent ones to make sure that they're going to overwrite the others.

+    } else {
+      AddRules(type, identifier, rules,
+               GetContentSettingSpecList(
+                   ext_it->first,
+                   extension_prefs_scope::kRegular));
     }
   }
 }
 
 ListValue* ExtensionContentSettingsStore::GetSettingsForExtension(
-    const std::string& extension_id) const {
+    const std::string& extension_id,
+    extension_prefs_scope::Scope scope) const {
   base::AutoLock lock(lock_);
   ListValue* settings = new ListValue();
   const ContentSettingSpecList* setting_spec_list =
-      GetContentSettingSpecList(extension_id, false);
+      GetContentSettingSpecList(extension_id, scope);
   ContentSettingSpecList::const_iterator it;
   for (it = setting_spec_list->begin(); it != setting_spec_list->end(); ++it) {
     DictionaryValue* setting_dict = new DictionaryValue();
     setting_dict->SetString(keys::kEmbeddedPatternKey,
                             it->embedded_pattern.ToString());
     setting_dict->SetString(keys::kTopLevelPatternKey,
                             it->top_level_pattern.ToString());
     setting_dict->SetString(
         keys::kContentSettingsTypeKey,
         helpers::ContentSettingsTypeToString(it->content_type));
     setting_dict->SetString(keys::kResourceIdentifierKey,
                             it->resource_identifier);
     setting_dict->SetString(keys::kContentSettingKey,
                             helpers::ContentSettingToString(it->setting));
     settings->Append(setting_dict);
   }
   return settings;
 }
 
 void ExtensionContentSettingsStore::SetExtensionContentSettingsFromList(
     const std::string& extension_id,
-    const ListValue* list) {
+    const ListValue* list,
+    extension_prefs_scope::Scope scope) {
   for (ListValue::const_iterator it = list->begin(); it != list->end(); ++it) {
     if ((*it)->GetType() != Value::TYPE_DICTIONARY) {
       NOTREACHED();
       continue;
     }
     DictionaryValue* dict = static_cast<DictionaryValue*>(*it);
     std::string pattern_str;
     dict->GetString(keys::kTopLevelPatternKey, &pattern_str);
     ContentSettingsPattern pattern =
         ContentSettingsPattern::FromString(pattern_str);
@@ skipping 20 matching lines @@
     bool result =
         helpers::StringToContentSetting(content_setting_string, &setting);
     DCHECK(result);
 
     SetExtensionContentSetting(extension_id,
                                pattern,
                                embedded_pattern,
                                content_settings_type,
                                resource_identifier,
                                setting,
-                               false);
+                               scope);
   }
 }
 
 void ExtensionContentSettingsStore::AddObserver(Observer* observer) {
   DCHECK(OnCorrectThread());
   observers_.AddObserver(observer);
 }
 
 void ExtensionContentSettingsStore::RemoveObserver(Observer* observer) {
   DCHECK(OnCorrectThread());
@@ skipping 25 matching lines @@
       ExtensionContentSettingsStore::Observer,
       observers_,
       OnContentSettingChanged(extension_id, incognito));
 }
 
 bool ExtensionContentSettingsStore::OnCorrectThread() {
   // If there is no UI thread, we're most likely in a unit test.
   return !BrowserThread::IsWellKnownThread(BrowserThread::UI) ||
          BrowserThread::CurrentlyOn(BrowserThread::UI);
 }