Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(117)

Issue 25719003: Fixing Node.clone not returning upgraded custom elements. (Closed)

Created:
7 years, 2 months ago by blois
Modified:
7 years, 2 months ago
Reviewers:
vsm
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fixing Node.clone not returning upgraded custom elements. This was an issue for Polymer as the templates are cloned with shallow clones. BUG= R=vsm@google.com Committed: https://code.google.com/p/dart/source/detail?r=28363

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -3 lines) Patch
M tests/html/custom/constructor_calls_created_synchronously_test.dart View 1 chunk +19 lines, -2 lines 0 comments Download
M tools/dom/idl/dart/dart.idl View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
blois
7 years, 2 months ago (2013-10-04 22:26:18 UTC) #1
vsm
Is there a corresponding Dartium CL with the custom implementation?
7 years, 2 months ago (2013-10-07 17:48:13 UTC) #2
blois
Oops- https://codereview.chromium.org/26380002/
7 years, 2 months ago (2013-10-07 23:10:27 UTC) #3
vsm
lgtm
7 years, 2 months ago (2013-10-08 04:38:43 UTC) #4
blois
7 years, 2 months ago (2013-10-08 16:02:54 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 manually as r28363 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698