Remove the scroll if the Reading List is empty.

Remove the scroll if the Reading List is empty.

BUG=678688, 680007, 680008

[gambard, 2017-01-10 08:53:47]

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

[gambard, 2017-01-10 08:53:47]

PTAL.

[Olivier, 2017-01-10 08:56:56]

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

https://codereview.chromium.org/2623853002/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_view_controller.mm:209:
self.collectionView.alwaysBounceVertical = NO;
Should this always stay NO?

[gambard, 2017-01-10 09:30:38]

Thanks, PTAL.

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

https://codereview.chromium.org/2623853002/diff/1/ios/chrome/browser/ui/readi...
ios/chrome/browser/ui/reading_list/reading_list_view_controller.mm:209:
self.collectionView.alwaysBounceVertical = NO;
On 2017/01/10 08:56:56, Olivier Robin wrote:
> Should this always stay NO?

Done.

[gambard, 2017-01-11 08:13:26]

Description was changed from

==========
Remove the scroll if the Reading List is empty.

BUG=678688
==========

to

==========
Remove the scroll if the Reading List is empty.

BUG=678688, 680007, 680008
==========

[Olivier, 2017-01-11 08:16:35]

lgtm

[gambard, 2017-01-11 08:16:54]

The CQ bit was checked by gambard@chromium.org

[commit-bot: I haz the power, 2017-01-11 08:17:19]

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-11 10:31:18]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484122614441500,
"parent_rev": "e5a88e66c2120ec14fe98ec23da580300064dbbf", "commit_rev":
"7e9f15b31cf27e8ebc5694dd417d15723c883367"}

[commit-bot: I haz the power, 2017-01-11 10:39:45]

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

[commit-bot: I haz the power, 2017-01-11 10:39:46]

Prior attempt to commit was detected, but we were not able to check whether the
issue was successfully committed. Please check Git history manually and re-check
CQ or close this issue as needed.

[gambard, 2017-01-11 12:05:00]

Landed as 7e9f15b31cf27e8ebc5694dd417d15723c883367.
Closing this.