Bug 59003

Summary: Update fast/canvas/font-update.html to use dumpAsText
Product: WebKit Reporter: Matthew Delaney <mdelaney7>
Component: New BugsAssignee: Matthew Delaney <mdelaney7>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch mitz: review+

Description Matthew Delaney 2011-04-20 09:39:14 PDT
Update fast/canvas/font-update.html to use dumpAsText
Comment 1 Matthew Delaney 2011-04-20 09:40:09 PDT
Created attachment 90350 [details]
Patch
Comment 2 Adam Roben (:aroben) 2011-04-20 09:42:48 PDT
Comment on attachment 90350 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=90350&action=review

> LayoutTests/ChangeLog:9
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Update fast/canvas/font-update.html to use dumpAsText
> +        https://bugs.webkit.org/show_bug.cgi?id=59003
> +
> +        * fast/canvas/font-update-expected.txt:
> +        * fast/canvas/font-update.html:

Your ChangeLog should explain why this is an OK thing to do. Explaining the purpose of the test will probably be part of that. (E.g., if the test is just detecting a crash, then dumpAsText is probably fine.)
Comment 3 Matthew Delaney 2011-04-20 10:07:02 PDT
Committed r84390: <http://trac.webkit.org/changeset/84390>