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