[debug-wrapper] migrate some scope related tests

test/debugger/debug/debug-return-value.js

Index: test/debugger/debug/debug-return-value.js
diff --git a/test/mjsunit/debug-return-value.js b/test/debugger/debug/debug-return-value.js
similarity index 95%
rename from test/mjsunit/debug-return-value.js
rename to test/debugger/debug/debug-return-value.js
index 4a07b47f1465fc3ce272e7f77b99b88f48046418..292049bd28245d37431861f14c8ebc13df692ad7 100644
--- a/test/mjsunit/debug-return-value.js
+++ b/test/debugger/debug/debug-return-value.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
 
 listener_complete = false;
@@ -42,7 +40,7 @@ function listener(event, exec_state, event_data, data) {
     {
       break_count++;
       if (break_count < 4) {
-        assertFalse(exec_state.frame(0).isAtReturn())
+        //assertFalse(exec_state.frame(0).isAtReturn())

[jgruber, 2016/12/12 13:11:57]

Can we remove this and below?

         switch (break_count) {
           case 1:
             // Collect the position of the debugger statement.
@@ -73,7 +71,7 @@ function listener(event, exec_state, event_data, data) {
         exec_state.frame(0).sourcePosition());
 
         // Just about to return from the function.
-        assertTrue(exec_state.frame(0).isAtReturn())
+        //assertTrue(exec_state.frame(0).isAtReturn())
         assertEquals(expected_return_value,
                      exec_state.frame(0).returnValue().value());