Bug 63372

Summary: convert editing/deleting/whitespace-pre-1.html to dumpAsText
Product: WebKit Reporter: Wyatt Carss <wcarss>
Component: HTML EditingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Enhancement CC: jparent, ojan, rniwa, tony, wcarss, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch none

Description Wyatt Carss 2011-06-24 18:09:23 PDT
convert editing/deleting/whitespace-pre-1.html to dumpAsText
Comment 1 Wyatt Carss 2011-06-24 18:12:08 PDT
Created attachment 98570 [details]
Patch
Comment 2 Ryosuke Niwa 2011-06-24 18:53:28 PDT
Comment on attachment 98570 [details]
Patch

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

> LayoutTests/editing/deleting/whitespace-pre-1.html:4
>  if (window.layoutTestController)
>       layoutTestController.dumpEditingCallbacks();
> +     layoutTestController.dumpAsText();

You need curly brackets around these two statements.  Otherwise, we'll get script errors when we run it manually on a browser.  Also I'd use dump-as-markup.js to see the markup.
Comment 3 Wyatt Carss 2011-06-27 12:11:24 PDT
Created attachment 98764 [details]
Patch
Comment 4 WebKit Review Bot 2011-06-27 14:38:51 PDT
Comment on attachment 98764 [details]
Patch

Clearing flags on attachment: 98764

Committed r89862: <http://trac.webkit.org/changeset/89862>
Comment 5 WebKit Review Bot 2011-06-27 14:38:55 PDT
All reviewed patches have been landed.  Closing bug.