Canonicalize dart:_blink resolver names

Canonicalize dart:_blink resolver names

BUG=
R=jacobr@google.com

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

[Leaf, 2014-05-14 21:18:22]

This is the dart half of the changes to canonicalize the native resolver
strings.  It should work in place with 1847 (layout tests pass), and will work
with 1916 when the paired webkit cl lands.

[Jacob, 2014-05-14 21:34:36]

https://codereview.chromium.org/289803004/diff/20001/sdk/lib/_blink/dartium/_...
File sdk/lib/_blink/dartium/_blink_dartium.dart (right):

https://codereview.chromium.org/289803004/diff/20001/sdk/lib/_blink/dartium/_...
sdk/lib/_blink/dartium/_blink_dartium.dart:977:
Native_CanvasRenderingContext2D__fill_1_Callback(mthis, winding) native
"CanvasRenderingContext2D_fill_Callback_1_CanvasWindingRule";
CanvasWindingRule is an enum.  I believe this should show up as DOMString not
String in this file.
If the bindings on the blink side also want to use CanvasWindingRule it is ok
but fragile but it would be more robust if these just showed up as DOMString.

[Leaf, 2014-05-14 22:57:23]

https://codereview.chromium.org/289803004/diff/20001/sdk/lib/_blink/dartium/_...
File sdk/lib/_blink/dartium/_blink_dartium.dart (right):

https://codereview.chromium.org/289803004/diff/20001/sdk/lib/_blink/dartium/_...
sdk/lib/_blink/dartium/_blink_dartium.dart:977:
Native_CanvasRenderingContext2D__fill_1_Callback(mthis, winding) native
"CanvasRenderingContext2D_fill_Callback_1_CanvasWindingRule";
Well, currently both generate CanvasWindingRule.  I believe I can robustly
generate DOMString on the blink side, but the dart:html side I'm not so sure
about.  I'm inclined to leave this for now, possibly with a FIXME to revisit
when we shift to generating everything from the blink side.  Especially since
it's not at all clear that this is our long term solution to overloading.

On 2014/05/14 21:34:37, Jacob wrote:
> CanvasWindingRule is an enum.  I believe this should show up as DOMString not
> String in this file.
> If the bindings on the blink side also want to use CanvasWindingRule it is ok
> but fragile but it would be more robust if these just showed up as DOMString.

[Jacob, 2014-05-14 23:23:03]

lgtm

[Leaf, 2014-05-14 23:28:03]

Committed patchset #3 manually as r36196 (presubmit successful).