Title: [202783] trunk/LayoutTests
Revision
202783
Author
[email protected]
Date
2016-07-02 20:32:18 -0700 (Sat, 02 Jul 2016)

Log Message

Web Inspector: Fix inspector/debugger/pause-reason.html
https://bugs.webkit.org/show_bug.cgi?id=159325

Patch by Joseph Pecoraro <[email protected]> on 2016-07-02
Reviewed by Timothy Hatcher.

* platform/mac/TestExpectations:
* inspector/debugger/pause-reason.html:
This test was pausing on inspector code (evaluateInPage) and
auto-resuming. Instead change the test to pause on the page
doing the setTimeout, so we get the expected pause on next
statement evaluation.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (202782 => 202783)


--- trunk/LayoutTests/ChangeLog	2016-07-02 17:52:09 UTC (rev 202782)
+++ trunk/LayoutTests/ChangeLog	2016-07-03 03:32:18 UTC (rev 202783)
@@ -1,3 +1,17 @@
+2016-07-02  Joseph Pecoraro  <[email protected]>
+
+        Web Inspector: Fix inspector/debugger/pause-reason.html
+        https://bugs.webkit.org/show_bug.cgi?id=159325
+
+        Reviewed by Timothy Hatcher.
+
+        * platform/mac/TestExpectations:
+        * inspector/debugger/pause-reason.html:
+        This test was pausing on inspector code (evaluateInPage) and
+        auto-resuming. Instead change the test to pause on the page
+        doing the setTimeout, so we get the expected pause on next
+        statement evaluation.
+
 2016-07-02  Frederic Wang  <[email protected]>
 
         Update EFL expectations for some MathML tests.

Modified: trunk/LayoutTests/inspector/debugger/pause-reason.html (202782 => 202783)


--- trunk/LayoutTests/inspector/debugger/pause-reason.html	2016-07-02 17:52:09 UTC (rev 202782)
+++ trunk/LayoutTests/inspector/debugger/pause-reason.html	2016-07-03 03:32:18 UTC (rev 202783)
@@ -16,7 +16,7 @@
         { _expression_: "setTimeout(triggerDebuggerStatement, 0)" },
         { _expression_: "setTimeout(triggerAssert, 0)" },
         { _expression_: "setTimeout(triggerBreakpoint, 0)" },
-        { _expression_: "setTimeout(function() { 1+1 }, 0)", setup: function() { WebInspector.debuggerManager.pause(); } },
+        { _expression_: "setTimeout(function() { 1+1; }, 50)", setup: function() { WebInspector.debuggerManager.pause(); } },
     ];
 
     function nextTestCase()
@@ -27,10 +27,10 @@
             return;
         }
 
+        InspectorTest.evaluateInPage(test._expression_);
+
         if (test.setup)
             test.setup();
-
-        InspectorTest.evaluateInPage(test._expression_);
     }
 
     function sanitizePauseData(data)

Modified: trunk/LayoutTests/platform/mac/TestExpectations (202782 => 202783)


--- trunk/LayoutTests/platform/mac/TestExpectations	2016-07-02 17:52:09 UTC (rev 202782)
+++ trunk/LayoutTests/platform/mac/TestExpectations	2016-07-03 03:32:18 UTC (rev 202783)
@@ -745,7 +745,6 @@
 inspector/debugger/setBreakpoint-dfg-callee-and-examine-dfg-local.html
 inspector/debugger/didSampleProbe-multiple-probes.html
 inspector/debugger/nested-inspectors.html
-inspector/debugger/pause-reason.html
 
 webkit.org/b/124311 compositing/regions/transform-transparent-positioned-video-inside-region.html [ ImageOnlyFailure ]
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to