WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
130353
Layout Test mathml/wbr-in-mroot-crash.html times out
https://bugs.webkit.org/show_bug.cgi?id=130353
Summary
Layout Test mathml/wbr-in-mroot-crash.html times out
Jer Noble
Reported
2014-03-17 12:17:46 PDT
The following layout test crashes on Apple MountainLion Release WK2, Apple Mavericks Release, and EFL Linux 64-bit Release. mathml/wbr-in-mroot-crash.html Probable cause:
http://trac.webkit.org/changeset/165702
from
https://bugs.webkit.org/show_bug.cgi?id=124128
Attachments
Patch
(3.36 KB, patch)
2016-06-28 01:27 PDT
,
Frédéric Wang (:fredw)
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Jer Noble
Comment 1
2014-03-17 12:29:00 PDT
Marked tests as failing in
https://trac.webkit.org/r165747
.
Alexey Proskuryakov
Comment 2
2014-03-17 12:46:26 PDT
Is there a crash log seen anywhere? On bots I checked, it looks like a timeout misreported as crash.
Frédéric Wang (:fredw)
Comment 3
2014-03-17 12:47:36 PDT
In
http://build.webkit.org/results/Apple%20MountainLion%20Release%20WK2%20%28Tests%29/r165745%20%2816869%29/mathml/wbr-in-mroot-crash-sample.txt
the crash seems to happen in RenderMathMLOperator, however no <mo>s are involved in the test so something is probably doing a wrong cast into a RenderMathMLOperator. I guess we can remove htmlElement.hasTagName(HTMLNames::wbrTag) from isPhrasingContent to workaround the crash, but I'd like to find what the problem is exactly.
Frédéric Wang (:fredw)
Comment 4
2014-03-18 03:27:28 PDT
I just tried again this morning and was not able to reproduce the crash on Linux. I'm no longer able to build a debug on my laptop and adding printf on RenderMathMLOperator does print anything so I'm not sure why they appear in the crash log. Does it happen only on Mac? @Brent: I think you worked on the security issue with HTML content badly cast to MathML. Do you know if that could happen again after
bug 124128
? Especially, this change allows <wbr> as child of token elements. (BTW, the change in
bug 128907
might also be involved here)
Frédéric Wang (:fredw)
Comment 5
2014-03-18 04:06:57 PDT
(In reply to
comment #2
)
> Is there a crash log seen anywhere? On bots I checked, it looks like a timeout misreported as crash.
This seems to be random:
r165702
has crash log but not
http://build.webkit.org/results/Apple%20MountainLion%20Release%20WK2%20%28Tests%29/r165702%20%2816840%29/mathml/wbr-in-mroot-crash-sample.txt
http://build.webkit.org/results/Apple%20MountainLion%20Release%20WK2%20%28Tests%29/r165703%20%2816841%29/mathml/
(In reply to
comment #4
)
> Does it happen only on Mac?
Apparently it was disabled on EFL too, so I'll try to build that port and see if I can reproduce the crash.
Frédéric Wang (:fredw)
Comment 6
2014-03-18 09:23:10 PDT
(In reply to
comment #5
)
> (In reply to
comment #4
) > > Does it happen only on Mac? > > Apparently it was disabled on EFL too, so I'll try to build that port and see if I can reproduce the crash.
I have not been able to reproduce any crash or hang...
Frédéric Wang (:fredw)
Comment 7
2014-06-17 23:52:46 PDT
Looking at the results for revision
r170050
-
r170094
, I know only see at worse time out. So let's say
bug 119038
improved the situation here, and mark the test as "Timeout". I'm also renaming the bug entry.
Frédéric Wang (:fredw)
Comment 8
2014-06-18 00:46:22 PDT
Committed
r170099
: <
http://trac.webkit.org/changeset/170099
>
Frédéric Wang (:fredw)
Comment 9
2014-06-18 00:47:34 PDT
Reopening, since the timeout is not actually fixed.
Frédéric Wang (:fredw)
Comment 10
2016-06-28 00:49:35 PDT
***
Bug 132267
has been marked as a duplicate of this bug. ***
Frédéric Wang (:fredw)
Comment 11
2016-06-28 01:23:00 PDT
Per
bug 157990 comment 4
and below, the problem actually seemed to be related to the previous test mathml/very-large-stretchy-operators.html. I can not reproduce the timeout on GTK with the command proposed there and the test expectation was removed for mac. So I assumed this has been fixed by the patch for
bug 155434
.
Frédéric Wang (:fredw)
Comment 12
2016-06-28 01:27:29 PDT
Created
attachment 282222
[details]
Patch
Frédéric Wang (:fredw)
Comment 13
2016-06-28 08:06:13 PDT
Committed
r202564
: <
http://trac.webkit.org/changeset/202564
>
Alexey Proskuryakov
Comment 14
2016-06-28 10:05:30 PDT
On Efl, the flakiness was "Crash Pass", which I don't think can be explained by this. But it's not happening any more, according to the flakiness dashboard, so it's fine to remove the expectation.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug