Makes -init and -setType: available in CollectionViewItem

ios/chrome/browser/ui/collection_view/cells/collection_view_item.mm

Index: ios/chrome/browser/ui/collection_view/cells/collection_view_item.mm
diff --git a/ios/chrome/browser/ui/collection_view/cells/collection_view_item.mm b/ios/chrome/browser/ui/collection_view/cells/collection_view_item.mm
index f4cc10ddd0d3fb2f6d1e702168e3f447a94cf8ee..3daa7ea5e14db44e979e7a19e1702fce52ce399a 100644
--- a/ios/chrome/browser/ui/collection_view/cells/collection_view_item.mm
+++ b/ios/chrome/browser/ui/collection_view/cells/collection_view_item.mm
@@ -5,6 +5,7 @@
 #import "ios/chrome/browser/ui/collection_view/cells/collection_view_item.h"
 
 #import "base/logging.h"
+#import "ios/chrome/browser/ui/collection_view/cells/collection_view_item+collection_view_controller.h"
 #import "ios/third_party/material_components_ios/src/components/CollectionCells/src/MaterialCollectionCells.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
@@ -26,8 +27,7 @@
 }
 
 - (instancetype)init {
-  NOTREACHED();
-  return nil;
+  return [self initWithType:0];
 }
 
 - (void)setCellClass:(Class)cellClass {
@@ -42,3 +42,11 @@
 }
 
 @end
+
+@implementation CollectionViewItem (CollectionViewController)
+

[lpromero, 2017/04/14 11:33:58]

Is this really needed, or does the synthesize take care of it?

If the compiler barfs, you could try:

@implementation CollectionViewItem (CollectionViewController)
@dynamic type;
@end

to tell the compiler to not freak out, there will be the proper getter/setter at
runtime.
Make sure that setting the type after init still works! → add a unit test :)

[Moe, 2017/04/17 16:34:14]

This setter is needed. I believe readonly tells the compiler not to synthesize a
setter method. So while @dynamic quiets the compiler, in runtime, setting the
type causes an
"unrecognized selector sent..." error because no synthesized setter exists.
Re: unit test, Done.

+- (void)setType:(NSInteger)type {
+  _type = type;
+}
+
+@end