[AiS] changing mac omnibox suggestions form NSMatrix to NSTableView

chrome/browser/ui/cocoa/omnibox/omnibox_popup_matrix.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_MATRIX_H_
 #define CHROME_BROWSER_UI_COCOA_OMNIBOX_OMNIBOX_POPUP_MATRIX_H_
 
 #import <Cocoa/Cocoa.h>
 
 #import "ui/base/cocoa/tracking_area.h"
 #include "ui/base/window_open_disposition.h"
 
+@class OmniboxPopupCell;
+
+@interface OmniboxPopupTableController
+    : NSViewController<NSTableViewDelegate, NSTableViewDataSource> {
+ @private
+  base::scoped_nsobject<NSArray> array_;
+  NSInteger hovered_index_;
+};
+
+// Set the hovered highlight.
+- (void)highlightRowAt:(NSInteger)rowIndex withView:(NSView*)view;
+
+// Which row has the hovered highlight.
+- (NSInteger)highlightedRow;
+
+// Get the drawing rectangle for the given row.
+- (NSRect)rectForRow:(NSInteger)rowIndex;
+
+@end
+
 @class OmniboxPopupMatrix;
 
 class OmniboxPopupMatrixDelegate {
  public:
   // Called when the selection in the matrix changes.
   virtual void OnMatrixRowSelected(OmniboxPopupMatrix* matrix, size_t row) = 0;
 
   // Called when the user clicks on a row.
   virtual void OnMatrixRowClicked(OmniboxPopupMatrix* matrix, size_t row) = 0;
 
   // Called when the user middle clicks on a row.
   virtual void OnMatrixRowMiddleClicked(OmniboxPopupMatrix* matrix,
                                         size_t row) = 0;
 };
 
 // Sets up a tracking area to implement hover by highlighting the cell the mouse
 // is over.
-@interface OmniboxPopupMatrix : NSMatrix {
+@interface OmniboxPopupMatrix : NSTableView {
   OmniboxPopupMatrixDelegate* delegate_;  // weak
   ui::ScopedCrTrackingArea trackingArea_;
+  base::scoped_nsobject<OmniboxPopupTableController> controller_;
 }
 
 // Create a zero-size matrix.
 - (id)initWithDelegate:(OmniboxPopupMatrixDelegate*)delegate;
 
-// Sets the delegate.
-- (void)setDelegate:(OmniboxPopupMatrixDelegate*)delegate;
+// Sets the OmniboxPopupMatrixDelegate which is not the same as the NSTableView
+// delegate.
+- (void)setTheDelegate:(OmniboxPopupMatrixDelegate*)delegate;
 
 // Return the currently highlighted row.  Returns -1 if no row is highlighted.
 - (NSInteger)highlightedRow;
 
+// Select the row by turning it on.  Turns off prior selection.

[Scott Hess - ex-Googler, 2015/05/07 22:35:42]

IMHO, using on and off are themselves weird, they're just implementation
details.  "Move the selection to |rowIndex|." seems reasonable.  Though I'm
always willing to fly without comments if the comment adds nothing above the
literal method signature.

[dschuyler, 2015/05/13 01:41:11]

Done.

+- (void)selectRowAt:(NSInteger)rowIndex;
+
+// Setup the information used by the NSTableView data source.
+- (void)setDataArray:(NSArray*)array;
+
 @end
 
 #endif  // CHROME_BROWSER_UI_COCOA_OMNIBOX_OMNIBOX_POPUP_MATRIX_H_