Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(408)

Unified Diff: chrome/browser/ui/zoom/zoom_controller.h

Issue 301733006: Zoom Extension API (chrome) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments, make zoom bubble icon loading safer. Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/zoom/zoom_controller.h
diff --git a/chrome/browser/ui/zoom/zoom_controller.h b/chrome/browser/ui/zoom/zoom_controller.h
index 3c427a716d6956b4603ee54f1d1fe67dd4780985..fbf48edfe35ae80a10d274e0eebaaf44e4d30e8b 100644
--- a/chrome/browser/ui/zoom/zoom_controller.h
+++ b/chrome/browser/ui/zoom/zoom_controller.h
@@ -7,6 +7,7 @@
#include "base/basictypes.h"
#include "base/compiler_specific.h"
+#include "base/observer_list.h"
#include "base/prefs/pref_member.h"
#include "content/public/browser/host_zoom_map.h"
#include "content/public/browser/web_contents_observer.h"
@@ -18,13 +19,38 @@ namespace content {
class WebContents;
}
-// Per-tab class to manage the Omnibox zoom icon.
+namespace extensions {
+class Extension;
+} // namespace extensions
+
+// Per-tab class to manage zoom changes and the Omnibox zoom icon.
class ZoomController : public content::WebContentsObserver,
public content::WebContentsUserData<ZoomController> {
public:
+ // Defines how zoom changes are handled.
+ // |kZoomModeDefault| results in default zoom behavior, i.e. zoom changes are
+ // handled automatically and on a per-origin basis, meaning that other
+ // tabs navigated to the same origin will also zoom.
+ // |kZoomModeIsolated| results in zoom changes being handled automatically,
+ // but on a per-tab basis. Tabs in this zoom mode will not be affected by
+ // zoom changes in other tabs, and vice versa.
+ // |kZoomModeManual| overrides the automatic handling of zoom changes. The
+ // |onZoomChange| event will still be dispatched, but the page will not
+ // actually be zoomed. These zoom changes can be handled manually by
+ // listening for the |onZoomChange| event. Zooming in this mode is also on
+ // a per-tab basis.
+ // |kZoomModeDisabled| disables all zooming in this tab. The tab will revert
+ // to default (100%) zoom, and all attempted zoom changes will be ignored.
+ enum ZoomMode {
+ kZoomModeDefault,
sky 2014/06/24 16:23:42 ZOOM_MODE_DEFAULT is the right name and style for
wjmaclean 2014/06/24 18:04:07 Done.
+ kZoomModeIsolated,
+ kZoomModeManual,
+ kZoomModeDisabled,
+ };
+
virtual ~ZoomController();
- int zoom_percent() const { return zoom_percent_; }
+ ZoomMode zoom_mode() const { return zoom_mode_; }
// Convenience method to quickly check if the tab's at default zoom.
bool IsAtDefaultZoom() const;
@@ -32,7 +58,29 @@ class ZoomController : public content::WebContentsObserver,
// Returns which image should be loaded for the current zoom level.
int GetResourceForZoomLevel() const;
- void set_observer(ZoomObserver* observer) { observer_ = observer; }
+ const extensions::Extension* last_extension() const {
+ return last_extension_.get();
+ }
+
+ void AddObserver(ZoomObserver* observer);
+ void RemoveObserver(ZoomObserver* observer);
+
+ double GetZoomLevel() const;
+ int GetZoomPercent() const;
+
+ // Sets the zoom level through HostZoomMap.
+ // Returns true on success.
+ bool SetZoomLevel(double zoom_level);
+
+ // Sets the zoom level through WebContents::SetZoomLevel(), and attributes the
+ // zoom to |extension|. Returns a boolean flag indicating success (true) or
+ // failure (false).
+ bool SetZoomLevelByExtension(
+ double zoom_level,
+ scoped_refptr<const extensions::Extension> extension);
+
+ // Sets the zoom mode, which defines zoom behavior (see enum ZoomMode).
+ void SetZoomMode(ZoomMode zoom_mode);
// content::WebContentsObserver overrides:
virtual void DidNavigateMainFrame(
@@ -51,15 +99,27 @@ class ZoomController : public content::WebContentsObserver,
// meaning the change should apply to ~all sites. If it is not empty, the
// change only affects sites with the given host.
void UpdateState(const std::string& host);
+ // Same as UpdateState, but takes into account whether a temporary zoom level
+ // has been set on |host| when deciding whether to show the zoom notification
+ // bubble.
+ void UpdateStateIncludingTemporary(const std::string& host,
+ bool is_temporary_zoom);
- // The current zoom percentage.
- int zoom_percent_;
+ // The current zoom mode.
+ ZoomMode zoom_mode_;
+
+ // Current zoom level.
+ double zoom_level_;
// Used to access the default zoom level preference.
DoublePrefMember default_zoom_level_;
+ // Keeps track of the extension (if any) that initiated the last zoom change
+ // that took effect.
+ scoped_refptr<const extensions::Extension> last_extension_;
+
// Observer receiving notifications on state changes.
- ZoomObserver* observer_;
+ ObserverList<ZoomObserver> observers_;
content::BrowserContext* browser_context_;

Powered by Google App Engine
This is Rietveld 408576698