Issue 1771773002: Handle '<something>' to 'none' changes of 'transform' for SVG

Issue 1771773002: Handle '<something>' to 'none' changes of 'transform' for SVG (Closed)

Created:
4 years, 9 months ago by fs

Modified:
4 years, 9 months ago

Reviewers:
davve, pdr.

CC:
fs, blink-reviews, blink-reviews-layout_chromium.org, chromium-reviews, krit, eae+blinkwatch, f(malita), gyuyoung2, jchaffraix+rendering, kouhei+svg_chromium.org, leviw+renderwatch, pdr+renderingwatchlist_chromium.org, pdr+svgwatchlist_chromium.org, rwlbuis, Stephen Chennney, szager+layoutwatch_chromium.org, zoltan1

Base URL:
https://chromium.googlesource.com/chromium/src.git@master

Target Ref:
refs/pending/heads/master

Project:
chromium

Visibility:
Public.

More Reviews

Description

Handle '<something>' to 'none' changes of 'transform' for SVG Only the value of the new style was observed and triggered an update. Check the StyleDifference flag for simplicity. BUG=592206 Committed: https://crrev.com/3530ee063bf77263b200026b37ad79b097229362 Cr-Commit-Position: refs/heads/master@{#379546}

Patch Set 1 #

Created: 4 years, 9 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+21 lines, -0 lines)			Patch
A	third_party/WebKit/LayoutTests/svg/transforms/change-transform-to-none-shape.html	View	1 chunk	+10 lines, -0 lines	0 comments	Download
A +	third_party/WebKit/LayoutTests/svg/transforms/change-transform-to-none-shape-expected.html	View	0 chunks	+-1 lines, --1 lines	0 comments	Download
A	third_party/WebKit/LayoutTests/svg/transforms/change-transform-to-none-text.html	View	1 chunk	+11 lines, -0 lines	0 comments	Download
A +	third_party/WebKit/LayoutTests/svg/transforms/change-transform-to-none-text-expected.html	View	0 chunks	+-1 lines, --1 lines	0 comments	Download
M	third_party/WebKit/Source/core/layout/svg/LayoutSVGBlock.cpp	View	1 chunk	+1 line, -1 line	0 comments	Download
M	third_party/WebKit/Source/core/layout/svg/LayoutSVGModelObject.cpp	View	1 chunk	+1 line, -1 line	0 comments	Download

Messages

Total messages: 8 (3 generated)

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1771773002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1771773002/1

4 years, 9 months ago (2016-03-07 12:58:44 UTC) #5

commit-bot: I haz the power

4 years, 9 months ago (2016-03-07 13:05:00 UTC) #8

Message was sent while issue was closed.

Patchset 1 (id:??) landed as
https://crrev.com/3530ee063bf77263b200026b37ad79b097229362
Cr-Commit-Position: refs/heads/master@{#379546}

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages