Rename ReadingListViewController to ReadingListCollectionViewController

Rename ReadingListViewController to ReadingListCollectionViewController

BUG=688392
Review-Url: https://codereview.chromium.org/2672823003
Cr-Commit-Position: refs/heads/master@{#448329}
Committed: https://chromium.googlesource.com/chromium/src/+/06742d0d6225a9b04cb79a1204630f9b828cedd2

[gambard, 2017-02-03 15:45:09]

gambard@chromium.org changed reviewers:
+ jif@chromium.org, olivierrobin@chromium.org, stkhapugin@chromium.org

[gambard, 2017-02-03 15:45:10]

PTAL/FYI

[Olivier, 2017-02-03 15:52:11]

https://codereview.chromium.org/2672823003/diff/1/ios/chrome/browser/ui/readi...
File ios/chrome/browser/ui/reading_list/reading_list_view_controller_container.h
(right):

https://codereview.chromium.org/2672823003/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_view_controller_container.h:17:
@class ReadingListCollectionViewController;
should be before (alphabetical order).

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

https://codereview.chromium.org/2672823003/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_view_controller_container.mm:154:
(ReadingListCollectionViewController*)readingListViewController
nit: here and below do you want to rename the variable?

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

https://codereview.chromium.org/2672823003/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_view_controller_unittest.mm:58:
#pragma mark - ReadingListCollectionViewControllerTest
I think you must rename the file.

[gambard, 2017-02-03 15:58:56]

Thanks, PTAL.

https://codereview.chromium.org/2672823003/diff/1/ios/chrome/browser/ui/readi...
File ios/chrome/browser/ui/reading_list/reading_list_view_controller_container.h
(right):

https://codereview.chromium.org/2672823003/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_view_controller_container.h:17:
@class ReadingListCollectionViewController;
On 2017/02/03 15:52:11, Olivier Robin wrote:
> should be before (alphabetical order).

Done.

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

https://codereview.chromium.org/2672823003/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_view_controller_container.mm:154:
(ReadingListCollectionViewController*)readingListViewController
On 2017/02/03 15:52:11, Olivier Robin wrote:
> nit: here and below do you want to rename the variable?

Done.

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

https://codereview.chromium.org/2672823003/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_view_controller_unittest.mm:58:
#pragma mark - ReadingListCollectionViewControllerTest
On 2017/02/03 15:52:11, Olivier Robin wrote:
> I think you must rename the file.

Done.

[stkhapugin, 2017-02-03 16:37:34]

lgtm \o/

[jif, 2017-02-06 08:29:44]

lgtm

[Olivier, 2017-02-06 08:32:19]

lgtm

[gambard, 2017-02-06 17:21:27]

The CQ bit was checked by gambard@chromium.org

[gambard, 2017-02-06 17:21:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
stkhapugin@chromium.org, olivierrobin@chromium.org, jif@chromium.org
Link to the patchset: https://codereview.chromium.org/2672823003/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2017-02-06 17:21:43]

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-02-06 18:54:36]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1486401687974680,
"parent_rev": "6b10ff77d359100c6fd3e287feccd0342211b51d", "commit_rev":
"06742d0d6225a9b04cb79a1204630f9b828cedd2"}

[commit-bot: I haz the power, 2017-02-06 18:55:12]

Description was changed from

==========
Rename ReadingListViewController to ReadingListCollectionViewController

BUG=688392
==========

to

==========
Rename ReadingListViewController to ReadingListCollectionViewController

BUG=688392

Review-Url: https://codereview.chromium.org/2672823003
Cr-Commit-Position: refs/heads/master@{#448329}
Committed:
https://chromium.googlesource.com/chromium/src/+/06742d0d6225a9b04cb79a120463...
==========

[commit-bot: I haz the power, 2017-02-06 18:55:13]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/06742d0d6225a9b04cb79a120463...