Revert "feat(platform): Make angular invoke web_component polyfills for browsers without native web_component implementations."

This reverts commit 5c0ed60.

Author: jbdeboer

example/pubspec.yaml

@@ -5,7 +5,6 @@ dependencies:
     path: ../
   browser: any
   unittest: any
-  web_components: any
 
 transformers:
 - angular

example/web/css/shadow_dom_bracket.css

@@ -5,7 +5,6 @@
       <li><a href="bouncing_balls.html">bouncing_balls.html</a></li>
       <li><a href="hello_world.html">hello_world.html</a></li>
       <li><a href="todo.html">todo.html</a></li>
-      <li><a href="shadow_dom_components.html">shadow_dom_components.html</a></li>
     </ul>
   </body>
 </html>

example/web/css/shadow_dom_components.css

@@ -61,11 +61,9 @@ export "package:angular/core_dom/module_internal.dart" show
 
 export "package:angular/core/module_internal.dart" show
     CacheStats,
-    ComponentCssRewriter,
     ExceptionHandler,
     Interpolate,
     VmTurnZone,
-    WebPlatform,
     PrototypeMap,
     RootScope,
     Scope,

example/web/index.html

@@ -3,7 +3,6 @@ library angular.core.dom_internal;
 import 'dart:async' as async;
 import 'dart:convert' show JSON;
 import 'dart:html' as dom;
-import 'dart:js' as js;
 
 import 'package:di/di.dart';
 import 'package:perf_api/perf_api.dart';
@@ -36,7 +35,6 @@ part 'event_handler.dart';
 part 'http.dart';
 part 'mustache.dart';
 part 'node_cursor.dart';
-part 'web_platform.dart';
 part 'selector.dart';
 part 'shadow_dom_component_factory.dart';
 part 'shadowless_shadow_root.dart';
@@ -68,8 +66,6 @@ class CoreDomModule extends Module {
     bind(TranscludingComponentFactory);
     bind(Content);
     bind(ContentPort, toValue: null);
-    bind(ComponentCssRewriter);
-    bind(WebPlatform);
 
     bind(Http);
     bind(UrlRewriter);

example/web/shadow_dom_components.dart

@@ -32,7 +32,7 @@ class ShadowDomComponentFactory implements ComponentFactory {
 
   ShadowDomComponentFactory(this._expando, this._config);
 
-  final Map<_ComponentAssetKey, async.Future<dom.StyleElement>> _styleElementCache = {};
+  final Map<String, async.Future<dom.StyleElement>> _styleElementCache = {};
 
 
 
@@ -46,16 +46,8 @@ class ShadowDomComponentFactory implements ComponentFactory {
         DirectiveMap directives = injector.getByKey(DIRECTIVE_MAP_KEY);
         NgBaseCss baseCss = component.useNgBaseCss ? injector.getByKey(NG_BASE_CSS_KEY) : null;
         // This is a bit of a hack since we are returning different type then we are.
-        var componentFactory = new _ComponentFactory(node,
-            ref.typeKey,
-            component,
-            injector.getByKey(NODE_TREE_SANITIZER_KEY),
-            injector.getByKey(WEB_PLATFORM_KEY),
-            injector.getByKey(COMPONENT_CSS_REWRITER_KEY),
-            _expando,
-            baseCss,
-            _styleElementCache,
-            _config);
+        var componentFactory = new _ComponentFactory(node, ref.typeKey, component,
+            injector.getByKey(NODE_TREE_SANITIZER_KEY), _expando, baseCss, _styleElementCache, _config);
         var controller = componentFactory.call(injector, scope, viewCache, http, templateCache,
             directives);
 
@@ -79,10 +71,7 @@ class _ComponentFactory implements Function {
   final dom.NodeTreeSanitizer treeSanitizer;
   final Expando _expando;
   final NgBaseCss _baseCss;
-  final Map<_ComponentAssetKey, async.Future<dom.StyleElement>>
-      _styleElementCache;
-  final ComponentCssRewriter componentCssRewriter;
-  final WebPlatform platform;
+  final Map<String, async.Future<dom.StyleElement>> _styleElementCache;
   final CompilerConfig _config;
 
   dom.ShadowRoot shadowDom;
@@ -91,8 +80,7 @@ class _ComponentFactory implements Function {
   var controller;
 
   _ComponentFactory(this.element, this.typeKey, this.component, this.treeSanitizer,
-                    this.platform, this.componentCssRewriter, this._expando,
-                    this._baseCss, this._styleElementCache, this._config);
+                    this._expando, this._baseCss, this._styleElementCache, this._config);
 
   dynamic call(Injector injector, Scope scope,
                ViewCache viewCache, Http http, TemplateCache templateCache,
@@ -111,57 +99,22 @@ class _ComponentFactory implements Function {
     var cssUrls = _baseCss != null ?
        ([]..addAll(_baseCss.urls)..addAll(component.cssUrls)) :
        component.cssUrls;
-    var tag = element.tagName.toLowerCase();
     if (cssUrls.isNotEmpty) {
-      cssFutures = cssUrls.map((cssUrl) => _styleElementCache.putIfAbsent(
-          new _ComponentAssetKey(tag, cssUrl), () =>
+      cssFutures = cssUrls.map((cssUrl) => _styleElementCache.putIfAbsent(cssUrl, () =>
         http.get(cssUrl, cache: templateCache)
           .then((resp) => resp.responseText,
             onError: (e) => '/*\n$e\n*/\n')
-          .then((String css) {
-
-            // Shim CSS if required
-            if (platform.cssShimRequired) {
-              css = platform.shimCss(css, selector: tag, cssUrl: cssUrl);
-            }
-
-            // If a css rewriter is installed, run the css through a rewriter
-            var styleElement = new dom.StyleElement()
-                ..appendText(componentCssRewriter(css, selector: tag,
-                    cssUrl: cssUrl));
-
-            // ensure there are no invalid tags or modifications
-            treeSanitizer.sanitizeTree(styleElement);
-
-            // If the css shim is required, it means that scoping does not
-            // work, and adding the style to the head of the document is
-            // preferrable.
-            if (platform.cssShimRequired) {
-              dom.document.head.append(styleElement);
-            }
-
-            return styleElement;
-          })
+          .then((styleContent) => new dom.StyleElement()..appendText(styleContent))
       )).toList();
     } else {
       cssFutures = [new async.Future.value(null)];
     }
-
-    var platformViewCache = new PlatformViewCache(viewCache, tag, platform);
-
-    var viewFuture = ComponentFactory._viewFuture(component, platformViewCache,
-        directives);
-
+    var viewFuture = ComponentFactory._viewFuture(component, viewCache, directives);
     TemplateLoader templateLoader = new TemplateLoader(
         async.Future.wait(cssFutures).then((Iterable<dom.StyleElement> cssList) {
-          // This prevents style duplication by only adding css to the shadow
-          // root if there is a native implementation of shadow dom.
-          if (!platform.cssShimRequired) {
-            cssList.where((styleElement) => styleElement != null)
-              .forEach((styleElement) {
-                shadowDom.append(styleElement.clone(true));
-              });
-          }
+          cssList
+              .where((styleElement) => styleElement != null)
+              .forEach((styleElement) => shadowDom.append(styleElement.clone(true)));
           if (viewFuture != null) {
             return viewFuture.then((ViewFactory viewFactory) {
               return (!shadowScope.isAttached) ?
@@ -209,33 +162,3 @@ class _ComponentFactory implements Function {
     return shadowInjector;
   }
 }
-
-class _ComponentAssetKey {
-  final String tag;
-  final String assetUrl;
-
-  final String _key;
-
-  _ComponentAssetKey(String tag, String assetUrl)
-      : _key = "$tag|$assetUrl",
-        this.tag = tag,
-        this.assetUrl = assetUrl;
-
-  @override
-  String toString() => _key;
-
-  @override
-  int get hashCode => _key.hashCode;
-
-  bool operator ==(key) =>
-      key is _ComponentAssetKey
-      && tag == key.tag
-      && assetUrl == key.assetUrl;
-}
-
-@Injectable()
-class ComponentCssRewriter {
-  String call(String css, { String selector, String cssUrl} ) {
-    return css;
-  }
-}