RefCounted types should not have public destructors, chrome/browser/extensions

chrome/browser/extensions/api/offscreen_tabs/offscreen_tabs_api.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_OFFSCREEN_TABS_OFFSCREEN_TABS_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_OFFSCREEN_TABS_OFFSCREEN_TABS_API_H_
 #pragma once
 
 #include <string>
 
@@ skipping 18 matching lines @@
 class BackingStore;
 class SkBitmap;
 class TabContentsWrapper;
 namespace content {
 class WebContents;
 }  // namespace content
 
 // Creates an offscreen tab.
 class CreateOffscreenTabFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.create")
+
   CreateOffscreenTabFunction();
+
+ protected:
+  virtual ~CreateOffscreenTabFunction();
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
+
  private:
-  virtual ~CreateOffscreenTabFunction();
-  virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.create")
   DISALLOW_COPY_AND_ASSIGN(CreateOffscreenTabFunction);
 };
 
 // Gets info about an offscreen tab.
 class GetOffscreenTabFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.get")
+
   GetOffscreenTabFunction();
+
+ protected:
+  virtual ~GetOffscreenTabFunction();
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
+
  private:
-  virtual ~GetOffscreenTabFunction();
-  virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.get")
   DISALLOW_COPY_AND_ASSIGN(GetOffscreenTabFunction);
 };
 
 // Gets all offscreen tabs created by the tab that invoked this function.
 class GetAllOffscreenTabFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.getAll")
+
   GetAllOffscreenTabFunction();
+
+ protected:
+  virtual ~GetAllOffscreenTabFunction();
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
+
  private:
-  virtual ~GetAllOffscreenTabFunction();
-  virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.getAll")
   DISALLOW_COPY_AND_ASSIGN(GetAllOffscreenTabFunction);
 };
 
 // Removes an offscreen tab.
 class RemoveOffscreenTabFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.remove")
+
   RemoveOffscreenTabFunction();
+
+ protected:
+  virtual ~RemoveOffscreenTabFunction();
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
+
  private:
-  virtual ~RemoveOffscreenTabFunction();
-  virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.remove")
   DISALLOW_COPY_AND_ASSIGN(RemoveOffscreenTabFunction);
 };
 
 // Synthesizes a keyboard event based on a passed-in JavaScript keyboard event.
 // TODO(jstritar): This would be useful on the chrome.tabs API.
 class SendKeyboardEventOffscreenTabFunction : public SyncExtensionFunction {
  public:
-  SendKeyboardEventOffscreenTabFunction();
- private:
-  virtual ~SendKeyboardEventOffscreenTabFunction();
-  virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION_NAME(
     "experimental.offscreenTabs.sendKeyboardEvent")
+
+  SendKeyboardEventOffscreenTabFunction();
+
+ protected:
+  virtual ~SendKeyboardEventOffscreenTabFunction();
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
+
+ private:
   DISALLOW_COPY_AND_ASSIGN(SendKeyboardEventOffscreenTabFunction);
 };
 
 // Synthesizes a mouse event based on a passed-in JavaScript mouse event.
 // Since only the application knows where the user clicks, x and y coordinates
 // need to be passed in as well in this case of click, mousedown, and mouseup.
 // TODO(jstritar): This would be useful on the chrome.tabs API.
 class SendMouseEventOffscreenTabFunction : public SyncExtensionFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.sendMouseEvent")
+
   SendMouseEventOffscreenTabFunction();
+
+ protected:
+  virtual ~SendMouseEventOffscreenTabFunction();
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
+
  private:
-  virtual ~SendMouseEventOffscreenTabFunction();
-  virtual bool RunImpl() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.sendMouseEvent")
   DISALLOW_COPY_AND_ASSIGN(SendMouseEventOffscreenTabFunction);
 };
 
 // Gets a snapshot of the offscreen tab and returns it as a data URL.
 class ToDataUrlOffscreenTabFunction : public CaptureVisibleTabFunction {
  public:
+  // TODO(jstritar): Rename to toDataUrl.
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.toDataUrl")
+
   ToDataUrlOffscreenTabFunction();
- private:
+
+ protected:
   virtual ~ToDataUrlOffscreenTabFunction();
+
+  // CaptureVisibleTabFunction:
   virtual bool GetTabToCapture(content::WebContents** web_contents,
                                TabContentsWrapper** wrapper) OVERRIDE;
-  // TODO(jstritar): Rename to toDataUrl.
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.toDataUrl")
+
+ private:
   DISALLOW_COPY_AND_ASSIGN(ToDataUrlOffscreenTabFunction);
 };
 
 // Updates an offscreen tab.
 class UpdateOffscreenTabFunction : public UpdateTabFunction {
  public:
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.update")
+
   UpdateOffscreenTabFunction();
+
+ protected:
+  virtual ~UpdateOffscreenTabFunction();
+
+  // ExtensionFunction:
+  virtual bool RunImpl() OVERRIDE;
+
+  // UpdateTabFunction:
+  virtual void PopulateResult() OVERRIDE;
+
  private:
-  virtual ~UpdateOffscreenTabFunction();
-  virtual bool RunImpl() OVERRIDE;
-  virtual void PopulateResult() OVERRIDE;
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.offscreenTabs.update")
   DISALLOW_COPY_AND_ASSIGN(UpdateOffscreenTabFunction);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_OFFSCREEN_TABS_OFFSCREEN_TABS_API_H_