vm/observatory: Clean up script access

runtime/vm/service.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/service.h"
 
 #include "include/dart_api.h"
 #include "platform/globals.h"
 
 #include "vm/compiler.h"
@@ skipping 137 matching lines @@
   Dart_ServiceRequestCallback callback_;
   void* user_data_;
   EmbedderServiceHandler* next_;
 };
 
 
 class LibraryCoverageFilter : public CoverageFilter {
  public:
   explicit LibraryCoverageFilter(const Library& lib) : lib_(lib) {}
   bool ShouldOutputCoverageFor(const Library& lib,
-                               const String& script_url,
+                               const Script& script,
                                const Class& cls,
                                const Function& func) const {
     return lib.raw() == lib_.raw();
   }
  private:
   const Library& lib_;
 };
 
 
 class ScriptCoverageFilter : public CoverageFilter {
  public:
-  explicit ScriptCoverageFilter(const String& script_url)
-      : script_url_(script_url) {}
+  explicit ScriptCoverageFilter(const Script& script)
+      : script_(script) {}
   bool ShouldOutputCoverageFor(const Library& lib,
-                               const String& script_url,
+                               const Script& script,
                                const Class& cls,
                                const Function& func) const {
-    return script_url_.Equals(script_url);
+    return script.raw() == script_.raw();
   }
  private:
-  const String& script_url_;
+  const Script& script_;
 };
 
 
 class ClassCoverageFilter : public CoverageFilter {
  public:
   explicit ClassCoverageFilter(const Class& cls) : cls_(cls) {}
   bool ShouldOutputCoverageFor(const Library& lib,
-                               const String& script_url,
+                               const Script& script,
                                const Class& cls,
                                const Function& func) const {
     return cls.raw() == cls_.raw();
   }
  private:
   const Class& cls_;
 };
 
 
 class FunctionCoverageFilter : public CoverageFilter {
  public:
   explicit FunctionCoverageFilter(const Function& func) : func_(func) {}
   bool ShouldOutputCoverageFor(const Library& lib,
-                               const String& script_url,
+                               const Script& script,
                                const Class& cls,
                                const Function& func) const {
     return func.raw() == func_.raw();
   }
  private:
   const Function& func_;
 };
 
 
 static uint8_t* allocator(uint8_t* ptr, intptr_t old_size, intptr_t new_size) {
@@ skipping 1189 matching lines @@
   }
   const String& expr_str = String::Handle(isolate, String::New(expr));
   const Object& result = Object::Handle(lib.Evaluate(expr_str,
                                                      Array::empty_array(),
                                                      Array::empty_array()));
   result.PrintJSON(js, true);
   return true;
 }
 
 
+static bool HandleLibrariesScriptsCoverage(
+    Isolate* isolate, const Script& script, JSONStream* js) {
+  ScriptCoverageFilter sf(script);
+  CodeCoverage::PrintJSON(isolate, js, &sf);
+  return true;
+}
+
+
+static bool HandleLibrariesScripts(Isolate* isolate,
+                                   const Library& lib,
+                                   JSONStream* js) {
+  if (js->num_arguments() > 5) {
+    PrintError(js, "Command too long");
+    return true;
+  } else if (js->num_arguments() < 4) {
+    PrintError(js, "Must specify collection object id: scripts/id");
+    return true;
+  }
+  const String& id = String::Handle(String::New(js->GetArgument(3)));
+  ASSERT(!id.IsNull());
+  // The id is the url of the script % encoded, decode it.
+  const String& requested_url = String::Handle(String::DecodeURI(id));
+  Script& script = Script::Handle();
+  String& script_url = String::Handle();
+  const Array& loaded_scripts = Array::Handle(lib.LoadedScripts());
+  ASSERT(!loaded_scripts.IsNull());
+  intptr_t i;
+  for (i = 0; i < loaded_scripts.Length(); i++) {
+    script ^= loaded_scripts.At(i);
+    ASSERT(!script.IsNull());
+    script_url ^= script.url();
+    if (script_url.Equals(requested_url)) {
+      break;
+    }
+  }
+  if (i == loaded_scripts.Length()) {
+    PrintError(js, "Script %s not found", requested_url.ToCString());
+    return true;
+  }
+  if (js->num_arguments() == 4) {
+    script.PrintJSON(js, false);
+    return true;
+  } else {
+    const char* subcollection = js->GetArgument(4);
+    if (strcmp(subcollection, "coverage") == 0) {
+      return HandleLibrariesScriptsCoverage(isolate, script, js);
+    } else {
+      PrintError(js, "Invalid sub collection %s", subcollection);
+      return true;
+    }
+  }
+  UNREACHABLE();
+  return true;
+}
+
+
 static bool HandleLibrariesCoverage(Isolate* isolate,
                                     const Library& lib,
                                     JSONStream* js) {
   LibraryCoverageFilter lf(lib);
   CodeCoverage::PrintJSON(isolate, js, &lf);
   return true;
 }
 
 
 static bool HandleLibraries(Isolate* isolate, JSONStream* js) {
   // TODO(johnmccutchan): Support fields and functions on libraries.
   REQUIRE_COLLECTION_ID("libraries");
   const GrowableObjectArray& libs =
       GrowableObjectArray::Handle(isolate->object_store()->libraries());
   ASSERT(!libs.IsNull());
   intptr_t id = 0;
   CHECK_COLLECTION_ID_BOUNDS("libraries", libs.Length(), js->GetArgument(1),
                              id, js);
   Library& lib = Library::Handle();
   lib ^= libs.At(id);
   ASSERT(!lib.IsNull());
   if (js->num_arguments() == 2) {
     lib.PrintJSON(js, false);
     return true;
   } else if (js->num_arguments() >= 3) {
     const char* second = js->GetArgument(2);
     if (strcmp(second, "eval") == 0) {
       return HandleLibrariesEval(isolate, lib, js);
+    } else if (strcmp(second, "scripts") == 0) {
+      return HandleLibrariesScripts(isolate, lib, js);
     } else if (strcmp(second, "coverage") == 0) {
       return HandleLibrariesCoverage(isolate, lib, js);
     } else {
       PrintError(js, "Invalid sub collection %s", second);
       return true;
     }
   }
   UNREACHABLE();
   return true;
 }
@@ skipping 153 matching lines @@
     intptr_t num_scripts = loaded_scripts.Length();
     for (intptr_t i = 0; i < num_scripts; i++) {
       script ^= loaded_scripts.At(i);
       members.AddValue(script);
     }
   }
   return true;
 }
 
 
-static bool HandleScriptsFetch(
-    Isolate* isolate, const Script& script, JSONStream* js) {
-  script.PrintJSON(js, false);
-  return true;
-}
-
-
-static bool HandleScriptsCoverage(
-    Isolate* isolate, const String& script_url, JSONStream* js) {
-  ScriptCoverageFilter sf(script_url);
-  CodeCoverage::PrintJSON(isolate, js, &sf);
-  return true;
-}
-
-
 static bool HandleScripts(Isolate* isolate, JSONStream* js) {
   if (js->num_arguments() == 1) {
     // Enumerate all scripts.
     return HandleScriptsEnumerate(isolate, js);
   }
-  // Subcommands of scripts require a valid script id.
-  const String& id = String::Handle(String::New(js->GetArgument(1)));
-  ASSERT(!id.IsNull());
-  // The id is the url of the script % encoded, decode it.
-  String& requested_url = String::Handle(String::DecodeURI(id));
-  Script& script = Script::Handle();
-  // There may exist more than one script object for a given url. Since they all
-  // have the same properties (source, tokens) we don't care which one we get.
-  const GrowableObjectArray& libs = GrowableObjectArray::Handle(
-      isolate, isolate->object_store()->libraries());
-  Library& lib = Library::Handle();
-  String& script_url = String::Handle();
-  bool found = false;
-  for (intptr_t i = 0; !found && (i < libs.Length()); i++) {
-    lib ^= libs.At(i);
-    ASSERT(!lib.IsNull());
-    const Array& loaded_scripts = Array::Handle(lib.LoadedScripts());
-    ASSERT(!loaded_scripts.IsNull());
-    for (intptr_t j = 0; !found && (j < loaded_scripts.Length()); j++) {
-      script ^= loaded_scripts.At(j);
-      ASSERT(!script.IsNull());
-      script_url ^= script.url();
-      if (script_url.Equals(requested_url)) {
-        found = true;
-      }
-    }
-  }
-  if (!found) {
-    PrintErrorWithKind(js, "NotFoundError", "Cannot find script %s",
-                       requested_url.ToCString());
-    return true;
-  }
-  if (js->num_arguments() == 2) {
-    // If no subcommand is given, just fetch the script.
-    return HandleScriptsFetch(isolate, script, js);
-  } else if (js->num_arguments() == 3) {
-    const char* arg = js->GetArgument(2);
-    if (strcmp(arg, "coverage") == 0) {
-      return HandleScriptsCoverage(isolate, requested_url, js);
-    }
-    PrintError(js, "Unrecognized subcommand '%s'", arg);
-    return true;
-  } else {
-    PrintError(js, "Command too long");
-    return true;
-  }
+  PrintError(js, "Command too long");
+  return true;
 }
 
 
 static bool HandleDebugResume(Isolate* isolate, JSONStream* js) {
   if (isolate->message_handler()->paused_on_start()) {
     isolate->message_handler()->set_pause_on_start(false);
     JSONObject jsobj(js);
     jsobj.AddProperty("type", "Success");
     jsobj.AddProperty("id", "");
     return true;
@@ skipping 693 matching lines @@
   while (current != NULL) {
     if (strcmp(name, current->name()) == 0) {
       return current;
     }
     current = current->next();
   }
   return NULL;
 }
 
 }  // namespace dart