Add signal strength indicator icon to WebBluetooth chooser on non-Mac desktops

chrome/browser/ui/views/chooser_content_view.cc

 // Copyright 2016 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 "chrome/browser/ui/views/chooser_content_view.h"
 
+#include "base/numerics/safe_conversions.h"
 #include "chrome/grit/generated_resources.h"
+#include "grit/ui_resources.h"
 #include "ui/base/l10n/l10n_util.h"
+#include "ui/base/resource/resource_bundle.h"
+#include "ui/gfx/image/image_skia.h"
 #include "ui/views/controls/link.h"
 #include "ui/views/controls/styled_label.h"
 #include "ui/views/controls/table/table_view.h"
 #include "ui/views/controls/throbber.h"
 #include "ui/views/layout/fill_layout.h"
 
 namespace {
 
 const int kChooserWidth = 330;
 
 const int kChooserHeight = 220;
 
 const int kThrobberDiameter = 24;
 
+// The lookup table for strength bar image.
+const int strength_bar_ids[5] = {IDR_SIGNAL_0_BAR, IDR_SIGNAL_1_BAR,
+                                 IDR_SIGNAL_2_BAR, IDR_SIGNAL_3_BAR,
+                                 IDR_SIGNAL_4_BAR};
+
 }  // namespace
 
 ChooserContentView::ChooserContentView(
     views::TableViewObserver* table_view_observer,
     std::unique_ptr<ChooserController> chooser_controller)
     : chooser_controller_(std::move(chooser_controller)) {
   chooser_controller_->set_view(this);
   std::vector<ui::TableColumn> table_columns;
   table_columns.push_back(ui::TableColumn());
-  table_view_ =
-      new views::TableView(this, table_columns, views::TEXT_ONLY, true);
+  table_view_ = new views::TableView(
+      this, table_columns,
+      chooser_controller_->ShouldShowIconBeforeText() ? views::ICON_AND_TEXT
+                                                      : views::TEXT_ONLY,
+      true /* single_selection */);
   table_view_->set_select_on_remove(false);
   table_view_->SetObserver(table_view_observer);
   table_view_->SetEnabled(chooser_controller_->NumOptions() > 0);
 
   SetLayoutManager(new views::FillLayout());
   views::View* table_parent = table_view_->CreateParentIfNecessary();
   AddChildView(table_parent);
 
   throbber_ = new views::Throbber();
   // Set the throbber in the center of the chooser.
@@ skipping 12 matching lines @@
     discovery_state_->set_listener(nullptr);
 }
 
 gfx::Size ChooserContentView::GetPreferredSize() const {
   return gfx::Size(kChooserWidth, kChooserHeight);
 }
 
 int ChooserContentView::RowCount() {
   // When there are no devices, the table contains a message saying there
   // are no devices, so the number of rows is always at least 1.
-  return std::max(static_cast<int>(chooser_controller_->NumOptions()), 1);
+  return std::max(base::checked_cast<int>(chooser_controller_->NumOptions()),
+                  1);
 }
 
 base::string16 ChooserContentView::GetText(int row, int column_id) {
-  int num_options = static_cast<int>(chooser_controller_->NumOptions());
+  int num_options = base::checked_cast<int>(chooser_controller_->NumOptions());
   if (num_options == 0) {
     DCHECK_EQ(0, row);
     return chooser_controller_->GetNoOptionsText();
   }
 
   DCHECK_GE(row, 0);
   DCHECK_LT(row, num_options);
   return chooser_controller_->GetOption(static_cast<size_t>(row));
 }
 
 void ChooserContentView::SetObserver(ui::TableModelObserver* observer) {}
 
+gfx::ImageSkia ChooserContentView::GetIcon(int row) {
+  DCHECK(chooser_controller_->ShouldShowIconBeforeText());
+
+  size_t num_options = chooser_controller_->NumOptions();
+  if (num_options == 0) {
+    DCHECK_EQ(0, row);
+    return gfx::ImageSkia();
+  }
+
+  DCHECK_GE(row, 0);
+  DCHECK_LT(row, base::checked_cast<int>(num_options));
+
+  int level =
+      chooser_controller_->GetSignalStrengthLevel(static_cast<size_t>(row));
+
+  if (level == -1)
+    return gfx::ImageSkia();
+
+  DCHECK_GE(level, 0);
+  DCHECK_LT(level,
+            base::checked_cast<int>(sizeof(strength_bar_ids) / sizeof(int)));

[msw, 2016/08/16 18:55:13]

nit: use base::checked_cast<int>(arraysize(strength_bar_ids)) see base/macros.h

[juncai, 2016/08/16 19:06:26]

Done.

+
+  return *ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
+      strength_bar_ids[level]);
+}
+
 void ChooserContentView::OnOptionsInitialized() {
   table_view_->OnModelChanged();
   UpdateTableView();
 }
 
 void ChooserContentView::OnOptionAdded(size_t index) {
-  table_view_->OnItemsAdded(static_cast<int>(index), 1);
+  table_view_->OnItemsAdded(base::checked_cast<int>(index), 1);
   UpdateTableView();
   table_view_->SetVisible(true);
   throbber_->SetVisible(false);
   throbber_->Stop();
 }
 
 void ChooserContentView::OnOptionRemoved(size_t index) {
-  table_view_->OnItemsRemoved(static_cast<int>(index), 1);
+  table_view_->OnItemsRemoved(base::checked_cast<int>(index), 1);
   UpdateTableView();
 }
 
 void ChooserContentView::OnAdapterEnabledChanged(bool enabled) {
   // No row is selected since the adapter status has changed.
   // This will also disable the OK button if it was enabled because
   // of a previously selected row.
   table_view_->Select(-1);
   UpdateTableView();
   table_view_->SetVisible(true);
@@ skipping 94 matching lines @@
 }
 
 void ChooserContentView::UpdateTableView() {
   if (chooser_controller_->NumOptions() == 0) {
     table_view_->OnModelChanged();
     table_view_->SetEnabled(false);
   } else {
     table_view_->SetEnabled(true);
   }
 }