Update device name in chooser when device changes name (desktops)

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 154 matching lines @@
   EXPECT_TRUE(ok_button_->enabled());
   EXPECT_TRUE(cancel_button_->enabled());
 
   // Remove option 1.
   mock_chooser_controller_->OptionRemoved(base::ASCIIToUTF16("b"));
   // OK button is disabled since the selected option is removed.
   EXPECT_FALSE(ok_button_->enabled());
   EXPECT_TRUE(cancel_button_->enabled());
 }
 
+TEST_F(ChooserDialogViewTest, SelectAnOptionAndUpdateTheSelectedOption) {
+  mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("a"));
+  mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("b"));
+  mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("c"));
+  EXPECT_FALSE(ok_button_->enabled());
+  EXPECT_TRUE(cancel_button_->enabled());
+
+  // Select option 1.
+  table_view_->Select(1);
+  EXPECT_TRUE(ok_button_->enabled());
+  EXPECT_TRUE(cancel_button_->enabled());
+
+  // Update option 1.
+  mock_chooser_controller_->OptionUpdated(base::ASCIIToUTF16("b"),
+                                          base::ASCIIToUTF16("d"));
+  EXPECT_TRUE(ok_button_->enabled());
+  EXPECT_TRUE(cancel_button_->enabled());

[msw, 2016/08/12 22:19:34]

optional nit: remove "d" after this and check that ok_button is disabled.

[juncai, 2016/08/15 18:11:19]

Done.

+}
+
 TEST_F(ChooserDialogViewTest,
        AddAnOptionAndSelectItAndRemoveTheSelectedOption) {
   mock_chooser_controller_->OptionAdded(base::ASCIIToUTF16("a"));
   EXPECT_FALSE(ok_button_->enabled());
   EXPECT_TRUE(cancel_button_->enabled());
 
   // Select option 0.
   table_view_->Select(0);
   EXPECT_TRUE(ok_button_->enabled());
   EXPECT_TRUE(cancel_button_->enabled());
@@ skipping 84 matching lines @@
   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());
 }