Bug 61681

Summary: editing/style/non-inheritable-styles should be dump-as-markup test
Product: WebKit Reporter: Annie Sullivan <sullivan>
Component: HTML EditingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 34564    
Attachments:
Description Flags
Patch
none
Patch none

Description Annie Sullivan 2011-05-27 19:25:56 PDT
The output would be easier to read as a dump-as-markup test.
Comment 1 Annie Sullivan 2011-05-27 19:28:40 PDT
Created attachment 95247 [details]
Patch
Comment 2 Ryosuke Niwa 2011-05-27 19:32:55 PDT
Comment on attachment 95247 [details]
Patch

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

> LayoutTests/editing/style/non-inheritable-styles.html:40
> +Markup.dump(document.body);

I think you want to print #paste instead.
Comment 3 Annie Sullivan 2011-05-27 19:36:37 PDT
Comment on attachment 95247 [details]
Patch

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

>> LayoutTests/editing/style/non-inheritable-styles.html:40
>> +Markup.dump(document.body);
> 
> I think you want to print #paste instead.

Yeah, that's a lot clearer. New patch on the way.
Comment 4 Annie Sullivan 2011-05-27 19:43:47 PDT
Created attachment 95249 [details]
Patch
Comment 5 WebKit Commit Bot 2011-05-28 00:18:13 PDT
Comment on attachment 95249 [details]
Patch

Clearing flags on attachment: 95249

Committed r87606: <http://trac.webkit.org/changeset/87606>
Comment 6 WebKit Commit Bot 2011-05-28 00:18:18 PDT
All reviewed patches have been landed.  Closing bug.