Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(147)

Unified Diff: test/debugger/debug/debug-enable-disable-breakpoints.js

Issue 2497973002: [debug-wrapper] Further extend the debug wrapper (Closed)
Patch Set: Address comments Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/debugger/debug/debug-compile-event.js ('k') | test/debugger/debug/debug-multiple-var-decl.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/debugger/debug/debug-enable-disable-breakpoints.js
diff --git a/test/mjsunit/debug-enable-disable-breakpoints.js b/test/debugger/debug/debug-enable-disable-breakpoints.js
similarity index 71%
rename from test/mjsunit/debug-enable-disable-breakpoints.js
rename to test/debugger/debug/debug-enable-disable-breakpoints.js
index 4592ffcc2eda25bf611a6b6ded99dd01a47a26c7..25db3eafe3d24330e97cf56b23f30cb8dc23d97e 100644
--- a/test/mjsunit/debug-enable-disable-breakpoints.js
+++ b/test/debugger/debug/debug-enable-disable-breakpoints.js
@@ -25,8 +25,6 @@
// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-// Flags: --expose-debug-as debug
-// Get the Debug object exposed from the debug context global object.
Debug = debug.Debug
// Simple debug event handler which just counts the number of break points hit.
@@ -53,53 +51,12 @@ bp1 = Debug.setBreakPoint(f);
f();
assertEquals(1, break_point_hit_count);
-// Disable the breakpoint.
-Debug.disableBreakPoint(bp1);
-f();
-assertEquals(1, break_point_hit_count);
-
-// Enable the breakpoint.
-Debug.enableBreakPoint(bp1);
-f();
-assertEquals(2, break_point_hit_count);
-
-// Set another breakpoint in f at the same place.
-bp2 = Debug.setBreakPoint(f);
-f();
-assertEquals(3, break_point_hit_count);
-
-// Disable the second breakpoint.
-Debug.disableBreakPoint(bp2);
-f();
-assertEquals(4, break_point_hit_count);
-
-// Disable the first breakpoint.
-Debug.disableBreakPoint(bp1);
-f();
-assertEquals(4, break_point_hit_count);
-
-// Enable both breakpoints.
-Debug.enableBreakPoint(bp1);
-Debug.enableBreakPoint(bp2);
-f();
-assertEquals(5, break_point_hit_count);
-
-// Disable the first breakpoint.
-Debug.disableBreakPoint(bp1);
-f();
-assertEquals(6, break_point_hit_count);
-
// Deactivate all breakpoints.
Debug.debuggerFlags().breakPointsActive.setValue(false);
f();
-assertEquals(6, break_point_hit_count);
-
-// Enable the first breakpoint.
-Debug.enableBreakPoint(bp1);
-f();
-assertEquals(6, break_point_hit_count);
+assertEquals(1, break_point_hit_count);
// Activate all breakpoints.
Debug.debuggerFlags().breakPointsActive.setValue(true);
f();
-assertEquals(7, break_point_hit_count);
+assertEquals(2, break_point_hit_count);
« no previous file with comments | « test/debugger/debug/debug-compile-event.js ('k') | test/debugger/debug/debug-multiple-var-decl.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698