Bug 20408

Summary:

REGRESSION: Crash loading WebKit blog post on HTML media elements

Product:

WebKit

Reporter:

Simon Fraser (smfr) <simon.fraser>

Component:

Layout and Rendering

Assignee:

Simon Fraser (smfr) <simon.fraser>

Status:

RESOLVED FIXED

Severity:

Major

CC:

dino

Priority:

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

URL:

http://webkit.org/blog/140/html5-media-support/

Attachments:

Description	Flags
Testcase	none
Patch, testcase, changelog	hyatt: review+

Simon Fraser (smfr)

Reported 2008-08-15 16:46:52 PDT

WebKit builds currently crash when loading the URL in the bug. This seems to be transition-related.

Attachments
Testcase (169 bytes, text/html) 2008-08-15 16:57 PDT, Simon Fraser (smfr)	no flags	Details
Patch, testcase, changelog (3.46 KB, patch) 2008-08-15 17:55 PDT, Simon Fraser (smfr)	hyatt: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Simon Fraser (smfr)

Comment 1 2008-08-15 16:57:14 PDT

Created attachment 22828 [details] Testcase

Dave Hyatt

Comment 2 2008-08-15 17:02:51 PDT

Is this a regression?

Simon Fraser (smfr)

Comment 3 2008-08-15 17:55:55 PDT

Created attachment 22830 [details] Patch, testcase, changelog Fixed the ugly macros to make sure there's an animation in the list. Is this if (isInherit) ... else if (isInitial) change OK?

Mark Rowe (bdash)

Comment 4 2008-08-15 18:48:14 PDT

Regressions are P1s.

Dave Hyatt

Comment 5 2008-08-18 13:48:46 PDT

Comment on attachment 22830 [details] Patch, testcase, changelog r=me

Simon Fraser (smfr)

Comment 6 2008-08-18 15:11:27 PDT

Committed r35825 M WebCore/ChangeLog M WebCore/css/CSSStyleSelector.cpp A LayoutTests/platform/mac/fast/css/transition_shorthand_parsing-expected.txt M LayoutTests/ChangeLog A LayoutTests/fast/css/transition_shorthand_parsing.html r35825 = 6040db395a99614c25b06ba9ea4fc7dfec42f1ea (trunk)

Note You need to log in before you can comment on or make changes to this bug.