views::Separator cleanup.

ash/common/system/chromeos/audio/volume_view.cc

 // Copyright 2014 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 "ash/common/system/chromeos/audio/volume_view.h"
 
 #include <algorithm>
 
 #include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/metrics/user_metrics_action.h"
@@ skipping 189 matching lines @@
                              .ToImageSkia());
   }
   more_button_->AddChildView(more_arrow);
 
   if (MaterialDesignController::IsSystemTrayMenuMaterial()) {
     more_button_->SetInkDropMode(views::InkDropHostView::InkDropMode::ON);
     tri_view_->AddView(TriView::Container::END, more_button_);
   } else {
     separator_ = new views::Separator(views::Separator::VERTICAL);
     separator_->SetColor(kButtonStrokeColor);
-    separator_->SetPreferredSize(kSeparatorSize);
     separator_->SetBorder(views::CreateEmptyBorder(kSeparatorVerticalInset, 0,
                                                    kSeparatorVerticalInset,
                                                    kBoxLayoutPadding));
 
     TrayPopupItemContainer* more_container =
         new TrayPopupItemContainer(separator_, true);
     more_container->SetLayoutManager(
         new views::BoxLayout(views::BoxLayout::kHorizontal, 0, 0, 0));
     more_container->AddChildView(more_button_);
     tri_view_->AddView(TriView::Container::END, more_container);
@@ skipping 125 matching lines @@
 void VolumeView::OnBoundsChanged(const gfx::Rect& previous_bounds) {
   // Separator's prefered size is based on set bounds. When an empty bounds is
   // set on first layout this causes BoxLayout to ignore the separator. Reset
   // its height on each bounds change so that it is laid out properly.
   if (separator_)
     separator_->SetSize(gfx::Size(kSeparatorSize, bounds().height()));
 }
 
 }  // namespace tray
 }  // namespace ash