views: implement width snapping for DialogDelegateViews

chrome/browser/ui/views/task_manager_view.cc

 // Copyright 2015 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/task_manager_view.h"
 
 #include <stddef.h>
 
 #include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 132 matching lines @@
 
   // If |sorted_column_id| is the default value, it means to clear the sort.
   if (descriptor.sorted_column_id != TableSortDescriptor().sorted_column_id) {
     descriptor_list.emplace_back(descriptor.sorted_column_id,
                                  descriptor.is_ascending);
   }
 
   tab_table_->SetSortDescriptors(descriptor_list);
 }
 
-gfx::Size TaskManagerView::GetPreferredSize() const {
+gfx::Size TaskManagerView::GetUnsnappedPreferredSize() const {
   return gfx::Size(460, 270);

[Peter Kasting, 2017/03/30 00:35:36]

This should probably be using a localized size instead of a pixel value (no bug
yet on file)

 }
 
 bool TaskManagerView::AcceleratorPressed(const ui::Accelerator& accelerator) {
   DCHECK_EQ(ui::VKEY_W, accelerator.key_code());
   DCHECK_EQ(ui::EF_CONTROL_DOWN, accelerator.modifiers());
   GetWidget()->Close();
   return true;
 }
 
 bool TaskManagerView::CanResize() const {
@@ skipping 197 matching lines @@
   if (!g_browser_process->local_state())
     return;
 
   const base::DictionaryValue* dictionary =
       g_browser_process->local_state()->GetDictionary(GetWindowName());
   if (dictionary)
     dictionary->GetBoolean("always_on_top", &is_always_on_top_);
 }
 
 }  // namespace task_manager