Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(224)

Unified Diff: chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc

Issue 2370001: Store blocked cookies in the tab contents. (Closed)
Patch Set: updates Created 10 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc
diff --git a/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc b/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc
index 63d6d82577394157815e2d36d450c2e5ab7c8406..dd01f79f4bd5ef097241b8e6d24cb7a3c3221f66 100644
--- a/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc
+++ b/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc
@@ -264,11 +264,11 @@ void DOMStorageDispatcherHost::OnSetItem(
// If content was blocked, tell the UI to display the blocked content icon.
if (reply_msg->routing_id() == MSG_ROUTING_CONTROL) {
DLOG(WARNING) << "setItem was not given a proper routing id";
- } else if (result == WebKit::WebStorageArea::ResultBlockedByPolicy) {
+ } else {
CallRenderViewHostContentSettingsDelegate(
process_id_, reply_msg->routing_id(),
- &RenderViewHostDelegate::ContentSettings::OnContentBlocked,
- CONTENT_SETTINGS_TYPE_COOKIES);
+ &RenderViewHostDelegate::ContentSettings::OnLocalStorageAccessed,
+ url, result == WebStorageArea::ResultBlockedByPolicy);
}
ViewHostMsg_DOMStorageSetItem::WriteReplyParams(reply_msg, result, old_value);

Powered by Google App Engine
This is Rietveld 408576698