Check temporary zoom status when sending zoom level in navigation.

content/public/browser/host_zoom_map.h

Index: content/public/browser/host_zoom_map.h
diff --git a/content/public/browser/host_zoom_map.h b/content/public/browser/host_zoom_map.h
index 653d24882fe0c5532e15524bf44129373748f276..eeb3b27111942fa6418c684dec5af199c660fbd1 100644
--- a/content/public/browser/host_zoom_map.h
+++ b/content/public/browser/host_zoom_map.h
@@ -137,6 +137,14 @@ class HostZoomMap {
   virtual bool UsesTemporaryZoomLevel(int render_process_id,
                                       int render_view_id) const = 0;
 
+  // Returns the temporary zoom level that's only valid for the lifetime of
+  // the given WebContents (i.e. isn't saved and doesn't affect other
+  // WebContentses) if it exists, the default zoom level otherwise.
+  //
+  // This may be called on any thread.
+  virtual double GetTemporaryZoomLevel(int render_process_id,

[Charlie Reis, 2015/02/11 18:34:31]

From the Content API guidelines:
http://www.chromium.org/developers/content-module/content-api

"only expose methods in the public API that embedders need. If a method is only
used by other code in content, it belongs in foo_impl.h and not foo.h."

[wjmaclean, 2015/02/11 20:29:34]

Done.

+                                       int render_view_id) const = 0;

[Charlie Reis, 2015/02/11 18:34:31]

From the Content API guidelines:
http://www.chromium.org/developers/content-module/content-api

"don't add the const identifier to interfaces. For interfaces implemented by the
embedder, we can't make assumptions about what the embedder needs to implement
it. For interfaces implemented by content, the implementation details doesn't
have to be exposed."

[wjmaclean, 2015/02/11 20:29:34]

Done.

+
   // Sets the temporary zoom level that's only valid for the lifetime of this
   // WebContents.
   //