Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1197)

Side by Side Diff: third_party/WebKit/Source/core/paint/BoxPainter.cpp

Issue 2194273002: Fix border radius on composited children. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Revised method based on PaintLayer switching Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "core/paint/BoxPainter.h" 5 #include "core/paint/BoxPainter.h"
6 6
7 #include "core/HTMLNames.h" 7 #include "core/HTMLNames.h"
8 #include "core/frame/Settings.h" 8 #include "core/frame/Settings.h"
9 #include "core/html/HTMLFrameOwnerElement.h" 9 #include "core/html/HTMLFrameOwnerElement.h"
10 #include "core/layout/ImageQualityController.h" 10 #include "core/layout/ImageQualityController.h"
(...skipping 774 matching lines...) Expand 10 before | Expand all | Expand 10 after
785 root.lineTop(), root.lineBottom()); 785 root.lineTop(), root.lineBottom());
786 } else { 786 } else {
787 // FIXME: this should only have an effect for the line box list within 787 // FIXME: this should only have an effect for the line box list within
788 // |obj|. Change this to create a LineBoxListPainter directly. 788 // |obj|. Change this to create a LineBoxListPainter directly.
789 LayoutSize localOffset = 789 LayoutSize localOffset =
790 obj.isBox() ? toLayoutBox(&obj)->locationOffset() : LayoutSize(); 790 obj.isBox() ? toLayoutBox(&obj)->locationOffset() : LayoutSize();
791 obj.paint(info, scrolledPaintRect.location() - localOffset); 791 obj.paint(info, scrolledPaintRect.location() - localOffset);
792 } 792 }
793 793
794 context.endLayer(); 794 context.endLayer();
795 context.endLayer();
796 } 795 }
797 } 796 }
798 797
799 void BoxPainter::paintMask(const PaintInfo& paintInfo, 798 void BoxPainter::paintMask(const PaintInfo& paintInfo,
800 const LayoutPoint& paintOffset) { 799 const LayoutPoint& paintOffset) {
801 if (m_layoutBox.style()->visibility() != EVisibility::Visible || 800 if (m_layoutBox.style()->visibility() != EVisibility::Visible ||
802 paintInfo.phase != PaintPhaseMask) 801 paintInfo.phase != PaintPhaseMask)
803 return; 802 return;
804 803
805 if (LayoutObjectDrawingRecorder::useCachedDrawingIfPossible( 804 if (LayoutObjectDrawingRecorder::useCachedDrawingIfPossible(
(...skipping 244 matching lines...) Expand 10 before | Expand all | Expand 10 after
1050 bool BoxPainter::shouldForceWhiteBackgroundForPrintEconomy( 1049 bool BoxPainter::shouldForceWhiteBackgroundForPrintEconomy(
1051 const ComputedStyle& style, 1050 const ComputedStyle& style,
1052 const Document& document) { 1051 const Document& document) {
1053 return document.printing() && 1052 return document.printing() &&
1054 style.getPrintColorAdjust() == PrintColorAdjust::Economy && 1053 style.getPrintColorAdjust() == PrintColorAdjust::Economy &&
1055 (!document.settings() || 1054 (!document.settings() ||
1056 !document.settings()->shouldPrintBackgrounds()); 1055 !document.settings()->shouldPrintBackgrounds());
1057 } 1056 }
1058 1057
1059 } // namespace blink 1058 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698