[AiS] changing mac omnibox suggestions form NSMatrix to NSTableView

chrome/browser/ui/cocoa/omnibox/omnibox_popup_cell.h

 // Copyright 2013 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.
 
 #ifndef CHROME_BROWSER_UI_COCOA_OMNIBOX_OMNIBOX_POPUP_CELL_H_
 #define CHROME_BROWSER_UI_COCOA_OMNIBOX_OMNIBOX_POPUP_CELL_H_
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/mac/scoped_nsobject.h"
 #include "components/omnibox/autocomplete_match.h"
 
 class OmniboxPopupViewMac;
+@class OmniboxPopupCell;
 
-// OmniboxPopupCell overrides how backgrounds are displayed to
-// handle hover versus selected.  So long as we're in there, it also
-// provides some default initialization.
-@interface OmniboxPopupCell : NSButtonCell {
+@interface OmniboxPopupCellData : NSObject {
  @private
-  // The popup view parent of this cell.
-  OmniboxPopupViewMac* parent_;
-
-  // The match which will be rendered for this row in omnibox dropdown.
-  AutocompleteMatch match_;
-
   // NSAttributedString instances for various match components.
+  base::scoped_nsobject<NSAttributedString> contents_;
   base::scoped_nsobject<NSAttributedString> separator_;
   base::scoped_nsobject<NSAttributedString> description_;
 
   // NOTE: While |prefix_| is used only for postfix suggestions, it still needs
   // to be a member of the class. This allows the |NSAttributedString| instance
   // to stay alive between the call to |drawTitle| and the actual paint event
   // which accesses the |NSAttributedString| instance.
   base::scoped_nsobject<NSAttributedString> prefix_;
 
+  base::scoped_nsobject<NSImage> image_;
+
   // The width of widest match contents in a set of infinite suggestions.
   CGFloat maxMatchContentsWidth_;
 
   // The offset at which the infinite suggestion contents should be displayed.
   CGFloat contentsOffset_;
+
+  bool isContentsRTL_;
+  AutocompleteMatch::Type matchType_;
 }
 
-- (void)setMatch:(const AutocompleteMatch&)match;
+- (id)initWithMatch:(const AutocompleteMatch&)match image:(NSImage*)image;
+
+- (void)drawMatchWithFrame:(NSRect)cellFrame
+                    inCell:(OmniboxPopupCell*)cell
+                    inView:(NSView*)controlView;
+
+- (CGFloat)rowHeight;
 
 - (void)setMaxMatchContentsWidth:(CGFloat)maxMatchContentsWidth;
 
 - (void)setContentsOffset:(CGFloat)contentsOffset;
 
 // Returns the width of the match contents.
 - (CGFloat)getMatchContentsWidth;
 
+@end
+
+// OmniboxPopupCell overrides how backgrounds are displayed to
+// handle hover versus selected.  So long as we're in there, it also
+// provides some default initialization.
+@interface OmniboxPopupCell : NSCell {

[groby-ooo-7-16, 2015/05/20 01:02:30]

NSCell already has a concept of an objectValue - can we use that to store
celldata? (And technically, the NSTableView should populate it anyways)

[dschuyler, 2015/05/21 00:38:39]

The objectValue requires that the value be parse-able by the NSCell.  There is a
similar property called representedObject that works well as a cellData_
alternative.

Done.

[groby-ooo-7-16, 2015/05/21 03:01:47]

You'd _think_ it has to be parsable, but it doesn't :) (The formatter can be
nil, and the world is still fine)

The objectValue must, however, obey the NSCopying protocol

[dschuyler, 2015/05/26 18:40:19]

That makes sense.  Should I change it from representedObject to objectValue?  I
kinda feel like representedObject is more representative of the relationship.

[groby-ooo-7-16, 2015/05/26 21:23:22]

Yes, please. -representedObject is the wrong choice. Quoth the docs: "This
feature is solely for the developer’s convenience. The cell itself does not use
the represented object, except to archive and restore it."

objectValue is the actual value displayed by the object. (Yes, there's some room
for semantics - but the general expectation is that if I change objectValue, the
display changes. If I change representedValue, nobody cares)

(https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/Contro...)

[dschuyler, 2015/05/28 20:34:22]

I love it when stuff is done for my convenience.

Acknowledged.

+ @private
+  base::scoped_nsobject<OmniboxPopupCellData> cellData_;
+}
+
+- (void)setCellData:(OmniboxPopupCellData*)cellData;
+
 // Returns the offset of the start of the contents in the input text for the
 // given match. It is costly to compute this offset, so it is computed once and
 // shared by all OmniboxPopupCell instances through OmniboxPopupViewMac parent.
 + (CGFloat)computeContentsOffset:(const AutocompleteMatch&)match;
 
 @end
 
 #endif  // CHROME_BROWSER_UI_COCOA_OMNIBOX_OMNIBOX_POPUP_CELL_H_