Bug 30815 - [GTK] Failing test http/tests/misc/isindex-with-no-form.html
Summary: [GTK] Failing test http/tests/misc/isindex-with-no-form.html
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Alejandro G. Castro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-10-27 06:21 PDT by Alejandro G. Castro
Modified: 2010-04-28 11:51 PDT (History)
2 users (show)

See Also:

Attachments
Proposed patch (1.06 KB, patch)
2010-04-28 04:57 PDT, Alejandro G. Castro 		xan.lopez: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alejandro G. Castro 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
Comment 1 Alejandro G. Castro 2009-10-27 06:22:41 PDT 
The patch that caused the problem was added in the bug 30491.
Comment 2 Sergio Villar Senin 2010-03-17 10:17:02 PDT 
Actually this test is currently working fine
Comment 3 Alejandro G. Castro 2010-04-28 04:35:47 PDT 
Yep, it is fixed also for me.
Comment 4 Alejandro G. Castro 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.
Comment 5 Alejandro G. Castro 2010-04-28 04:57:20 PDT 
Created attachment 54549 [details]
Proposed patch

Patch that removes the test from the Skipped file.
Comment 6 Alejandro G. Castro 2010-04-28 11:51:32 PDT 
Landed r58417, closing.