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

chrome/browser/ui/views/page_info/permission_selector_row.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/permission_selector_row.h"
 
 #include "base/i18n/rtl.h"
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/page_info/page_info_ui.h"
@@ skipping 191 matching lines @@
   if (ui::MaterialDesignController::IsSecondaryUiMaterial()) {
     set_size_to_largest_label(false);
     ModelChanged();
   }
 }
 
 PermissionCombobox::~PermissionCombobox() {}
 
 void PermissionCombobox::UpdateSelectedIndex(bool use_default) {
   int index = model_->GetCheckedIndex();
-  if (use_default && index == -1)
+  if (use_default && index == -1) {
     index = 0;
+  }
   SetSelectedIndex(index);
 }
 
 void PermissionCombobox::OnPaintBorder(gfx::Canvas* canvas) {
   // No border except a focus indicator for MD mode.
-  if (ui::MaterialDesignController::IsSecondaryUiMaterial() && !HasFocus())
+  if (ui::MaterialDesignController::IsSecondaryUiMaterial() && !HasFocus()) {
     return;
+  }
   Combobox::OnPaintBorder(canvas);
 }
 
 void PermissionCombobox::OnPerformAction(Combobox* combobox) {
   model_->OnPerformAction(combobox->selected_index());
 }
 
 }  // namespace internal
 
 ///////////////////////////////////////////////////////////////////////////////
@@ skipping 24 matching lines @@
       base::Bind(&PermissionSelectorRow::PermissionChanged,
                  base::Unretained(this))));
 
 // Create the permission menu button.
 #if defined(OS_MACOSX)
   bool use_real_combobox = true;
 #else
   bool use_real_combobox =
       ui::MaterialDesignController::IsSecondaryUiMaterial();
 #endif
-  if (use_real_combobox)
+  if (use_real_combobox) {
     InitializeComboboxView(layout, permission);
-  else
+  } else {
     InitializeMenuButtonView(layout, permission);
+  }
 
   // Show the permission decision reason, if it was not the user.
   base::string16 reason =
       PageInfoUI::PermissionDecisionReasonToUIString(profile, permission, url);
   if (!reason.empty()) {
     layout->StartRow(1, 1);
     layout->SkipColumns(1);
     views::Label* permission_decision_reason = new views::Label(reason);
     permission_decision_reason->SetEnabledColor(
         PageInfoUI::GetPermissionDecisionTextColor());
@@ skipping 14 matching lines @@
   // Gross. On paper the Combobox and the ComboboxModelAdapter are both owned by
   // this class, but actually, the Combobox is owned by View and will be
   // destroyed in ~View(), which runs *after* ~PermissionSelectorRow() is done,
   // which means the Combobox gets destroyed after its ComboboxModel, which
   // causes an explosion when the Combobox attempts to stop observing the
   // ComboboxModel. This hack ensures the Combobox is deleted before its
   // ComboboxModel.
   //
   // Technically, the MenuButton has the same problem, but MenuButton doesn't
   // use its model in its destructor.
-  if (combobox_)
+  if (combobox_) {
     combobox_->parent()->RemoveChildView(combobox_);
+  }
 }
 
 void PermissionSelectorRow::InitializeMenuButtonView(
     views::GridLayout* layout,
     const PageInfoUI::PermissionInfo& permission) {
   bool button_enabled =
       permission.source == content_settings::SETTING_SOURCE_USER;
   menu_button_ = new internal::PermissionMenuButton(
       PageInfoUI::PermissionActionToUIString(
           profile_, permission.type, permission.setting,
@@ skipping 34 matching lines @@
     menu_button_->SizeToPreferredSize();
     // Re-layout will be done at the |PageInfoBubbleView| level, since
     // that view may need to resize itself to accomodate the new sizes of its
     // contents.
     menu_button_->InvalidateLayout();
   } else if (combobox_) {
     bool use_default = permission.setting == CONTENT_SETTING_DEFAULT;
     combobox_->UpdateSelectedIndex(use_default);
   }
 
-  for (PermissionSelectorRowObserver& observer : observer_list_)
+  for (PermissionSelectorRowObserver& observer : observer_list_) {
     observer.OnPermissionChanged(permission);
+  }
 }
 
 views::View* PermissionSelectorRow::button() {
   // These casts are required because the two arms of a ?: cannot have different
   // types T1 and T2, even if the resulting value of the ?: is about to be a T
   // and T1 and T2 are both subtypes of T.
   return menu_button_ ? static_cast<views::View*>(menu_button_)
                       : static_cast<views::View*>(combobox_);
 }