Invalidate the frame view when changing the background image of the document or body.

Source/core/rendering/RenderView.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 149 matching lines @@
 {
     // It's hard to predict here which of full paint invalidation or per-descendant paint invalidation costs less.
     // For vertical writing mode or width change it's more likely that per-descendant paint invalidation
     // eventually turns out to be full paint invalidation but with the cost to handle more layout states
     // and discrete paint invalidation rects, so marking full paint invalidation here is more likely to cost less.
     // Otherwise, per-descendant paint invalidation is more likely to avoid unnecessary full paint invalidation.
 
     if (shouldUsePrintingLayout())
         return true;
 
-    if (!style()->isHorizontalWritingMode() || width() != viewWidth())
+    if (!style()->isHorizontalWritingMode() || width() != viewWidth()) {
+        fprintf(stderr, "change view width\n");

[dsinclair, 2014/10/31 00:14:34]

Remove.

[chrishtr, 2014/10/31 18:22:24]

Fixed.

         return true;
+    }
 
     if (height() != viewHeight()) {
         if (RenderObject* backgroundRenderer = this->backgroundRenderer()) {
             // When background-attachment is 'fixed', we treat the viewport (instead of the 'root'
             // i.e. html or body) as the background positioning area, and we should full paint invalidation
             // viewport resize if the background image is not composited and needs full paint invalidation on
             // background positioning area resize.
             if (!m_compositor || !m_compositor->needsFixedRootBackgroundLayer(layer())) {
                 if (backgroundRenderer->style()->hasFixedBackgroundImage()
                     && mustInvalidateFillLayersPaintOnHeightChange(backgroundRenderer->style()->backgroundLayers()))
-                return true;
+                    return true;
             }
         }
     }
 
     return false;
 }
 
 void RenderView::layout()
 {
     if (!document().paginated())
@@ skipping 679 matching lines @@
     return viewWidth(IncludeScrollbars) / scale;
 }
 
 double RenderView::layoutViewportHeight() const
 {
     float scale = m_frameView ? m_frameView->frame().pageZoomFactor() : 1;
     return viewHeight(IncludeScrollbars) / scale;
 }
 
 } // namespace blink