Bug 92944

Summary: Use testRunner instead of layoutTestController in fast/notifications, overflow, parser, performance, preloader, profiler, reflections, regex, regions, repaint, ruby and runin tests
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: New BugsAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: tkent, tony
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 88210    
Bug Blocks:    
Attachments:
Description Flags
Patch tkent: review+

Description Ryosuke Niwa 2012-08-01 22:31:29 PDT
Use testRunner instead of layoutTestController in fast/notifications, overflow, parser, performance, preloader, profiler, reflections, regex, regions, repaint, ruby and runin tests
Comment 1 Ryosuke Niwa 2012-08-01 22:35:18 PDT
Created attachment 155974 [details]
Patch
Comment 2 Kent Tamura 2012-08-01 22:43:34 PDT
Comment on attachment 155974 [details]
Patch

> Index: LayoutTests/fast/parser/html-whitespace.html
> ===================================================================
> Cannot display: file marked as a binary type.
> svn:mime-type = application/binary

What's this?
Comment 3 Ryosuke Niwa 2012-08-01 22:44:23 PDT
(In reply to comment #2)
> (From update of attachment 155974 [details])
> > Index: LayoutTests/fast/parser/html-whitespace.html
> > ===================================================================
> > Cannot display: file marked as a binary type.
> > svn:mime-type = application/binary
> 
> What's this?

Apparently that file is a binary file :( It contains some exotic characters in them.
Comment 4 Kent Tamura 2012-08-01 22:46:33 PDT
(In reply to comment #3)
> (In reply to comment #2)
> > (From update of attachment 155974 [details] [details])
> > > Index: LayoutTests/fast/parser/html-whitespace.html
> > > ===================================================================
> > > Cannot display: file marked as a binary type.
> > > svn:mime-type = application/binary
> > 
> > What's this?
> 
> Apparently that file is a binary file :( It contains some exotic characters in them.

Oh, I see.  Ok, I believe your changed it correctly :-P
Comment 5 Kent Tamura 2012-08-01 22:46:46 PDT
Comment on attachment 155974 [details]
Patch

rubber stamped
Comment 6 Ryosuke Niwa 2012-08-01 22:49:26 PDT
Thanks.
Comment 7 Ryosuke Niwa 2012-08-01 22:53:55 PDT
Committed r124410: <http://trac.webkit.org/changeset/124410>