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

Unified Diff: test/mjsunit/debug-evaluate-locals-optimized.js

Issue 10234007: Remove more assumptions from debug tests. Even though a function (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: Created 8 years, 8 months 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
Index: test/mjsunit/debug-evaluate-locals-optimized.js
===================================================================
--- test/mjsunit/debug-evaluate-locals-optimized.js (revision 11449)
+++ test/mjsunit/debug-evaluate-locals-optimized.js (working copy)
@@ -46,11 +46,6 @@
return arr.reduce(function (a, b) { return a + b; }, 0);
}
-function isCurrentlyOptimized(fun) {
- // See runtime.cc.
- return (%GetOptimizationStatus(fun) & 1) != 0;
-}
-
function listener(event, exec_state, event_data, data) {
try {
if (event == Debug.DebugEvent.Break)
@@ -149,16 +144,6 @@
}
}
- // When function f is optimized we expect an optimized frame for f. We
- // can't say whether or not the top 3 frames (g1, g2 and g3) are
- // optimized and inlined.
- var frame4 = exec_state.frame(4);
-
- if (isCurrentlyOptimized(f)) {
- assertTrue(frame4.isOptimizedFrame());
- assertFalse(frame4.isInlinedFrame());
- }
-
// Indicate that all was processed.
listenerComplete = true;
}

Powered by Google App Engine
This is Rietveld 408576698