Don't set expression attributes before they're bound.

Don't set expression attributes before they're bound.

We were cloning elements with all the attributes that contained
expressions like attrName="{{ foo }}" which meant we'd go through the
reflection process converting that value and also call the
attrNameChanged() callback and the attributeChanged() with the braced
string which the element didn't really want to know about.

After this patch we create a "clone source node" which is a copy of the
original element without the attributes that have the expressions and
use that when cloning, then we assign the properties using data binding
later.

R=abarth@chromium.org, ojan@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/1a1e9bee08f42b600cb42a7e07c84618a837f977

[abarth-chromium, 2015-01-23 00:55:02]

lgtm

[ojan, 2015-01-23 00:55:09]

ojan@chromium.org changed reviewers:
+ ojan@chromium.org

[ojan, 2015-01-23 00:55:09]

lgtm

[esprehn, 2015-01-23 01:04:17]

Committed patchset #1 (id:1) manually as
1a1e9bee08f42b600cb42a7e07c84618a837f977 (presubmit successful).