|Summary:||[Mac] [WK2] inspector/device-orientation-success.html, added by r124484, is failing|
|Product:||WebKit||Reporter:||Dean Jackson <dino>|
|Component:||Web Inspector (Deprecated)||Assignee:||Nobody <webkit-unassigned>|
|Severity:||Normal||CC:||apavlov, bweinstein, dino, joepeck, keishi, kpiascik, loislo, pfeldman, pmuellr, rik, rniwa, yurys|
|Version:||528+ (Nightly build)|
|Bug Depends on:||91008|
Description Dean Jackson 2012-08-08 16:53:36 PDT
http://trac.webkit.org/changeset/124484 added inspector/device-orientation-success.html which is failing on Mac ports. e.g. http://build.webkit.org/results/Apple%20Lion%20Release%20WK1%20(Tests)/r125103%20(2133)/results.html --- /Volumes/Data/slave/lion-release-tests-wk1/build/layout-test-results/inspector/device-orientation-success-expected.txt +++ /Volumes/Data/slave/lion-release-tests-wk1/build/layout-test-results/inspector/device-orientation-success-actual.txt @@ -1,3 +1,1 @@ -CONSOLE MESSAGE: line 17: alpha: 20 beta: 30 gamma: 40 -CONSOLE MESSAGE: line 17: alpha: 1.1 beta: 2.2 gamma: 3.3
Comment 1 Dean Jackson 2012-08-08 17:02:54 PDT
Test expectations updated: http://trac.webkit.org/changeset/125119
Comment 2 Pavel Feldman 2012-08-09 23:35:43 PDT
@kpiascik: do you want to look into it?
Comment 3 Konrad Piascik 2012-08-14 09:51:06 PDT
Are inspector layout tests on mac-lion supported? I see a bunch of timed out tests for inspector with this results set. Also I want to double check whether this is for safari or chromium?
Comment 4 Pavel Feldman 2012-08-14 11:15:15 PDT
(In reply to comment #3) > Are inspector layout tests on mac-lion supported? Yes > I see a bunch of timed out tests for inspector with this results set. Also I want to double check whether this is for safari or chromium? It is Safari
Comment 5 Konrad Piascik 2012-08-14 13:15:01 PDT
navigating to http://www.html5rocks.com/en/tutorials/device/orientation/deviceorientationsample.html in safari says that device orientation is not supported. Since this is not supported the test should be skipped on mac. Apologies for not doing this in the original patch. Unless there are objections we can close this as invalid and update the TestExpectations file with a proper comment.
Comment 6 Ryosuke Niwa 2013-01-03 13:34:29 PST
This test is now only failing on WK2. Updated the test expectations accordingly in http://trac.webkit.org/changeset/138734.