Bug 30815

Summary: [GTK] Failing test http/tests/misc/isindex-with-no-form.html
Product: WebKit Reporter: Alejandro G. Castro <alex>
Component: Tools / TestsAssignee: Alejandro G. Castro <alex>
Status: RESOLVED FIXED    
Severity: Normal CC: svillar, xan.lopez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Proposed patch xan.lopez: review+

Alejandro G. Castro
Reported 2009-10-27 06:21:02 PDT
Apparently the string is not correct and we just have to change it: --- layout-test-results/http/tests/misc/isindex-with-no-form-expected.txt 2009-10-27 03:25:13.558652312 -0700 +++ layout-test-results/http/tests/misc/isindex-with-no-form-actual.txt 2009-10-27 03:25:13.558652312 -0700 @@ -2,6 +2,6 @@ If you are running this test by hand, press the enter/return key on your keyboard to submit. -This is a searchable index. Enter search keywords: +_Searchable Index PASS
Attachments
Proposed patch (1.06 KB, patch)
2010-04-28 04:57 PDT, Alejandro G. Castro 		xan.lopez: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alejandro G. Castro
Comment 1 2009-10-27 06:22:41 PDT
The patch that caused the problem was added in the bug 30491.
Sergio Villar Senin
Comment 2 2010-03-17 10:17:02 PDT
Actually this test is currently working fine
Alejandro G. Castro
Comment 3 2010-04-28 04:35:47 PDT
Yep, it is fixed also for me.
Alejandro G. Castro
Comment 4 2010-04-28 04:42:19 PDT
Reopening because the test is still in the Skipped file and I want to upload a patch to remove it.
Alejandro G. Castro
Comment 5 2010-04-28 04:57:20 PDT
Created attachment 54549 [details] Proposed patch Patch that removes the test from the Skipped file.
Alejandro G. Castro
Comment 6 2010-04-28 11:51:32 PDT
Landed r58417, closing.
Note You need to log in before you can comment on or make changes to this bug.