Harmony - Use FocusableBorder for comboboxes on Mac

ui/views/controls/combobox/combobox.cc

Index: ui/views/controls/combobox/combobox.cc
diff --git a/ui/views/controls/combobox/combobox.cc b/ui/views/controls/combobox/combobox.cc
index cc474726ba960ea528b48f698a89ba36e401370c..1d4ba52a31db5253ca124e04c36a74950dbbdf60 100644
--- a/ui/views/controls/combobox/combobox.cc
+++ b/ui/views/controls/combobox/combobox.cc
@@ -768,7 +768,7 @@ void Combobox::ButtonPressed(Button* sender, const ui::Event& event) {
 
 void Combobox::UpdateBorder() {
   std::unique_ptr<FocusableBorder> border(
-      PlatformStyle::CreateComboboxBorder());
+      UseMd() ? new FocusableBorder() : PlatformStyle::CreateComboboxBorder());

[Elly Fong-Jones, 2016/10/06 14:15:36]

this won't build - FocusableBorder* and unique_ptr<FocusableBorder> don't match

[Evan Stade, 2016/10/06 15:51:08]

good call, fixed.

It also occurs to me we probably don't want a different BG on mac, right? Since
for whatever reason we are only setting that in the UseMd case, we can go ahead
and ditch the mac combobox bg, yes? I've made that change as well, ptal.

   if (style_ == STYLE_ACTION)
     border->SetInsets(5, 10, 5, 10);
   if (invalid_)