Get ride of chrome.storage.* data when removing an application.

chrome/browser/extensions/data_deleter.h

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_DATA_DELETER_H_
 #define CHROME_BROWSER_EXTENSIONS_DATA_DELETER_H_
 
-#include "url/gurl.h"
+#include <string>
+#include "base/basictypes.h"
 
 class Profile;
+class GURL;
+
+namespace content {
+class StoragePartition;
+}
 
 namespace extensions {
 
+class Extension;
+
 class DataDeleter {
  public:
   // Starts removing data. The extension should not be running when this is
   // called. Cookies are deleted on the current thread, local storage and
   // databases/settings are deleted asynchronously on the webkit and file
   // threads, respectively. This function must be called from the UI thread.
-  static void StartDeleting(
-      Profile* profile,
-      const std::string& extension_id,
-      const GURL& storage_origin);
+  static void StartDeleting(Profile* profile, const Extension* extenion);
+
+ private:
+  // Helper function that deletes data of a given |storage_origin| in a given
+  // |partition|.
+  static void DeleteOrigin(

[not at google - send to devlin, 2014/01/08 23:59:53]

just realised that this doesn't need to be on the interface, it can be in an
anonymous namespace within the .cc file.

[mlamouri (slow - plz ping), 2014/01/09 00:09:29]

Sure. But it makes me wonder why we keep this as a class instead of having a
namespace.

+    Profile* profile,
+    content::StoragePartition* partition,
+    const GURL& origin);
 
   DISALLOW_COPY_AND_ASSIGN(DataDeleter);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_DATA_DELETER_H_