Bug 185979 - REGRESSION: Assertion failure compiling 1Blocker
Summary: REGRESSION: Assertion failure compiling 1Blocker
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tadeu Zagallo
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-05-25 02:40 PDT by Tadeu Zagallo
Modified: 2018-05-29 11:10 PDT (History)
5 users (show)

See Also:


Attachments
Patch (5.37 KB, patch)
2018-05-27 15:04 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Patch (5.28 KB, patch)
2018-05-28 01:33 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tadeu Zagallo 2018-05-25 02:40:57 PDT
ToT debug builds are crashing when compiling 1Blocker due to an assertion failure in DFANode::setActions.
Comment 1 Tadeu Zagallo 2018-05-25 02:41:49 PDT
<rdar://problem/39669458>
Comment 2 Tadeu Zagallo 2018-05-27 15:04:04 PDT
Created attachment 341430 [details]
Patch
Comment 3 Daniel Bates 2018-05-27 20:41:57 PDT
Comment on attachment 341430 [details]
Patch

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

> LayoutTests/http/tests/contentextensions/root-actions-expected.txt:2
> +layer at (0,0) size 800x600

I do not see the need for this test to dump the render tree. It seems sufficient to make this a text-only trust. See my remark below on how to achieve this.

> LayoutTests/http/tests/contentextensions/root-actions.html:1
> +<body>

Unless it is necessary to use quirks mode for this test please add <!DOCTYPE html> at the top of this file and make this a well defined HTML document with <html> and </html>. Please make this a text-only test as opposed to a render tree dump. You can do this by adding the following markup to the <head> or <body>:

<script>
if (window.testRunner)
    testRunner.dumpAsText();
</script>
Comment 4 Tadeu Zagallo 2018-05-28 01:33:47 PDT
Created attachment 341449 [details]
Patch
Comment 5 Geoffrey Garen 2018-05-29 10:43:47 PDT
Comment on attachment 341449 [details]
Patch

r=me
Comment 6 Geoffrey Garen 2018-05-29 10:44:14 PDT
Comment on attachment 341449 [details]
Patch

cq+
Comment 7 WebKit Commit Bot 2018-05-29 11:10:56 PDT
Comment on attachment 341449 [details]
Patch

Clearing flags on attachment: 341449

Committed r232265: <https://trac.webkit.org/changeset/232265>
Comment 8 WebKit Commit Bot 2018-05-29 11:10:58 PDT
All reviewed patches have been landed.  Closing bug.