Bug 106883 – New inspector/editor/text-editor-formatter.html introduced in r139617 fails on EFL, Mac, QT

• Component: Tools / Tests
• Status: RESOLVED INVALID

[Ádám Kallai]

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 ============

[Csaba Osztrogonác]

Mac isn't an acronym. ;)

[Ádám Kallai]

Ohh, typo. Thanks.

[Andrey Lushnikov]

Created attachment 182728 [details]
Patch

[Andrey Lushnikov]

It looks like it's already skipped on qt and mac, so I skip it on efl as well. Working on a fix.

[Vsevolod Vlasov]

Comment on attachment 182728 [details]
Patch

I have landed it for you.

[Brian Burg]

The related test(s) have been removed from trunk.