Fix tests compilation

plugins/org.chromium.sdk.tests/src/org/chromium/sdk/internal/v8native/DebugContextImplTest.java

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.sdk.internal.v8native;
 
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.fail;
 
 import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
 import java.util.concurrent.CountDownLatch;
 
 import org.chromium.sdk.Breakpoint;
 import org.chromium.sdk.JsObject;
 import org.chromium.sdk.JsScope;
+import org.chromium.sdk.JsScope.Declarative;
+import org.chromium.sdk.JsScope.ObjectBased;
 import org.chromium.sdk.JsVariable;
 import org.chromium.sdk.DebugContext.StepAction;
 import org.chromium.sdk.JavascriptVm.BreakpointCallback;
 import org.chromium.sdk.internal.browserfixture.AbstractAttachedTest;
 import org.chromium.sdk.internal.transport.FakeConnection;
 import org.junit.Test;
 
 /**
  * A test for the DebugContextImpl class.
  */
@@ skipping 32 matching lines @@
 
     {
       CountDownLatch latch = expectSuspend();
       messageResponder.hitBreakpoints(Collections.singleton(bp[0].getId()));
       latch.await();
     }
 
     List<? extends JsScope> variableScopes =
         suspendContext.getCallFrames().get(0).getVariableScopes();
 
-    Collection<? extends JsVariable> variables = variableScopes.get(0).getVariables();
+    Collection<? extends JsVariable> variables = getScopeVariables(variableScopes.get(0));
 
     // This call invalidates the debug context for the "lookup" operation that is invoked
     // inside "ensureProperties".
     suspendContext.continueVm(StepAction.CONTINUE, 1, null);
     JsObject jsObject = variables.iterator().next().getValue().asObject();
     try {
       jsObject.getProperties();
       fail();
     } catch (RuntimeException e) {
       // this exception is expected
@@ skipping 33 matching lines @@
 
     {
       CountDownLatch latch = expectSuspend();
       messageResponder.hitBreakpoints(Collections.singleton(bp[0].getId()));
       latch.await();
     }
 
     List<? extends JsScope> variableScopes =
         suspendContext.getCallFrames().get(0).getVariableScopes();
 
-    Collection<? extends JsVariable> variables = variableScopes.get(0).getVariables();
+    Collection<? extends JsVariable> variables = getScopeVariables(variableScopes.get(0));
 
     JsObject jsObject = variables.iterator().next().getValue().asObject();
     // This call should finish OK
     jsObject.getProperties();
   }
 
   @Override
   protected FakeConnection createConnection() {
     return new FakeConnection(messageResponder);
   }
+
+  private static Collection<? extends JsVariable> getScopeVariables(JsScope scope) {
+    return scope.accept(new JsScope.Visitor<Collection<? extends JsVariable>>() {
+      @Override
+      public Collection<? extends JsVariable> visitDeclarative(Declarative declarativeScope) {
+        return declarativeScope.getVariables();
+      }
+
+      @Override
+      public Collection<? extends JsVariable> visitObject(ObjectBased objectScope) {
+        return objectScope.getScopeObject().getProperties();
+      }
+
+    });
+  }
 }