Stretching NativeViewHost, and misc tab capture fixes.

ui/views/controls/webview/webview.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 "ui/views/controls/webview/webview.h"
 
 #include <utility>
 
 #include "build/build_config.h"
 #include "content/public/browser/browser_accessibility_state.h"
@@ skipping 120 matching lines @@
   // will engage the special layout/sizing behavior.
   gfx::Rect holder_bounds(bounds().size());
   if (!embed_fullscreen_widget_mode_enabled_ ||
       !web_contents() ||
       web_contents()->GetCapturerCount() == 0 ||
       web_contents()->GetPreferredSize().IsEmpty() ||
       !(is_embedding_fullscreen_widget_ ||
         (web_contents()->GetDelegate() &&
          web_contents()->GetDelegate()->
              IsFullscreenForTabOrPending(web_contents())))) {
+    holder_->SetRenderingSize(gfx::Size());
     holder_->SetBoundsRect(holder_bounds);
     return;
   }
 
-  // Size the holder to the capture video resolution and center it.  If this
-  // WebView is not large enough to contain the holder at the preferred size,
-  // scale down to fit (preserving aspect ratio).
+  // For screen-captured fullscreened content, scale the |holder_| to fit within
+  // this View and center it.
   const gfx::Size capture_size = web_contents()->GetPreferredSize();
-  if (capture_size.width() <= holder_bounds.width() &&
-      capture_size.height() <= holder_bounds.height()) {
-    // No scaling, just centering.
-    holder_bounds.ClampToCenteredSize(capture_size);
+  const int64_t x =
+      static_cast<int64_t>(capture_size.width()) * holder_bounds.height();
+  const int64_t y =
+      static_cast<int64_t>(capture_size.height()) * holder_bounds.width();
+  if (y < x) {
+    holder_bounds.ClampToCenteredSize(gfx::Size(
+        holder_bounds.width(), static_cast<int>(y / capture_size.width())));
   } else {
-    // Scale down, preserving aspect ratio, and center.
-    // TODO(miu): This is basically media::ComputeLetterboxRegion(), and it
-    // looks like others have written this code elsewhere.  Let's considate
-    // into a shared function ui/gfx/geometry or around there.
-    const int64_t x =
-        static_cast<int64_t>(capture_size.width()) * holder_bounds.height();
-    const int64_t y =
-        static_cast<int64_t>(capture_size.height()) * holder_bounds.width();
-    if (y < x) {
-      holder_bounds.ClampToCenteredSize(gfx::Size(
-          holder_bounds.width(), static_cast<int>(y / capture_size.width())));
-    } else {
-      holder_bounds.ClampToCenteredSize(gfx::Size(
-          static_cast<int>(x / capture_size.height()), holder_bounds.height()));
-    }
+    holder_bounds.ClampToCenteredSize(gfx::Size(
+        static_cast<int>(x / capture_size.height()), holder_bounds.height()));
   }
-
+  holder_->SetRenderingSize(capture_size);
   holder_->SetBoundsRect(holder_bounds);
 }
 
 void WebView::ViewHierarchyChanged(
     const ViewHierarchyChangedDetails& details) {
   if (details.is_add)
     AttachWebContents();
 }
 
 bool WebView::SkipDefaultKeyEventProcessing(const ui::KeyEvent& event) {
@@ skipping 202 matching lines @@
   if (!contents) {
     content::WebContents::CreateParams create_params(
         browser_context, NULL);
     return content::WebContents::Create(create_params);
   }
 
   return contents;
 }
 
 }  // namespace views