Tab Search: Disable Tab Search button in incognito mode

Bug: 1099917
Change-Id: Ie74de530af3fdb7aa87ed14ac14aba47c7fd5b4d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432677
Commit-Queue: Thomas Lukaszewicz <tluk@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#811078}

chrome/browser/ui/views/frame/tab_strip_region_view.cc

@@ -8,6 +8,7 @@
 #include "chrome/browser/ui/layout_constants.h"
 #include "chrome/browser/ui/ui_features.h"
 #include "chrome/browser/ui/views/tabs/tab_search_button.h"
+#include "chrome/browser/ui/views/tabs/tab_strip.h"
 #include "chrome/browser/ui/views/tabs/tab_strip_controller.h"
 #include "chrome/grit/generated_resources.h"
 #include "ui/accessibility/ax_node_data.h"
@@ -47,7 +48,8 @@ TabStripRegionView::TabStripRegionView(std::unique_ptr<TabStrip> tab_strip) {
                                     tab_strip_container_flex_spec);
 
   if (base::FeatureList::IsEnabled(features::kTabSearch) &&
-      base::FeatureList::IsEnabled(features::kTabSearchFixedEntrypoint)) {
+      base::FeatureList::IsEnabled(features::kTabSearchFixedEntrypoint) &&
+      !tab_strip_->controller()->GetProfile()->IsIncognitoProfile()) {
     // TODO(tluk): |tab_search_container| is only needed here so the tab search
     // button can be vertically centered. This can be removed if FlexLayout is
     // updated to support per-child cross-axis alignment.

chrome/browser/ui/views/frame/tab_strip_region_view.h

@@ -9,6 +9,7 @@
 #include "ui/views/accessible_pane_view.h"
 
 class TabSearchButton;
+class TabStrip;
 
 // Container for the tabstrip, new tab button, and reserved grab handle space.
 // TODO (https://crbug.com/949660) Under construction.