Bug 47350 - REGRESSION: WebKit hangs when editing page in Atlassian Confluence
Summary: REGRESSION: WebKit hangs when editing page in Atlassian Confluence
Status: CLOSED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac (Intel) OS X 10.6
: P1 Major
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-07 07:18 PDT by Dimitris Apostolou
Modified: 2010-10-09 12:31 PDT (History)
3 users (show)

See Also:


Attachments
Activity Monitor sample. (95.59 KB, text/plain)
2010-10-07 07:18 PDT, Dimitris Apostolou
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitris Apostolou 2010-10-07 07:18:44 PDT
Created attachment 70083 [details]
Activity Monitor sample.

Safari 5.0.2 (6533.18.5, r69221)
Atlassian Confluence 3.1.2

Reproducibility: always

Steps:
Edit a page in Confluence wiki.

What happened:
WebKit hangs and beachballs forever until you force quit.

Expected result:
WebKit does not hang.

Notes:
Does not occur in Safari 5.0.2 (6533.18.5)
Comment 1 Alexey Proskuryakov 2010-10-07 11:44:47 PDT
Looks hanging in HTMLDocumentParser::pumpTokenizer().

But I cannot reproduce this on <http://www.atlassian.com/software/confluence/try.jsp> with r69221. Can you provide steps to reproduce that we could follow?
Comment 2 Dimitris Apostolou 2010-10-07 11:53:54 PDT
I tried this page too and it's OK. But this is Confluence 3.3

My report was for Confluence 3.1.2

There are no other steps. Whatever seems to work for 3.3, hangs for 3.1.2
Comment 3 Dimitris Apostolou 2010-10-07 12:02:07 PDT
If you have any tips to try things to get more info, I'd be happy to follow :)

Unfortunately the wiki instance I'm using is https and needs authorization as it belongs to private company. Otherwise I would just provide the exact link.
Comment 4 Alexey Proskuryakov 2010-10-07 12:05:54 PDT
Could you try to isolate with which nightly this started? If we knew what the very first one with the problem was, that would help a lot.
Comment 5 Dimitris Apostolou 2010-10-07 13:04:54 PDT
Regression was introduced with .r68242
Comment 6 Adam Barth 2010-10-07 13:11:35 PDT
1104 WebCore::HTMLElementStack::inButtonScope(WTF::AtomicString const&) const

That number is super big.  I wonder if we're running into depth problems?  I'm not sure whether we've implemented a depth limit for the new parser.
Comment 7 Alexey Proskuryakov 2010-10-07 14:05:13 PDT
Hmm. I don't see any parser changes in r68204-r68242. There are many JavaScriptCore changes, but that's about it.
Comment 8 Dimitris Apostolou 2010-10-09 02:54:52 PDT
Verified fixed with r69429
Comment 9 Dimitris Apostolou 2010-10-09 02:55:07 PDT
Closing.