Chooser spinner should disappear as soon as one device shows up

chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.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/extensions/chooser_dialog_view.h"
 
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chooser_controller/mock_chooser_controller.h"
 #include "chrome/browser/ui/views/chooser_content_view.h"
@@ skipping 216 matching lines @@
   // becomes invalid, the OK button is disabled.
   EXPECT_FALSE(ok_button_->enabled());
   EXPECT_TRUE(cancel_button_->enabled());
 
   mock_chooser_controller_->OnAdapterPresenceChanged(
       content::BluetoothChooser::AdapterPresence::POWERED_ON);
   EXPECT_FALSE(ok_button_->enabled());
   EXPECT_TRUE(cancel_button_->enabled());
 }
 
-TEST_F(ChooserDialogViewTest, DiscoveringAndIdle) {
+TEST_F(ChooserDialogViewTest, DiscoveringAndNoOptionAddedAndIdle) {
   mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("a"));
   mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("b"));
   mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("c"));
   table_view_->Select(1);
   EXPECT_TRUE(ok_button_->enabled());
   EXPECT_TRUE(cancel_button_->enabled());
 
   mock_chooser_controller_->OnDiscoveryStateChanged(
       content::BluetoothChooser::DiscoveryState::DISCOVERING);
   // OK button is disabled since the chooser is refreshing options.
   EXPECT_FALSE(ok_button_->enabled());
   EXPECT_TRUE(cancel_button_->enabled());
 
   mock_chooser_controller_->OnDiscoveryStateChanged(
       content::BluetoothChooser::DiscoveryState::IDLE);
   // OK button is disabled since the chooser refreshed options.
   EXPECT_FALSE(ok_button_->enabled());
   EXPECT_TRUE(cancel_button_->enabled());
 }
+
+TEST_F(ChooserDialogViewTest, DiscoveringAndOneOptionAddedAndSelectedAndIdle) {
+  mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("a"));
+  mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("b"));
+  mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("c"));
+  table_view_->Select(1);
+
+  mock_chooser_controller_->OnDiscoveryStateChanged(
+      content::BluetoothChooser::DiscoveryState::DISCOVERING);
+  mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("d"));
+  // OK button is disabled since no option is selected.
+  EXPECT_FALSE(ok_button_->enabled());
+  EXPECT_TRUE(cancel_button_->enabled());
+  table_view_->Select(0);
+  EXPECT_TRUE(ok_button_->enabled());
+  EXPECT_TRUE(cancel_button_->enabled());
+
+  mock_chooser_controller_->OnDiscoveryStateChanged(
+      content::BluetoothChooser::DiscoveryState::IDLE);
+  EXPECT_TRUE(ok_button_->enabled());
+  EXPECT_TRUE(cancel_button_->enabled());
+}