Bug 169795 - Initial letter does not paginate properly.
Summary: Initial letter does not paginate properly.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dave Hyatt
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-03-16 15:46 PDT by Dave Hyatt
Modified: 2017-03-17 11:55 PDT (History)
3 users (show)

See Also:


Attachments
Patch (43.29 KB, patch)
2017-03-16 15:54 PDT, Dave Hyatt
no flags Details | Formatted Diff | Diff
Patch (43.59 KB, patch)
2017-03-16 16:02 PDT, Dave Hyatt
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews106 for mac-elcapitan-wk2 (2.68 MB, application/zip)
2017-03-16 16:52 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews101 for mac-elcapitan (2.47 MB, application/zip)
2017-03-16 17:00 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews112 for mac-elcapitan (3.31 MB, application/zip)
2017-03-16 17:20 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews121 for ios-simulator-wk2 (4.82 MB, application/zip)
2017-03-16 17:32 PDT, Build Bot
no flags Details
Patch (47.85 KB, patch)
2017-03-17 07:59 PDT, Dave Hyatt
no flags Details | Formatted Diff | Diff
Patch (61.65 KB, patch)
2017-03-17 08:16 PDT, Dave Hyatt
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews103 for mac-elcapitan (1.87 MB, application/zip)
2017-03-17 09:23 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews106 for mac-elcapitan-wk2 (2.64 MB, application/zip)
2017-03-17 09:27 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews117 for mac-elcapitan (2.59 MB, application/zip)
2017-03-17 09:34 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews121 for ios-simulator-wk2 (14.38 MB, application/zip)
2017-03-17 09:49 PDT, Build Bot
no flags Details
Patch (65.79 KB, patch)
2017-03-17 10:59 PDT, Dave Hyatt
zalan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Hyatt 2017-03-16 15:46:53 PDT
Initial letter does not paginate properly.
Comment 1 Dave Hyatt 2017-03-16 15:54:13 PDT
Created attachment 304711 [details]
Patch
Comment 2 Simon Fraser (smfr) 2017-03-16 15:58:25 PDT
Comment on attachment 304711 [details]
Patch

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

> Source/WebCore/rendering/FloatingObjects.h:98
>      RootInlineBox* m_originatingLine;

Might as well convert this to { nullptr }

> Source/WebCore/rendering/RenderBlockFlow.cpp:1886
> +            // Increase our logical height to ensure that lines all get pushed along with the
> +            // letter.

Comment can go one one line.

> Source/WebCore/rendering/RenderBlockFlow.cpp:2263
> +            LayoutRect floatBox(offsetFromRootBlock.width(), offsetFromRootBlock.height(),                 floatingObject.renderer().width(), floatingObject.renderer().height());

Weird huge space.
Comment 3 Dave Hyatt 2017-03-16 16:02:05 PDT
Created attachment 304714 [details]
Patch
Comment 4 Build Bot 2017-03-16 16:52:04 PDT
Comment on attachment 304714 [details]
Patch

Attachment 304714 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/3343917

New failing tests:
fast/regions/overflow/overflow-in-variable-width-regions-inline-bt.html
imported/blink/fast/multicol/vertical-rl/float-break.html
imported/blink/fast/multicol/vertical-rl/float-edge.html
fast/regions/overflow/overflow-in-variable-width-regions-inline-rl.html
fast/writing-mode/vertical-float-margin.html
imported/blink/fast/multicol/vertical-rl/float-content-break.html
fast/regions/selection/selecting-text-ignoring-region-vert-rl.html
fast/regions/selection/selecting-text-ignoring-region-horiz-bt.html
Comment 5 Build Bot 2017-03-16 16:52:09 PDT
Created attachment 304722 [details]
Archive of layout-test-results from ews106 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 6 Build Bot 2017-03-16 17:00:03 PDT
Comment on attachment 304714 [details]
Patch

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

New failing tests:
fast/regions/overflow/overflow-in-variable-width-regions-inline-bt.html
imported/blink/fast/multicol/vertical-rl/float-break.html
imported/blink/fast/multicol/vertical-rl/float-edge.html
fast/regions/overflow/overflow-in-variable-width-regions-inline-rl.html
fast/writing-mode/vertical-float-margin.html
imported/blink/fast/multicol/vertical-rl/float-big-line.html
imported/blink/fast/multicol/vertical-rl/float-content-break.html
fast/regions/selection/selecting-text-ignoring-region-vert-rl.html
fast/regions/selection/selecting-text-ignoring-region-horiz-bt.html
Comment 7 Build Bot 2017-03-16 17:00:07 PDT
Created attachment 304723 [details]
Archive of layout-test-results from ews101 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 8 Build Bot 2017-03-16 17:20:18 PDT
Comment on attachment 304714 [details]
Patch

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

New failing tests:
fast/regions/overflow/overflow-in-variable-width-regions-inline-bt.html
imported/blink/fast/multicol/vertical-rl/float-break.html
imported/blink/fast/multicol/vertical-rl/float-edge.html
fast/regions/overflow/overflow-in-variable-width-regions-inline-rl.html
fast/writing-mode/vertical-float-margin.html
imported/blink/fast/multicol/vertical-rl/float-big-line.html
imported/blink/fast/multicol/vertical-rl/float-content-break.html
fast/regions/selection/selecting-text-ignoring-region-vert-rl.html
fast/regions/selection/selecting-text-ignoring-region-horiz-bt.html
Comment 9 Build Bot 2017-03-16 17:20:25 PDT
Created attachment 304726 [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 10 Build Bot 2017-03-16 17:32:11 PDT
Comment on attachment 304714 [details]
Patch

Attachment 304714 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/3344127

New failing tests:
fast/css-generated-content/initial-letter-pagination-raised.html
imported/blink/fast/multicol/vertical-rl/float-edge.html
fast/writing-mode/vertical-float-margin.html
fast/css-generated-content/initial-letter-pagination-sunken.html
imported/blink/fast/multicol/vertical-rl/float-break.html
imported/blink/fast/multicol/vertical-rl/float-big-line.html
imported/blink/fast/multicol/vertical-rl/float-content-break.html
fast/regions/selection/selecting-text-ignoring-region-vert-rl.html
Comment 11 Build Bot 2017-03-16 17:32:14 PDT
Created attachment 304727 [details]
Archive of layout-test-results from ews121 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews121  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6
Comment 12 Dave Hyatt 2017-03-17 07:59:17 PDT
Created attachment 304778 [details]
Patch
Comment 13 WebKit Commit Bot 2017-03-17 08:01:19 PDT
Attachment 304778 [details] did not pass style-queue:


ERROR: LayoutTests/platform/ios-simulator/TestExpectations:2919:  Path does not exist.  [test/expectations] [5]
ERROR: LayoutTests/platform/ios-simulator/TestExpectations:2920:  Path does not exist.  [test/expectations] [5]
Total errors found: 2 in 13 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 14 Dave Hyatt 2017-03-17 08:16:22 PDT
Created attachment 304779 [details]
Patch
Comment 15 Build Bot 2017-03-17 09:23:11 PDT
Comment on attachment 304779 [details]
Patch

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

New failing tests:
fast/block/lineboxcontain/block-with-ideographs.xhtml
fast/css-generated-content/initial-letter-pagination-sunken-rl.html
fast/multicol/tall-image-behavior-rl.html
fast/multicol/tall-image-behavior-lr.html
Comment 16 Build Bot 2017-03-17 09:23:17 PDT
Created attachment 304784 [details]
Archive of layout-test-results from ews103 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 17 Build Bot 2017-03-17 09:27:34 PDT
Comment on attachment 304779 [details]
Patch

Attachment 304779 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/3350039

New failing tests:
fast/block/lineboxcontain/block-with-ideographs.xhtml
fast/css-generated-content/initial-letter-pagination-sunken-rl.html
fast/multicol/tall-image-behavior-rl.html
fast/css-generated-content/initial-letter-pagination-raised-rl.html
fast/multicol/tall-image-behavior-lr.html
Comment 18 Build Bot 2017-03-17 09:27:38 PDT
Created attachment 304785 [details]
Archive of layout-test-results from ews106 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 19 Build Bot 2017-03-17 09:34:55 PDT
Comment on attachment 304779 [details]
Patch

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

New failing tests:
fast/block/lineboxcontain/block-with-ideographs.xhtml
fast/css-generated-content/initial-letter-pagination-sunken-rl.html
fast/multicol/tall-image-behavior-rl.html
fast/multicol/tall-image-behavior-lr.html
Comment 20 Build Bot 2017-03-17 09:34:59 PDT
Created attachment 304786 [details]
Archive of layout-test-results from ews117 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 21 Build Bot 2017-03-17 09:49:15 PDT
Comment on attachment 304779 [details]
Patch

Attachment 304779 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/3350090

New failing tests:
fast/multicol/tall-image-behavior-rl.html
fast/multicol/tall-image-behavior-lr.html
Comment 22 Build Bot 2017-03-17 09:49:19 PDT
Created attachment 304788 [details]
Archive of layout-test-results from ews121 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews121  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6
Comment 23 Dave Hyatt 2017-03-17 10:59:51 PDT
Created attachment 304794 [details]
Patch
Comment 24 Dave Hyatt 2017-03-17 11:55:19 PDT
Landed in r214110.