|Summary:||REGRESSION: affects Blogger, GMail & Wikipedia: empty textarea's which should have content|
|Product:||WebKit||Reporter:||Daniele Metilli <daniele.metilli>|
|Component:||New Bugs||Assignee:||Anders Carlsson <andersca>|
|Severity:||Critical||CC:||jorge, matrix9180, Neil, tonyarnold|
|OS:||OS X 10.4|
Description Daniele Metilli 2005-06-28 18:06:46 PDT
With the last version of WebKit (June 28), if you try to edit some pages on Wikipedia you obtain a blank form (see the linked URL). The form should contain the current revision of the article instead. Some pages, like http://en.wikipedia.org/w/index.php?title=Current_events&action=edit work correctly. This bug was not present in the last official version of WebKit (412).
Comment 1 Joost de Valk (AlthA) 2005-06-29 01:12:30 PDT
Confirmed, marked p1 and major since it's a major site and thus a major problem :P
Comment 2 Daniele Metilli 2005-06-29 12:06:40 PDT
I've noticed another quite strange thing about this bug. The content of the form appears if you do the following: 1 - Click on the background of the page 2 - Reload the page (If you just reload immediately after loading, or if you click on the form then reload this doesn't work) This could be useful if someone wants to edit Wikipedia pages before the bug is fixed.
Comment 3 Tony Arnold 2005-06-29 14:06:24 PDT
This also seems to be affecting Backpack. Clicking "Edit" on any element results in an empty field.
Comment 4 Joost de Valk (AlthA) 2005-06-30 11:23:35 PDT
*** Bug 3772 has been marked as a duplicate of this bug. ***
Comment 5 Joost de Valk (AlthA) 2005-07-01 05:23:42 PDT
*** Bug 3785 has been marked as a duplicate of this bug. ***
Comment 6 Joost de Valk (AlthA) 2005-07-01 05:25:11 PDT
This one get's reported more, please look into it, as it is breaking some major sites.
Comment 7 Joost de Valk (AlthA) 2005-07-04 01:10:42 PDT
Affects Gmail too, changing summary.
Comment 9 Joost de Valk (AlthA) 2005-07-04 01:54:57 PDT
Affects every Mambo CMS generated site too.
Comment 10 Derek Butterfield 2005-07-04 02:38:38 PDT
Affects Gmail when replying to an e-mail. Address box blank: should be pre-filled with recipient.
Comment 11 Jasenko Blazevic 2005-07-04 20:11:11 PDT
Just pulled the latest code (July 5th) and still not fixed. For people thinking of updating their webkit, stay clear of this unless you are not using Gmail, or other JS text area related sites.
Comment 12 Joost de Valk (AlthA) 2005-07-05 00:11:04 PDT
Testcase crashes Safari when you click on the page. This one get's nicer and nicer :)
Comment 13 Chad Ingram 2005-07-05 02:02:15 PDT
terminal always shows /Users/matrix9180/WebCore/khtml/xml/dom_position.cpp:494: failed assertion `currentPos == start' Abort trap when it crashes
Comment 15 mitz 2005-07-05 03:19:03 PDT
(In reply to comment #13) > terminal always shows > /Users/matrix9180/WebCore/khtml/xml/dom_position.cpp:494: failed assertion `currentPos == start' > Abort trap > > when it crashes This assertion failure also shows in bug 3624.
Comment 16 Jasenko Blazevic 2005-07-06 07:31:33 PDT
The attached patch is working in GMail. Cheers guys.
Comment 17 Tony Arnold 2005-07-06 13:13:22 PDT
I can also confirm that the patch works with Gmail, and (thankfully) Backpack too.
Comment 18 Anders Carlsson 2005-07-08 15:20:29 PDT
Created attachment 2870 [details] Address comments
Comment 19 Anders Carlsson 2005-07-08 15:45:11 PDT
Comment on attachment 2870 [details] Address comments This breaks some of the layout tests.
Comment 20 Anders Carlsson 2005-07-08 16:46:29 PDT
Created attachment 2871 [details] Call updateValue We need to call updateValue, otherwise we'll have the same problem when setValue calls updateFromElement
Comment 21 Maciej Stachowiak 2005-07-08 18:08:50 PDT
Comment on attachment 2871 [details] Call updateValue r=me
Comment 22 Neil Lee 2005-07-09 07:42:45 PDT
Don't know if the patch was pulled, but the 0708 and 0709 builds are regressing back to the old, broken behaviour.
Comment 23 Tony Arnold 2005-07-10 17:11:10 PDT
Neil, it was never applied - AFAIK the patch is still in review, and hasn't been merged with ToT. Those of us reporting success have applied the patch manually.
Comment 24 Tony Arnold 2005-07-19 20:54:35 PDT
Why has this been marked as fixed? A build I pulled and compiled 5 minutes ago still has this issue.