Test accessibility of Reading List

Test accessibility of Reading List

BUG=none

Review-Url: https://codereview.chromium.org/2610923010
Cr-Commit-Position: refs/heads/master@{#441910}
Committed: https://chromium.googlesource.com/chromium/src/+/78b9be0cd4da61618c21e4003f7cc84092684ccd

[gambard, 2017-01-06 09:35:09]

gambard@chromium.org changed reviewers:
+ olivierrobin@chromium.org

[gambard, 2017-01-06 09:35:10]

PTAL.

[Olivier, 2017-01-06 09:44:41]

LGTM one question

https://codereview.chromium.org/2610923010/diff/1/ios/chrome/browser/ui/readi...
File ios/chrome/browser/ui/reading_list/reading_list_egtest.mm (right):

https://codereview.chromium.org/2610923010/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_egtest.mm:184:
AddEntriesAndEnterEdit();
Should we test out of edit mode?

[gambard, 2017-01-06 09:51:53]

Thanks.

https://codereview.chromium.org/2610923010/diff/1/ios/chrome/browser/ui/readi...
File ios/chrome/browser/ui/reading_list/reading_list_egtest.mm (right):

https://codereview.chromium.org/2610923010/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_egtest.mm:184:
AddEntriesAndEnterEdit();
On 2017/01/06 09:44:41, Olivier Robin wrote:
> Should we test out of edit mode?

Done.

[gambard, 2017-01-06 09:51:57]

The CQ bit was checked by gambard@chromium.org

[gambard, 2017-01-06 09:51:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
olivierrobin@chromium.org
Link to the patchset: https://codereview.chromium.org/2610923010/#ps20001
(title: "Test accessibility out of edit")

[commit-bot: I haz the power, 2017-01-06 09:52:09]

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, 2017-01-06 10:00:15]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483696317431980,
"parent_rev": "c5f4d7239d10069882d99f20d100992e520126a2", "commit_rev":
"78b9be0cd4da61618c21e4003f7cc84092684ccd"}

[commit-bot: I haz the power, 2017-01-06 10:00:44]

Description was changed from

==========
Test accessibility of Reading List

BUG=none
==========

to

==========
Test accessibility of Reading List

BUG=none

Review-Url: https://codereview.chromium.org/2610923010
Cr-Commit-Position: refs/heads/master@{#441910}
Committed:
https://chromium.googlesource.com/chromium/src/+/78b9be0cd4da61618c21e4003f7c...
==========

[commit-bot: I haz the power, 2017-01-06 10:00:44]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/78b9be0cd4da61618c21e4003f7c...