Adding Sign-in promo for bookmark view.

ios/chrome/browser/ui/bookmarks/bookmark_folder_collection_view.mm

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "ios/chrome/browser/ui/bookmarks/bookmark_folder_collection_view.h"
 
 #include "base/logging.h"
 #include "base/strings/sys_string_conversions.h"
 #include "components/bookmarks/browser/bookmark_model.h"
 #include "ios/chrome/browser/bookmarks/bookmarks_utils.h"
 #include "ios/chrome/browser/experimental_flags.h"
+#import "ios/chrome/browser/ui/authentication/signin_promo_view.h"
+#import "ios/chrome/browser/ui/authentication/signin_promo_view_configurator.h"
+#import "ios/chrome/browser/ui/authentication/signin_promo_view_consumer.h"
+#import "ios/chrome/browser/ui/authentication/signin_promo_view_mediator.h"
 #import "ios/chrome/browser/ui/bookmarks/bookmark_collection_cells.h"
 #import "ios/chrome/browser/ui/bookmarks/bookmark_promo_cell.h"
+#import "ios/chrome/browser/ui/bookmarks/bookmark_signin_promo_cell.h"
 #import "ios/chrome/browser/ui/bookmarks/bookmark_utils_ios.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
 using bookmarks::BookmarkNode;
 
-@interface BookmarkFolderCollectionView ()<BookmarkPromoCellDelegate> {
+namespace {
+// Computes the cell size based on width.
+CGSize PreferredCellSizeForWidth(UICollectionViewCell* cell, CGFloat width) {

[msarda, 2017/04/28 11:53:14]

Personal preference (I find that you need less local variables): How about?
CGFrame cellFrame = cell.frame;
cellFrame.size.width = width;
cell.frame = cellFrame;
[cell setNeedsLayout];
[cell layoutIfNeeded];
CGSize result = ...
cellFrame.size = result;
cell.frame = cellFrame;
return result;

[jlebel, 2017/04/28 13:03:17]

Done.

+  CGPoint initialCellOrigin = cell.frame.origin;
+  CGRect fixedWidthMaxHeightFrame =
+      CGRectMake(initialCellOrigin.x, initialCellOrigin.y, width, CGFLOAT_MAX);
+  cell.frame = fixedWidthMaxHeightFrame;
+  [cell setNeedsLayout];
+  [cell layoutIfNeeded];
+  CGSize result =
+      [cell systemLayoutSizeFittingSize:UILayoutFittingCompressedSize
+          withHorizontalFittingPriority:UILayoutPriorityRequired
+                verticalFittingPriority:UILayoutPriorityDefaultLow];
+  cell.frame = CGRectMake(initialCellOrigin.x, initialCellOrigin.y,
+                          result.width, result.height);
+  return result;
+}
+}
+
+@interface BookmarkFolderCollectionView ()<BookmarkPromoCellDelegate,
+                                           SigninPromoViewConsumer> {
   // A vector of folders to display in the collection view.
   std::vector<const BookmarkNode*> _subFolders;
   // A vector of bookmark urls to display in the collection view.
   std::vector<const BookmarkNode*> _subItems;
 
   // True if the promo is visible.
   BOOL _promoVisible;
+
+  // Mediator, helper for the sign-in promo view.
+  SigninPromoViewMediator* _signinPromoViewMediator;
 }
 @property(nonatomic, assign) const bookmarks::BookmarkNode* folder;
 
 // Section indices.
 @property(nonatomic, readonly, assign) NSInteger promoSection;
 @property(nonatomic, readonly, assign) NSInteger folderSection;
 @property(nonatomic, readonly, assign) NSInteger itemsSection;
 @property(nonatomic, readonly, assign) NSInteger sectionCount;
 
-// Keep a reference to the promo cell to deregister as delegate.
-@property(nonatomic, strong) BookmarkPromoCell* promoCell;
-
 @end
 
 @implementation BookmarkFolderCollectionView
 @synthesize delegate = _delegate;
 @synthesize folder = _folder;
-@synthesize promoCell = _promoCell;
 
 - (instancetype)initWithBrowserState:(ios::ChromeBrowserState*)browserState
                                frame:(CGRect)frame {
   self = [super initWithBrowserState:browserState frame:frame];
   if (self) {
     [self updateCollectionView];
   }
   return self;
 }
 
-- (void)dealloc {
-  _promoCell.delegate = nil;
-}
-
 - (void)setDelegate:(id<BookmarkFolderCollectionViewDelegate>)delegate {
   _delegate = delegate;
   [self promoStateChangedAnimated:NO];
 }
 
 - (NSInteger)promoSection {
   return [self shouldShowPromoCell] ? 0 : -1;
 }
 
 - (NSInteger)folderSection {
@@ skipping 277 matching lines @@
   else if (section == self.itemsSection)
     insets.top = [self interitemSpacingForSectionAtIndex:section] / 2.;
   else if (section == self.promoSection)
     (void)0;  // No insets to update.
   else
     NOTREACHED();
   return insets;
 }
 
 // Parent class override.
+- (CGSize)cellSizeForIndexPath:(NSIndexPath*)indexPath {
+  if ([self isPromoSection:indexPath.section]) {
+    UICollectionViewCell* cell =
+        [self.collectionView cellForItemAtIndexPath:indexPath];
+    if (!cell) {
+      // -[UICollectionView
+      // dequeueReusableCellWithReuseIdentifier:forIndexPath:] cannot be used
+      // here since this method is called by -[id<UICollectionViewDelegate>
+      // collectionView:layout:sizeForItemAtIndexPath:]. This would generate
+      // crash: SIGFPE, EXC_I386_DIV.
+      if (experimental_flags::IsSigninPromoEnabled()) {
+        DCHECK(_signinPromoViewMediator);
+        BookmarkSigninPromoCell* signinPromoCell =
+            [[BookmarkSigninPromoCell alloc]
+                initWithFrame:CGRectMake(0, 0, 1000, 1000)];

[msarda, 2017/04/28 11:53:14]

I would prefer CGFLOAT_MAX, CGFLOAT_MAX instead of 1000 (1000 seems very
arbitrary to me).

[jlebel, 2017/04/28 13:03:17]

It is totally arbitrary. But it seems more reasonable than CGFLOAT_MAX.

[lpromero, 2017/05/02 15:25:20]

CGFLOAT_MAX*CGFLOAT_MAX would have a large (huge!) memory impact :p

[jlebel, 2017/05/03 11:02:31]

Done.

+        [[_signinPromoViewMediator createConfigurator]
+            configureSigninPromoView:signinPromoCell.signinPromoView];
+        cell = signinPromoCell;
+      } else {
+        cell = [[BookmarkPromoCell alloc] init];
+      }
+    }
+    return PreferredCellSizeForWidth(cell, CGRectGetWidth(self.bounds));
+  }
+  return [super cellSizeForIndexPath:indexPath];
+}
+
+// Parent class override.
 - (UICollectionViewCell*)cellAtIndexPath:(NSIndexPath*)indexPath {
   if (indexPath.section == self.promoSection) {
-    self.promoCell = [self.collectionView
-        dequeueReusableCellWithReuseIdentifier:[BookmarkPromoCell
-                                                   reuseIdentifier]
-                                  forIndexPath:indexPath];
-    self.promoCell.delegate = self;
-    return self.promoCell;
+    if (experimental_flags::IsSigninPromoEnabled()) {
+      BookmarkSigninPromoCell* signinPromoCell = [self.collectionView
+          dequeueReusableCellWithReuseIdentifier:[BookmarkSigninPromoCell
+                                                     reuseIdentifier]
+                                    forIndexPath:indexPath];
+      signinPromoCell.signinPromoView.sendChromeCommand = YES;
+      [[_signinPromoViewMediator createConfigurator]
+          configureSigninPromoView:signinPromoCell.signinPromoView];
+      return signinPromoCell;
+    } else {
+      BookmarkPromoCell* promoCell = [self.collectionView
+          dequeueReusableCellWithReuseIdentifier:[BookmarkPromoCell
+                                                     reuseIdentifier]
+                                    forIndexPath:indexPath];
+      promoCell.delegate = self;
+      return promoCell;
+    }
   }
   const BookmarkNode* node = [self nodeAtIndexPath:indexPath];
 
   if (indexPath.section == self.folderSection)
     return [self cellForFolder:node indexPath:indexPath];
 
   BookmarkItemCell* cell = [self cellForBookmark:node indexPath:indexPath];
   return cell;
 }
 
@@ skipping 61 matching lines @@
 - (void)promoStateChangedAnimated:(BOOL)animate {
   BOOL newPromoState =
       !self.editing && self.folder &&
       self.folder->type() == BookmarkNode::MOBILE &&
       [self.delegate bookmarkCollectionViewShouldShowPromoCell:self];
   if (newPromoState != _promoVisible) {
     // This is awful, but until the old code to do the refresh when switching
     // in and out of edit mode is fixed, this is probably the cleanest thing to
     // do.
     _promoVisible = newPromoState;
+    if (experimental_flags::IsSigninPromoEnabled()) {
+      if (!_promoVisible) {
+        _signinPromoViewMediator.consumer = nil;
+        _signinPromoViewMediator = nil;
+      } else {
+        _signinPromoViewMediator = [[SigninPromoViewMediator alloc] init];
+        _signinPromoViewMediator.consumer = self;
+      }
+    }
     [self.collectionView reloadData];
   }
 }
 
 #pragma mark - BookmarkPromoCellDelegate
 
 - (void)bookmarkPromoCellDidTapSignIn:(BookmarkPromoCell*)bookmarkPromoCell {
   [self.delegate bookmarkCollectionViewShowSignIn:self];
 }
 
 - (void)bookmarkPromoCellDidTapDismiss:(BookmarkPromoCell*)bookmarkPromoCell {
   [self.delegate bookmarkCollectionViewDismissPromo:self];
 }
 
 #pragma mark - Promo Cell
 
 - (BOOL)shouldShowPromoCell {
   return _promoVisible;
 }
 
+#pragma mark - SigninPromoViewConsumer
+
+- (void)configureSigninPromoViewWithNewIdentity:(BOOL)newIdentity
+                                   configurator:(SigninPromoViewConfigurator*)
+                                                    configurator {
+  DCHECK(_signinPromoViewMediator);
+  if (newIdentity) {
+    NSIndexSet* indexSet = [NSIndexSet indexSetWithIndex:self.promoSection];
+    [self.collectionView reloadSections:indexSet];
+    return;
+  }
+  NSIndexPath* indexPath =
+      [NSIndexPath indexPathForRow:0 inSection:self.promoSection];
+  BookmarkSigninPromoCell* signinPromoCell =
+      static_cast<BookmarkSigninPromoCell*>(
+          [self.collectionView cellForItemAtIndexPath:indexPath]);
+  if (!signinPromoCell)
+    return;
+  [configurator configureSigninPromoView:signinPromoCell.signinPromoView];
+}
+
 @end