Support aria-dropeffect attribute.

Support aria-dropeffect attribute.

This patch exposes aria-dropeffect attribute through WebAXObject
adding WebAXObject::ariaDropEffect().

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

[je_julie(Not used), 2015-01-25 16:27:15]

Hi Dominic and Mike,

This patch is to expose aria-dropeffect attribute at blink.
FYI, the patch from chromium is http://crrev.com/874893003/.

The aria-dropeffect attribute allows more than one drop effect may be supported
for a given element.
as described in the spec,
http://www.w3.org/TR/wai-aria/states_and_properties#aria-dropeffect.

When I look into implementation from WebKit, it handles drop-effect with vector
and I tried to handle it with vector as well.
But I checked Firefox exposes drop effect with just string such as “popup|copy”
on one attribute field.
I also referred to some attributes which return values as a token list such as
aria-relevant.
After checking them, I created a patch with returning a simple string type.

WDYT?

[je_julie(Not used), 2015-01-25 16:31:20]

je_julie.kim@samsung.com changed reviewers:
+ dmazzoni@chromium.org, mkwst@chromium.org

[je_julie(Not used), 2015-01-26 00:06:22]

Hi Dominic and Mike,

This patch is to expose aria-dropeffect attribute at blink.
FYI, the patch from chromium is http://crrev.com/874893003/.

The aria-dropeffect attribute allows more than one drop effect may be supported
for a given element.
as described in the spec,
http://www.w3.org/TR/wai-aria/states_and_properties#aria-dropeffect.

When I look into implementation from WebKit, it handles drop-effect with vector
and I tried to handle it with vector as well.
But I checked Firefox exposes drop effect with just string such as “popup|copy”
on one attribute field.
I also referred to some attributes which return values as a token list such as
aria-relevant.
After checking them, I created a patch with returning a simple string type.

WDYT?

[Mike West, 2015-01-27 10:57:54]

mkwst@chromium.org changed required reviewers:
+ dmazzoni@chromium.org

[Mike West, 2015-01-27 10:57:55]

The code change LGTM, but please wait for Dominic to take a look before landing.
Marking him as required.

[dmazzoni, 2015-01-27 19:54:53]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2015-01-27 19:54:54]

lgtm

[commit-bot: I haz the power, 2015-01-27 19:55:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/863283005/1

[commit-bot: I haz the power, 2015-01-27 21:21:04]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=189056