Zoom Extension API (content changes)

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index a52697b5a8bedea54494f03058da2c728832e588..511a91aeec081c72430bdd61dd42ffea025b04cf 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -671,8 +671,7 @@ RenderViewImpl::RenderViewImpl(RenderViewImplParams* params)
 #endif
       enumeration_completion_id_(0),
       session_storage_namespace_id_(params->session_storage_namespace_id),
-      next_snapshot_id_(0) {
-}
+      next_snapshot_id_(0) {}

[Fady Samuel, 2014/06/09 15:43:21]

Can we undo this change?

[wjmaclean, 2014/06/09 17:23:29]

Done. Leftover debugging artifact I suspect.

 
 void RenderViewImpl::Initialize(RenderViewImplParams* params) {
   routing_id_ = params->routing_id;
@@ -1073,6 +1072,8 @@ bool RenderViewImpl::OnMessageReceived(const IPC::Message& message) {
     IPC_MESSAGE_HANDLER(ViewMsg_Zoom, OnZoom)
     IPC_MESSAGE_HANDLER(ViewMsg_SetZoomLevelForLoadingURL,
                         OnSetZoomLevelForLoadingURL)
+    IPC_MESSAGE_HANDLER(ViewMsg_SetZoomLevelForView,
+                        OnSetZoomLevelForView)
     IPC_MESSAGE_HANDLER(ViewMsg_SetPageEncoding, OnSetPageEncoding)
     IPC_MESSAGE_HANDLER(ViewMsg_ResetPageEncodingToDefault,
                         OnResetPageEncodingToDefault)
@@ -2660,6 +2661,11 @@ void RenderViewImpl::OnSetZoomLevelForLoadingURL(const GURL& url,
 #endif
 }
 
+void RenderViewImpl::OnSetZoomLevelForView(double level) {
+  webview()->setZoomLevel(level);

[Fady Samuel, 2014/06/09 15:43:21]

Do we need to close popups here?

[wjmaclean, 2014/06/09 17:23:29]

Probably ... consider it added (unless I discover a good reason why we shouldn't
... which I vaguely think there might be ...).

+//  zoomLevelChanged();

[Fady Samuel, 2014/06/09 15:43:21]

Remove this? 

[wjmaclean, 2014/06/09 17:23:29]

Done.

+}
+
 void RenderViewImpl::OnSetPageEncoding(const std::string& encoding_name) {
   webview()->setPageEncoding(WebString::fromUTF8(encoding_name));
 }