Use Reflect on attributes in xml namespace

Use Reflect on attributes in xml namespace

Use Reflect on attributes in xml namespace, in order make this work teach v8_attributes.py about XMLNames namespace.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=177847

[rwlbuis, 2014-07-09 17:32:29]

PTAL.

[haraken, 2014-07-09 23:45:53]

LGTM.

Just to confirm: Adding [Reflect] won't change any existing web-exposed
behavior, right? I'm asking this because [Reflect] can have a couple of side
effects (See various special-casings for [Reflect] in bindings/scripts/).

[rwlbuis, 2014-07-10 11:50:53]

On 2014/07/09 23:45:53, haraken wrote:
> LGTM.
> 
> Just to confirm: Adding [Reflect] won't change any existing web-exposed
> behavior, right? I'm asking this because [Reflect] can have a couple of side
> effects (See various special-casings for [Reflect] in bindings/scripts/).

It should not change web-exposed behavior, it just moves the
fastGetAttribute/setAttribute calls into the bindings.

[haraken, 2014-07-10 11:57:43]

On 2014/07/10 11:50:53, rwlbuis wrote:
> On 2014/07/09 23:45:53, haraken wrote:
> > LGTM.
> > 
> > Just to confirm: Adding [Reflect] won't change any existing web-exposed
> > behavior, right? I'm asking this because [Reflect] can have a couple of side
> > effects (See various special-casings for [Reflect] in bindings/scripts/).
> 
> It should not change web-exposed behavior, it just moves the
> fastGetAttribute/setAttribute calls into the bindings.

ok, thanks for the clarification.

[rwlbuis, 2014-07-10 14:32:39]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-07-10 14:33:46]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rob.buis@samsung.com/378343004/1

[commit-bot: I haz the power, 2014-07-10 15:41:45]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/16255)

[commit-bot: I haz the power, 2014-07-10 16:25:40]

Change committed as 177847