Strip leading/trailing whitespaces from script URLs in ScriptLoader

third_party/WebKit/Source/core/dom/ScriptLoader.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2008 Nikolas Zimmermann <zimmermann@kde.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 326 matching lines @@
       // TODO(hiroshige): Make this asynchronous. Currently we fire the error
       // event synchronously to keep the existing behavior.
       DispatchErrorEvent();
       return false;
     }
 
     // 21.3. "Set the element's from an external file flag."
     is_external_script_ = true;
 
     // 21.4. "Parse src relative to the element's node document."
-    // TODO(hiroshige): Use CompleteURL(src) instead.
-    KURL url = element_document.CompleteURL(element_->SourceAttributeValue());
+    KURL url = element_document.CompleteURL(src);
 
     // 21.5. "If the previous step failed, queue a task to
     //        fire an event named error at the element, and abort these steps."
     if (!url.IsValid()) {
       // TODO(hiroshige): Make this asynchronous. Currently we fire the error
       // event synchronously to keep the existing behavior.
       DispatchErrorEvent();
       return false;
     }
 
@@ skipping 493 matching lines @@
   //     then abort these steps at this point. The script is not executed.
   return DeprecatedEqualIgnoringCase(event_attribute, "onload") ||
          DeprecatedEqualIgnoringCase(event_attribute, "onload()");
 }
 
 String ScriptLoader::ScriptContent() const {
   return element_->TextFromChildren();
 }
 
 }  // namespace blink