REGRESSION(r115594): Caused 10 plus tests crash on Win/Linux debug (Requested by zhenyao on #webkit).

• Component: New Bugs
• Status: NEW ---

[WebKit Review Bot]

http://trac.webkit.org/changeset/115594 broke the build:
Caused 10 plus tests crash on Win/Linux debug (Requested by zhenyao on #webkit).

This is an automatic bug report generated by the sheriff-bot. If this bug
report was created because of a flaky test, please file a bug for the flaky
test (if we don't already have one on file) and dup this bug against that bug
so that we can track how often these flaky tests case pain.

"Only you can prevent forest fires." -- Smokey the Bear

[WebKit Review Bot]

Created attachment 139474 [details]
ROLLOUT of r115594

Any committer can land this patch automatically by marking it commit-queue+.  The commit-queue will build and test the patch before landing to ensure that the rollout will be successful.  This process takes approximately 15 minutes.

If you would like to land the rollout faster, you can use the following command:

  webkit-patch land-attachment ATTACHMENT_ID

where ATTACHMENT_ID is the ID of this attachment.

[Ojan Vafai]

Can you list which tests it caused to crash? Or point to the build where it failed? We expect this assert to get hit since it points out a known bug. So, the right thing is probably to just mark the tests as crashing in debug.

[Kentaro Hara]

(In reply to comment #2)
> Can you list which tests it caused to crash? Or point to the build where it failed? We expect this assert to get hit since it points out a known bug. So, the right thing is probably to just mark the tests as crashing in debug.

Yes! We want to know it!

For the time being, I landed a patch to remove ASSERT() from the code to avoid the crashes.