Page Info: Add brackets in to one line if and for statements

chrome/browser/ui/views/page_info/page_info_bubble_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/page_info/page_info_bubble_view.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <utility>
@@ skipping 252 matching lines @@
 
   std::vector<size_t> offsets;
 
   base::string16 text = base::ReplaceStringPlaceholders(
       base::ASCIIToUTF16("$1 $2"), subst, &offsets);
   security_details_label_->SetText(text);
   gfx::Range details_range(offsets[1], text.length());
 
   views::StyledLabel::RangeStyleInfo link_style =
       views::StyledLabel::RangeStyleInfo::CreateForLink();
-  if (!ui::MaterialDesignController::IsSecondaryUiMaterial())
+  if (!ui::MaterialDesignController::IsSecondaryUiMaterial()) {
     link_style.font_style |= gfx::Font::FontStyle::UNDERLINE;
+  }
   link_style.disable_line_wrapping = false;
 
   security_details_label_->AddStyleRange(details_range, link_style);
 }
 
 void BubbleHeaderView::AddResetDecisionsLabel() {
   std::vector<base::string16> subst;
   subst.push_back(
       l10n_util::GetStringUTF16(IDS_PAGE_INFO_INVALID_CERTIFICATE_DESCRIPTION));
   subst.push_back(l10n_util::GetStringUTF16(
       IDS_PAGE_INFO_RESET_INVALID_CERTIFICATE_DECISIONS_BUTTON));
 
   std::vector<size_t> offsets;
 
   base::string16 text = base::ReplaceStringPlaceholders(
       base::ASCIIToUTF16("$1 $2"), subst, &offsets);
   reset_cert_decisions_label_ =
       new views::StyledLabel(text, styled_label_listener_);
   reset_cert_decisions_label_->set_id(
       VIEW_ID_PAGE_INFO_LABEL_RESET_CERTIFICATE_DECISIONS);
   gfx::Range link_range(offsets[1], text.length());
 
   views::StyledLabel::RangeStyleInfo link_style =
       views::StyledLabel::RangeStyleInfo::CreateForLink();
-  if (!ui::MaterialDesignController::IsSecondaryUiMaterial())
+  if (!ui::MaterialDesignController::IsSecondaryUiMaterial()) {
     link_style.font_style |= gfx::Font::FontStyle::UNDERLINE;
+  }
   link_style.disable_line_wrapping = false;
 
   reset_cert_decisions_label_->AddStyleRange(link_range, link_style);
   // Fit the styled label to occupy available width.
   reset_cert_decisions_label_->SizeToFit(0);
   reset_decisions_label_container_->AddChildView(reset_cert_decisions_label_);
 
   // Now that it contains a label, the container needs padding at the top.
   reset_decisions_label_container_->SetBorder(
       views::CreateEmptyBorder(8, 0, 0, 0));
@@ skipping 79 matching lines @@
     const security_state::SecurityInfo& security_info) {
   gfx::NativeView parent_window =
       anchor_view ? nullptr : web_contents->GetNativeView();
   if (url.SchemeIs(content::kChromeUIScheme) ||
       url.SchemeIs(content::kChromeDevToolsScheme) ||
       url.SchemeIs(extensions::kExtensionScheme) ||
       url.SchemeIs(content::kViewSourceScheme)) {
     // Use the concrete type so that |SetAnchorRect| can be called as a friend.
     InternalPageInfoBubbleView* bubble =
         new InternalPageInfoBubbleView(anchor_view, parent_window, url);
-    if (!anchor_view)
+    if (!anchor_view) {
       bubble->SetAnchorRect(anchor_rect);
-    if (widget_observer)
+    }
+    if (widget_observer) {
       bubble->GetWidget()->AddObserver(widget_observer);
+    }
     bubble->GetWidget()->Show();
     return bubble;
   }
   PageInfoBubbleView* bubble = new PageInfoBubbleView(
       anchor_view, parent_window, profile, web_contents, url, security_info);
-  if (!anchor_view)
+  if (!anchor_view) {
     bubble->SetAnchorRect(anchor_rect);
-  if (widget_observer)
+  }
+  if (widget_observer) {
     bubble->GetWidget()->AddObserver(widget_observer);
+  }
   bubble->GetWidget()->Show();
   return bubble;
 }
 
 // static
 PageInfoBubbleView::BubbleType PageInfoBubbleView::GetShownBubbleType() {
   return g_shown_bubble_type;
 }
 
 // static
@@ skipping 70 matching lines @@
   }
   views::BubbleDialogDelegateView::CreateBubble(this);
 
   presenter_.reset(new PageInfo(
       this, profile, TabSpecificContentSettings::FromWebContents(web_contents),
       web_contents, url, security_info));
 }
 
 void PageInfoBubbleView::RenderFrameDeleted(
     content::RenderFrameHost* render_frame_host) {
-  if (render_frame_host == web_contents()->GetMainFrame())
+  if (render_frame_host == web_contents()->GetMainFrame()) {
     GetWidget()->Close();
+  }
 }
 
 void PageInfoBubbleView::WebContentsDestroyed() {
   weak_factory_.InvalidateWeakPtrs();
 }
 
 void PageInfoBubbleView::WasHidden() {
   GetWidget()->Close();
 }
 
@@ skipping 48 matching lines @@
   // The bubble closes automatically when the collected cookies dialog or the
   // certificate viewer opens. So delay handling of the link clicked to avoid
   // a crash in the base class which needs to complete the mouse event handling.
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE,
       base::BindOnce(&PageInfoBubbleView::HandleLinkClickedAsync,
                      weak_factory_.GetWeakPtr(), source));
 }
 
 gfx::Size PageInfoBubbleView::CalculatePreferredSize() const {
-  if (header_ == nullptr && site_settings_view_ == nullptr)
+  if (header_ == nullptr && site_settings_view_ == nullptr) {
     return views::View::CalculatePreferredSize();
+  }
 
   int height = 0;
-  if (header_)
+  if (header_) {
     height += header_->GetPreferredSize().height() + kHeaderMarginBottom;
-  if (separator_)
+  }
+  if (separator_) {
     height += separator_->GetPreferredSize().height();
+  }
 
-  if (site_settings_view_)
+  if (site_settings_view_) {
     height += site_settings_view_->GetPreferredSize().height();
+  }
 
   int width = kMinBubbleWidth;
-  if (site_settings_view_)
+  if (site_settings_view_) {
     width = std::max(width, site_settings_view_->GetPreferredSize().width());
+  }
   width = std::min(width, kMaxBubbleWidth);
   return gfx::Size(width, height);
 }
 
 void PageInfoBubbleView::SetCookieInfo(const CookieInfoList& cookie_info_list) {
   // |cookie_info_list| should only ever have 2 items: first- and third-party
   // cookies.
   DCHECK_EQ(cookie_info_list.size(), 2u);
   int total_allowed = 0;
-  for (const auto& i : cookie_info_list)
+  for (const auto& i : cookie_info_list) {
     total_allowed += i.allowed;
+  }
   base::string16 label_text = l10n_util::GetPluralStringFUTF16(
       IDS_PAGE_INFO_NUM_COOKIES, total_allowed);
 
   cookie_dialog_link_->SetText(label_text);
   Layout();
   SizeToContents();
 }
 
 void PageInfoBubbleView::SetPermissionInfo(
     const PermissionInfoList& permission_info_list,
     ChosenObjectInfoList chosen_object_info_list) {
   // When a permission is changed, PageInfo::OnSitePermissionChanged()
   // calls this method with updated permissions. However, PermissionSelectorRow
   // will have already updated its state, so it's already reflected in the UI.
   // In addition, if a permission is set to the default setting, PageInfo
   // removes it from |permission_info_list|, but the button should remain.
-  if (permissions_view_)
+  if (permissions_view_) {
     return;
+  }
 
   permissions_view_ = new views::View();
   views::GridLayout* layout = new views::GridLayout(permissions_view_);
   permissions_view_->SetLayoutManager(layout);
 
   site_settings_view_->AddChildView(permissions_view_);
 
   const int content_column = 0;
   views::ColumnSet* column_set = layout->AddColumnSet(content_column);
   column_set->AddColumn(views::GridLayout::FILL, views::GridLayout::FILL, 1,
@@ skipping 53 matching lines @@
 void PageInfoBubbleView::SetIdentityInfo(const IdentityInfo& identity_info) {
   std::unique_ptr<PageInfoUI::SecurityDescription> security_description =
       identity_info.GetSecurityDescription();
 
   summary_text_ = security_description->summary;
   GetWidget()->UpdateWindowTitle();
 
   if (identity_info.certificate) {
     certificate_ = identity_info.certificate;
 
-    if (identity_info.show_ssl_decision_revoke_button)
+    if (identity_info.show_ssl_decision_revoke_button) {
       header_->AddResetDecisionsLabel();
+    }
 
     if (PageInfoUI::ShouldShowCertificateLink()) {
       // The text of link to the Certificate Viewer varies depending on the
       // validity of the Certificate.
       const bool valid_identity = (identity_info.identity_status !=
                                    PageInfo::SITE_IDENTITY_STATUS_ERROR);
       const base::string16 link_title = l10n_util::GetStringUTF16(
           valid_identity ? IDS_PAGE_INFO_CERTIFICATE_VALID_LINK
                          : IDS_PAGE_INFO_CERTIFICATE_INVALID_LINK);
 
@@ skipping 47 matching lines @@
   const gfx::ImageSkia icon = PageInfoUI::GetPermissionIcon(info).AsImageSkia();
   // Add the Cookies section.
   site_settings_view->AddChildView(CreateInspectLinkSection(
       icon, IDS_PAGE_INFO_COOKIES, cookie_dialog_link_));
 
   return site_settings_view;
 }
 
 void PageInfoBubbleView::HandleLinkClickedAsync(views::Link* source) {
   // All switch cases require accessing web_contents(), so we check it here.
-  if (web_contents() == nullptr || web_contents()->IsBeingDestroyed())
+  if (web_contents() == nullptr || web_contents()->IsBeingDestroyed()) {
     return;
+  }
   switch (source->id()) {
     case VIEW_ID_PAGE_INFO_LINK_SITE_SETTINGS:
       presenter_->OpenSiteSettingsView();
       break;
     case VIEW_ID_PAGE_INFO_LINK_COOKIE_DIALOG:
       // Count how often the Collected Cookies dialog is opened.
       presenter_->RecordPageInfoAction(
           PageInfo::PAGE_INFO_COOKIES_DIALOG_OPENED);
       new CollectedCookiesViews(web_contents());
       break;
@@ skipping 24 matching lines @@
           PageInfo::PAGE_INFO_CONNECTION_HELP_OPENED);
       break;
     case VIEW_ID_PAGE_INFO_LABEL_RESET_CERTIFICATE_DECISIONS:
       presenter_->OnRevokeSSLErrorBypassButtonPressed();
       GetWidget()->Close();
       break;
     default:
       NOTREACHED();
   }
 }