Cherry-pick d60efa8

Cherry-pick b4b02da
Cherry-pick bbfa1da
Cherry-pick 0bb2d5e

Author: alan-knight

pkg/dev_compiler/lib/js/amd/dart_sdk.js

@@ -6033,8 +6033,7 @@ String jsLibraryName(String libraryRoot, LibraryElement library) {
     // E.g., "foo/bar.dart" and "foo$47bar.dart" would collide.
     qualifiedPath = uri.pathSegments.skip(1).join(encodedSeparator);
   } else if (isWithin(libraryRoot, uri.toFilePath())) {
-    qualifiedPath = uri.path
-        .substring(libraryRoot.length)
+    qualifiedPath = relative(uri.toFilePath(), from: libraryRoot)
         .replaceAll(separator, encodedSeparator);
   } else {
     // We don't have a unique name.
@@ -6059,30 +6058,6 @@ String jsLibraryDebuggerName(String libraryRoot, LibraryElement library) {
   return relative(filePath, from: libraryRoot);
 }
 
-String jsDebuggingLibraryName(String libraryRoot, LibraryElement library) {
-  var uri = library.source.uri;
-  if (uri.scheme == 'dart') {
-    return uri.path;
-  }
-  // TODO(vsm): This is not necessarily unique if '__' appears in a file name.
-  var separator = '__';
-  String qualifiedPath;
-  if (uri.scheme == 'package') {
-    // Strip the package name.
-    // TODO(vsm): This is not unique if an escaped '/'appears in a filename.
-    // E.g., "foo/bar.dart" and "foo$47bar.dart" would collide.
-    qualifiedPath = uri.pathSegments.skip(1).join(separator);
-  } else if (uri.toFilePath().startsWith(libraryRoot)) {
-    qualifiedPath =
-        uri.path.substring(libraryRoot.length).replaceAll('/', separator);
-  } else {
-    // We don't have a unique name.
-    throw 'Invalid library root. $libraryRoot does not contain ${uri
-        .toFilePath()}';
-  }
-  return pathToJSIdentifier(qualifiedPath);
-}
-
 /// Shorthand for identifier-like property names.
 /// For now, we emit them as strings and the printer restores them to
 /// identifiers if it can.

pkg/dev_compiler/lib/js/amd/dart_sdk.js.map

@@ -114,7 +114,7 @@ define(['dart_sdk', 'async_helper', 'expect', 'unittest', 'is', 'require'],
       'const_switch_test_02_multi': fail,
       'const_switch_test_04_multi': fail,
       'constructor12_test': fail,
-      'covariant_subtyping_tearoff2_test': fail,     
+      'covariant_subtyping_tearoff2_test': fail,
       'covariant_subtyping_tearoff3_test': fail,
       'covariant_subtyping_unsafe_call2_test': fail,
       'covariant_subtyping_unsafe_call3_test': fail,
@@ -496,7 +496,6 @@ define(['dart_sdk', 'async_helper', 'expect', 'unittest', 'is', 'require'],
 
       // was https://github.com/dart-lang/sdk/issues/27578, needs triage
       'speechrecognition_test': firefox_fail,
-      'svgelement_test': chrome_fail,
       'text_event_test': firefox_fail,
 
       // was https://github.com/dart-lang/sdk/issues/27578, needs triage

pkg/dev_compiler/lib/js/common/dart_sdk.js

@@ -2402,7 +2402,6 @@ final notYetStrongTests = new HashSet<String>.from([
   'lib/html/custom/document_register_basic_test',
   'lib/html/custom/document_register_type_extensions_test',
   'lib/html/custom/element_upgrade_test',
-  'lib/html/element_dimensions_test',
   'lib/html/element_test',
   'lib/html/events_test',
   'lib/html/fileapi_test',