Created attachment 274828[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.10.5
Created attachment 277392[details]
Archive of layout-test-results from ews123 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123 Port: ios-simulator-wk2 Platform: Mac OS X 10.11.4
Created attachment 279010[details]
Archive of layout-test-results from ews122 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews122 Port: ios-simulator-wk2 Platform: Mac OS X 10.11.4
Created attachment 282062[details]
Archive of layout-test-results from ews100 for mac-yosemite
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100 Port: mac-yosemite Platform: Mac OS X 10.10.5
Created attachment 282064[details]
Archive of layout-test-results from ews107 for mac-yosemite-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107 Port: mac-yosemite-wk2 Platform: Mac OS X 10.10.5
Created attachment 282066[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.4
Created attachment 282068[details]
Archive of layout-test-results from ews116 for mac-yosemite
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116 Port: mac-yosemite Platform: Mac OS X 10.10.5
Created attachment 283072[details]
Archive of layout-test-results from ews100 for mac-yosemite
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100 Port: mac-yosemite Platform: Mac OS X 10.10.5
Created attachment 283073[details]
Archive of layout-test-results from ews105 for mac-yosemite-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105 Port: mac-yosemite-wk2 Platform: Mac OS X 10.10.5
Created attachment 283076[details]
Archive of layout-test-results from ews116 for mac-yosemite
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116 Port: mac-yosemite Platform: Mac OS X 10.10.5
Created attachment 283081[details]
Archive of layout-test-results from ews122 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews122 Port: ios-simulator-wk2 Platform: Mac OS X 10.11.5
(In reply to comment #23)
> This change doesn't seem to work. Does it depend on any of your other MathML
> changes to work properly?
The change still works and the basic tests from the MathML in HTML5 test suite pass. The problem was with these two more advanced tests
mathml/presentation/href-enter.html
mathml/presentation/href-style.html
which use our test API to emulate keyboard navigation. They were skipped in the patch from mid May but it seems the change in TestExpectations were lost after later rebasing. Should be fixed now.
Comment on attachment 283303[details]
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=283303&action=review
r=me. I suggest you file bugzillas for the two skip cases so we don't forget to investigate them.
> Source/WebCore/mathml/MathMLElement.cpp:228
> + setNeedsStyleRecalc();
Should you be doing the DNS prefetch logic here, just like we do for HTMLAnchorElement?
> Source/WebCore/mathml/MathMLElement.cpp:335
> + Frame* frame = document().frame();
Maybe better as:
if (Frame* frame = document().frame())
frame->loader()......
return;
> LayoutTests/platform/ios-simulator/TestExpectations:680
> +mathml/presentation/href-style.html [ Skip ]
Is there a bugzilla for this problem?
> LayoutTests/platform/mac/TestExpectations:810
> +mathml/presentation/href-style.html [ Skip ]
Ditto.
Comment on attachment 283303[details]
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=283303&action=review>> Source/WebCore/mathml/MathMLElement.cpp:228
>> + setNeedsStyleRecalc();
>
> Should you be doing the DNS prefetch logic here, just like we do for HTMLAnchorElement?
Maybe someone who is more familiar with the code can comment but I believe we could land the patch without that, given that SVGAElement does not do that stuff either.
(In reply to comment #37)
> Comment on attachment 283303[details]
> Patch
>
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=283303&action=review
>
> >> Source/WebCore/mathml/MathMLElement.cpp:228
> >> + setNeedsStyleRecalc();
> >
> > Should you be doing the DNS prefetch logic here, just like we do for HTMLAnchorElement?
>
> Maybe someone who is more familiar with the code can comment but I believe
> we could land the patch without that, given that SVGAElement does not do
> that stuff either.
It looks like the prefetch code is pretty old, so I guess SVG would have been updated to include it if it was extremely important. Let's just proceed with the code as-is.
2016-03-23 10:16 PDT, Frédéric Wang (:fredw)
2016-03-23 10:18 PDT, Frédéric Wang (:fredw)
2016-03-24 03:30 PDT, Frédéric Wang (:fredw)
2016-03-24 04:23 PDT, Build Bot
2016-04-26 09:07 PDT, Frédéric Wang (:fredw)
2016-04-26 09:59 PDT, Build Bot
2016-05-16 00:54 PDT, Frédéric Wang (:fredw)
2016-05-16 01:58 PDT, Build Bot
2016-05-16 02:07 PDT, Frédéric Wang (:fredw)
2016-06-25 00:41 PDT, Frédéric Wang (:fredw)
2016-06-25 01:28 PDT, Build Bot
2016-06-25 01:30 PDT, Build Bot
2016-06-25 01:37 PDT, Build Bot
2016-06-25 01:40 PDT, Build Bot
2016-07-07 15:02 PDT, Frédéric Wang (:fredw)
2016-07-07 15:51 PDT, Build Bot
2016-07-07 15:55 PDT, Build Bot
2016-07-07 16:01 PDT, Build Bot
2016-07-07 16:14 PDT, Build Bot
2016-07-10 23:56 PDT, Frédéric Wang (:fredw)