Zoom Extension API (chrome)

chrome/browser/guest_view/web_view/web_view_guest.cc

Index: chrome/browser/guest_view/web_view/web_view_guest.cc
diff --git a/chrome/browser/guest_view/web_view/web_view_guest.cc b/chrome/browser/guest_view/web_view/web_view_guest.cc
index 7b8e4f4995b962de8645191ad74249f3fc57163b..39e3aacf49808cd2237b3bb9692dc3ef8285c9ea 100644
--- a/chrome/browser/guest_view/web_view/web_view_guest.cc
+++ b/chrome/browser/guest_view/web_view/web_view_guest.cc
@@ -21,6 +21,7 @@
 #include "chrome/browser/guest_view/web_view/web_view_permission_types.h"
 #include "chrome/browser/renderer_context_menu/context_menu_delegate.h"
 #include "chrome/browser/renderer_context_menu/render_view_context_menu.h"
+#include "chrome/browser/ui/zoom/zoom_controller.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/render_messages.h"
 #include "content/public/browser/browser_thread.h"
@@ -878,7 +879,7 @@ void WebViewGuest::DidCommitProvisionalLoadForFrame(
 
   // Update the current zoom factor for the new page.
   current_zoom_factor_ = content::ZoomLevelToZoomFactor(
-      content::HostZoomMap::GetZoomLevel(guest_web_contents()));
+      ZoomController::FromWebContents(guest_web_contents())->GetZoomLevel());
 
   if (is_main_frame) {
     chromevox_injected_ = false;
@@ -1325,8 +1326,11 @@ void WebViewGuest::SetName(const std::string& name) {
 }
 
 void WebViewGuest::SetZoom(double zoom_factor) {
+  ZoomController* zoom_controller =
+      ZoomController::FromWebContents(guest_web_contents());
   double zoom_level = content::ZoomFactorToZoomLevel(zoom_factor);
-  content::HostZoomMap::SetZoomLevel(guest_web_contents(), zoom_level);
+  //  TODO(wjmaclean) Should we talk to HostZoomMap directly instead?

[Fady Samuel, 2014/06/12 19:38:58]

I don't think so. WebViews should always be in isolated mode btw. It should not
be the case that zooming a tab is independent of zooming a webview and vice
versa.

[wjmaclean, 2014/06/12 19:42:22]

Sorry about this ... this comment died (in my mind ;-) ) a long time ago, and
should have been removed. Agreed, should not be talking directly to HostZoomMap
in this case.

+  zoom_controller->SetZoomLevel(zoom_level);
 
   scoped_ptr<base::DictionaryValue> args(new base::DictionaryValue());
   args->SetDouble(webview::kOldZoomFactor, current_zoom_factor_);