Update to use one less proxy per element

lib/src/js/polymer_base_dart.html

 <!--
 Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 for details. All rights reserved. Use of this source code is governed by a
 BSD-style license that can be found in the LICENSE file.
 -->
 <script>
   (function() {
     Polymer.Base.originalPolymerCreatedCallback =
         Polymer.Base.createdCallback;
     Polymer.Base.createdCallback = function() {
       if (this.__isPolymerDart__) return;
       this.originalPolymerCreatedCallback();
     };
 
-    // Store serialize/deserialize to retain the original behavior when the
-    // methods are overriden by PolymerSerialize.
-    Polymer.Base.originalSerialize = Polymer.Base.serialize;
-    Polymer.Base.originalDeserialize = Polymer.Base.deserialize;
-
     Polymer.Dart = {};
 
     // Placeholder for `undefined` return values. This should eventually go
     // away, once we have dart:js support for `undefined`.
     Polymer.Dart.undefined = {};
 
     // Used to get an empty constructor function which doesn't call into dart.
     Polymer.Dart.functionFactory = function() { return function() {}; };
 
     // Generates a function which accesses a particular property on a dart
@@ skipping 25 matching lines @@
         // recognize.
         var args = [];
         for (var i = 0; i < arguments.length; i++) {
           args[i] = arguments[i];
         }
         var value = dartFunction(thisArg, args);
         if (value === Polymer.Dart.undefined) return undefined;
         return value;
       }
     };
+
+    // Remove any `__dartClass__` instances during serialization.
+    Polymer.Dart.serialize = function(value, type) {
+      var dartClass = value['__dartClass__'];

[Siggi Cherem (dart-lang), 2015/09/17 21:11:59]

Since this is JS, use the field directly?

  var dartClass = value.__dartClass__;
  delete value.__dartClass__;
  var serialized = ...
  value.__dartClass__ = dartClass;
  return serialized;

[jakemac, 2015/09/23 17:37:08]

done

+      value['__dartClass__'] = undefined;

[Siggi Cherem (dart-lang), 2015/09/17 21:11:59]

Does Polymer.Base.serialize depend on Object.keys? if so, do you need to change
this to delete the key entirely (delete value.__dartClass__)?

[jakemac, 2015/09/23 17:37:08]

It seemed to work fine, but deleting it entirely is probably better. I think it
was explicitly skipping undefined values.

+      var serialized = Polymer.Base.serialize(value, type);
+      value['__dartClass__'] = dartClass;
+      return serialized;
+    };
+
+    Polymer.Dart.Behavior = {

[Siggi Cherem (dart-lang), 2015/09/17 21:11:59]

instead of a behavior, would it make sense to make these properly be part of
Polymer.Base?

[jakemac, 2015/09/23 17:37:08]

I think the behavior actually makes a lot of sense. It only affects Polymer Dart
elements, and the behaviors are low cost.

+      // Secret hook into property changes. Pretty hacky but its more efficient
+      // than using a JsProxy object for the element.
+      _propertyChanged: function(path, newValue, oldValue) {
+        var parts = path.split('.');
+        var thisArg = parts.length == 1 ?
+            this : this.get(parts.slice(0, parts.length - 1));
+        thisArg = thisArg.__dartClass__ ? thisArg.__dartClass__ : thisArg;

[Siggi Cherem (dart-lang), 2015/09/17 21:11:59]

Or:

thisArg = thisArg.__dartClass__ || thisArg;

[jakemac, 2015/09/23 17:37:08]

Done.

+        Polymer.Dart.propertyChanged(
+            thisArg, parts[parts.length - 1], newValue);
+      },
+
+      notifyPath: function(path, value, fromAbove) {
+        var notified =
+            Polymer.Base.notifyPath.call(this, path, value, fromAbove);
+        if (fromAbove) return notified;
+
+        // If you call notifyPath from dart we need to do manual modifications
+        // on any JsArray or JsObject instances.
+        var parts = this._getPathParts(path);
+        if (parts.length > 1) {
+          var model = this.get(parts.splice(0, parts.length - 1));
+          if (model[parts[0]] !== value) model[parts[0]] = value;

[Siggi Cherem (dart-lang), 2015/09/17 21:11:59]

do we expect the value to always be a string literal, or could it be the result
of `new String`? If we might have the latter, maybe we need to use != instead of
!==?

[jakemac, 2015/09/23 17:37:08]

good catch, I don't think it would ever be created with `new String` but better
safe than sorry

+        }
+        return notified;
+      },
+
+      serialize: function(value, type) {
+        return Polymer.Dart.serialize(value, type);
+      }
+    }
   })();
 </script>