WIP - Gutterless resize on Windows

chrome/browser/ui/views/frame/contents_web_view.cc

 // Copyright 2014 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/browser/ui/views/frame/contents_web_view.h"
 
 #include "chrome/browser/themes/theme_properties.h"
 #include "chrome/browser/ui/views/status_bubble_views.h"
 #include "content/public/browser/web_contents.h"
 #include "ui/base/theme_provider.h"
@@ skipping 38 matching lines @@
 
 void ContentsWebView::OnThemeChanged() {
   const ui::ThemeProvider* const theme = GetThemeProvider();
   if (!theme)
     return;
 
   // Set the background color to a dark tint of the new tab page's background
   // color.  This is the color filled within the WebView's bounds when its child
   // view is sized specially for fullscreen tab capture.  See WebView header
   // file comments for more details.
+  // TODO(ericrk): This color causes ugly guttering when the window is resized,
+  // investigate changing color to match window color. May need to special case
+  // tab capture to not break the behavior described above. crbug.com/568317
   const int kBackgroundBrightness = 0x33;  // 20%
   const SkColor ntp_background =
       theme->GetColor(ThemeProperties::COLOR_NTP_BACKGROUND);
   set_background(views::Background::CreateSolidBackground(
       SkColorGetR(ntp_background) * kBackgroundBrightness / 0xFF,
       SkColorGetG(ntp_background) * kBackgroundBrightness / 0xFF,
       SkColorGetB(ntp_background) * kBackgroundBrightness / 0xFF,
       SkColorGetA(ntp_background)));
 }
 
@@ skipping 38 matching lines @@
   cloned_layer_tree_->root()->SetBounds(
       gfx::Rect(origin, cloned_layer_tree_->root()->bounds().size()));
   layer()->Add(cloned_layer_tree_->root());
 }
 
 void ContentsWebView::DestroyClonedLayer() {
   cloned_layer_tree_.reset();
   SetPaintToLayer(false);
   set_layer_owner_delegate(nullptr);
 }