Ctrl-Enter on Bookmark bar link should open in the New Tab.

ui/views/controls/button/custom_button.cc

Index: ui/views/controls/button/custom_button.cc
diff --git a/ui/views/controls/button/custom_button.cc b/ui/views/controls/button/custom_button.cc
index 5040c4ea9cf13cf814f091766d2af59e6f9bceb7..c1c7c27f37ff03955b2fae46ec011e365b7391d4 100644
--- a/ui/views/controls/button/custom_button.cc
+++ b/ui/views/controls/button/custom_button.cc
@@ -200,7 +200,7 @@ bool CustomButton::OnKeyPressed(const ui::KeyEvent& event) {
     // TODO(beng): remove once NotifyClick takes ui::Event.
     ui::MouseEvent synthetic_event(ui::ET_MOUSE_RELEASED, gfx::Point(),
                                    gfx::Point(), ui::EventTimeForNow(),
-                                   ui::EF_LEFT_MOUSE_BUTTON,
+                                   ui::EF_LEFT_MOUSE_BUTTON | event.flags(),

[sadrul, 2015/08/14 14:52:46]

KeyEvent can have KeyEventFlags, which can mean different things for a
MouseEvent (e.g. KeyEventFlags::EF_IME_FABRICATED_KEY ==
MouseEventFlags::EF_IS_DOUBLE_CLICK). So we should set only a white-listed set
of flags from event.flags().

Also, please add a test.

[Deepak, 2015/08/17 06:32:51]

Done.

                                    ui::EF_LEFT_MOUSE_BUTTON);
     NotifyClick(synthetic_event);
   } else {