Issue 326753002: Fix use of LayoutStateDisabler for RenderView invalidation

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 326753002: Fix use of LayoutStateDisabler for RenderView invalidation (Closed)

Created:
6 years, 6 months ago by leviw_travelin_and_unemployed

Modified:
6 years, 6 months ago

Reviewers:
dsinclair, Julien - ping for review

CC:
blink-reviews, blink-reviews-rendering, eae+blinkwatch, jchaffraix+rendering, leviw+renderwatch, pdr., rune+blink, zoltan1

Base URL:
svn://svn.chromium.org/blink/trunk

Visibility:
Public.

More Reviews

Description

Fix use of LayoutStateDisabler for RenderView invalidation In an attempt to optimize pushing and popping LayoutState, the entire optimization ended up being turned off. Fix the use of LayoutState for RenderView-relative renderers. BUG=381138 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175832

Patch Set 1 #

Patch Set 2 : Add test expectations #

Created: 6 years, 6 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+5 lines, -1 line)			Patch
	M	LayoutTests/TestExpectations	View	1	1 chunk	+4 lines, -0 lines	0 comments	Download
	M	Source/core/rendering/RenderBox.cpp	View		1 chunk	+1 line, -1 line	0 comments	Download