Bug 96065

Summary: REGRESSION(127810): It broke inspector/debugger/debugger-expand-scope.html
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, ggaren, oliver, ossy, sam, thorton
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 79666, 95659    

Description Csaba Osztrogonác 2012-09-06 21:50:41 PDT
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.
Comment 1 Csaba Osztrogonác 2012-09-06 22:32:50 PDT
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.
Comment 2 Tim Horton 2012-09-06 22:33:22 PDT
Heh, sorry for the noise :\ Hopefully Geoff will know what's really up.
Comment 3 Geoffrey Garen 2012-09-07 14:56:24 PDT
That's the right change.

Sorry, I thought Tim was in the process of making that change when I landed.
Comment 4 Tim Horton 2012-09-07 14:58:39 PDT
(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 :\