Update WebView/FrameView size from LayoutView::UpdateAfterLayout

third_party/WebKit/Source/core/paint/FramePainter.cpp

 // 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 "core/paint/FramePainter.h"
 
 #include "core/editing/markers/DocumentMarkerController.h"
 #include "core/frame/FrameView.h"
 #include "core/inspector/InspectorTraceEvents.h"
 #include "core/layout/LayoutView.h"
@@ skipping 198 matching lines @@
   if (GetFrameView().LayerForScrollCorner() ||
       !GetFrameView().IsScrollCornerVisible()) {
     return;
   }
 
   PaintScrollCorner(context, GetFrameView().ScrollCornerRect());
 }
 
 void FramePainter::PaintScrollCorner(GraphicsContext& context,
                                      const IntRect& corner_rect) {
+  if (corner_rect.IsEmpty())

[cbiesinger, 2017/05/05 18:20:01]

This seems fine but why is that necessary as part of this change?

[szager1, 2017/05/08 16:27:06]

This is a mistake, I had another CL to fix this issue; reverted this diff.

+    return;
+
   if (GetFrameView().ScrollCorner()) {
     bool needs_background = GetFrameView().GetFrame().IsMainFrame();
     if (needs_background &&
         !LayoutObjectDrawingRecorder::UseCachedDrawingIfPossible(
             context, *GetFrameView().GetLayoutView(),
             DisplayItem::kScrollbarCorner)) {
       LayoutObjectDrawingRecorder drawing_recorder(
           context, *GetFrameView().GetLayoutView(),
           DisplayItem::kScrollbarCorner, FloatRect(corner_rect));
       context.FillRect(corner_rect, GetFrameView().BaseBackgroundColor());
@@ skipping 31 matching lines @@
 
   bar.Paint(context, CullRect(rect));
 }
 
 const FrameView& FramePainter::GetFrameView() {
   DCHECK(frame_view_);
   return *frame_view_;
 }
 
 }  // namespace blink