[subresource_filter] Update the desktop UI according to mocks

chrome/browser/ui/views/content_setting_bubble_contents.cc

 // Copyright (c) 2012 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/content_setting_bubble_contents.h"
 
 #include <algorithm>
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 154 matching lines @@
     ContentSettingBubbleModel* content_setting_bubble_model,
     content::WebContents* web_contents,
     views::View* anchor_view,
     views::BubbleBorder::Arrow arrow)
     : content::WebContentsObserver(web_contents),
       BubbleDialogDelegateView(anchor_view, arrow),
       content_setting_bubble_model_(content_setting_bubble_model),
       custom_link_(nullptr),
       manage_link_(nullptr),
       manage_button_(nullptr),
+      manage_checkbox_(nullptr),
       learn_more_link_(nullptr) {
   // Compensate for built-in vertical padding in the anchor view's image.
   set_anchor_view_insets(gfx::Insets(
       GetLayoutConstant(LOCATION_BAR_BUBBLE_ANCHOR_VERTICAL_INSET), 0));
 }
 
 ContentSettingBubbleContents::~ContentSettingBubbleContents() {
   // Must remove the children here so the comboboxes get destroyed before
   // their associated models.
   RemoveAllChildViews(true);
@@ skipping 224 matching lines @@
         new views::Link(base::UTF8ToUTF16(bubble_content.custom_link));
     custom_link_->SetEnabled(bubble_content.custom_link_enabled);
     custom_link_->set_listener(this);
     if (!bubble_content_empty)
       layout->AddPaddingRow(0, related_control_vertical_spacing);
     layout->StartRow(0, kSingleColumnSetId);
     layout->AddView(custom_link_);
     bubble_content_empty = false;
   }
 
+  if (content_setting_bubble_model_->bubble_content()

[msw, 2017/04/05 22:08:34]

Use |bubble_content| here and below.

[Charlie Harrison, 2017/04/05 22:35:32]

Done.

+          .show_manage_text_as_checkbox) {
+    manage_checkbox_ = new views::Checkbox(base::UTF8ToUTF16(
+        content_setting_bubble_model_->bubble_content().manage_text));
+    manage_checkbox_->set_listener(this);
+    layout->AddPaddingRow(0, related_control_vertical_spacing);
+    layout->StartRow(0, indented_kSingleColumnSetId);
+    layout->AddView(manage_checkbox_);
+  }
+
   if (!bubble_content_empty) {
     if (!layout_delegate->IsHarmonyMode()) {
       layout->AddPaddingRow(0, related_control_vertical_spacing);
       layout->StartRow(0, kSingleColumnSetId);
       layout->AddView(new views::Separator(), 1, 1, GridLayout::FILL,
                       GridLayout::FILL);
     }
     layout->AddPaddingRow(0, related_control_vertical_spacing);
   }
 }
 
 views::View* ContentSettingBubbleContents::CreateExtraView() {
   if (content_setting_bubble_model_->bubble_content()

[msw, 2017/04/05 22:08:34]

optional nit: make and use a local |bubble_content| like above.
(less important if you remove |show_manage_text_as_button|)

[Charlie Harrison, 2017/04/05 22:35:32]

Done.

           .show_manage_text_as_button) {
     manage_button_ = views::MdTextButton::CreateSecondaryUiButton(
         this, base::UTF8ToUTF16(
                   content_setting_bubble_model_->bubble_content().manage_text));
     return manage_button_;
+  } else if (content_setting_bubble_model_->bubble_content()

[msw, 2017/04/05 22:08:34]

No else after return here and below

[Charlie Harrison, 2017/04/05 22:35:32]

Done.

+                 .show_manage_text_as_checkbox) {
+    // Added as part of the primary view.
+    return nullptr;
   } else {
     manage_link_ = new views::Link(base::UTF8ToUTF16(
         content_setting_bubble_model_->bubble_content().manage_text));
     manage_link_->set_listener(this);
     return manage_link_;
   }
 }
 
 bool ContentSettingBubbleContents::Accept() {
   content_setting_bubble_model_->OnDoneClicked();
   return true;
 }
 
 bool ContentSettingBubbleContents::Close() {
   return true;
 }
 
 int ContentSettingBubbleContents::GetDialogButtons() const {
   return ui::DIALOG_BUTTON_OK;
 }
 
 base::string16 ContentSettingBubbleContents::GetDialogButtonLabel(
     ui::DialogButton button) const {
+  base::string16 done_text =

[msw, 2017/04/05 22:08:34]

const &

[Charlie Harrison, 2017/04/05 22:35:32]

Done.

+      content_setting_bubble_model_->bubble_content().done_text;
+  if (!done_text.empty())
+    return done_text;
   return l10n_util::GetStringUTF16(IDS_DONE);
 }
 
 void ContentSettingBubbleContents::DidFinishNavigation(
     content::NavigationHandle* navigation_handle) {
   if (!navigation_handle->IsInMainFrame() || !navigation_handle->HasCommitted())
     return;
 
   // Content settings are based on the main frame, so if it switches then
   // close up shop.
   GetWidget()->Close();
 }
 
 void ContentSettingBubbleContents::ButtonPressed(views::Button* sender,
                                                  const ui::Event& event) {
   if (manage_button_ == sender) {
     GetWidget()->Close();
     content_setting_bubble_model_->OnManageLinkClicked();
+  } else if (manage_checkbox_ == sender) {
+    content_setting_bubble_model_->OnChecked(manage_checkbox_->checked());
   } else {
     RadioGroup::const_iterator i(
         std::find(radio_group_.begin(), radio_group_.end(), sender));
     DCHECK(i != radio_group_.end());
     content_setting_bubble_model_->OnRadioClicked(i - radio_group_.begin());
   }
 }
 
 void ContentSettingBubbleContents::LinkClicked(views::Link* source,
                                                int event_flags) {
@@ skipping 19 matching lines @@
   DCHECK(i != list_item_links_.end());
   content_setting_bubble_model_->OnListItemClicked(i->second);
 }
 
 void ContentSettingBubbleContents::OnPerformAction(views::Combobox* combobox) {
   MediaComboboxModel* model =
       static_cast<MediaComboboxModel*>(combobox->model());
   content_setting_bubble_model_->OnMediaMenuClicked(
       model->type(), model->GetDevices()[combobox->selected_index()].id);
 }