Delete panel metrics and define insets in terms of distance metrics.

chrome/browser/ui/views/task_manager_view.cc

Index: chrome/browser/ui/views/task_manager_view.cc
diff --git a/chrome/browser/ui/views/task_manager_view.cc b/chrome/browser/ui/views/task_manager_view.cc
index 00bdd99e6190edec6ef21711a886a298438afb08..cf3bb9315f79f7cda14ff11aa692f43b5e3e569b 100644
--- a/chrome/browser/ui/views/task_manager_view.cc
+++ b/chrome/browser/ui/views/task_manager_view.cc
@@ -336,10 +336,11 @@ void TaskManagerView::Init() {
   AddChildView(tab_table_parent_);
 
   SetLayoutManager(new views::FillLayout());
+  const gfx::Insets content_insets =
+      ChromeLayoutProvider::Get()->GetInsetsMetric(
+          views::INSETS_DIALOG_CONTENTS);
   SetBorder(views::CreateEmptyBorder(
-      ChromeLayoutProvider::Get()->GetDistanceMetric(
-          DISTANCE_PANEL_CONTENT_MARGIN),
-      views::kButtonHEdgeMarginNew, 0, views::kButtonHEdgeMarginNew));
+      content_insets.top(), content_insets.left(), 0, content_insets.right()));

[Peter Kasting, 2017/05/18 00:00:06]

The 0 here looks odd.  Why are we not just using the normal content insets?

[Bret, 2017/05/18 00:50:05]

I guess someone wanted the "End Process" button closer to the process table?
Changed it to do the correct thing in Harmony at least.

[Peter Kasting, 2017/05/18 00:56:28]

Mrph.  Can we just Do The Correct Thing all the time?  I'm OK with behavioral
changes to pre-Harmony when it seems correct and simpler.

[Bret, 2017/05/18 01:20:30]

Okay, done.

 
   table_model_->RetrieveSavedColumnsSettingsAndUpdateTable();