Remove non-MD code from Bluetooth tray.

ash/common/system/chromeos/bluetooth/tray_bluetooth.cc

 // Copyright (c) 2012 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.
 
 #include "ash/common/system/chromeos/bluetooth/tray_bluetooth.h"
 
-#include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/session/session_state_delegate.h"
 #include "ash/common/system/tray/hover_highlight_view.h"
 #include "ash/common/system/tray/system_tray.h"
 #include "ash/common/system/tray/system_tray_delegate.h"
 #include "ash/common/system/tray/system_tray_notifier.h"
 #include "ash/common/system/tray/throbber_view.h"

[tdanderson, 2017/02/10 15:51:01]

I think this can be removed too

[fukino, 2017/02/11 00:25:01]

ThrobberView is still in use for connecting rows.

[tdanderson, 2017/02/13 17:14:59]

Acknowledged.

 #include "ash/common/system/tray/tray_constants.h"
 #include "ash/common/system/tray/tray_details_view.h"
 #include "ash/common/system/tray/tray_item_more.h"
 #include "ash/common/system/tray/tray_popup_header_button.h"

[tdanderson, 2017/02/10 15:51:01]

Can also be removed

[fukino, 2017/02/11 00:25:01]

Done.

 #include "ash/common/system/tray/tray_popup_item_style.h"
 #include "ash/common/system/tray/tray_popup_utils.h"
 #include "ash/common/system/tray/tri_view.h"
 #include "ash/common/wm_shell.h"
 #include "ash/resources/vector_icons/vector_icons.h"
 #include "device/bluetooth/bluetooth_common.h"
 #include "grit/ash_resources.h"
 #include "grit/ash_strings.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/color_palette.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/gfx/vector_icons_public.h"
 #include "ui/views/controls/button/toggle_button.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/controls/progress_bar.h"
 #include "ui/views/controls/scroll_view.h"
 #include "ui/views/controls/separator.h"
 #include "ui/views/layout/box_layout.h"
 
 namespace ash {
 namespace tray {
 namespace {
 
-bool UseMd() {
-  return MaterialDesignController::IsSystemTrayMenuMaterial();
-}
-
 // Updates bluetooth device |device| in the |list|. If it is new, append to the
 // end of the |list|; otherwise, keep it at the same place, but update the data
 // with new device info provided by |device|.
 void UpdateBluetoothDeviceListHelper(BluetoothDeviceList* list,
                                      const BluetoothDeviceInfo& device) {
   for (BluetoothDeviceList::iterator it = list->begin(); it != list->end();
        ++it) {
     if ((*it).address == device.address) {
       *it = device;
       return;
@@ skipping 54 matching lines @@
                    : ash::kSystemMenuBluetoothIcon;
 }
 
 const int kDisabledPanelLabelBaselineY = 20;
 
 }  // namespace
 
 class BluetoothDefaultView : public TrayItemMore {
  public:
   BluetoothDefaultView(SystemTrayItem* owner, bool show_more)
-      : TrayItemMore(owner, show_more) {
-    if (!UseMd()) {
-      // The icon doesn't change in non-md.
-      ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
-      SetImage(
-          *bundle.GetImageNamed(IDR_AURA_UBER_TRAY_BLUETOOTH).ToImageSkia());
-    }
-  }
+      : TrayItemMore(owner, show_more) {}
 
   ~BluetoothDefaultView() override {}
 
   void Update() {
     SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
     const bool enabled = delegate->GetBluetoothEnabled();
     if (delegate->GetBluetoothAvailable()) {
       ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
       const base::string16 label = rb.GetLocalizedString(
           enabled ? IDS_ASH_STATUS_TRAY_BLUETOOTH_ENABLED
@@ skipping 17 matching lines @@
             ? TrayPopupItemStyle::ColorStyle::ACTIVE
             : delegate->GetBluetoothAvailable()
                   ? TrayPopupItemStyle::ColorStyle::INACTIVE
                   : TrayPopupItemStyle::ColorStyle::DISABLED);
 
     return style;
   }
 
   void UpdateStyle() override {
     TrayItemMore::UpdateStyle();
-
-    if (!UseMd())
-      return;
-
     std::unique_ptr<TrayPopupItemStyle> style = CreateStyle();
     SetImage(gfx::CreateVectorIcon(GetCurrentIcon(), style->GetIconColor()));
   }
 
  private:
   const gfx::VectorIcon& GetCurrentIcon() {
     SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
     if (!delegate->GetBluetoothEnabled())
       return kSystemMenuBluetoothDisabledIcon;
 
@@ skipping 11 matching lines @@
   }
 
   DISALLOW_COPY_AND_ASSIGN(BluetoothDefaultView);
 };
 
 class BluetoothDetailedView : public TrayDetailsView {
  public:
   BluetoothDetailedView(SystemTrayItem* owner, LoginStatus login)
       : TrayDetailsView(owner),
         login_(login),
-        manage_devices_(nullptr),
-        throbber_(nullptr),
-        toggle_bluetooth_(nullptr),
         enable_bluetooth_(nullptr),
         toggle_(nullptr),
         settings_(nullptr),
         disabled_panel_(nullptr) {
     CreateItems();
   }
 
   ~BluetoothDetailedView() override {
     // Stop discovering bluetooth devices when exiting BT detailed view.
     BluetoothStopDiscovering();
   }
 
   void Update() {
     BluetoothStartDiscovering();
     UpdateBluetoothDeviceList();
 
     // Update UI.
     UpdateDeviceScrollList();
     UpdateHeaderEntry();
     Layout();
   }
 
  private:
   void CreateItems() {
     CreateScrollableList();
-    AppendSettingsEntries();
     CreateTitleRow(IDS_ASH_STATUS_TRAY_BLUETOOTH);
   }
 
   void BluetoothStartDiscovering() {
     SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
     if (delegate->GetBluetoothDiscovering()) {
       ShowLoadingIndicator();
       return;
     }
     HideLoadingIndicator();
@@ skipping 12 matching lines @@
   void UpdateBluetoothDeviceList() {
     std::set<std::string> new_connecting_devices;
     std::set<std::string> new_connected_devices;
     std::set<std::string> new_paired_not_connected_devices;
     std::set<std::string> new_discovered_not_paired_devices;
 
     BluetoothDeviceList list;
     WmShell::Get()->system_tray_delegate()->GetAvailableBluetoothDevices(&list);
     for (size_t i = 0; i < list.size(); ++i) {
       if (list[i].connecting) {
-        if (!UseMd()) {
-          list[i].display_name = l10n_util::GetStringFUTF16(
-              IDS_ASH_STATUS_TRAY_BLUETOOTH_CONNECTING, list[i].display_name);

[tdanderson, 2017/02/10 15:51:01]

I think you can remove this string resource too since it doesn't look to be used
in MD

[fukino, 2017/02/11 00:25:01]

You are right. Thank you for pointing this out!
Done.

-        }
         new_connecting_devices.insert(list[i].address);
         UpdateBluetoothDeviceListHelper(&connecting_devices_, list[i]);
       } else if (list[i].connected && list[i].paired) {
         new_connected_devices.insert(list[i].address);
         UpdateBluetoothDeviceListHelper(&connected_devices_, list[i]);
       } else if (list[i].paired) {
         new_paired_not_connected_devices.insert(list[i].address);
         UpdateBluetoothDeviceListHelper(&paired_not_connected_devices_,
                                         list[i]);
       } else {
@@ skipping 10 matching lines @@
                                            new_paired_not_connected_devices);
     RemoveObsoleteBluetoothDevicesFromList(&discovered_not_paired_devices_,
                                            new_discovered_not_paired_devices);
   }
 
   void UpdateHeaderEntry() {
     bool is_bluetooth_enabled =
         WmShell::Get()->system_tray_delegate()->GetBluetoothEnabled();
     if (toggle_)
       toggle_->SetIsOn(is_bluetooth_enabled, false);
-    else if (toggle_bluetooth_)
-      toggle_bluetooth_->SetToggled(!is_bluetooth_enabled);
   }
 
   void UpdateDeviceScrollList() {
     device_map_.clear();
     scroll_content()->RemoveAllChildViews(true);
     enable_bluetooth_ = nullptr;
 
     SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
     bool bluetooth_enabled = delegate->GetBluetoothEnabled();
     bool bluetooth_available = delegate->GetBluetoothAvailable();
-    if (bluetooth_available && !bluetooth_enabled && toggle_bluetooth_) {
-      enable_bluetooth_ = AddScrollListItem(
-          l10n_util::GetStringUTF16(IDS_ASH_STATUS_TRAY_ENABLE_BLUETOOTH),

[tdanderson, 2017/02/10 15:51:01]

This string also looks like it's no longer used

[fukino, 2017/02/11 00:25:01]

DONE.
I removed IDS_ASH_STATUS_TRAY_DISABLE_BLUETOOTH and
IDS_ASH_STATUS_TRAY_BLUETOOTH_MANAGE_DEVICES as well.

[tdanderson, 2017/02/13 17:14:59]

Great, thanks for catching those other two strings.

[fukino, 2017/02/13 19:56:17]

I should have noticed earlier. Thank you for catching them!

-          false /* highlight */, false /* checked */, true /* enabled */);
-    }
 
     // If Bluetooth is disabled, show a panel which only indicates that it is
     // disabled, instead of the scroller with Bluetooth devices.
-    if (UseMd()) {
-      if (bluetooth_enabled) {
-        HideDisabledPanel();
-      } else {
-        ShowDisabledPanel();
-        return;
-      }
+    if (bluetooth_enabled) {
+      HideDisabledPanel();
+    } else {
+      ShowDisabledPanel();
+      return;
     }
 
     // Add paired devices (and their section header in MD) in the list.
     size_t num_paired_devices = connected_devices_.size() +
                                 connecting_devices_.size() +
                                 paired_not_connected_devices_.size();
     if (num_paired_devices > 0) {
-      if (UseMd())
-        AddSubHeader(IDS_ASH_STATUS_TRAY_BLUETOOTH_PAIRED_DEVICES);
+      AddSubHeader(IDS_ASH_STATUS_TRAY_BLUETOOTH_PAIRED_DEVICES);
       AppendSameTypeDevicesToScrollList(connected_devices_, true, true,
                                         bluetooth_enabled);
       AppendSameTypeDevicesToScrollList(connecting_devices_, true, false,
                                         bluetooth_enabled);
       AppendSameTypeDevicesToScrollList(paired_not_connected_devices_, false,
                                         false, bluetooth_enabled);
     }
 
     // Add paired devices (and their section header in MD) in the list.
     if (discovered_not_paired_devices_.size() > 0) {
-      if (UseMd()) {
-        if (num_paired_devices > 0)
-          AddSubHeader(IDS_ASH_STATUS_TRAY_BLUETOOTH_UNPAIRED_DEVICES);
-      } else {
-        AddScrollSeparator();
-      }
+      if (num_paired_devices > 0)
+        AddSubHeader(IDS_ASH_STATUS_TRAY_BLUETOOTH_UNPAIRED_DEVICES);
       AppendSameTypeDevicesToScrollList(discovered_not_paired_devices_, false,
                                         false, bluetooth_enabled);
     }
 
     // Show user Bluetooth state if there is no bluetooth devices in list.
     if (device_map_.size() == 0) {
       if (bluetooth_available && bluetooth_enabled) {
-        AddScrollListItem(l10n_util::GetStringUTF16(
-                              IDS_ASH_STATUS_TRAY_BLUETOOTH_DISCOVERING),
-                          false /* highlight */, false /* checked */,
-                          true /* enabled */);
+        HoverHighlightView* container = new HoverHighlightView(this);

[tdanderson, 2017/02/10 15:51:01]

If I understand correctly, is this just displaying a simple "Scanning for
devices..." message at the top of the detailed view? Is there any way you can
grab a screenshot of what this looks like?

We should maybe be displaying something nicer for MD, with the message centered
along with an icon of some sort (similar to what we do for the "Bluetooth
disabled" case. But that can be done in a separate patch set. Can you please
file a bug and attach what this looks like currently? We can then discuss with
Sebastien what should happen.

[fukino, 2017/02/11 00:25:01]

I filed crbug.com/691138 and attached a screenshot on it.
Thank you for the suggestion.

[tdanderson, 2017/02/13 17:14:59]

Thank you!

+        container->AddLabel(l10n_util::GetStringUTF16(
+                                IDS_ASH_STATUS_TRAY_BLUETOOTH_DISCOVERING),
+                            gfx::ALIGN_LEFT, false);
+        scroll_content()->AddChildView(container);
       }
     }
 
     scroll_content()->InvalidateLayout();
   }
 
   void AppendSameTypeDevicesToScrollList(const BluetoothDeviceList& list,
                                          bool highlight,
                                          bool checked,
                                          bool enabled) {
     for (size_t i = 0; i < list.size(); ++i) {
       HoverHighlightView* container = nullptr;
-      if (UseMd()) {
-        gfx::ImageSkia icon_image = CreateVectorIcon(
-            GetBluetoothDeviceIcon(list[i].device_type, list[i].connected),
-            kMenuIconColor);
-        container = AddScrollListItemMd(list[i].display_name, icon_image,
-                                        list[i].connected, list[i].connecting);
-      } else {
-        container = AddScrollListItem(list[i].display_name, highlight, checked,
-                                      enabled);
-      }
+      gfx::ImageSkia icon_image = CreateVectorIcon(
+          GetBluetoothDeviceIcon(list[i].device_type, list[i].connected),
+          kMenuIconColor);
+      container = AddScrollListItem(list[i].display_name, icon_image,
+                                    list[i].connected, list[i].connecting);
       device_map_[container] = list[i].address;
     }
   }
 
-  // TODO(fukino): Remove this code when material design is enabled by default,
-  // since AddScrollListItem should be used only in the old design.
-  // See crbug.com/614453".
   HoverHighlightView* AddScrollListItem(const base::string16& text,
-                                        bool highlight,
-                                        bool checked,
-                                        bool enabled) {
-    HoverHighlightView* container = new HoverHighlightView(this);
-    views::Label* label =
-        container->AddCheckableLabel(text, highlight, checked);
-    label->SetEnabled(enabled);
-    scroll_content()->AddChildView(container);
-    return container;
-  }
-
-  HoverHighlightView* AddScrollListItemMd(const base::string16& text,
-                                          const gfx::ImageSkia& image,
-                                          bool connected,
-                                          bool connecting) {
-    DCHECK(UseMd());
+                                        const gfx::ImageSkia& image,
+                                        bool connected,
+                                        bool connecting) {
     HoverHighlightView* container = new HoverHighlightView(this);
     if (connected) {
-      SetupConnectedItemMd(container, text, image);
+      SetupConnectedItem(container, text, image);
     } else if (connecting) {
-      SetupConnectingItemMd(container, text, image);
+      SetupConnectingItem(container, text, image);
     } else {
       container->AddIconAndLabel(image, text, false);
     }
     scroll_content()->AddChildView(container);
     return container;
   }
 
   void AddSubHeader(int message_id) {
     TriView* header = TrayPopupUtils::CreateSubHeaderRowView();
     TrayPopupUtils::ConfigureAsStickyHeader(header);
 
     views::Label* label = TrayPopupUtils::CreateDefaultLabel();
     label->SetText(l10n_util::GetStringUTF16(message_id));
     TrayPopupItemStyle style(TrayPopupItemStyle::FontStyle::SUB_HEADER);
     style.SetupLabel(label);
     header->AddView(TriView::Container::CENTER, label);
 
     scroll_content()->AddChildView(header);
   }
 
-  void SetupConnectedItemMd(HoverHighlightView* container,
-                            const base::string16& text,
-                            const gfx::ImageSkia& image) {
+  void SetupConnectedItem(HoverHighlightView* container,
+                          const base::string16& text,
+                          const gfx::ImageSkia& image) {
     container->AddIconAndLabels(
         image, text, l10n_util::GetStringUTF16(
                          IDS_ASH_STATUS_TRAY_NETWORK_STATUS_CONNECTED));
     TrayPopupItemStyle style(TrayPopupItemStyle::FontStyle::CAPTION);
     style.set_color_style(TrayPopupItemStyle::ColorStyle::CONNECTED);
     style.SetupLabel(container->sub_text_label());
   }
 
-  void SetupConnectingItemMd(HoverHighlightView* container,
-                             const base::string16& text,
-                             const gfx::ImageSkia& image) {
+  void SetupConnectingItem(HoverHighlightView* container,
+                           const base::string16& text,
+                           const gfx::ImageSkia& image) {
     container->AddIconAndLabels(
         image, text, l10n_util::GetStringUTF16(
                          IDS_ASH_STATUS_TRAY_NETWORK_STATUS_CONNECTING));
     ThrobberView* throbber = new ThrobberView;
     throbber->Start();
     container->AddRightView(throbber);
   }
 
-  // Add settings entries.
-  void AppendSettingsEntries() {
-    // Do not append the bottom button row in material design; this is replaced
-    // by the settings button in the header row.
-    if (UseMd())
-      return;
-
-    if (!WmShell::Get()->system_tray_delegate()->ShouldShowSettings())
-      return;
-
-    // Add bluetooth device requires a browser window, hide it for non logged in
-    // user.
-    if (!TrayPopupUtils::CanOpenWebUISettings(login_))
-      return;
-
-    SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
-    ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
-    HoverHighlightView* container = new HoverHighlightView(this);
-    container->AddLabel(
-        rb.GetLocalizedString(IDS_ASH_STATUS_TRAY_BLUETOOTH_MANAGE_DEVICES),
-        gfx::ALIGN_LEFT, false /* highlight */);
-    container->SetEnabled(delegate->GetBluetoothAvailable());
-    AddChildView(container);
-    manage_devices_ = container;
-  }
-
   // Returns true if the device with |device_id| is found in |device_list|,
   // and the display_name of the device will be returned in |display_name| if
   // it's not NULL.
   bool FoundDevice(const std::string& device_id,
                    const BluetoothDeviceList& device_list,
                    base::string16* display_name,
                    device::BluetoothDeviceType* device_type) {
     for (size_t i = 0; i < device_list.size(); ++i) {
       if (device_list[i].address == device_id) {
         if (display_name)
@@ skipping 10 matching lines @@
   // or disconnected if such an operation is going to be performed underway.
   void UpdateClickedDevice(const std::string& device_id,
                            views::View* item_container) {
     base::string16 display_name;
     device::BluetoothDeviceType device_type;
     if (FoundDevice(device_id, paired_not_connected_devices_, &display_name,
                     &device_type)) {
       item_container->RemoveAllChildViews(true);
       HoverHighlightView* container =
           static_cast<HoverHighlightView*>(item_container);
-      if (UseMd()) {
-        TrayPopupItemStyle style(
-            TrayPopupItemStyle::FontStyle::DETAILED_VIEW_LABEL);
-        gfx::ImageSkia icon_image = CreateVectorIcon(
-            GetBluetoothDeviceIcon(device_type, false), style.GetIconColor());
-        SetupConnectingItemMd(container, display_name, icon_image);
-      } else {
-        display_name = l10n_util::GetStringFUTF16(
-            IDS_ASH_STATUS_TRAY_BLUETOOTH_CONNECTING, display_name);
-        container->AddCheckableLabel(display_name, true /* highlight */, false);
-      }
+      TrayPopupItemStyle style(
+          TrayPopupItemStyle::FontStyle::DETAILED_VIEW_LABEL);
+      gfx::ImageSkia icon_image = CreateVectorIcon(
+          GetBluetoothDeviceIcon(device_type, false), style.GetIconColor());
+      SetupConnectingItem(container, display_name, icon_image);
       scroll_content()->SizeToPreferredSize();
       scroller()->Layout();
     }
   }
 
   // TrayDetailsView:
   void HandleViewClicked(views::View* view) override {
-    if (view == manage_devices_) {
-      ShowSettings();
-      return;
-    }
-
     SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
     if (view == enable_bluetooth_) {

[tdanderson, 2017/02/10 15:51:01]

On line 310 you are removing the instantiation of |enable_bluetooth_|. I think
this member was only relevant to non-MD and can now be removed, can you please
double check? Note the metrics collection below should also probably be
collected when the MD toggle is clicked.

[fukino, 2017/02/11 00:25:01]

Removed |enable_bluetooth_| and its related code.
As for the metrics, I filed crbug.com/691152 to make the issue clear.

[tdanderson, 2017/02/13 17:14:59]

Acknowledged, thanks.

       WmShell::Get()->RecordUserMetricsAction(
           delegate->GetBluetoothEnabled() ? UMA_STATUS_AREA_BLUETOOTH_DISABLED
                                           : UMA_STATUS_AREA_BLUETOOTH_ENABLED);
       delegate->ToggleBluetooth();
       return;
     }
 
     if (!delegate->GetBluetoothEnabled())
       return;
 
     std::map<views::View*, std::string>::iterator find;
     find = device_map_.find(view);
     if (find == device_map_.end())
       return;
 
     const std::string device_id = find->second;
     if (FoundDevice(device_id, connecting_devices_, nullptr, nullptr))
       return;
 
     UpdateClickedDevice(device_id, view);
     delegate->ConnectToBluetoothDevice(device_id);
   }
 
   void HandleButtonPressed(views::Button* sender,
                            const ui::Event& event) override {
-    if (UseMd()) {
-      if (sender == toggle_)
-        WmShell::Get()->system_tray_delegate()->ToggleBluetooth();
-      else if (sender == settings_)
-        ShowSettings();
-      else
-        NOTREACHED();
-      return;
-    }
+    if (sender == toggle_)
+      WmShell::Get()->system_tray_delegate()->ToggleBluetooth();
+    else if (sender == settings_)
+      ShowSettings();
 
-    SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
-    if (sender == toggle_bluetooth_)
-      delegate->ToggleBluetooth();
-    else
-      NOTREACHED();
+    NOTREACHED();
   }
 
   void CreateExtraTitleRowButtons() override {
     if (login_ == LoginStatus::LOCKED)
       return;
 
-    if (UseMd()) {
-      DCHECK(!toggle_);
-      DCHECK(!settings_);
+    DCHECK(!toggle_);
+    DCHECK(!settings_);
 
-      tri_view()->SetContainerVisible(TriView::Container::END, true);
+    tri_view()->SetContainerVisible(TriView::Container::END, true);
 
-      toggle_ = TrayPopupUtils::CreateToggleButton(
-          this, IDS_ASH_STATUS_TRAY_BLUETOOTH);
-      tri_view()->AddView(TriView::Container::END, toggle_);
+    toggle_ =
+        TrayPopupUtils::CreateToggleButton(this, IDS_ASH_STATUS_TRAY_BLUETOOTH);
+    tri_view()->AddView(TriView::Container::END, toggle_);
 
-      settings_ = CreateSettingsButton(login_);
-      tri_view()->AddView(TriView::Container::END, settings_);
-      return;
-    }
-
-    throbber_ = new ThrobberView;
-    throbber_->SetTooltipText(
-        l10n_util::GetStringUTF16(IDS_ASH_STATUS_TRAY_BLUETOOTH_DISCOVERING));
-    title_row()->AddViewToRowNonMd(throbber_, false);
-
-    // Do not allow toggling bluetooth in the lock screen.
-    SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
-    toggle_bluetooth_ =
-        new TrayPopupHeaderButton(this, IDR_AURA_UBER_TRAY_BLUETOOTH_ENABLED,
-                                  IDR_AURA_UBER_TRAY_BLUETOOTH_DISABLED,
-                                  IDR_AURA_UBER_TRAY_BLUETOOTH_ENABLED_HOVER,
-                                  IDR_AURA_UBER_TRAY_BLUETOOTH_DISABLED_HOVER,
-                                  IDS_ASH_STATUS_TRAY_BLUETOOTH);
-    toggle_bluetooth_->SetToggled(!delegate->GetBluetoothEnabled());
-    toggle_bluetooth_->SetTooltipText(
-        l10n_util::GetStringUTF16(IDS_ASH_STATUS_TRAY_DISABLE_BLUETOOTH));
-    toggle_bluetooth_->SetToggledTooltipText(
-        l10n_util::GetStringUTF16(IDS_ASH_STATUS_TRAY_ENABLE_BLUETOOTH));
-    toggle_bluetooth_->EnableCanvasFlippingForRTLUI(false);
-    title_row()->AddViewToRowNonMd(toggle_bluetooth_, true);
+    settings_ = CreateSettingsButton(login_);
+    tri_view()->AddView(TriView::Container::END, settings_);
   }
 
   void ShowSettings() {
     if (TrayPopupUtils::CanOpenWebUISettings(login_)) {
       WmShell::Get()->system_tray_delegate()->ManageBluetoothDevices();
       owner()->system_tray()->CloseSystemBubble();
     }
   }
 
   void ShowLoadingIndicator() {
-    if (throbber_) {
-      throbber_->Start();
-    } else if (UseMd()) {
-      // Setting a value of -1 gives progress_bar an infinite-loading behavior.
-      ShowProgress(-1, true);
-    }
+    // Setting a value of -1 gives progress_bar an infinite-loading behavior.
+    ShowProgress(-1, true);
   }
 
-  void HideLoadingIndicator() {
-    if (throbber_)
-      throbber_->Stop();
-    else if (UseMd())
-      ShowProgress(0, false);
-  }
+  void HideLoadingIndicator() { ShowProgress(0, false); }

[tdanderson, 2017/02/10 15:51:01]

Please format as

void ...() {
  ...
}

[fukino, 2017/02/11 00:25:01]

Done, but I had to ignore presubmit warning "Please run git cl format ash" to
upload the patch.
Can I ignore it?

[tdanderson, 2017/02/13 17:14:59]

Oh, that's strange. In this case just go with what 'git cl format ash' says.

[fukino, 2017/02/13 19:56:18]

Got it. I did git cl format for PS3.

 
   void ShowDisabledPanel() {
-    DCHECK(UseMd());
     DCHECK(scroller());
     if (!disabled_panel_) {
       disabled_panel_ = CreateDisabledPanel();
       // Insert |disabled_panel_| before the scroller, since the scroller will
       // have unnecessary bottom border when it is not the last child.
       AddChildViewAt(disabled_panel_, GetIndexOf(scroller()));
       // |disabled_panel_| need to fill the remaining space below the title row
       // so that the inner contents of |disabled_panel_| are placed properly.
       box_layout()->SetFlexForView(disabled_panel_, 1);
     }
     disabled_panel_->SetVisible(true);
     scroller()->SetVisible(false);
   }
 
   void HideDisabledPanel() {
-    DCHECK(UseMd());
     DCHECK(scroller());
     if (disabled_panel_)
       disabled_panel_->SetVisible(false);
     scroller()->SetVisible(true);
   }
 
   views::View* CreateDisabledPanel() {
     views::View* container = new views::View;
     views::BoxLayout* box_layout =
         new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0);
@@ skipping 23 matching lines @@
     // icon is vertically aligned to center of the container.
     image_view->SetBorder(
         views::CreateEmptyBorder(label->GetPreferredSize().height(), 0, 0, 0));
     return container;
   }
 
   LoginStatus login_;
 
   std::map<views::View*, std::string> device_map_;
 
-  // Not used in material design.
-  views::View* manage_devices_;
-
-  // Not used in material design.
-  ThrobberView* throbber_;
-
-  // Not used in material design.
-  TrayPopupHeaderButton* toggle_bluetooth_;
-
   HoverHighlightView* enable_bluetooth_;
   BluetoothDeviceList connected_devices_;
   BluetoothDeviceList connecting_devices_;
   BluetoothDeviceList paired_not_connected_devices_;
   BluetoothDeviceList discovered_not_paired_devices_;
 
-  // The on/off toggle button used in material design.
   views::ToggleButton* toggle_;
-
-  // Only used in material design.
   views::Button* settings_;
 
-  // Only used in material design.
   // The container of the message "Bluetooth is disabled" and an icon. It should
   // be shown instead of Bluetooth device list when Bluetooth is disabled.
   views::View* disabled_panel_;
 
   DISALLOW_COPY_AND_ASSIGN(BluetoothDetailedView);
 };
 
 }  // namespace tray
 
 TrayBluetooth::TrayBluetooth(SystemTray* system_tray)
@@ skipping 49 matching lines @@
     detailed_->Update();
 }
 
 void TrayBluetooth::OnBluetoothDiscoveringChanged() {
   if (!detailed_)
     return;
   detailed_->Update();
 }
 
 }  // namespace ash