Fixed the problem that cookie blocking notification bubble is not displayed in the omnibox.

chrome/browser/content_settings/tab_specific_content_settings.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/content_settings/tab_specific_content_settings.h"
 
 #include <list>
 
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
@@ skipping 246 matching lines @@
 }
 
 void TabSpecificContentSettings::OnContentBlocked(
     ContentSettingsType type,
     const std::string& resource_identifier) {
   DCHECK(type != CONTENT_SETTINGS_TYPE_GEOLOCATION)
       << "Geolocation settings handled by OnGeolocationPermissionSet";
   if (type < 0 || type >= CONTENT_SETTINGS_NUM_TYPES)
     return;
 
-  content_allowed_[type] = false;
+  // Media is different from other content setting types since it allows new
+  // setting to kick in without reloading the page, and the UI for media is
+  // always reflecting the newest permission setting.
+  if (type == CONTENT_SETTINGS_TYPE_MEDIASTREAM)
+    content_allowed_[type] = false;
+  else
+    content_allowed_[type] = true;
 
   // Unless UI for resource content settings is enabled, ignore the resource
   // identifier.
   // TODO(bauerb): The UI to unblock content should be disabled if the content
   // setting was not set by the user.
   std::string identifier;
   if (CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kEnableResourceContentSettings)) {
     identifier = resource_identifier;
   }
@@ skipping 18 matching lines @@
         chrome::NOTIFICATION_WEB_CONTENT_SETTINGS_CHANGED,
         content::Source<WebContents>(web_contents()),
         content::NotificationService::NoDetails());
   }
 }
 
 void TabSpecificContentSettings::OnContentAllowed(ContentSettingsType type) {
   DCHECK(type != CONTENT_SETTINGS_TYPE_GEOLOCATION)
       << "Geolocation settings handled by OnGeolocationPermissionSet";
   bool access_changed = false;
-  if (content_blocked_[type]) {
-    content_blocked_[type] = false;
-    access_changed = true;
-  }
+  if (type == CONTENT_SETTINGS_TYPE_MEDIASTREAM) {
+    // The setting for media is overwritten here because media does not need to
+    // reload the page to have the new setting kick in. See issue/175993.
+    if (content_blocked_[type]) {
+      content_blocked_[type] = false;
+      access_changed = true;
+    }

[jochen (gone - plz use gerrit), 2013/04/02 13:08:53]

nit, you can just close the if block here, no need for the else block then

[no longer working on chromium, 2013/04/02 14:12:06]

Done.

 
-  if (!content_allowed_[type]) {
-    content_allowed_[type] = true;
-    access_changed = true;
+    if (!content_allowed_[type]) {
+      content_allowed_[type] = true;
+      access_changed = true;
+    }
+  } else {
+    if (!content_allowed_[type]) {
+      content_allowed_[type] = true;
+      access_changed = true;
+    }
   }
 
   if (access_changed) {
     content::NotificationService::current()->Notify(
         chrome::NOTIFICATION_WEB_CONTENT_SETTINGS_CHANGED,
         content::Source<WebContents>(web_contents()),
         content::NotificationService::NoDetails());
   }
 }
 
@@ skipping 262 matching lines @@
 }
 
 void TabSpecificContentSettings::RemoveSiteDataObserver(
     SiteDataObserver* observer) {
   observer_list_.RemoveObserver(observer);
 }
 
 void TabSpecificContentSettings::NotifySiteDataObservers() {
   FOR_EACH_OBSERVER(SiteDataObserver, observer_list_, OnSiteDataAccessed());
 }