Add WebsiteSettingsUI for Windows

chrome/browser/ui/views/website_settings_popup_view.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/website_settings_popup_view.h"
+
+#include "base/string_number_conversions.h"
+#include "base/utf_string_conversions.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/website_settings/website_settings.h"
+#include "chrome/browser/ui/views/collected_cookies_views.h"
+#include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
+#include "chrome/common/content_settings_types.h"
+#include "content/public/browser/cert_store.h"
+#include "googleurl/src/gurl.h"
+#include "grit/generated_resources.h"
+#include "ui/base/l10n/l10n_util.h"
+#include "ui/gfx/font.h"
+#include "ui/views/controls/combobox/combobox.h"
+#include "ui/views/controls/label.h"
+#include "ui/views/controls/link.h"
+#include "ui/views/controls/tabbed_pane/tabbed_pane.h"
+#include "ui/views/layout/box_layout.h"
+#include "ui/views/layout/grid_layout.h"
+#include "ui/views/layout/layout_manager.h"
+
+namespace {
+
+// In order to make the arrow of the bubble point directly at the location icon
+// in the Omnibox rather then the bottom border of the Omnibox, the position of
+// the bubble must be adjusted. This is the number of pixel the bubble must be
+// moved towards the top of the screen (starting from the bottom border of the
+// Omnibox).
+const int kLocationIconBottomMargin = 5;
+
+string16 PermissionTypeToString(ContentSettingsType type) {
+  return l10n_util::GetStringUTF16(
+      WebsiteSettingsUI::PermissionTypeToUIStringID(type));
+}
+
+string16 PermissionValueToString(ContentSetting value) {
+  return l10n_util::GetStringUTF16(
+      WebsiteSettingsUI::PermissionValueToUIStringID(value));
+}
+
+}  // namespace
+
+// |PopupHeader| is the UI element (view) that represents the header of the
+// |WebsiteSettingsPopupView|. The header shows the status of the site's
+// identity check and the name of the site's identity.
+class PopupHeader : public views::View {
+ public:
+  PopupHeader();
+  virtual ~PopupHeader();
+
+  // Sets the name of the site's identity.
+  void SetIdentityName(const string16& name);
+
+  // Sets the status text for the identity check of this site.
+  void SetIdentityStatus(const string16& status_text);
+
+ private:
+  // The label that displays the name of the site's identity.
+  views::Label* name_;
+  // The label that displays the status of the identity check for this site.
+  views::Label* status_;
+
+  DISALLOW_COPY_AND_ASSIGN(PopupHeader);
+};
+
+// A |ComboboxModel| implementation that is used for |Combobox|es that allow
+// selecting a setting for a given site permission.
+class PermissionComboboxModel : public ui::ComboboxModel {
+ public:
+  // Creates a combobox model that provides all possible settings for the given
+  // |site_permission|.
+  PermissionComboboxModel(ContentSettingsType site_permission,
+                          ContentSetting default_setting);
+  virtual ~PermissionComboboxModel();
+
+  // Returns the setting for the given |index|.
+  ContentSetting GetSettingAt(int index) const;
+
+  // Returns the site permission for which the combobox model provides
+  // settings.
+  ContentSettingsType site_permission() const {
+    return site_permission_;
+  }
+
+  // ui::ComboboxModel implementations.
+  virtual int GetItemCount() const OVERRIDE;
+  virtual string16 GetItemAt(int index) OVERRIDE;
+
+ private:
+  // The site permission (the |ContentSettingsType|) for which the combobox
+  // model provides settings.
+  ContentSettingsType site_permission_;
+
+  // The global default setting for the |site_permission_|.
+  ContentSetting default_setting_;
+
+  // All possible valid setting for the |site_permission_|.
+  std::vector<ContentSetting> settings_;
+
+  DISALLOW_COPY_AND_ASSIGN(PermissionComboboxModel);
+};
+
+////////////////////////////////////////////////////////////////////////////////
+// Popup Header
+////////////////////////////////////////////////////////////////////////////////
+
+PopupHeader::PopupHeader() : name_(NULL), status_(NULL) {
+  views::GridLayout* layout = new views::GridLayout(this);
+  SetLayoutManager(layout);
+  const int label_column = 0;
+  views::ColumnSet* column_set = layout->AddColumnSet(label_column);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+
+  layout->AddPaddingRow(0, 8);
+
+  layout->StartRow(0, label_column);
+  name_ = new views::Label(string16());
+  name_->SetFont(name_->font().DeriveFont(0, gfx::Font::BOLD));
+  layout->AddView(name_, 1, 1, views::GridLayout::LEADING,
+                  views::GridLayout::CENTER);
+
+  layout->AddPaddingRow(0, 4);
+
+  status_ = new views::Label(string16());
+  layout->StartRow(0, label_column);
+  layout->AddView(status_,
+                  1,
+                  1,
+                  views::GridLayout::LEADING,
+                  views::GridLayout::CENTER);
+}
+
+PopupHeader::~PopupHeader() {
+}
+
+void PopupHeader::SetIdentityName(const string16& name) {
+  name_->SetText(name);
+}
+
+void PopupHeader::SetIdentityStatus(const string16& status) {
+  status_->SetText(status);
+}
+
+////////////////////////////////////////////////////////////////////////////////
+// PermissionComboboxModel
+////////////////////////////////////////////////////////////////////////////////
+
+PermissionComboboxModel::PermissionComboboxModel(
+    ContentSettingsType site_permission, ContentSetting default_setting)

[tfarina, 2012/06/06 16:58:29]

nit: I'd wrap it argument in its own line.

[markusheintz_, 2012/06/07 20:10:38]

Done.

+    : site_permission_(site_permission),
+      default_setting_(default_setting){
+  settings_.push_back(CONTENT_SETTING_DEFAULT);
+  settings_.push_back(CONTENT_SETTING_ALLOW);
+  settings_.push_back(CONTENT_SETTING_BLOCK);
+  if (site_permission == CONTENT_SETTINGS_TYPE_GEOLOCATION ||
+      site_permission == CONTENT_SETTINGS_TYPE_NOTIFICATIONS) {
+    settings_.push_back(CONTENT_SETTING_ASK);
+  }

[tfarina, 2012/06/06 16:58:29]

nit: Peter (pkasting) insists that single-line statements should not require { }
even though the condition spans multiple lines. The opposite is not true, if
there are more than one statement for the condition then it requires { }

[markusheintz_, 2012/06/07 20:10:38]

Done.

+}
+
+PermissionComboboxModel::~PermissionComboboxModel() {
+}
+
+ContentSetting PermissionComboboxModel::GetSettingAt(int index) const {
+  if (index < static_cast<int>(settings_.size()))
+    return settings_[index];
+  NOTREACHED();
+  return CONTENT_SETTING_DEFAULT;
+}
+
+int PermissionComboboxModel::GetItemCount() const {
+ return settings_.size();
+}
+
+string16 PermissionComboboxModel::GetItemAt(int index) {
+  if (index == 0) {
+    return l10n_util::GetStringFUTF16(
+        IDS_WEBSITE_SETTINGS_DEFAULT_PERMISSION_LABEL,
+        PermissionValueToString(default_setting_));
+  }
+  if (index < static_cast<int>(settings_.size())) {
+    return l10n_util::GetStringFUTF16(
+        IDS_WEBSITE_SETTINGS_PERMISSION_LABEL,
+        PermissionValueToString(settings_[index]));
+  }
+  NOTREACHED();
+  return string16();
+}
+
+///////////////////////////////////////////////////////////////////////////////
+// WebsiteSettingsPopupView
+///////////////////////////////////////////////////////////////////////////////
+
+// static
+void WebsiteSettingsPopupView::ShowPopup(views::View* anchor_view,
+                                         Profile* profile,
+                                         TabContentsWrapper* wrapper,
+                                         const GURL& url,
+                                         const content::SSLStatus& ssl) {
+  new WebsiteSettingsPopupView(anchor_view, profile, wrapper, url, ssl);
+}
+
+WebsiteSettingsPopupView::WebsiteSettingsPopupView(
+    views::View* anchor_view,
+    Profile* profile,
+    TabContentsWrapper* wrapper,
+    const GURL& url,
+    const content::SSLStatus& ssl)
+    : BubbleDelegateView(anchor_view, views::BubbleBorder::TOP_LEFT),
+      tab_contents_wrapper_(wrapper),
+      presenter_(NULL),
+      header_(NULL),
+      tabbed_pane_(NULL),
+      site_data_content_(NULL),
+      cookie_dialog_link_(NULL),
+      permissions_content_(NULL),
+      identity_info_text_(NULL),
+      connection_info_text_(NULL),
+      page_info_text_(NULL) {
+  views::GridLayout* layout = new views::GridLayout(this);
+  SetLayoutManager(layout);
+  const int content_column = 0;
+  views::ColumnSet* column_set = layout->AddColumnSet(content_column);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+
+  header_ = new PopupHeader();
+  layout->StartRow(1, content_column);
+  layout->AddView(header_);
+
+  layout->AddPaddingRow(1, 10);
+
+  tabbed_pane_ = new views::TabbedPane();
+  layout->StartRow(1, content_column);
+  layout->AddView(tabbed_pane_);
+  // Tabs must be added after the tabbed_pane_ was added to the views hierachy.
+  // Adding the |tabbed_pane_| to the views hierachy triggers the
+  // initialization of the native tab UI element. If the native tab UI element
+  // is not initalized adding a tab will result in a NULL pointer excetion.
+  tabbed_pane_->AddTab(
+      l10n_util::GetStringUTF16(IDS_WEBSITE_SETTINGS_TAB_LABEL_PERMISSIONS),
+      CreatePermissionsTab());
+  tabbed_pane_->AddTab(
+      l10n_util::GetStringUTF16(IDS_WEBSITE_SETTINGS_TAB_LABEL_IDENTITY),
+      CreateIdentityTab());
+  tabbed_pane_->SelectTabAt(0);
+  tabbed_pane_->set_listener(this);
+
+  views::BubbleDelegateView::CreateBubble(this);
+  this->Show();
+  SizeToContents();
+
+  presenter_.reset(new WebsiteSettings(this, profile,
+                                       wrapper->content_settings(), url, ssl,
+                                       content::CertStore::GetInstance()));
+}
+
+WebsiteSettingsPopupView::~WebsiteSettingsPopupView() {
+}
+
+void WebsiteSettingsPopupView::SetCookieInfo(
+    const CookieInfoList& cookie_info_list) {
+  site_data_content_->RemoveAllChildViews(true);
+
+  views::GridLayout* layout = new views::GridLayout(site_data_content_);
+  site_data_content_->SetLayoutManager(layout);
+
+  const int site_data_content_column = 0;
+  views::ColumnSet* column_set =
+      layout->AddColumnSet(site_data_content_column);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+
+  for (CookieInfoList::const_iterator it = cookie_info_list.begin();
+       it != cookie_info_list.end();
+       ++it) {
+    string16 label_text = l10n_util::GetStringFUTF16(
+        IDS_WEBSITE_SETTINGS_SITE_DATA_STATS_LINE,
+        UTF8ToUTF16(it->cookie_source),
+        base::IntToString16(it->allowed),
+        base::IntToString16(it->allowed));
+    layout->StartRow(1, site_data_content_column);
+    layout->AddView(new views::Label(label_text),
+                    1,
+                    1,
+                    views::GridLayout::LEADING,
+                    views::GridLayout::CENTER);
+
+    layout->AddPaddingRow(1, 4);
+  }
+
+  layout->Layout(site_data_content_);
+  SizeToContents();
+}
+
+void WebsiteSettingsPopupView::SetPermissionInfo(
+    const PermissionInfoList& permission_info_list) {
+  permissions_content_->RemoveAllChildViews(true);
+
+  views::GridLayout* layout =
+      new views::GridLayout(permissions_content_);
+  permissions_content_->SetLayoutManager(layout);
+  const int content_column = 0;
+  views::ColumnSet* column_set =
+      layout->AddColumnSet(content_column);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+  for (PermissionInfoList::const_iterator permission =
+           permission_info_list.begin();
+       permission != permission_info_list.end();
+       ++permission) {
+    views::Label* label =
+        new views::Label(PermissionTypeToString(permission->type));
+    views::Combobox* combobox = new views::Combobox(
+        new PermissionComboboxModel(permission->type,

[tfarina, 2012/06/06 16:58:29]

nit: Please, do not do this way. I have written many patches fixing this leak.
views::ComboboxModel does NOT own the model, thus we are leaking here.

One way I have fixed some chromeos leaks was:
- In header file:
namespace internal {
class PermissionComboboxModel;
}
- scoped_ptr<PermissionComboboxModel> permission_combobox_model_;

- In source file:
namespace internal {
class PermissionComboboxModel : public ui::ComboboxModel {
...
};

PermissionComboboxModel::PermissionComboboxModel() {}
...

And that way you will be able to avoid the static_cast below.

[markusheintz_, 2012/06/07 20:10:38]

Thanks a lot for pointing this out! I didn't know this.
Since I store the permission type and other permission type related information
in the PermissionComboboxModel I need to keep a vector for all Combobox models.
I use a ScopedVector now that deletes all the ComboboxModels when the popup is
destroyed.

I chose "website_settings" as namespace.

+                                    permission->default_setting));
+    switch (permission->setting) {
+      case CONTENT_SETTING_DEFAULT: combobox->SetSelectedIndex(0);
+        break;
+      case CONTENT_SETTING_ALLOW: combobox->SetSelectedIndex(1);
+        break;
+      case CONTENT_SETTING_BLOCK: combobox->SetSelectedIndex(2);
+        break;
+      default: combobox->SetSelectedIndex(4);
+        break;
+    }
+    combobox->set_listener(this);
+
+    layout->StartRow(1, content_column);
+    layout->AddView(CreatePermissionRow(label, combobox),
+                    1,
+                    1,
+                    views::GridLayout::LEADING,
+                    views::GridLayout::CENTER);
+
+  }
+
+  SizeToContents();
+}
+
+void WebsiteSettingsPopupView::SetIdentityInfo(
+    const IdentityInfo& identity_info) {
+  string16 identity_status_text;
+  switch (identity_info.identity_status) {
+    case WebsiteSettings::SITE_IDENTITY_STATUS_CERT:
+    case WebsiteSettings::SITE_IDENTITY_STATUS_DNSSEC_CERT:
+    case WebsiteSettings::SITE_IDENTITY_STATUS_EV_CERT:
+      identity_status_text =
+          l10n_util::GetStringUTF16(IDS_WEBSITE_SETTINGS_IDENTITY_VERIFIED);
+      break;
+    default:
+      identity_status_text =
+         l10n_util::GetStringUTF16(IDS_WEBSITE_SETTINGS_IDENTITY_NOT_VERIFIED);
+      break;
+  }
+  header_->SetIdentityName(UTF8ToUTF16(identity_info.site_identity));
+  header_->SetIdentityStatus(identity_status_text);
+
+  identity_info_text_->SetText(
+      UTF8ToUTF16(identity_info.identity_status_description));
+
+  connection_info_text_->SetText(
+      UTF8ToUTF16(identity_info.connection_status_description));
+
+  GetLayoutManager()->Layout(this);
+  SizeToContents();
+}
+
+gfx::Size WebsiteSettingsPopupView::GetPreferredSize() {
+  int height = 0;
+  int width = 300;
+  if (header_)
+    height += header_->GetPreferredSize().height();
+  if (tabbed_pane_)
+    height += tabbed_pane_->GetPreferredSize().height();
+
+  gfx::Size size(width, height);
+  return size;
+}
+
+void WebsiteSettingsPopupView::SetFirstVisit(const string16& first_visit) {
+  page_info_text_->SetText(first_visit);
+}
+
+gfx::Rect WebsiteSettingsPopupView::GetAnchorRect() {
+  // Compensate for some built-in padding in the icon. This will make the arrow
+  // point to the middle of the icon.
+  gfx::Rect anchor(BubbleDelegateView::GetAnchorRect());
+  anchor.Inset(0, anchor_view() ? kLocationIconBottomMargin : 0);
+  return anchor;
+}
+
+void WebsiteSettingsPopupView::OnSelectedIndexChanged(
+    views::Combobox* combobox) {
+  PermissionComboboxModel* model =
+    static_cast<PermissionComboboxModel*>(combobox->model());

[tfarina, 2012/06/06 16:58:29]

nice: although I like this trick, see my comment about the ownership/leak of
ComboboxModel.

[markusheintz_, 2012/06/07 20:10:38]

:)
I'll keep it this way since this allows me to use a ScopedVector instead of a
combobox -> model map (see above for why I need to store several models).

+  DCHECK(model);
+  presenter_->OnSitePermissionChanged(
+      model->site_permission(),
+      model->GetSettingAt(combobox->selected_index()));
+}
+
+void WebsiteSettingsPopupView::LinkClicked(views::Link* source,
+                                           int event_flags) {
+  if (source != cookie_dialog_link_)
+    NOTREACHED();
+  new CollectedCookiesViews(tab_contents_wrapper_);
+  GetWidget()->CloseNow();
+}
+
+void WebsiteSettingsPopupView::TabSelectedAt(int index) {
+  tabbed_pane_->GetSelectedTab()->Layout();
+  SizeToContents();
+}
+
+views::View* WebsiteSettingsPopupView::CreatePermissionsTab() {
+  views::View* pane = new views::View();
+  pane->SetLayoutManager(
+      new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 1));
+
+  cookie_dialog_link_ = new views::Link(
+      l10n_util::GetStringUTF16(IDS_WEBSITE_SETTINGS_SHOW_SITE_DATA));
+  cookie_dialog_link_->set_listener(this);
+  site_data_content_ = new views::View();
+  views::View* site_data_section =
+      CreateSection(l10n_util::GetStringUTF16(
+                        IDS_WEBSITE_SETTINGS_TITLE_SITE_DATA),
+                    site_data_content_,
+                    cookie_dialog_link_);
+  pane->AddChildView(site_data_section);
+
+  permissions_content_ = new views::View();
+  views::View* permissions_section =
+      CreateSection(l10n_util::GetStringUTF16(
+                        IDS_WEBSITE_SETTINGS_TITLE_SITE_PERMISSIONS),
+                    permissions_content_,
+                    NULL);
+  pane->AddChildView(permissions_section);
+  return pane;
+}
+
+views::View* WebsiteSettingsPopupView::CreateIdentityTab() {
+  views::View* pane = new views::View();
+  pane->SetLayoutManager(
+      new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 1));
+
+  // Add Identity section.
+  views::View* section_content = new views::View();
+  views::GridLayout* layout =
+      new views::GridLayout(section_content);
+  section_content->SetLayoutManager(layout);
+  const int content_column = 0;
+  views::ColumnSet* column_set =
+      layout->AddColumnSet(content_column);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+  identity_info_text_ = CreateTextLabel(string16());
+  layout->StartRow(1, content_column);
+  layout->AddView(identity_info_text_, 1, 1, views::GridLayout::LEADING,
+                  views::GridLayout::CENTER);
+  views::View* section =
+      CreateSection(l10n_util::GetStringUTF16(
+                        IDS_WEBSITE_SETTINGS_TITEL_IDENTITY),
+                    section_content,
+                    NULL);
+  pane->AddChildView(section);
+
+  // Add connection section.
+  section_content = new views::View();
+  layout = new views::GridLayout(section_content);
+  section_content->SetLayoutManager(layout);
+  column_set = layout->AddColumnSet(content_column);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+  connection_info_text_ = CreateTextLabel(string16());
+  layout->StartRow(1, content_column);
+  layout->AddView(connection_info_text_, 1, 1, views::GridLayout::LEADING,
+                  views::GridLayout::CENTER);
+  section = CreateSection(l10n_util::GetStringUTF16(
+                              IDS_WEBSITE_SETTINGS_TITEL_CONNECTION),
+                          section_content,
+                          NULL);
+  pane->AddChildView(section);
+
+  // Add page info section.
+  section_content = new views::View();
+  layout = new views::GridLayout(section_content);
+  section_content->SetLayoutManager(layout);
+  column_set = layout->AddColumnSet(content_column);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+  page_info_text_ = CreateTextLabel(string16());
+  layout->StartRow(1, content_column);
+  layout->AddView(page_info_text_, 1, 1, views::GridLayout::LEADING,
+                  views::GridLayout::CENTER);
+  section = CreateSection(l10n_util::GetStringUTF16(
+                              IDS_PAGE_INFO_SITE_INFO_TITLE),
+                          section_content,
+                          NULL);
+  pane->AddChildView(section);
+
+  return pane;
+}
+
+views::View* WebsiteSettingsPopupView::CreateSection(
+    const string16& headline_text,
+    views::View* content,
+    views::Link* link) {
+  views::View* container = new views::View();
+  views::GridLayout* layout = new views::GridLayout(container);
+  container->SetLayoutManager(layout);
+  const int content_column = 0;
+  views::ColumnSet* column_set = layout->AddColumnSet(content_column);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+
+  layout->AddPaddingRow(1, 4);
+  layout->StartRow(1, content_column);
+  views::Label* headline = new views::Label(headline_text);
+  headline->SetFont(headline->font().DeriveFont(0, gfx::Font::BOLD));
+  layout->AddView(headline, 1, 1, views::GridLayout::LEADING,
+                  views::GridLayout::CENTER);
+
+  layout->AddPaddingRow(1, 4);
+  layout->StartRow(1, content_column);
+  layout->AddView(content, 1, 1, views::GridLayout::LEADING,
+                  views::GridLayout::CENTER);
+
+  if (link) {
+    layout->AddPaddingRow(1, 4);
+    layout->StartRow(1, content_column);
+    layout->AddView(link, 1, 1, views::GridLayout::LEADING,
+                    views::GridLayout::CENTER);
+  }
+
+  return container;
+}
+
+views::View* WebsiteSettingsPopupView::CreatePermissionRow(
+    views::Label* label,
+    views::Combobox* combobox) {
+  views::View* container = new views::View();
+  views::GridLayout* layout = new views::GridLayout(container);
+  container->SetLayoutManager(layout);
+  const int two_column_layout = 0;
+  views::ColumnSet* column_set = layout->AddColumnSet(two_column_layout);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+  column_set->AddPaddingColumn(0, 8);
+  column_set->AddColumn(views::GridLayout::FILL,
+                        views::GridLayout::FILL,
+                        1,
+                        views::GridLayout::USE_PREF,
+                        0,
+                        0);
+
+  layout->StartRow(1, two_column_layout);
+  layout->AddView(label);
+  layout->AddView(combobox);
+  return container;
+}
+
+views::Label* WebsiteSettingsPopupView::CreateTextLabel(const string16& text) {
+  views::Label* label = new views::Label(text);
+  label->SetMultiLine(true);
+  label->SetAllowCharacterBreak(true);
+  label->SetHorizontalAlignment(views::Label::ALIGN_LEFT);
+  return label;
+}