Implement WebStorageArea::containsItem(key) in chrome and drt/test_shell.

webkit/tools/test_shell/simple_dom_storage_system.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 "webkit/tools/test_shell/simple_dom_storage_system.h"
 
 #include "base/auto_reset.h"
 #include "googleurl/src/gurl.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebURL.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebStorageArea.h"
@@ skipping 36 matching lines @@
 
   base::WeakPtr<SimpleDomStorageSystem> parent_;
   int namespace_id_;
 };
 
 class SimpleDomStorageSystem::AreaImpl : public WebStorageArea {
  public:
   AreaImpl(const base::WeakPtr<SimpleDomStorageSystem>& parent,
           int namespace_id, const GURL& origin);
   virtual ~AreaImpl();
-  virtual unsigned length() OVERRIDE;

[jochen (gone - plz use gerrit), 2012/06/25 15:21:26]

just wondering, why did you remove the OVERRIDEs?

-  virtual WebString key(unsigned index) OVERRIDE;
-  virtual WebString getItem(const WebString& key) OVERRIDE;
+  virtual unsigned length();
+  virtual WebString key(unsigned index);
+  virtual WebString getItem(const WebString& key);
+  virtual bool containsItem(const WebString& key);
   virtual void setItem(const WebString& key, const WebString& newValue,
-                       const WebURL& pageUrl, Result&) OVERRIDE;
+                       const WebURL& pageUrl, Result&);
   virtual void removeItem(const WebString& key,
-                          const WebURL& pageUrl) OVERRIDE;
-  virtual void clear(const WebURL& pageUrl) OVERRIDE;
+                          const WebURL& pageUrl);
+  virtual void clear(const WebURL& pageUrl);
 
  private:
   DomStorageHost* Host() {
     if (!parent_.get())
       return NULL;
     return parent_->host_.get();
   }
 
   base::WeakPtr<SimpleDomStorageSystem> parent_;
   int connection_id_;
@@ skipping 72 matching lines @@
     return Host()->GetAreaKey(connection_id_, index);
   return NullableString16(true);
 }
 
 WebString SimpleDomStorageSystem::AreaImpl::getItem(const WebString& key) {
   if (Host())
     return Host()->GetAreaItem(connection_id_, key);
   return NullableString16(true);
 }
 
+bool SimpleDomStorageSystem::AreaImpl::containsItem(const WebString& key) {
+  return !getItem(key).isNull();
+}
+
 void SimpleDomStorageSystem::AreaImpl::setItem(
     const WebString& key, const WebString& newValue,
     const WebURL& pageUrl, Result& result) {
   result = ResultBlockedByQuota;
   if (!Host())
     return;
 
   AutoReset<AreaImpl*> auto_reset(&parent_->area_being_processed_, this);
   NullableString16 unused;
   if (!Host()->SetAreaItem(connection_id_, key, newValue, pageUrl,
@@ skipping 107 matching lines @@
         key,
         old_value,
         new_value,
         area->origin(),
         page_url,
         session_namespace_for_event_dispatch,
         area_being_processed_,
         true  /* originatedInProcess */);
   }
 }