Changeset 220781 in webkit
- Timestamp:
- Aug 15, 2017 7:05:28 PM (7 years ago)
- Location:
- trunk/Source/WebCore
- Files:
-
- 4 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r220780 r220781 1 2017-08-15 Simon Fraser <simon.fraser@apple.com> 2 3 Remove ScrollView::clipsRepaints() which was only used by Chromium 4 https://bugs.webkit.org/show_bug.cgi?id=175594 5 6 Reviewed by Tim Horton. 7 8 clipsRepaints() was added in r74568 but was only used by Chromium. It's always 9 true for other platforms, so remove conditionals that test for it and assume true. 10 11 * page/FrameView.cpp: 12 (WebCore::FrameView::scrollContentsFastPath): 13 * platform/ScrollView.cpp: 14 (WebCore::ScrollView::repaintContentRectangle): 15 (WebCore::ScrollView::setClipsRepaints): Deleted. 16 * platform/ScrollView.h: 17 (WebCore::ScrollView::clipsRepaints const): Deleted. 18 1 19 2017-08-15 Simon Fraser <simon.fraser@apple.com> 2 20 -
trunk/Source/WebCore/page/FrameView.cpp
r220503 r220781 2310 2310 IntRect updateRect = enclosingIntRect(layer->repaintRectIncludingNonCompositingDescendants()); 2311 2311 updateRect = contentsToRootView(updateRect); 2312 if (!isCompositedContentLayer && clipsRepaints())2312 if (!isCompositedContentLayer) 2313 2313 updateRect.intersect(rectToScroll); 2314 2314 if (!updateRect.isEmpty()) … … 2330 2330 continue; 2331 2331 } 2332 if (clipsRepaints()) 2333 updateRect.intersect(rectToScroll); 2332 updateRect.intersect(rectToScroll); 2334 2333 frame().page()->chrome().invalidateContentsAndRootView(updateRect); 2335 2334 } -
trunk/Source/WebCore/platform/ScrollView.cpp
r220503 r220781 193 193 } 194 194 195 void ScrollView::setClipsRepaints(bool clipsRepaints)196 {197 m_clipsRepaints = clipsRepaints;198 }199 200 195 void ScrollView::setDelegatesScrolling(bool delegatesScrolling) 201 196 { … … 1072 1067 { 1073 1068 IntRect paintRect = rect; 1074 if ( clipsRepaints() &&!paintsEntireContents())1069 if (!paintsEntireContents()) 1075 1070 paintRect.intersect(visibleContentRect(LegacyIOSDocumentVisibleRect)); 1076 1071 if (paintRect.isEmpty()) -
trunk/Source/WebCore/platform/ScrollView.h
r216541 r220781 130 130 WEBCORE_EXPORT void setPaintsEntireContents(bool); 131 131 132 // By default, paint events are clipped to the visible area. If set to133 // false, paint events are no longer clipped. paintsEntireContents() implies !clipsRepaints().134 bool clipsRepaints() const { return m_clipsRepaints; }135 void setClipsRepaints(bool);136 137 132 // By default programmatic scrolling is handled by WebCore and not by the UI application. 138 133 // In the case of using a tiled backing store, this mode can be set, so that the scroll requests … … 482 477 483 478 bool m_paintsEntireContents { false }; 484 bool m_clipsRepaints { true };485 479 bool m_delegatesScrolling { false }; 486 480
Note: See TracChangeset
for help on using the changeset viewer.