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

Unified Diff: chrome/browser/in_process_webkit/dom_storage_area.h

Issue 5722003: Make DOMStorageDispatcherHost be a message filter (and rename it accordingly)... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: get rid of _DELAY_HANDLE macro Created 10 years 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_area.h
===================================================================
--- chrome/browser/in_process_webkit/dom_storage_area.h (revision 69012)
+++ chrome/browser/in_process_webkit/dom_storage_area.h (working copy)
@@ -15,7 +15,6 @@
#include "googleurl/src/gurl.h"
#include "third_party/WebKit/WebKit/chromium/public/WebStorageArea.h"
-class DOMStorageDispatcherHost;
class DOMStorageNamespace;
class HostContentSettingsMap;
@@ -32,14 +31,9 @@
unsigned Length();
NullableString16 Key(unsigned index);
NullableString16 GetItem(const string16& key);
- // The DOMStorageDispatcherHost parameter is required in case we get a
- // CONTENT_SETTING_ASK response from the HostContentettingsMap. If we do,
- // we'll need to let the renderer know that it'll need to run a nested message
- // loop.
NullableString16 SetItem(
const string16& key, const string16& value,
- WebKit::WebStorageArea::Result* result,
- DOMStorageDispatcherHost* sender);
+ WebKit::WebStorageArea::Result* result);
NullableString16 RemoveItem(const string16& key);
bool Clear();
void PurgeMemory();
@@ -53,8 +47,7 @@
void CreateWebStorageAreaIfNecessary();
// Used to see if setItem has permission to do its thing.
- bool CheckContentSetting(const string16& key, const string16& value,
- DOMStorageDispatcherHost* sender);
+ bool CheckContentSetting(const string16& key, const string16& value);
// The origin this storage area represents.
string16 origin_;

Powered by Google App Engine
This is Rietveld 408576698