Changeset 206063 in webkit

Timestamp:

Sep 16, 2016 8:53:29 PM (8 years ago)

Author:

Joseph Pecoraro

Message:

Unreviewed cleanup of some inspector tests.

• TestExpectations:

Start skipping some debugger stepping test flakeyness on
Debug builds until that bug is addressed.

• inspector/debugger/break-on-exception-throw-in-promise.html:

Remove debug only.

• inspector/debugger/paused-scopes.html: Added.

This test was missing for 3 months. Its expectations got added
but the test itself got lost after a rollout and re-land.

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• TestExpectations (modified) (1 diff)
• inspector/debugger/break-on-exception-throw-in-promise.html (modified) (1 diff)
• inspector/debugger/paused-scopes.html (added)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2016-09-16  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Unreviewed cleanup of some inspector tests.
+
+        * TestExpectations:
+        Start skipping some debugger stepping test flakeyness on
+        Debug builds until that bug is addressed.
+
+        * inspector/debugger/break-on-exception-throw-in-promise.html:
+        Remove debug only.
+
+        * inspector/debugger/paused-scopes.html: Added.
+        This test was missing for 3 months. Its expectations got added
+        but the test itself got lost after a rollout and re-land.
+
 2016-09-16  Chris Dumez  <cdumez@apple.com>
 

trunk/LayoutTests/TestExpectations

@@ -144 +144 @@
 # This test is fast enough in release but quite slow in debug builds.
 [ Debug ] inspector/debugger/debugger-stack-overflow.html [ Skip ]
+
+# Debugger stepping tests can timeout if they run slowly due to a short timer scheduled in the frontend.
+webkit.org/b/161951 [ Debug ] inspector/debugger/paused-scopes.html [ Skip ]
 
 # Doesn't work yet, relies on network replay functionality (webkit.org/b/130728, webkit.org/b/129391)

trunk/LayoutTests/inspector/debugger/break-on-exception-throw-in-promise.html

@@ -12 +12 @@
 
     let suite = InspectorTest.createAsyncSuite("BreakOnAnyException.Promise");
-
-    InspectorTest.debug();
 
     function addTestCase({name, description, expression}) {