Bug 166641 - Reimplement RenderCounter.
Summary: Reimplement RenderCounter.
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: zalan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-02 10:04 PST by zalan
Modified: 2020-09-08 13:41 PDT (History)
2 users (show)

See Also:


Attachments
Patch (73.98 KB, patch)
2017-01-02 10:08 PST, zalan
no flags Details | Formatted Diff | Diff
Patch (77.17 KB, patch)
2017-01-03 22:06 PST, zalan
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-elcapitan (884.84 KB, application/zip)
2017-01-04 11:45 PST, Build Bot
no flags Details
Archive of layout-test-results from ews106 for mac-elcapitan-wk2 (898.03 KB, application/zip)
2017-01-04 11:50 PST, Build Bot
no flags Details
Archive of layout-test-results from ews113 for mac-elcapitan (1.66 MB, application/zip)
2017-01-04 11:52 PST, Build Bot
no flags Details
Archive of layout-test-results from ews126 for ios-simulator-wk2 (9.71 MB, application/zip)
2017-01-04 12:01 PST, Build Bot
no flags Details
Patch (82.58 KB, patch)
2017-01-06 22:11 PST, zalan
no flags Details | Formatted Diff | Diff
Patch (81.46 KB, patch)
2017-01-07 18:36 PST, zalan
no flags Details | Formatted Diff | Diff
Patch (80.83 KB, patch)
2017-01-08 20:39 PST, zalan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description zalan 2017-01-02 10:04:50 PST
It is mutating the render tree during layout.
Comment 1 zalan 2017-01-02 10:08:34 PST
Created attachment 297903 [details]
Patch
Comment 2 zalan 2017-01-02 10:08:55 PST
^^WIP patch.
Comment 3 zalan 2017-01-03 22:06:20 PST
Created attachment 297991 [details]
Patch
Comment 4 Build Bot 2017-01-04 11:45:27 PST
Comment on attachment 297991 [details]
Patch

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

New failing tests:
css2.1/t1204-implied-02-d.html
Comment 5 Build Bot 2017-01-04 11:45:30 PST
Created attachment 298033 [details]
Archive of layout-test-results from ews102 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 6 Build Bot 2017-01-04 11:50:28 PST
Comment on attachment 297991 [details]
Patch

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

New failing tests:
css2.1/t1204-implied-02-d.html
Comment 7 Build Bot 2017-01-04 11:50:32 PST
Created attachment 298034 [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 8 Build Bot 2017-01-04 11:52:51 PST
Comment on attachment 297991 [details]
Patch

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

New failing tests:
css2.1/t1204-implied-02-d.html
Comment 9 Build Bot 2017-01-04 11:52:54 PST
Created attachment 298035 [details]
Archive of layout-test-results from ews113 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews113  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 10 Build Bot 2017-01-04 12:01:39 PST
Comment on attachment 297991 [details]
Patch

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

New failing tests:
css2.1/t1204-implied-02-d.html
Comment 11 Build Bot 2017-01-04 12:01:42 PST
Created attachment 298036 [details]
Archive of layout-test-results from ews126 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews126  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6
Comment 12 zalan 2017-01-06 22:11:34 PST
Created attachment 298258 [details]
Patch
Comment 13 zalan 2017-01-07 18:36:45 PST
Created attachment 298296 [details]
Patch
Comment 14 zalan 2017-01-08 20:39:29 PST
Created attachment 298331 [details]
Patch
Comment 15 zalan 2020-09-04 19:23:29 PDT
Comment on attachment 298331 [details]
Patch

test
Comment 16 zalan 2020-09-04 19:28:15 PDT
Comment on attachment 298331 [details]
Patch

test
Comment 17 zalan 2020-09-04 19:28:54 PDT
Comment on attachment 298331 [details]
Patch

test
Comment 18 zalan 2020-09-04 19:30:08 PDT
Comment on attachment 298331 [details]
Patch

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

> Source/WebCore/ChangeLog:8
> +

test
Comment 19 zalan 2020-09-04 19:34:51 PDT
Comment on attachment 298331 [details]
Patch

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

> Source/WebCore/rendering/RenderCounter.cpp:-194
> -        if (searchEndRenderer == currentRenderer) {

test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test.
Comment 20 zalan 2020-09-04 19:35:50 PDT
Comment on attachment 298331 [details]
Patch

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

> Source/WebCore/rendering/RenderElement.h:-197
> -    void setHasCounterNodeMap(bool f) { m_hasCounterNodeMap = f; }

some more test test test test test test test test test test test test test test test.
Comment 21 zalan 2020-09-04 19:36:38 PDT
Comment on attachment 298331 [details]
Patch

test some more
Comment 22 Darin Adler 2020-09-04 22:45:03 PDT
Comment on attachment 298331 [details]
Patch

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

> Source/WebCore/ChangeLog:7
> +        Next iteration will take care of the vast amount of counter data on RenderElement.

test
Comment 23 Simon Fraser (smfr) 2020-09-08 13:27:19 PDT
Comment on attachment 298331 [details]
Patch

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

>> Source/WebCore/ChangeLog:7
>> +        Next iteration will take care of the vast amount of counter data on RenderElement.
> 
> test

Test

>> Source/WebCore/ChangeLog:8
>> +
> 
> test

Test

>> Source/WebCore/rendering/RenderCounter.cpp:-194
>> -        if (searchEndRenderer == currentRenderer) {
> 
> test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test.

i am also testing.
Comment 24 zalan 2020-09-08 13:39:11 PDT
Comment on attachment 298331 [details]
Patch

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

>>> Source/WebCore/ChangeLog:8
>>> +
>> 
>> test
> 
> Test

Test here.

>>> Source/WebCore/rendering/RenderCounter.cpp:-194
>>> -        if (searchEndRenderer == currentRenderer) {
>> 
>> test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test.
> 
> i am also testing.

Some more testing
Comment 25 zalan 2020-09-08 13:41:57 PDT
Comment on attachment 298331 [details]
Patch

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

>>> Source/WebCore/ChangeLog:7
>>> +        Next iteration will take care of the vast amount of counter data on RenderElement.
>> 
>> test
> 
> Test

Test

>>>> Source/WebCore/ChangeLog:8
>>>> +
>>> 
>>> test
>> 
>> Test
> 
> Test here.

Test

>>>> Source/WebCore/rendering/RenderCounter.cpp:-194
>>>> -        if (searchEndRenderer == currentRenderer) {
>>> 
>>> test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test test.
>> 
>> i am also testing.
> 
> Some more testing

Testing here too.