Bug 168864

Summary: Text might wrap when its preferred logical width is used for sizing the containing block.
Product: WebKit Reporter: zalan <zalan>
Component: Layout and RenderingAssignee: zalan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, koivisto, mmaxfield, simon.fraser, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews112 for mac-elcapitan none

Description zalan 2017-02-24 21:26:33 PST
<rdar://problem/30690734>
Comment 1 zalan 2017-02-24 22:01:45 PST
Created attachment 302734 [details]
Patch
Comment 2 zalan 2017-02-24 22:02:40 PST
Alternative solutions are welcome!
Comment 3 Build Bot 2017-02-24 23:49:48 PST
Comment on attachment 302734 [details]
Patch

Attachment 302734 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/3189792

New failing tests:
media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-drag.html
media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-drag-is-prevented-over-button.html
Comment 4 Build Bot 2017-02-24 23:49:52 PST
Created attachment 302745 [details]
Archive of layout-test-results from ews112 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 5 zalan 2017-02-25 06:14:51 PST
Comment on attachment 302745 [details]
Archive of layout-test-results from ews112 for mac-elcapitan

Timeouts -> totally unrelated.
Comment 6 WebKit Commit Bot 2017-02-25 14:20:13 PST
Comment on attachment 302734 [details]
Patch

Clearing flags on attachment: 302734

Committed r213008: <http://trac.webkit.org/changeset/213008>
Comment 7 WebKit Commit Bot 2017-02-25 14:20:18 PST
All reviewed patches have been landed.  Closing bug.
Comment 8 Darin Adler 2017-02-27 08:50:03 PST
Comment on attachment 302734 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=302734&action=review

> Source/WebCore/platform/graphics/FontCascade.cpp:386
> +    if (text.isNull() || text.isEmpty())
> +        return 0;

Checking isNull is redundant. If it's null, then it's also empty.