diff options
author | jorlow@chromium.org <jorlow@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-08 19:37:07 +0000 |
---|---|---|
committer | jorlow@chromium.org <jorlow@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-08 19:37:07 +0000 |
commit | d028f1ae334a261dd42ba9783b32ce6396910934 (patch) | |
tree | 0ed4b52e59939c21cedf69f7e2d6d4fc7b2f5470 /chrome/browser/in_process_webkit | |
parent | 13c340e130d5fa54b3abe5f3001bda94d4569009 (diff) | |
download | chromium_src-d028f1ae334a261dd42ba9783b32ce6396910934.zip chromium_src-d028f1ae334a261dd42ba9783b32ce6396910934.tar.gz chromium_src-d028f1ae334a261dd42ba9783b32ce6396910934.tar.bz2 |
Revert 40920 - Use the WebViewClient* passed in from the StorageAreaProxy instead of using
RenderThread::RoutingIDForCurrentContext() whenever possible. In addition,
call OnContentBlocked directly from within the browser instead of sending
another IPC message. Lastly, make sure that we don't crash in the event
that we can't determine the routing id.
WebKit side: https://bugs.webkit.org/show_bug.cgi?id=35758
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/669084
TBR=jorlow@chromium.org
Review URL: http://codereview.chromium.org/669293
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40922 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/in_process_webkit')
-rw-r--r-- | chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc | 18 |
1 files changed, 3 insertions, 15 deletions
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 299787a..ddd9439 100644 --- a/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc +++ b/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc @@ -1,6 +1,6 @@ -// Copyright (c) 2010 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. +// Copyright (c) 2010 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/in_process_webkit/dom_storage_dispatcher_host.h" @@ -12,7 +12,6 @@ #include "chrome/browser/in_process_webkit/webkit_thread.h" #include "chrome/browser/net/chrome_url_request_context.h" #include "chrome/browser/renderer_host/browser_render_process_host.h" -#include "chrome/browser/renderer_host/render_view_host_notification_task.h" #include "chrome/browser/renderer_host/resource_message_filter.h" #include "chrome/common/render_messages.h" #include "googleurl/src/gurl.h" @@ -272,17 +271,6 @@ void DOMStorageDispatcherHost::OnSetItem( ScopedStorageEventContext scope(this, &url); WebStorageArea::Result result; NullableString16 old_value = storage_area->SetItem(key, value, &result, this); - - // 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) { - CallRenderViewHostResourceDelegate( - process_handle_, reply_msg->routing_id(), - &RenderViewHostDelegate::Resource::OnContentBlocked, - CONTENT_SETTINGS_TYPE_COOKIES); - } - ViewHostMsg_DOMStorageSetItem::WriteReplyParams(reply_msg, result, old_value); Send(reply_msg); } |