Fix Harmony popover buttons being flush with their content.

chrome/browser/ui/views/extensions/chooser_dialog_view.h

Index: chrome/browser/ui/views/extensions/chooser_dialog_view.h
diff --git a/chrome/browser/ui/views/extensions/chooser_dialog_view.h b/chrome/browser/ui/views/extensions/chooser_dialog_view.h
index 231dd9fada692bd6e4ec8a110f547f1ec16e8967..dcb4f6e411f165f8c1d9490158345ab66bc334ba 100644
--- a/chrome/browser/ui/views/extensions/chooser_dialog_view.h
+++ b/chrome/browser/ui/views/extensions/chooser_dialog_view.h
@@ -32,9 +32,6 @@ class ChooserDialogView : public views::DialogDelegateView,
   base::string16 GetDialogButtonLabel(ui::DialogButton button) const override;
   bool IsDialogButtonEnabled(ui::DialogButton button) const override;
   views::View* CreateFootnoteView() override;
-  views::ClientView* CreateClientView(views::Widget* widget) override;

[Bret, 2017/06/07 22:52:18]

Removing these changes the layout very slightly pre-Harmony, but it's hard to
see.

[Peter Kasting, 2017/06/10 01:51:11]

Mostly because the net button spacing went from "unrelated" (in the .cc file
here) to "related" (in your new layout provider constant), right?

[Bret, 2017/06/11 00:14:09]

Oops, I went to double-check this and it turns out I was invoking the wrong
dialog (ChooserBubbleUiViewDelegate 😬). I split this off into
crrev.com/2932243002 since it's a bit more involved.

-  views::NonClientFrameView* CreateNonClientFrameView(
-      views::Widget* widget) override;
   bool Accept() override;
   bool Cancel() override;
   bool Close() override;