Compute pageScaleFactor on page so that fixed layout page fits width of window.

chrome/renderer/chrome_render_view_observer.cc

 // 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.
 
 #include "chrome/renderer/chrome_render_view_observer.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
 #include "base/string_util.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/icon_messages.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/thumbnail_score.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/renderer/about_handler.h"
 #include "chrome/renderer/automation/dom_automation_controller.h"
 #include "chrome/renderer/chrome_render_process_observer.h"
 #include "chrome/renderer/content_settings_observer.h"
 #include "chrome/renderer/extensions/extension_dispatcher.h"
+#include "chrome/renderer/extensions/extension_helper.h"
 #include "chrome/renderer/external_host_bindings.h"
 #include "chrome/renderer/frame_sniffer.h"
 #include "chrome/renderer/prerender/prerender_helper.h"
 #include "chrome/renderer/safe_browsing/phishing_classifier_delegate.h"
 #include "chrome/renderer/translate_helper.h"
 #include "chrome/renderer/webview_color_overlay.h"
 #include "content/public/common/bindings_policy.h"
+#include "content/public/common/view_type.h"
 #include "content/public/renderer/render_view.h"
 #include "content/public/renderer/content_renderer_client.h"
 #include "net/base/data_url.h"
 #include "skia/ext/image_operations.h"
 #include "skia/ext/platform_canvas.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebAccessibilityObject.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebCString.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebDataSource.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebDocument.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
@@ skipping 44 matching lines @@
 // #define TIME_TEXT_RETRIEVAL
 
 // maximum number of characters in the document to index, any text beyond this
 // point will be clipped
 static const size_t kMaxIndexChars = 65535;
 
 // Size of the thumbnails that we'll generate
 static const int kThumbnailWidth = 212;
 static const int kThumbnailHeight = 132;
 
+// The default layout width for pages when fixed layout is enabled.
+static const int kDefaultLayoutWidth = 980;
+static const int kDefaultLayoutHeight = 640;
+
 // Constants for UMA statistic collection.
 static const char kWWWDotGoogleDotCom[] = "www.google.com";
 static const char kMailDotGoogleDotCom[] = "mail.google.com";
 static const char kPlusDotGoogleDotCom[] = "plus.google.com";
 static const char kDocsDotGoogleDotCom[] = "docs.google.com";
 static const char kSitesDotGoogleDotCom[] = "sites.google.com";
 static const char kPicasawebDotGoogleDotCom[] = "picasaweb.google.com";
 static const char kCodeDotGoogleDotCom[] = "code.google.com";
 static const char kGroupsDotGoogleDotCom[] = "groups.google.com";
 static const char kMapsDotGoogleDotCom[] = "maps.google.com";
@@ skipping 259 matching lines @@
 void ChromeRenderViewObserver::OnAddStrictSecurityHost(
     const std::string& host) {
   strict_security_hosts_.insert(host);
 }
 
 void ChromeRenderViewObserver::OnSetAsInterstitial() {
   content_settings_->SetAsInterstitial();
 }
 
 void ChromeRenderViewObserver::Navigate(const GURL& url) {
+  ExtensionHelper* extension_helper = ExtensionHelper::Get(render_view());

[Aaron Boodman, 2012/01/20 18:49:23]

What are you trying to do here? It looks to me like ExtensionHelper exists for
all render views, so this code will run for all tab contents - web or extension.

Is that what you want?

+  bool is_tab_contents = extension_helper &&
+      extension_helper->view_type() == content::VIEW_TYPE_TAB_CONTENTS;

[darin (slow to review), 2012/01/20 19:10:20]

it seems a little hacky to be using something from the extension system for
this.  it seems like you are trying to test if the RenderView belongs to a
TabContents.  surely, there must be or could be a more direct way of determining
that via the Content API.  if not, then perhaps we should add one.  it seems
wrong to entangle the extension system here when this code really has nothing to
do with extensions.

[Fady Samuel, 2012/01/20 21:05:04]

I don't know what the answer to that question is...perhaps jam might know? I'll
ask him.

[jam, 2012/01/20 21:08:05]

code in the renderer doesn't generally know about TabContents, by design. can
you have the browser process (TabContents?) to send relevant ipcs to
enable/disable ? it also seems that this isn't chrome related, but something
that completely belongs in content (since all the other code in this cl is
there)

+  const CommandLine* cmd_line = CommandLine::ForCurrentProcess();
+  // Only use fixed layout on tab contents.
+  if (is_tab_contents && cmd_line->HasSwitch(switches::kEnableFixedLayout)) {
+    WebSize viewSize = render_view()->GetWebView()->size();

[darin (slow to review), 2012/01/20 19:10:20]

nit: view_size

[Fady Samuel, 2012/01/20 21:05:04]

Done.

+    int layoutHeight =

[darin (slow to review), 2012/01/20 19:10:20]

nit: layout_height

[Fady Samuel, 2012/01/20 21:05:04]

Done.

+        viewSize.height ?
+            viewSize.height * kDefaultLayoutWidth / viewSize.width :
+            kDefaultLayoutHeight;
+    WebSize layout_size(kDefaultLayoutWidth, layoutHeight);
+    render_view()->GetWebView()->enableFixedLayoutMode(true);
+    render_view()->GetWebView()->setFixedLayoutSize(layout_size);
+    render_view()->GetWebView()->
+        setShouldLayoutFixedElementsRelativeToFrame(true);

[darin (slow to review), 2012/01/20 19:10:20]

isn't this done through WebSettings now?

[Fady Samuel, 2012/01/20 21:05:04]

Patch hasn't landed yet. Will fix now.  But once it does, this isn't necessary,
so I've removed those lines.

+  } else {
+    render_view()->GetWebView()->enableFixedLayoutMode(false);
+    render_view()->GetWebView()->
+        setShouldLayoutFixedElementsRelativeToFrame(false);
+  }
   // Execute cache clear operations that were postponed until a navigation
   // event (including tab reload).
   if (chrome_render_process_observer_)
     chrome_render_process_observer_->ExecutePendingClearCache();
   AboutHandler::MaybeHandle(url);
 }
 
 void ChromeRenderViewObserver::OnSetClientSidePhishingDetection(
     bool enable_phishing_detection) {
 #if defined(ENABLE_SAFE_BROWSING) && !defined(OS_CHROMEOS)
@@ skipping 690 matching lines @@
         reinterpret_cast<const unsigned char*>(&data[0]);
 
     return decoder.Decode(src_data, data.size());
   }
   return SkBitmap();
 }
 
 bool ChromeRenderViewObserver::IsStrictSecurityHost(const std::string& host) {
   return (strict_security_hosts_.find(host) != strict_security_hosts_.end());
 }