Bug 131592

Summary:	Consider merging: Pass the correct value for the available width to updateLogicalWidthForAlignment() after placing an ellipsis
Product:	WebKit	Reporter:	Daniel Trebbien <dtrebbien>
Component:	CSS	Assignee:	Myles C. Maxfield <mmaxfield>
Status:	RESOLVED DUPLICATE
Severity:	Normal	CC:	bfulgham, dtrebbien, mario, mitz, mmaxfield, rniwa, simon.fraser
Priority:	P2	Keywords:	BlinkMergeCandidate
Version:	528+ (Nightly build)
Hardware:	All
OS:	iOS 6.0

Description Daniel Trebbien 2014-04-13 10:50:07 PDT

Consider merging:  http://src.chromium.org/viewvc/blink?revision=167194&view=revision

mario.prada's patch fixes an issue with text-overflow:ellipsis demonstrated by the test case that I attached to Bug 75277.

I am seeing the issue in iOS 7.1 Mobile Safari, Safari 7.0.3, and WebKit nightly r167020 built on 09 April 2014.  The issue is fixed in Google Chrome 34.0.1847.116 (Official Build 260972).  I used Chromium's bisect-builds.py script and found that the issue was fixed after 167188 (known bad), but no later than 167194 (first known good).
BLINK CHANGELOG URL:
  http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog_blink.html?url=/trunk&range=167194%3A167188
.. which led me to Mario's patch.

Comment 1 Mario Sanchez Prada 2014-04-22 04:06:02 PDT

Just to mention that a WebKit version of this patch is already attached in this bugzilla, waiting for a final review:
https://bugs.webkit.org/show_bug.cgi?id=129313

It's just pending, IIRC, in answering some final doubt exposed in there on how to write the new test for that specific case. My vote goes for useing regular tests to complete the backport and think about converting all of them to reftests later.

Ryosuke, would you mind commenting on that?
https://bugs.webkit.org/show_bug.cgi?id=129313#c20

Thanks!

Comment 2 Brent Fulgham 2022-07-13 17:14:27 PDT


*** This bug has been marked as a duplicate of bug 129313 ***