106883 2013-01-15 01:20:09 -0800 New inspector/editor/text-editor-formatter.html introduced in r139617 fails on EFL, Mac, QT 2014-12-17 17:48:27 -0800 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED INVALID P2 Normal --- 87008 106705 1 kadam lushnikov d-r gyuyoung.kim kkristof lushnikov ossy pfeldman rakuco szledan webkit.review.bot zan oldest_to_newest 807097 0 kadam 2013-01-15 01:20:09 -0800 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 ============ 807110 1 ossy 2013-01-15 01:34:12 -0800 Mac isn't an acronym. ;) 807118 2 kadam 2013-01-15 01:44:20 -0800 Ohh, typo. Thanks. 807155 3 182728 lushnikov 2013-01-15 03:03:28 -0800 Created attachment 182728 Patch 807156 4 lushnikov 2013-01-15 03:04:39 -0800 It looks like it's already skipped on qt and mac, so I skip it on efl as well. Working on a fix. 807306 5 182728 vsevik 2013-01-15 07:59:07 -0800 Comment on attachment 182728 Patch I have landed it for you. 1056671 6 burg 2014-12-17 17:48:27 -0800 The related test(s) have been removed from trunk. 182728 2013-01-15 03:03:28 -0800 2013-01-15 07:59:19 -0800 Patch bug-106883-20130115150027.patch text/plain 1449 lushnikov U3VidmVyc2lvbiBSZXZpc2lvbjogMTM5NzMyCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggOTg1NGIyNzhhOThmYTdmMDc0Zjlh YzkxMTc4MTZhZDk2MjlhODk4Ny4uNWRhMzc5NzAxYjRlOThmOTlkNjYxODNkNWFjN2IxNTRmMTFj OTM4MiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE0IEBACisyMDEzLTAxLTE1ICBBbmRyZXkgTHVzaG5pa292 ICA8bHVzaG5pa292QGNocm9taXVtLm9yZz4KKworICAgICAgICBOZXcgaW5zcGVjdG9yL2VkaXRv ci90ZXh0LWVkaXRvci1mb3JtYXR0ZXIuaHRtbCBpbnRyb2R1Y2VkIGluIHIxMzk2MTcgZmFpbHMg b24gRUZMLCBNYWMsIFFUCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn LmNnaT9pZD0xMDY4ODMKKworICAgICAgICBTa2lwIGZhaWxpbmcgaW5zcGVjdG9yL2VkaXRvci90 ZXh0LWVkaXRvci1mb3JtYXR0ZXIuaHRtbCB0ZXN0LgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P Qk9EWSAoT09QUyEpLgorCisgICAgICAgICogcGxhdGZvcm0vZWZsL1Rlc3RFeHBlY3RhdGlvbnM6 CisKIDIwMTMtMDEtMTUgIFRvbW15IFdpZGVuZmx5Y2h0ICA8dG9tbXl3QGdvb2dsZS5jb20+CiAK ICAgICAgICAgTWVkaWFTdHJlYW0gQVBJOiBGaXhpbmcgY3Jhc2hpbmcgYnVnIGluIE1lZGlhU3Ry ZWFtCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9lZmwvVGVzdEV4cGVjdGF0aW9u cyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2VmbC9UZXN0RXhwZWN0YXRpb25zCmluZGV4IGQ2MTZm ZGUwMjM3ZGY0YzgzNjg0NzFmZTY4OTU0Mjk3OTgyZjI0MGEuLmJiNjNjMWQxOGJlOWFiYjliMGZh ODZkODg2NDc4MDY4ZjE3N2VhMTMgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL3BsYXRmb3JtL2Vm bC9UZXN0RXhwZWN0YXRpb25zCisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2VmbC9UZXN0RXhw ZWN0YXRpb25zCkBAIC0xODA2LDMgKzE4MDYsNiBAQCB3ZWJraXQub3JnL2IvMTAzOTU1IGZhc3Qv cmVwYWludC9jYXJldC13aXRoLXRyYW5zZm9ybWF0aW9uLmh0bWwgWyBNaXNzaW5nIF0KICMgU3Rh cnRlZCBmYWlsaW5nIGR1ZSB0byBuZXcgY2hlY2tzIGFkZGVkIGluIHIxMzk1NDcuCiB3ZWJraXQu b3JnL2IvMTA2NzQzIG1lZGlhL3ZpZGVvLWNvbnRyb2xzLWNhcHRpb25zLmh0bWwgWyBGYWlsdXJl IF0KIAorIyBOZXcgaW5zcGVjdG9yL2VkaXRvci90ZXh0LWVkaXRvci1mb3JtYXR0ZXIuaHRtbCBp bnRyb2R1Y2VkIGluIHIxMzk2MTcgZmFpbHMgb24gRUZMLCBNYWMsIFFUCisjIGh0dHBzOi8vYnVn cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDY4ODMKK2luc3BlY3Rvci9lZGl0b3IvdGV4 dC1lZGl0b3ItZm9ybWF0dGVyLmh0bWwK