Bug 106883 - New inspector/editor/text-editor-formatter.html introduced in r139617 fails on EFL, Mac, QT
Summary: New inspector/editor/text-editor-formatter.html introduced in r139617 fails o...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Andrey Lushnikov
URL:
Keywords:
Depends on:
Blocks: 87008 106705
  Show dependency treegraph
 
Reported: 2013-01-15 01:20 PST by Ádám Kallai
Modified: 2014-12-17 17:48 PST (History)
10 users (show)

See Also:


Attachments
Patch (1.42 KB, patch)
2013-01-15 03:03 PST, Andrey Lushnikov
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ádám Kallai 2013-01-15 01:20:09 PST
This test fails: 

Diff is here:

--- /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/editor/text-editor-formatter-expected.txt 
+++ /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/editor/text-editor-formatter-actual.txt 
@@ -24,31 +24,8 @@
         }
     }
 }
+Uncaught exception in inspector front-end: ReferenceError: Can't find variable: textInputController [undefined:448]
 ============ editor contents start ============
-/**
-*.Multi-line.comment
-*
-*/
-function.foo(n).{
-....//.one-line.comment
-....function.bar().{
-........return.42;
-....}
-....
-....var.sum.=.0;
-....for(var.i.=.0;.i.<.n;.++i).{
-........for(var.j.=.0;.j.<.n;.++j).{
-............sum.+=.i.+.j;
-........}
-....}
-....
-....
-....if.(sum.>.1000).{
-........while.(sum.>.0).{
-............--sum;
-........}
-....}
-}
 
 ============ editor contents end ============
Comment 1 Csaba Osztrogonác 2013-01-15 01:34:12 PST
Mac isn't an acronym. ;)
Comment 2 Ádám Kallai 2013-01-15 01:44:20 PST
Ohh, typo. Thanks.
Comment 3 Andrey Lushnikov 2013-01-15 03:03:28 PST
Created attachment 182728 [details]
Patch
Comment 4 Andrey Lushnikov 2013-01-15 03:04:39 PST
It looks like it's already skipped on qt and mac, so I skip it on efl as well. Working on a fix.
Comment 5 Vsevolod Vlasov 2013-01-15 07:59:07 PST
Comment on attachment 182728 [details]
Patch

I have landed it for you.
Comment 6 Brian Burg 2014-12-17 17:48:27 PST
The related test(s) have been removed from trunk.