Mac Picker UI: Screen Auto Selection

chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.h

Index: chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.h
diff --git a/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.h b/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.h
index 55191ef1cdb3256e62a513bca7ceb66b0cd6ce5d..a520119c774f9bf34d4f73f7ab10110809f58340 100644
--- a/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.h
+++ b/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.h
@@ -20,6 +20,7 @@
   base::scoped_nsobject<NSString> imageTitle_;
   base::scoped_nsobject<NSImage> image_;
   NSUInteger imageVersion_;
+  bool hiddenTitle_;

[tapted, 2016/07/22 03:43:36]

bool -> BOOL  (and hiddenTitle_ -> titleHidden_)

[qiangchen, 2016/07/22 18:33:54]

Done.

 }
 
 // Designated initializer.
@@ -41,6 +42,9 @@
 // Sets the label of the item to be displayed in |IKImageBrowserView|.
 - (void)setImageTitle:(NSString*)imageTitle;
 
+// Sets true, if we do not want to show the title.
+- (void)setTitleHidden:(bool)hidden;

[tapted, 2016/07/22 03:43:36]

for simply types we usually just use properties. In the header (before the
initializer above)

@property(assign, nonatomic) BOOL titleHidden;


then in the .mm (also just before the initializer)

@synthesize titleHidden = titleHidden_;

you don't need an implementaiton of - (void)setTitleHidden:

[qiangchen, 2016/07/22 18:33:54]

Done.

+
 @end
 
 #endif  // CHROME_BROWSER_UI_COCOA_MEDIA_PICKER_DESKTOP_MEDIA_PICKER_ITEM_H_