Get rid of postToElement argument for accessibility notifications.

Get rid of postToElement argument for accessibility notifications.

It's a confusing argument because it's "true" 90% of the time.
This changes the four cases where "false" was passed to take
care of mapping to the "observable" object explicitly.

BUG=none

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

[dmazzoni, 2015-03-25 22:03:30]

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

[dmazzoni, 2015-03-25 22:03:31]

Depends on:
https://codereview.chromium.org/1032183003/

[je_julie(Not used), 2015-03-27 09:37:21]

lgtm

https://codereview.chromium.org/1014573010/diff/20001/Source/modules/accessib...
File Source/modules/accessibility/AXObjectCacheImpl.cpp (right):

https://codereview.chromium.org/1014573010/diff/20001/Source/modules/accessib...
Source/modules/accessibility/AXObjectCacheImpl.cpp:754: obj =
obj->parentObject();
How about adding a helper function for this?

https://codereview.chromium.org/1014573010/diff/20001/Source/modules/accessib...
Source/modules/accessibility/AXObjectCacheImpl.cpp:1009: postNotification(obj,
AXObjectCache::AXValueChanged);
ditto.

[dmazzoni, 2015-03-27 18:31:35]

https://codereview.chromium.org/1014573010/diff/20001/Source/modules/accessib...
File Source/modules/accessibility/AXObjectCacheImpl.cpp (right):

https://codereview.chromium.org/1014573010/diff/20001/Source/modules/accessib...
Source/modules/accessibility/AXObjectCacheImpl.cpp:754: obj =
obj->parentObject();
On 2015/03/27 09:37:21, je_julie wrote:
> How about adding a helper function for this?

Done.

https://codereview.chromium.org/1014573010/diff/20001/Source/modules/accessib...
Source/modules/accessibility/AXObjectCacheImpl.cpp:1009: postNotification(obj,
AXObjectCache::AXValueChanged);
On 2015/03/27 09:37:21, je_julie wrote:
> ditto.

Sure, in this case I'll just call handleEditableTextContentChanged since they're
the same.

[dmazzoni, 2015-03-27 18:31:42]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2015-03-27 18:31:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
je_julie.kim@samsung.com
Link to the patchset: https://codereview.chromium.org/1014573010/#ps40001
(title: "Reuse more code")

[commit-bot: I haz the power, 2015-03-27 18:32:08]

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

[commit-bot: I haz the power, 2015-03-27 22:04:57]

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

[commit-bot: I haz the power, 2015-03-27 22:04:58]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/5...)

[dmazzoni, 2015-03-29 07:31:07]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2015-03-29 07:31:08]

The patchset sent to the CQ was uploaded after l-g-t-m from
je_julie.kim@samsung.com
Link to the patchset: https://codereview.chromium.org/1014573010/#ps60001
(title: "Rebaseline test")

[commit-bot: I haz the power, 2015-03-29 07:31:19]

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

[commit-bot: I haz the power, 2015-03-29 11:24:11]

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