Bugfix in DOM bindings: canonicalize URLs even if no ApplicationLoader is available

DartApplicationLoader.cpp

 // Copyright (c) 2009, Google Inc.
 // All rights reserved.
 //
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 // notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
@@ skipping 68 matching lines @@
 
 void DartApplicationLoader::reportDartError(Dart_Handle error)
 {
     m_scriptHasError = true;
     DartUtilities::reportProblem(m_originDocument, error, m_libraryUrl);
 }
 
 Dart_Handle DartApplicationLoader::libraryTagHandlerCallback(Dart_LibraryTag tag, Dart_Handle library, Dart_Handle urlHandle)
 {
     ASSERT(Dart_CurrentIsolate());
-    ASSERT(DartDOMData::current()->applicationLoader());
-    return DartDOMData::current()->applicationLoader()->libraryTagHandler(tag, library, urlHandle);
-}
-
-Dart_Handle DartApplicationLoader::libraryTagHandler(Dart_LibraryTag tag, Dart_Handle library, Dart_Handle urlHandle)
-{
     ASSERT(Dart_IsLibrary(library));
 
     const String url = DartUtilities::toString(urlHandle);
 
+    if (tag == Dart_kCanonicalizeUrl) {
+        // If a dart application calls spawnUri, the DartVM will call this
+        // libraryTagHandler to canonicalize the url.
+        // DartDOMData::current()->applicationLoader() may be 0 at this point.
+        return DartApplicationLoader::CanonicalizeUrl(library, urlHandle, url);
+    }
+
+    ASSERT(DartDOMData::current()->applicationLoader());
+    return DartDOMData::current()->applicationLoader()->libraryTagHandler(tag, library, urlHandle, url);
+}
+
+Dart_Handle DartApplicationLoader::CanonicalizeUrl(Dart_Handle library, Dart_Handle urlHandle, String url)
+{
+    if (url.startsWith("dart:") || url.startsWith("package:"))
+        return urlHandle;
+
     Dart_Handle libraryURLHandle = Dart_LibraryUrl(library);
     ASSERT(!Dart_IsError(libraryURLHandle));
     String libraryURL = DartUtilities::toString(libraryURLHandle);
 
-    if (tag == Dart_kCanonicalizeUrl) {
-        if (url.startsWith("dart:") || url.startsWith("package:"))
-            return urlHandle;
+    bool packageScheme = false;
 
-        bool packageScheme = false;
-        if (libraryURL.startsWith("package:")) {
-            // KURL have problems concating package:foo/bar (without slashes right after colon)
-            // and relative urls. Therefore pretend to be a standard absolute URL.
-            packageScheme = true;
-            libraryURL = "http://" + libraryURL.substring(8);
-        }
+    const char* kPackagePrefix = "package:";
+    const int kPackagePrefixLength = strlen(kPackagePrefix);
 
-        const KURL canonical = KURL(KURL(KURL(), libraryURL), url);
-        String result = canonical.string();
-        if (packageScheme)
-            result = "package:" + result.substring(7);
-        return DartUtilities::stringToDartString(result);
+    const char* kHttpPrefix = "http://";
+    const int kHttpPrefixLength = strlen(kHttpPrefix);
+
+    if (libraryURL.startsWith(kPackagePrefix)) {
+        // KURL has problems concating package:foo/bar (without slashes right after colon)
+        // and relative urls. Therefore pretend to be a standard absolute URL.
+        packageScheme = true;
+        libraryURL = kHttpPrefix + libraryURL.substring(kPackagePrefixLength);
     }
 
+    const KURL canonical = KURL(KURL(KURL(), libraryURL), url);
+    String result = canonical.string();
+    if (packageScheme)
+        result = kPackagePrefix + result.substring(kHttpPrefixLength);
+    return DartUtilities::stringToDartString(result);
+}
+
+Dart_Handle DartApplicationLoader::libraryTagHandler(Dart_LibraryTag tag, Dart_Handle library, Dart_Handle urlHandle, String url)
+{
     ASSERT(url != "dart:html");
 
     // Record the importer.
     if (tag == Dart_kImportTag)
         m_importedLibraries.add(url);
-    else if (tag == Dart_kSourceTag)
+    else if (tag == Dart_kSourceTag) {
+        Dart_Handle libraryURLHandle = Dart_LibraryUrl(library);
+        ASSERT(!Dart_IsError(libraryURLHandle));
+        String libraryURL = DartUtilities::toString(libraryURLHandle);
+
         add(m_importersForSource, url, libraryURL);
-    else
+    } else
         ASSERT_NOT_REACHED();
 
     loadScriptResource(url);
     return Dart_NewBoolean(true);
 }
 
 void DartApplicationLoader::load(const String& url, const String& source)
 {
     DartIsolateScope isolateScope(m_isolate);
     DartApiScope dartApiScope;
@@ skipping 243 matching lines @@
     // We should never have a self dependence.
     ASSERT(key != value);
     UrlMultiMap::iterator iter = map.find(key);
     if (iter == map.end())
         iter = map.add(key, new UrlSet()).iterator;
     UrlSet* set = iter->value;
     set->add(value);
 }
 
 }