Eliminate unused url mapping parameter in dartutils

runtime/vm/unit_test.cc

Index: runtime/vm/unit_test.cc
===================================================================
--- runtime/vm/unit_test.cc	(revision 35851)
+++ runtime/vm/unit_test.cc	(working copy)
@@ -81,7 +81,7 @@
     Dart_Handle builtin_lib =
         Builtin::LoadAndCheckLibrary(Builtin::kBuiltinLibrary);
     DART_CHECK_VALID(builtin_lib);
-    return DartUtils::CanonicalizeURL(NULL, library, url_chars);
+    return DartUtils::CanonicalizeURL(library, url_chars);

[Ivan Posva, 2014/05/07 22:04:06]

This is the only surviving call to CanonicalizeURL and it can probably be
morphed into ResolveUri.

[hausner, 2014/05/08 23:10:03]

Eliminated this use of CanonicalizeURL, which required a series of changes to
the library tag handler and unit tests.

   }
   if (is_dart_scheme_url) {
     ASSERT(tag == Dart_kImportTag);
@@ -101,11 +101,7 @@
                            Builtin::PartSource(Builtin::kIOLibrary,
                                                url_chars));
   }
-  return DartUtils::LoadSource(NULL,
-                               library,
-                               url,
-                               tag,
-                               url_chars);
+  return DartUtils::LoadSource(library, url, tag, url_chars);
 }