Bug 3751 – REGRESSION: affects Blogger, GMail & Wikipedia: empty textarea's which should have content

• Component: New Bugs
• Status: RESOLVED FIXED

[Daniele Metilli]

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).

[Joost de Valk (AlthA)]

Confirmed, marked p1 and major since it's a major site and thus a major problem :P

[Daniele Metilli]

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.

[Tony Arnold]

This also seems to be affecting Backpack. Clicking "Edit" on any element results in an empty field.

[Joost de Valk (AlthA)]

*** Bug 3772 has been marked as a duplicate of this bug. ***

[Joost de Valk (AlthA)]

*** Bug 3785 has been marked as a duplicate of this bug. ***

[Joost de Valk (AlthA)]

This one get's reported more, please look into it, as it is breaking some major sites.

[Joost de Valk (AlthA)]

Affects Gmail too, changing summary.

[Chad Ingram]

Created attachment 2782 [details]
Test Case

[Joost de Valk (AlthA)]

Affects every Mambo CMS generated site too.

[Derek Butterfield]

Affects Gmail when replying to an e-mail. Address box blank: should be pre-filled with recipient.

[Jasenko Blazevic]

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.

[Joost de Valk (AlthA)]

Testcase crashes Safari when you click on the page. This one get's nicer and nicer :)

[Chad Ingram]

terminal always shows 
/Users/matrix9180/WebCore/khtml/xml/dom_position.cpp:494: failed assertion `currentPos == start'
Abort trap

when it crashes

[Anders Carlsson]

Created attachment 2803 [details]
Fix bug

(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.

[Jasenko Blazevic]

The attached patch is working in GMail. Cheers guys.

[Tony Arnold]

I can also confirm that the patch works with Gmail, and (thankfully) Backpack too.

[Anders Carlsson]

Created attachment 2870 [details]
Address comments

[Anders Carlsson]

Comment on attachment 2870 [details]
Address comments

This breaks some of the layout tests.

[Anders Carlsson]

Created attachment 2871 [details]
Call updateValue

We need to call updateValue, otherwise we'll have the same problem when
setValue calls updateFromElement

[Maciej Stachowiak]

Comment on attachment 2871 [details]
Call updateValue

r=me

[Neil Lee]

Don't know if the patch was pulled, but the 0708 and 0709 builds are regressing back to the old, broken 
behaviour.

[Tony Arnold]

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.

[Tony Arnold]

Why has this been marked as fixed? A build I pulled and compiled 5 minutes ago still has this issue.