Implementing TabManager extensions API Discard Function.

chrome/browser/extensions/api/tabs/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_TABS_TABS_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_API_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 118 matching lines @@
 class TabsDuplicateFunction : public ChromeSyncExtensionFunction {
   ~TabsDuplicateFunction() override {}
   bool RunSync() override;
   DECLARE_EXTENSION_FUNCTION("tabs.duplicate", TABS_DUPLICATE)
 };
 class TabsHighlightFunction : public ChromeSyncExtensionFunction {
   ~TabsHighlightFunction() override {}
   bool RunSync() override;
   bool HighlightTab(TabStripModel* tabstrip,
                     ui::ListSelectionModel* selection,
-                    int *active_index,
+                    int* active_index,
                     int index);
   DECLARE_EXTENSION_FUNCTION("tabs.highlight", TABS_HIGHLIGHT)
 };
 class TabsUpdateFunction : public ChromeAsyncExtensionFunction {
  public:
   TabsUpdateFunction();
 
  protected:
   ~TabsUpdateFunction() override {}
   virtual bool UpdateURL(const std::string& url,
@@ skipping 165 matching lines @@
 
 class TabsGetZoomSettingsFunction : public ZoomAPIFunction {
  private:
   ~TabsGetZoomSettingsFunction() override {}
 
   bool RunAsync() override;
 
   DECLARE_EXTENSION_FUNCTION("tabs.getZoomSettings", TABS_GETZOOMSETTINGS)
 };
 
+class TabsDiscardFunction : public ChromeSyncExtensionFunction {
+  ~TabsDiscardFunction() override {}

[Georges Khalil, 2016/07/19 19:39:45]

missing private.

[Anderson Silva, 2016/07/19 20:28:42]

Hmm. I think private is not necessary. Most of the functions in the API doesn't
have it. Why do you think so?
The functions above (Zoom API) might be private because it inherits from another
class than ChromeSyncExtensionFunction.

[Georges Khalil, 2016/07/19 20:35:40]

Yeah, it seems to be inconsistent through out the file. I'll defer to the owner
to make the call.

+
+  bool RunSync() override;
+
+  DECLARE_EXTENSION_FUNCTION("tabs.discard", TABS_DISCARD)
+};
+
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_API_H_