Issue 1444973002: Collapse whitespace after list markers

Issue 1444973002: Collapse whitespace after list markers (Closed)

Created:
5 years, 1 month ago by rhogan

Modified:
5 years, 1 month ago

Reviewers:
eae

CC:
blink-reviews, blink-reviews-layout_chromium.org, chromium-reviews, eae+blinkwatch, jchaffraix+rendering, leviw+renderwatch, pdr+renderingwatchlist_chromium.org, 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

Collapse whitespace after list markers Peak inside any 'empty' inlines after the next object on the line when looking for whitespace to collapse. BUG=554903 Committed: https://crrev.com/74fce33853729f49a3574a930862aeed91274b09 Cr-Commit-Position: refs/heads/master@{#360143}

Patch Set 1 #

Patch Set 2 : Updated #

Created: 5 years, 1 month ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+33 lines, -5 lines)			Patch
M	third_party/WebKit/LayoutTests/TestExpectations	View	1	1 chunk	+2 lines, -0 lines	0 comments	Download
A	third_party/WebKit/LayoutTests/fast/inline/collapsing-text-after-list-markers.html	View		1 chunk	+6 lines, -0 lines	0 comments	Download
A	third_party/WebKit/LayoutTests/fast/inline/collapsing-text-after-list-markers-expected.html	View		1 chunk	+6 lines, -0 lines	0 comments	Download
M	third_party/WebKit/LayoutTests/platform/linux/css2.1/t0505-c16-descendant-01-e-expected.png	View	1	Binary file		0 comments	Download
M	third_party/WebKit/LayoutTests/platform/linux/css2.1/t0505-c16-descendant-01-e-expected.txt	View	1	1 chunk	+4 lines, -5 lines	0 comments	Download
M	third_party/WebKit/Source/core/layout/line/BreakingContextInlineHeaders.h	View	1	1 chunk	+3 lines, -0 lines	0 comments	Download
M	third_party/WebKit/Source/core/layout/line/InlineIterator.h	View		2 chunks	+12 lines, -0 lines	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/1444973002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1444973002/20001

5 years, 1 month ago (2015-11-17 20:06:38 UTC) #6

commit-bot: I haz the power

5 years, 1 month ago (2015-11-17 20:13:48 UTC) #8

Message was sent while issue was closed.

Patchset 2 (id:??) landed as
https://crrev.com/74fce33853729f49a3574a930862aeed91274b09
Cr-Commit-Position: refs/heads/master@{#360143}

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