Zoom Extension API

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 24 matching lines @@
 namespace ui {
 class ListSelectionModel;
 }
 
 namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
 namespace extensions {
 
+// Converts a ZoomMode to its zoom settings dictionary object representation.
+void ZoomModeToZoomSettings(content::ZoomMode zoom_mode,
+                            base::DictionaryValue* zoom_settings);

[not at google - send to devlin, 2014/04/09 03:52:11]

doesn't seem like this needs to be in the header file, could be in an anon
namespace in the .cc file.

however, you probably don't need it at all.

[paulmeyer, 2014/04/11 03:01:02]

Talked about this over IM. I fixed up the function to use the schema-generated
data structure.

On 2014/04/09 03:52:11, kalman wrote:

+
 // Windows
 class WindowsGetFunction : public ChromeSyncExtensionFunction {
   virtual ~WindowsGetFunction() {}
   virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION("windows.get", WINDOWS_GET)
 };
 class WindowsGetCurrentFunction : public ChromeSyncExtensionFunction {
   virtual ~WindowsGetCurrentFunction() {}
   virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION("windows.getCurrent", WINDOWS_GETCURRENT)
@@ skipping 194 matching lines @@
 
 class TabsInsertCSSFunction : public ExecuteCodeInTabFunction {
  private:
   virtual ~TabsInsertCSSFunction() {}
 
   virtual bool ShouldInsertCSS() const OVERRIDE;
 
   DECLARE_EXTENSION_FUNCTION("tabs.insertCSS", TABS_INSERTCSS)
 };
 
+class ZoomAPIFunction : public ChromeAsyncExtensionFunction {
+ protected:
+  virtual ~ZoomAPIFunction() {}
+
+  // Gets the WebContents for |tab_id| if it is specified. Otherwise get the
+  // WebContents for the active tab in the current window.
+  bool GetWebContents(int* tab_id, content::WebContents** web_contents);

[not at google - send to devlin, 2014/04/09 03:52:11]

hopefully this functionality is already provided, so you won't need this
ZoomAPIFunction interface.

if it *isn't* already provided - if every tab function implements it itself -
that sounds like a nice cleanup.

[paulmeyer, 2014/04/11 03:01:02]

This functionality unfortunately isn't provided already. This might be a good
candidate for cleanup, though perhaps in a different patch.

On 2014/04/09 03:52:11, kalman wrote:

+};
+
+class TabsSetZoomFunction : public ZoomAPIFunction {
+ private:
+  virtual ~TabsSetZoomFunction() {}
+
+  virtual bool RunImpl() OVERRIDE;
+
+  // Called when the zoom change has completed.
+  void Callback();
+
+  DECLARE_EXTENSION_FUNCTION("tabs.setZoom", TABS_SETZOOM)
+};
+
+class TabsGetZoomFunction : public ZoomAPIFunction {
+ private:
+  virtual ~TabsGetZoomFunction() {}
+
+  virtual bool RunImpl() OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION("tabs.getZoom", TABS_GETZOOM)
+};
+
+class TabsSetZoomSettingsFunction : public ZoomAPIFunction {
+ private:
+  virtual ~TabsSetZoomSettingsFunction() {}
+
+  virtual bool RunImpl() OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION("tabs.setZoomSettings", TABS_SETZOOMSETTINGS)
+};
+
+class TabsGetZoomSettingsFunction : public ZoomAPIFunction {
+ private:
+  virtual ~TabsGetZoomSettingsFunction() {}
+
+  virtual bool RunImpl() OVERRIDE;
+
+  DECLARE_EXTENSION_FUNCTION("tabs.getZoomSettings", TABS_GETZOOMSETTINGS)
+};
+
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_API_H_