Omnibox UI Experiments: Port Vertical Layout experiment to Mac Cocoa.

chrome/browser/ui/cocoa/omnibox/omnibox_popup_matrix.mm

Index: chrome/browser/ui/cocoa/omnibox/omnibox_popup_matrix.mm
diff --git a/chrome/browser/ui/cocoa/omnibox/omnibox_popup_matrix.mm b/chrome/browser/ui/cocoa/omnibox/omnibox_popup_matrix.mm
index ca66db2cf4d6bc966e97f3621cdcd0225630ed8d..3596a252864c65bc533a9fd4b020332cd7d70c19 100644
--- a/chrome/browser/ui/cocoa/omnibox/omnibox_popup_matrix.mm
+++ b/chrome/browser/ui/cocoa/omnibox/omnibox_popup_matrix.mm
@@ -4,12 +4,14 @@
 
 #import "chrome/browser/ui/cocoa/omnibox/omnibox_popup_matrix.h"
 
+#include "base/feature_list.h"
 #include "base/logging.h"
 #include "base/mac/foundation_util.h"
 #import "chrome/browser/ui/cocoa/omnibox/omnibox_popup_cell.h"
 #include "chrome/browser/ui/cocoa/omnibox/omnibox_popup_view_mac.h"
 #include "chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.h"
 #include "components/omnibox/browser/autocomplete_result.h"
+#include "components/omnibox/browser/omnibox_field_trial.h"
 
 namespace {
 
@@ -106,8 +108,12 @@ const NSInteger kMiddleButtonNumber = 2;
 }
 
 - (CGFloat)tableView:(NSTableView*)tableView heightOfRow:(NSInteger)row {
-  CGFloat height = [OmniboxPopupCell getContentTextHeight];
-  if ([[array_ objectAtIndex:row] isAnswer]) {
+  bool isAnswer = [[array_ objectAtIndex:row] isAnswer];

[groby-ooo-7-16, 2017/05/31 00:14:51]

BOOL, please :)

[tommycli, 2017/05/31 18:20:40]

Done.

+  bool isTwoLine = !isAnswer && base::FeatureList::IsEnabled(

[groby-ooo-7-16, 2017/05/31 00:14:51]

isDoubleLine or usesTwoLines, please. (isTwoLine seems like it's missing a word)

[tommycli, 2017/05/31 18:20:40]

Done.

+                                    omnibox::kUIExperimentVerticalLayout);
+  CGFloat height = [OmniboxPopupCell getContentTextHeight:isTwoLine];
+
+  if (isAnswer) {
     OmniboxPopupMatrix* matrix =
         base::mac::ObjCCastStrict<OmniboxPopupMatrix>(tableView);
     NSRect rowRect = [tableView rectOfColumn:0];