CLOSED DUPLICATE of bug 92653 90515
skip file-URL-with-port-number.html Layout Test on android 
https://bugs.webkit.org/show_bug.cgi?id=90515
Summary skip file-URL-with-port-number.html Layout Test on android
Wei James (wistoch)
Reported 2012-07-03 23:17:24 PDT
skip file-URL-with-port-number.html Layout Test on android
Attachments
Patch (1.38 KB, patch)
2012-07-03 23:17 PDT, Wei James (wistoch) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Wei James (wistoch)
Comment 1 2012-07-03 23:17:44 PDT
Created attachment 150716 [details] Patch
Wei James (wistoch)
Comment 2 2012-07-03 23:20:06 PDT
as mentioned at http://code.google.com/p/chromium/issues/detail?id=10342 and http://trac.webkit.org/changeset/86622 also skip this test for android.
Laszlo Gombos
Comment 3 2012-08-05 18:26:40 PDT
It seems to make sense to skip this test for Android, however I am wondering what is the best way to manage the TestExpectations file. I would speculate that most of the entries that are marked for LINUX are also applicable to ANDROID as well, not just the entry in the patch. What is the reason that the patch singles out this entry for Android ?
Wei James (wistoch)
Comment 4 2012-08-05 18:41:10 PDT
(In reply to comment #3) > It seems to make sense to skip this test for Android, however I am wondering what is the best way to manage the TestExpectations file. > > I would speculate that most of the entries that are marked for LINUX are also applicable to ANDROID as well, not just the entry in the patch. > > What is the reason that the patch singles out this entry for Android ? I don't know whether there is any plan to clean the expectation file for android. I met this issue so point it out. If there is any plan to clean the android expectation file, this patch can be obsoleted. thanks
Xianzhu Wang
Comment 5 2012-08-06 09:16:40 PDT
(In reply to comment #4) > (In reply to comment #3) > > It seems to make sense to skip this test for Android, however I am wondering what is the best way to manage the TestExpectations file. > > > > I would speculate that most of the entries that are marked for LINUX are also applicable to ANDROID as well, not just the entry in the patch. > > > > What is the reason that the patch singles out this entry for Android ? > > I don't know whether there is any plan to clean the expectation file for android. > > I met this issue so point it out. If there is any plan to clean the android expectation file, this patch can be obsoleted. > > thanks Thanks for the change. Our current plan is to mark all LINUX expectations also ANDROID if applicable (bug 92653). So this patch can be a step to the goal.
Wei James (wistoch)
Comment 6 2012-08-08 18:42:37 PDT
*** This bug has been marked as a duplicate of bug 92653 ***
Note You need to log in before you can comment on or make changes to this bug.