Expose thumbnails of pages to iOS share extensions.

ios/chrome/browser/ui/activity_services/share_to_data.mm

 // Copyright 2012 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.
 
 #include "ios/chrome/browser/ui/activity_services/share_to_data.h"
 
 #include "base/logging.h"
 #include "base/strings/sys_string_conversions.h"
 #include "ios/chrome/browser/tabs/tab.h"
 #import "net/base/mac/url_conversions.h"
@@ skipping 20 matching lines @@
 
 @property(nonatomic, readwrite, copy) NSString* title;
 @property(nonatomic, readwrite, assign) BOOL isOriginalTitle;
 @property(nonatomic, readwrite, assign) BOOL isPagePrintable;
 @end
 
 @implementation ShareToData
 
 @synthesize title = title_;
 @synthesize image = image_;
+@synthesize thumbnailGenerator = thumbnailGenerator_;
 @synthesize isOriginalTitle = isOriginalTitle_;
 @synthesize isPagePrintable = isPagePrintable_;
 
 - (id)init {
   NOTREACHED();
   return nil;
 }
 
 - (id)initWithURL:(const GURL&)url
-              title:(NSString*)title
-    isOriginalTitle:(BOOL)isOriginalTitle
-    isPagePrintable:(BOOL)isPagePrintable {
+                 title:(NSString*)title
+       isOriginalTitle:(BOOL)isOriginalTitle
+       isPagePrintable:(BOOL)isPagePrintable
+    thumbnailGenerator:(ThumbnailGeneratorBlock)thumbnailGenerator {
   DCHECK(url.is_valid());
   DCHECK(title);
   self = [super init];
   if (self) {
     url_ = url;
     self.title = title;
     self.isOriginalTitle = isOriginalTitle;
     self.isPagePrintable = isPagePrintable;
+    self.thumbnailGenerator = thumbnailGenerator;
   }
   return self;
 }
 
 - (const GURL&)url {
   return url_;
 }
 
 - (NSURL*)nsurl {
   return net::NSURLWithGURL(url_);
 }
 
 - (BOOL)isEqual:(id)object {
+  // Used for testing.
   if (![object isMemberOfClass:self.class])
     return NO;
   DCHECK(self.url.is_valid());
   DCHECK(self.title);
   ShareToData* other = (ShareToData*)object;
+  CGSize size = CGSizeMake(40, 40);

[Olivier, 2017/01/24 16:54:05]

Add comment.
Block are equal if they give the same thumbnail.

[jif, 2017/01/25 15:17:29]

Ack.
I moved the code which makes this comment obsolete I think.

+  NSData* thumbnailData =
+      UIImagePNGRepresentation(self.thumbnailGenerator(size));
+  NSData* otherThumbnailData =
+      UIImagePNGRepresentation(other.thumbnailGenerator(size));
+  BOOL thumbnailDataIsEqual = thumbnailData == otherThumbnailData ||

[Olivier, 2017/01/24 16:54:05]

I don't think I see anywhere in tests where this can be false (you always have a
single snapshot in tests).
Is this ever effectively used?

[jif, 2017/01/25 15:17:29]

Done.
The bitmap comparison is used in the bvc_unittest.mm.

+                              [thumbnailData isEqual:otherThumbnailData];
   return self.url == other.url && [self.title isEqual:other.title] &&
          self.image == other.image &&
-         self.isOriginalTitle == other.isOriginalTitle;
-}
-
-- (NSUInteger)hash {
-  DCHECK(self.url.is_valid());
-  DCHECK(self.title);
-  const NSUInteger kPrime = 31;
-  NSString* urlString = base::SysUTF8ToNSString(self.url.spec());
-  return kPrime * kPrime * kPrime * urlString.hash +
-         kPrime * kPrime * self.title.hash + kPrime * self.image.hash +
-         (self.isOriginalTitle ? 0 : 1);
+         self.isOriginalTitle == other.isOriginalTitle &&
+         self.isPagePrintable == other.isPagePrintable && thumbnailDataIsEqual;
 }
 
 @end