96065 2012-09-06 21:50:41 -0700 REGRESSION(127810): It broke inspector/debugger/debugger-expand-scope.html 2012-09-07 14:58:39 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 79666 95659 1 ossy webkit-unassigned barraclough ggaren oliver ossy sam thorton oldest_to_newest 714086 0 ossy 2012-09-06 21:50:41 -0700 Steps of fixing https://bugs.webkit.org/show_bug.cgi?id=95659 : - https://trac.webkit.org/changeset/127698 (with updating expected txt) - https://trac.webkit.org/changeset/127774 (reverting r127698 without expected file) - https://trac.webkit.org/changeset/127808 (reverting the expected file change too) - https://trac.webkit.org/changeset/127810 (fixed patch without expected update) - https://trac.webkit.org/changeset/127814 (update the expected file - reverting 127808) The updated expected file passed on Qt after r127698, but if fails after r127814. So it needed updated with the first patch r127698, but it doesn't with the second patch r127810. Could you check it, please? Now I don't have any idea if it is a progression or a regression. I totally confused. :) It fails on EFL,Qt and Mountail Lion now: (skipped on mac-lion) ---------------------------------------------------------------- --- /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/debugger/debugger-expand-scope-expected.txt +++ /ramdisk/qt-linux-64-release/build/layout-test-results/inspector/debugger/debugger-expand-scope-actual.txt @@ -19,8 +19,6 @@ arguments: Arguments[1] makeClosureLocalVar: "local.TextParam" n: "TextParam" -WindowWith Block - makeClosure: function makeClosure(n) { WindowGlobal <section collapsed> Script execution resumed. 714115 1 ossy 2012-09-06 22:32:50 -0700 I reverted the expected file change to paint the bots green - https://trac.webkit.org/changeset/127827 Please check if it is proper fix or not. 714117 2 thorton 2012-09-06 22:33:22 -0700 Heh, sorry for the noise :\ Hopefully Geoff will know what's really up. 715017 3 ggaren 2012-09-07 14:56:24 -0700 That's the right change. Sorry, I thought Tim was in the process of making that change when I landed. 715021 4 thorton 2012-09-07 14:58:39 -0700 (In reply to comment #3) > That's the right change. > > Sorry, I thought Tim was in the process of making that change when I landed. I did! And then I un-made it when you landed :\