Strawman proposal for chrome.experimental.clear.*

chrome/browser/extensions/extension_clear_api.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.
+
+// Defines the Chrome Extensions Clear API functions, which entail
+// clearing browsing data, and clearing the browser's cache (which, let's be
+// honest, are the same thing), as specified in
+// chrome/common/extensions/api/extension_api.json.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_CLEAR_API_H_
+#define CHROME_BROWSER_EXTENSIONS_EXTENSION_CLEAR_API_H_
+#pragma once
+
+#include <string>
+
+#include "chrome/browser/browsing_data_remover.h"
+#include "chrome/browser/extensions/extension_function.h"
+
+namespace base {
+  class DictionaryValue;

[Bernhard Bauer, 2011/08/16 09:49:36]

Nit: no indent.

[Mike West, 2011/08/16 11:05:56]

Done.

+}
+
+// This serves as a base class from which the browsing data API functions will
+// inherit. Each needs to be an observer of BrowsingDataRemover events, and each
+// will handle those events in the same way (by calling the passed-in callback
+// function).
+//
+// Each child class must implement removal_mask(), which returns the bitmask of
+// data types to remove.
+class BrowsingDataExtensionFunction : public AsyncExtensionFunction,
+                                      public BrowsingDataRemover::Observer {
+ public:
+  // BrowsingDataRemover::Observer interface method.
+  virtual void OnBrowsingDataRemoverDone() OVERRIDE;
+
+  // AsyncExtensionFunction interface method.
+  virtual bool RunImpl() OVERRIDE;
+
+  // Children should override this method to provide the proper removal mask
+  // based on the API call they represent.
+  virtual int removal_mask() const = 0;
+
+ protected:
+  // Converts the JavaScript API's string input ("last_week") into the
+  // appropriate BrowsingDataRemover::TimePeriod (in this case,
+  // BrowsingDataRemover::LAST_WEEK).
+  virtual BrowsingDataRemover::TimePeriod ParseTimePeriod(
+      const std::string& period) const;
+
+  // Convert the JavaScript API's object input ({ cookies: true }) into the
+  // appropriate removal mask for the BrowsingDataRemover object.
+  virtual int ParseRemovalMask(base::DictionaryValue* value) const;
+};
+
+class ClearBrowsingDataFunction : public BrowsingDataExtensionFunction {
+ public:
+  ClearBrowsingDataFunction() {}
+  virtual ~ClearBrowsingDataFunction() {}
+
+  // BrowsingDataExtensionFunction interface method.
+  virtual int removal_mask() const OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION_NAME(

[Bernhard Bauer, 2011/08/16 09:49:36]

Nit: does this fit on one line?

[Mike West, 2011/08/16 11:05:56]

Done.

+      "experimental.clear.browsingData")
+};
+
+class ClearCacheFunction : public BrowsingDataExtensionFunction {
+ public:
+  ClearCacheFunction() {}
+  virtual ~ClearCacheFunction() {}
+
+  // BrowsingDataTypeExtensionFunction interface method.
+  virtual int removal_mask() const OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.clear.cache")
+};
+
+class ClearCookiesFunction : public BrowsingDataExtensionFunction {
+ public:
+  ClearCookiesFunction() {}
+  virtual ~ClearCookiesFunction() {}
+
+  // BrowsingDataTypeExtensionFunction interface method.
+  virtual int removal_mask() const OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.clear.cookies")
+};
+
+class ClearDownloadsFunction : public BrowsingDataExtensionFunction {
+ public:
+  ClearDownloadsFunction() {}
+  virtual ~ClearDownloadsFunction() {}
+
+  // BrowsingDataTypeExtensionFunction interface method.
+  virtual int removal_mask() const OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.clear.downloads")
+};
+
+class ClearFormDataFunction : public BrowsingDataExtensionFunction {
+ public:
+  ClearFormDataFunction() {}
+  virtual ~ClearFormDataFunction() {}
+
+  // BrowsingDataTypeExtensionFunction interface method.
+  virtual int removal_mask() const OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.clear.formData")
+};
+
+class ClearHistoryFunction : public BrowsingDataExtensionFunction {
+ public:
+  ClearHistoryFunction() {}
+  virtual ~ClearHistoryFunction() {}
+
+  // BrowsingDataTypeExtensionFunction interface method.
+  virtual int removal_mask() const OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.clear.history")
+};
+
+class ClearPasswordsFunction : public BrowsingDataExtensionFunction {
+ public:
+  ClearPasswordsFunction() {}
+  virtual ~ClearPasswordsFunction() {}
+
+  // BrowsingDataTypeExtensionFunction interface method.
+  virtual int removal_mask() const OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.clear.passwords")
+};
+
+#endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_CLEAR_API_H_