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..8f453a0fa69b40ac891182846e7024681a82fcfa 100644
--- a/ui/views/controls/button/custom_button.cc
+++ b/ui/views/controls/button/custom_button.cc
@@ -197,10 +197,13 @@ bool CustomButton::OnKeyPressed(const ui::KeyEvent& event) {
     SetState(STATE_PRESSED);
   } else if (event.key_code() == ui::VKEY_RETURN) {
     SetState(STATE_NORMAL);
+    // Keeping only white - listed set of flags and resetting the KeyEventFlags.
+    int flags = event.flags() &
+                ~(ui::EF_IME_FABRICATED_KEY | ui::EF_IS_REPEAT | ui::EF_FINAL);
     // TODO(beng): remove once NotifyClick takes ui::Event.

[sadrul, 2015/08/17 14:40:11]

NotifyClick() now takes a ui::Event. Can we address this TODO here, and just
send event, instead of synthesizing the mouse-release event?

[Deepak, 2015/08/18 04:34:56]

Done.

     ui::MouseEvent synthetic_event(ui::ET_MOUSE_RELEASED, gfx::Point(),
                                    gfx::Point(), ui::EventTimeForNow(),
-                                   ui::EF_LEFT_MOUSE_BUTTON,
+                                   ui::EF_LEFT_MOUSE_BUTTON | flags,
                                    ui::EF_LEFT_MOUSE_BUTTON);
     NotifyClick(synthetic_event);
   } else {