Add throbber and status text to WebBluetooth chooser UI on non-Mac desktops

chrome/browser/ui/views/chooser_content_view.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_CHOOSER_CONTENT_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_CHOOSER_CONTENT_VIEW_H_
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/strings/string16.h"
-#include "ui/base/ui_base_types.h"
+#include "chrome/browser/chooser_controller/chooser_controller.h"
+#include "ui/base/models/table_model.h"
+#include "ui/views/controls/link_listener.h"
+#include "ui/views/controls/styled_label_listener.h"
 #include "ui/views/view.h"
 
-class ChooserController;
-class ChooserTableModel;
-
-namespace ui {
-class TableModel;
-}
-
 namespace views {
+class Link;
 class StyledLabel;
-class StyledLabelListener;
 class TableView;
 class TableViewObserver;
+class Throbber;
 }
 
 // A bubble or dialog view for choosing among several options in a table.
 // Used for WebUSB/WebBluetooth device selection for Chrome and extensions.
-class ChooserContentView : public views::View {
+class ChooserContentView : public views::View,
+                           public ui::TableModel,
+                           public ChooserController::Observer,
+                           public views::LinkListener,
+                           public views::StyledLabelListener {
  public:
-  ChooserContentView(views::TableViewObserver* observer,
+  ChooserContentView(views::TableViewObserver* table_view_observer,
                      std::unique_ptr<ChooserController> chooser_controller);
   ~ChooserContentView() override;
 
   // views::View:
   gfx::Size GetPreferredSize() const override;
 
+  // ui::TableModel:
+  int RowCount() override;
+  base::string16 GetText(int row, int column_id) override;
+  void SetObserver(ui::TableModelObserver* observer) override;
+
+  // ChooserController::Observer:
+  void OnOptionsInitialized() override;
+  void OnOptionAdded(size_t index) override;
+  void OnOptionRemoved(size_t index) override;
+  void OnAdapterEnabledChanged(bool enabled) override;
+  void OnRefreshStateChanged(bool refreshing) override;
+
+  // views::LinkListener:
+  void LinkClicked(views::Link* source, int event_flags) override;
+
+  // views::StyledLabelListener:
+  void StyledLabelLinkClicked(views::StyledLabel* label,
+                              const gfx::Range& range,
+                              int event_flags) override;
+
   base::string16 GetWindowTitle() const;
   base::string16 GetDialogButtonLabel(ui::DialogButton button) const;
   bool IsDialogButtonEnabled(ui::DialogButton button) const;
   // Ownership of the view is passed to the caller.
-  views::StyledLabel* CreateFootnoteView(
-      views::StyledLabelListener* listener) const;
+  views::View* CreateExtraView();
+  // Ownership of the view is passed to the caller.
+  views::StyledLabel* CreateFootnoteView();
   void Accept();
   void Cancel();
   void Close();
-  void StyledLabelLinkClicked();
-  void UpdateTableModel();
+  void UpdateTableView();
 
   views::TableView* table_view_for_test() const { return table_view_; }
+  views::StyledLabel* styled_label_for_test() const { return styled_label_; }
 
  private:
   std::unique_ptr<ChooserController> chooser_controller_;
-  std::unique_ptr<ChooserTableModel> chooser_table_model_;
-  views::TableView* table_view_;
+  views::TableView* table_view_;                // Weak.
+  views::Throbber* throbber_;                   // Weak.
+  views::Link* discovery_state_ = nullptr;      // Weak.
+  views::StyledLabel* styled_label_ = nullptr;  // Weak. For testing only.

[msw, 2016/07/19 22:08:11]

nit: I'm not sure I'd mark |styled_label_| "For testing only.", even if the only
reason to cache the pointer is for test, the object is still used in production.

[juncai, 2016/07/19 22:58:24]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(ChooserContentView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_CHOOSER_CONTENT_VIEW_H_