Rename mdv -> template_binding, remove experiemntal apis from dart:html

Rename mdv -> template_binding, remove experiemntal apis from dart:html

essentially, we trade off reduce dart:html API surface area for more annoying usage of these APIs. Since Polymer hides most of this, I suspect it won't be too annoying in practice.

Sample usage:

    nodeBind(node).bind(...)

    templateBind(node).model = ...;

R=sigmund@google.com

Committed: https://code.google.com/p/dart/source/detail?r=29057

[Jennifer Messerly, 2013-10-23 01:16:11]

apologies for "node.dart" and "template.dart". The code moved from impl_Node and
implElement/impl_HTMLTemplateElement, but unfortunately the diff tool doesn't
figure that out. There are no implementation changes other than what is needed
to keep everything working with these "extension apis".

[Siggi Cherem (dart-lang), 2013-10-23 01:41:11]

lgtm!

https://codereview.chromium.org/34453003/diff/60001/pkg/polymer/lib/src/decla...
File pkg/polymer/lib/src/declaration.dart (right):

https://codereview.chromium.org/34453003/diff/60001/pkg/polymer/lib/src/decla...
pkg/polymer/lib/src/declaration.dart:72: return template != null ?
templateBind(template).content : null;
I've been thinking about what we discussed in person. Is there a way we could
include a comment in to alert users about the fact that they shouldn't use
template.content and use templateBind(template).content instead? Could this go
in the docs of TemplateELement.content?

https://codereview.chromium.org/34453003/diff/60001/pkg/polymer_expressions/t...
File pkg/polymer_expressions/test/eval_test.dart (right):

https://codereview.chromium.org/34453003/diff/60001/pkg/polymer_expressions/t...
pkg/polymer_expressions/test/eval_test.dart:9: // NOTE: this import is unused,
but we use it to cause all mirrors to be
rereading this, sounds funny =): it's unused, but we use it

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
File pkg/template_binding/test/custom_element_bindings_test.dart (right):

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
pkg/template_binding/test/custom_element_bindings_test.dart:33: return
document.body.on['WebComponentsReady'].first;
I believe the auto-wrapping of this test puts the script tag in body, but
otherwise, consider removing '.body' like in my other change.

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
pkg/template_binding/test/custom_element_bindings_test.dart:178: print('!!!
createBinding $name $model $path');
delete

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
File pkg/template_binding/test/template_element_test.dart (right):

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
pkg/template_binding/test/template_element_test.dart:591: var div =
createTestHtml('<template bind="{{}}">Hi {{\nname\n}}</template>');
while you are here, consider wrapping at 80

[Jennifer Messerly, 2013-10-23 02:05:02]

thanks!

https://codereview.chromium.org/34453003/diff/60001/pkg/polymer/lib/src/decla...
File pkg/polymer/lib/src/declaration.dart (right):

https://codereview.chromium.org/34453003/diff/60001/pkg/polymer/lib/src/decla...
pkg/polymer/lib/src/declaration.dart:72: return template != null ?
templateBind(template).content : null;
On 2013/10/23 01:41:11, Siggi Cherem (dart-lang) wrote:
> I've been thinking about what we discussed in person. Is there a way we could
> include a comment in to alert users about the fact that they shouldn't use
> template.content and use templateBind(template).content instead? Could this go
> in the docs of TemplateELement.content?

using ".content" could be okay as long as you don't mix it with this package. I
like the idea of improving the docs tho. maybe we can do in a follow up (i'm not
very familiar with how to edit HTML docs)

https://codereview.chromium.org/34453003/diff/60001/pkg/polymer_expressions/t...
File pkg/polymer_expressions/test/eval_test.dart (right):

https://codereview.chromium.org/34453003/diff/60001/pkg/polymer_expressions/t...
pkg/polymer_expressions/test/eval_test.dart:9: // NOTE: this import is unused,
but we use it to cause all mirrors to be
On 2013/10/23 01:41:11, Siggi Cherem (dart-lang) wrote:
> rereading this, sounds funny =): it's unused, but we use it

Done.

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
File pkg/template_binding/test/custom_element_bindings_test.dart (right):

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
pkg/template_binding/test/custom_element_bindings_test.dart:33: return
document.body.on['WebComponentsReady'].first;
On 2013/10/23 01:41:11, Siggi Cherem (dart-lang) wrote:
> I believe the auto-wrapping of this test puts the script tag in body, but
> otherwise, consider removing '.body' like in my other change.

sure. FYI it's copied from HTML tests

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
pkg/template_binding/test/custom_element_bindings_test.dart:178: print('!!!
createBinding $name $model $path');
On 2013/10/23 01:41:11, Siggi Cherem (dart-lang) wrote:
> delete 

good catch!

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
File pkg/template_binding/test/template_element_test.dart (right):

https://codereview.chromium.org/34453003/diff/60001/pkg/template_binding/test...
pkg/template_binding/test/template_element_test.dart:591: var div =
createTestHtml('<template bind="{{}}">Hi {{\nname\n}}</template>');
On 2013/10/23 01:41:11, Siggi Cherem (dart-lang) wrote:
> while you are here, consider wrapping at 80

Done.

[Jennifer Messerly, 2013-10-23 02:12:28]

Committed patchset #4 manually as r29057 (presubmit successful).