Attempted to fix crash in BrowserAccessibilityWin::RemoveBidirectionalRelation.

Attempted to fix crash in BrowserAccessibilityWin::RemoveBidirectionalRelation.

BUG=624626
R=dmazzoni@chromium.org

Committed: https://crrev.com/02970ff8bb63db38e86cceb455a298d4d1eb5162
Cr-Commit-Position: refs/heads/master@{#403323}

[dmazzoni, 2016-06-30 21:13:29]

lgtm

https://codereview.chromium.org/2119643002/diff/20001/content/browser/accessi...
File content/browser/accessibility/browser_accessibility_win.cc (right):

https://codereview.chromium.org/2119643002/diff/20001/content/browser/accessi...
content/browser/accessibility/browser_accessibility_win.cc:4477: const
std::vector<int32_t>& attributes = GetIntListAttribute(attribute);
How about "target_ids" and "filtered_target_ids" instead of "attributes" and
"filtered_attributes".

[chromium-reviews, 2016-06-30 21:58:24]

How about "target_ids" and "filtered_target_ids" instead of "attributes"
> and "filtered_attributes".
Done.

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[nektarios, 2016-06-30 21:59:38]

The CQ bit was checked by nektar@chromium.org

[nektarios, 2016-06-30 21:59:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2119643002/#ps40001
(title: "Changed attributes to target_ids.")

[commit-bot: I haz the power, 2016-06-30 22:00:03]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-30 23:05:56]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-30 23:06:09]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 23:09:20]

Description was changed from

==========
Attempted to fix crash in BrowserAccessibilityWin::RemoveBidirectionalRelation.

BUG=624626
R=dmazzoni@chromium.org
==========

to

==========
Attempted to fix crash in BrowserAccessibilityWin::RemoveBidirectionalRelation.

BUG=624626
R=dmazzoni@chromium.org

Committed: https://crrev.com/02970ff8bb63db38e86cceb455a298d4d1eb5162
Cr-Commit-Position: refs/heads/master@{#403323}
==========

[commit-bot: I haz the power, 2016-06-30 23:09:20]

Patchset 3 (id:??) landed as
https://crrev.com/02970ff8bb63db38e86cceb455a298d4d1eb5162
Cr-Commit-Position: refs/heads/master@{#403323}