Bug 58791

Summary: REGRESSION: [Chromium Win] details-add-summary-1.html start crashing after r84039
Product: WebKit Reporter: Hajime Morrita <morrita>
Component: DOMAssignee: Hajime Morrita <morrita>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dglazkov, eric, jknotten, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Patch eric: review+, eric: commit-queue+

Attachments
Patch (1.42 KB, patch)
2011-04-18 09:29 PDT, Hajime Morrita 		eric: review+
eric: commit-queue+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Hajime Morrita
Comment 1 2011-04-18 09:29:29 PDT
Created attachment 90044 [details] Patch
John Knottenbelt
Comment 2 2011-04-18 09:32:59 PDT
LGTM
Eric Seidel (no email)
Comment 3 2011-04-18 09:39:02 PDT
Comment on attachment 90044 [details] Patch I take it the chromium is the only port with <details> enabled?
Hajime Morrita
Comment 4 2011-04-18 09:46:47 PDT
> (From update of attachment 90044 [details]) > I take it the chromium is the only port with <details> enabled? No, there is no ifdefs for <details>. But the crash is reproduced only there.
Hajime Morrita
Comment 5 2011-04-18 09:47:40 PDT
Anyway, thank you for reviewing Eric! I'm landing this.
Hajime Morrita
Comment 6 2011-04-18 09:49:11 PDT
Committed r84152: <http://trac.webkit.org/changeset/84152>
WebKit Review Bot
Comment 7 2011-04-18 12:20:55 PDT
http://trac.webkit.org/changeset/84152 might have broken GTK Linux 32-bit Debug
Note You need to log in before you can comment on or make changes to this bug.