Exposing ARIA state/property aria-orientation on Windows correctly.

Exposing ARIA state/property aria-orientation on Windows correctly.

According to latest editor draft http://rawgit.com/w3c/aria/master/aria/aria.html#aria-orientation

On windows aria orientation should be exposed as below ia2_states.
IAccessible2: IA2_STATE_HORIZONTAL & IA2_STATE_VERTICAL

This CL depends on https://codereview.chromium.org/730113003/

BUG=416763
Committed: https://crrev.com/c562d1d528441b80119252a6bd16f685990b7a29
Cr-Commit-Position: refs/heads/master@{#306154}

[shreeramk, 2014-11-28 08:28:41]

shreeram.k@samsung.com changed reviewers:
+ aboxhall@chromium.org, dmazzoni@chromium.org

[shreeramk, 2014-11-28 08:28:42]

Please review the changes.

Thanks!!!

[dmazzoni, 2014-11-29 07:48:19]

lgtm

This was a really great bug fix, well done.

https://codereview.chromium.org/744623002/diff/80001/content/renderer/accessi...
File content/renderer/accessibility/blink_ax_enum_conversion.cc (right):

https://codereview.chromium.org/744623002/diff/80001/content/renderer/accessi...
content/renderer/accessibility/blink_ax_enum_conversion.cc:77: if (orientation)
{
Nit: I'd drop the if (orientation) check; it's not really necessary since you're
just comparing it to blink::WebAXOrientationVertical and
blink::WebAXOrientationHorizontal below.

[shreeramk, 2014-11-30 15:15:11]

The CQ bit was checked by shreeram.k@samsung.com

[commit-bot: I haz the power, 2014-11-30 15:15:35]

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

[commit-bot: I haz the power, 2014-11-30 15:18:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-30 15:18:59]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[shreeramk, 2014-12-01 05:01:40]

The CQ bit was checked by shreeram.k@samsung.com

[commit-bot: I haz the power, 2014-12-01 05:02:10]

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

[commit-bot: I haz the power, 2014-12-01 06:00:08]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2014-12-01 06:00:48]

Patchset 7 (id:??) landed as
https://crrev.com/c562d1d528441b80119252a6bd16f685990b7a29
Cr-Commit-Position: refs/heads/master@{#306154}