Material Page Info (Views, 3/3): Update site settings section.

chrome/browser/ui/views/website_settings/chosen_object_row.h

 // Copyright 2015 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_WEBSITE_SETTINGS_CHOSEN_OBJECT_ROW_H_
 #define CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_CHOSEN_OBJECT_ROW_H_
 
 #include "base/macros.h"
 #include "chrome/browser/ui/website_settings/website_settings_ui.h"
 #include "ui/views/controls/button/button.h"
+#include "ui/views/layout/grid_layout.h"
 #include "ui/views/view.h"
 
 namespace views {
 class ImageButton;
 class ImageView;
 }
 
 class ChosenObjectRowObserver;
 
 // A ChosenObjectRow is a row in the Page Info bubble that shows an individual
 // object (e.g. a Bluetooth device, a USB device) that the current site has
 // access to.
-class ChosenObjectRow : public views::View, public views::ButtonListener {
+class ChosenObjectRow : public views::ButtonListener {
  public:
+  // The constructor adds two views to the current row of |layout|:
+  //  1. An icon.
+  //  2. A label with a delete button.
   explicit ChosenObjectRow(

[msw, 2016/09/26 20:47:55]

nit: remove explicit

[lgarron, 2016/09/28 21:11:46]

Done.

At the risk of asking in the wrong place, why was it appropriate for the
views::View subclass but not now?

[msw, 2016/09/30 00:37:04]

The explicit keyword is a guard against implicit construction invocation when
the ctor only has one argument. Now that the ctor has two arguments, the
explicit keyword guard is no longer needed.

[lgarron, 2016/09/30 05:05:55]

Ah, that makes sense, thanks.

-      std::unique_ptr<WebsiteSettingsUI::ChosenObjectInfo> info);
+      std::unique_ptr<WebsiteSettingsUI::ChosenObjectInfo> info,
+      views::GridLayout* layout);
 
   void AddObserver(ChosenObjectRowObserver* observer);
 
  private:
   ~ChosenObjectRow() override;
 
   // views::ButtonListener implementation.
   void ButtonPressed(views::Button* sender, const ui::Event& event) override;
 
   views::ImageView* icon_;             // Owned by the views hierarchy.
+  views::View* label_with_delete_;     // Owned by the views hierarchy.
   views::ImageButton* delete_button_;  // Owned by the views hierarchy.
 
   base::ObserverList<ChosenObjectRowObserver> observer_list_;
   std::unique_ptr<WebsiteSettingsUI::ChosenObjectInfo> info_;
 
   DISALLOW_COPY_AND_ASSIGN(ChosenObjectRow);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_WEBSITE_SETTINGS_CHOSEN_OBJECT_ROW_H_