Notify ReadingListCollectionViewController during dismissal

Notify ReadingListCollectionViewController during dismissal

Make sure ReadingListCollectionViewController is notified when it is about to
be dismissed after opening a link.

BUG=697003
Review-Url: https://codereview.chromium.org/2721313002
Cr-Commit-Position: refs/heads/master@{#453914}
Committed: https://chromium.googlesource.com/chromium/src/+/eaa4981576e913012f79990d3920956ecafd13fb

[gambard, 2017-03-01 09:01:51]

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

[gambard, 2017-03-01 09:01:52]

PTAL.

[Olivier, 2017-03-01 09:42:17]

lgtm

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

https://codereview.chromium.org/2721313002/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_coordinator.mm:295:
[readingListCollectionViewController willBeDismissed];
nit question: should this not be on line 301 (just before stop)?

[gambard, 2017-03-01 11:59:38]

Thanks!

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

https://codereview.chromium.org/2721313002/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_coordinator.mm:295:
[readingListCollectionViewController willBeDismissed];
On 2017/03/01 09:42:16, Olivier Robin wrote:
> nit question: should this not be on line 301 (just before stop)? 

It could be.

[gambard, 2017-03-01 11:59:45]

The CQ bit was checked by gambard@chromium.org

[commit-bot: I haz the power, 2017-03-01 11:59:56]

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-03-01 12:10:43]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488369585001450, "parent_rev":
"c61e077736727cf9ed705045d4d2e244efad315a", "commit_rev":
"eaa4981576e913012f79990d3920956ecafd13fb"}

[commit-bot: I haz the power, 2017-03-01 12:11:22]

Description was changed from

==========
Notify ReadingListCollectionViewController during dismissal

Make sure ReadingListCollectionViewController is notified when it is about to
be dismissed after opening a link.

BUG=697003
==========

to

==========
Notify ReadingListCollectionViewController during dismissal

Make sure ReadingListCollectionViewController is notified when it is about to
be dismissed after opening a link.

BUG=697003

Review-Url: https://codereview.chromium.org/2721313002
Cr-Commit-Position: refs/heads/master@{#453914}
Committed:
https://chromium.googlesource.com/chromium/src/+/eaa4981576e913012f79990d3920...
==========

[commit-bot: I haz the power, 2017-03-01 12:11:23]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/eaa4981576e913012f79990d3920...