Implementing TabManager extensions API Discard Function.

chrome/browser/extensions/api/tabs/tabs_api.h

Index: chrome/browser/extensions/api/tabs/tabs_api.h
diff --git a/chrome/browser/extensions/api/tabs/tabs_api.h b/chrome/browser/extensions/api/tabs/tabs_api.h
index 7457aad43d8c735c3904923a207cb3b18ffbee07..07b8df8fe1c6ae61ad98462c363793ddca0ebc0d 100644
--- a/chrome/browser/extensions/api/tabs/tabs_api.h
+++ b/chrome/browser/extensions/api/tabs/tabs_api.h
@@ -136,7 +136,7 @@ class TabsHighlightFunction : public ChromeSyncExtensionFunction {
   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)
 };
@@ -322,6 +322,14 @@ class TabsGetZoomSettingsFunction : public ZoomAPIFunction {
   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_