Popup open and hide using Alt+KeyDown

Source/core/html/HTMLSelectElement.cpp

 /*
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2009, 2010, 2011 Apple Inc. All rights reserved.
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
@@ skipping 1096 matching lines @@
     }
 
     if (!selectedOption && firstOption && !m_multiple && m_size <= 1)
         firstOption->setSelectedState(true);
 
     setOptionsChangedOnRenderer();
     setNeedsStyleRecalc(SubtreeStyleChange);
     setNeedsValidityCheck();
 }
 
-#if !OS(WIN)
-bool HTMLSelectElement::platformHandleKeydownEvent(KeyboardEvent* event)
+bool HTMLSelectElement::handlePopupOpenKeyboardEvent(Event* event)
 {
-    if (!RenderTheme::theme().popsMenuByArrowKeys())
-        return false;
+    const KeyboardEvent* keyEvent = toKeyboardEvent(event);
+    const String& keyIdentifier = keyEvent->keyIdentifier();
+    RenderTheme& renderTheme = RenderTheme::theme();
+    int keyCode = toKeyboardEvent(event)->keyCode();
 
-    if (!isSpatialNavigationEnabled(document().frame())) {
-        if (event->keyIdentifier() == "Down" || event->keyIdentifier() == "Up") {
-            focus();
-            // Calling focus() may cause us to lose our renderer. Return true so
-            // that our caller doesn't process the event further, but don't set
-            // the event as handled.
-            if (!renderer() || !renderer()->isMenuList() || isDisabledFormControl())
-                return true;
+    // downUpKey and spaceKey works for mac.
+    bool downUpKey = (event->type() == EventTypeNames::keydown) && !isSpatialNavigationEnabled(document().frame()) && RenderTheme::theme().popsMenuByArrowKeys() && (keyIdentifier == "Down" || keyIdentifier == "Up");
+    bool spaceKey = (event->type() == EventTypeNames::keypress) &&  renderTheme.popsMenuByArrowKeys() && keyCode == ' ';

[keishi, 2014/07/04 01:33:50]

Using RenderTheme.popsMenuByArrowKeys here seems wrong. Maybe you can split
popsMenuBySpaceOrReturn to popsMenuBySpace and popsMenuByReturn

[Habib Virji, 2014/07/08 12:21:32]

Done.

+    // spaceOrEnterKey, altDownUpKey and f4Key all work for linux and windows but not for mac.
+    bool spaceOrEnterKey = (event->type() == EventTypeNames::keypress) && renderTheme.popsMenuBySpaceOrReturn() && (keyCode == ' ' || keyCode == '\r');
+    bool altDownUpKey = (event->type() == EventTypeNames::keydown) && renderTheme.shouldOpenDropDownWithAltDownUpOrF4Key() && (keyIdentifier == "Down" || keyIdentifier == "Up") && (keyEvent->altKey() || keyEvent->altGraphKey());
+    bool f4Key = (event->type() == EventTypeNames::keydown) && renderTheme.shouldOpenDropDownWithAltDownUpOrF4Key() && (!keyEvent->altKey() && !keyEvent->ctrlKey() && keyIdentifier == "F4");
 
-            // Save the selection so it can be compared to the new selection
-            // when dispatching change events during selectOption, which
-            // gets called from RenderMenuList::valueChanged, which gets called
-            // after the user makes a selection from the menu.
-            saveLastSelection();
-            if (RenderMenuList* menuList = toRenderMenuList(renderer()))
-                menuList->showPopup();
-            event->setDefaultHandled();
-        }
+    if (downUpKey || spaceKey || spaceOrEnterKey || altDownUpKey || f4Key) {

[keishi, 2014/07/04 01:33:50]

nit: I think this is kind of confusing because spaceKey may be false even when
space is pressed.
Maybe we could have a single shouldOpenPopup boolean and do |=.

bool shouldOpenPopup = false;
if (event->type() == EventTypeNames::keypress)
  shouldOpenPopup |= renderTheme.popsMenuByArrowKeys() && keyCode == ' ';
  ...
} else if (event->type() == EventTypeNames::keydown)
  ...
}
if (shouldOpenPopup) {
  // open popup
} else {
  // Maybe you could event handle form submission and spatial navigation
selecting in here too.
}

[Habib Virji, 2014/07/08 12:21:32]

Done.

+        focus();
+        // Calling focus() may cause us to lose our renderer. Return true so
+        // that our caller doesn't process the event further, but don't set
+        // the event as handled.
+        if (!renderer() || !renderer()->isMenuList() || isDisabledFormControl())
+            return true;
+        // Save the selection so it can be compared to the new selection
+        // when dispatching change events during selectOption, which
+        // gets called from RenderMenuList::valueChanged, which gets called
+        // after the user makes a selection from the menu.
+        saveLastSelection();
+        if (RenderMenuList* menuList = toRenderMenuList(renderer()))
+            menuList->showPopup();
+        int index = selectedIndex();
+        ASSERT(index >= 0);
+        ASSERT_WITH_SECURITY_IMPLICATION(index < static_cast<int>(listItems().size()));
+        setSelectedIndex(index);
+        event->setDefaultHandled();
         return true;
     }
 
     return false;
 }
-#endif
 
 void HTMLSelectElement::menuListDefaultEventHandler(Event* event)
 {
     RenderTheme& renderTheme = RenderTheme::theme();
 
+    if (renderer() && event->isKeyboardEvent() && handlePopupOpenKeyboardEvent(event))
+        return;
+
     if (event->type() == EventTypeNames::keydown) {
         if (!renderer() || !event->isKeyboardEvent())
             return;
 
-        if (platformHandleKeydownEvent(toKeyboardEvent(event)))
-            return;
-
         // When using spatial navigation, we want to be able to navigate away
         // from the select element when the user hits any of the arrow keys,
         // instead of changing the selection.
         if (isSpatialNavigationEnabled(document().frame())) {
             if (!m_activeSelectionState)
                 return;
         }
 
         const String& keyIdentifier = toKeyboardEvent(event)->keyIdentifier();
         bool handled = true;
@@ skipping 22 matching lines @@
             event->setDefaultHandled();
     }
 
     // Use key press event here since sending simulated mouse events
     // on key down blocks the proper sending of the key press event.
     if (event->type() == EventTypeNames::keypress) {
         if (!renderer() || !event->isKeyboardEvent())
             return;
 
         int keyCode = toKeyboardEvent(event)->keyCode();
-        bool handled = false;
 
         if (keyCode == ' ' && isSpatialNavigationEnabled(document().frame())) {
             // Use space to toggle arrow key handling for selection change or spatial navigation.
             m_activeSelectionState = !m_activeSelectionState;
             event->setDefaultHandled();
             return;
         }
 
-        if (renderTheme.popsMenuBySpaceOrReturn()) {
-            if (keyCode == ' ' || keyCode == '\r') {
-                focus();
-
-                // Calling focus() may remove the renderer or change the
-                // renderer type.
-                if (!renderer() || !renderer()->isMenuList() || isDisabledFormControl())
-                    return;
-
-                // Save the selection so it can be compared to the new selection
-                // when dispatching change events during selectOption, which
-                // gets called from RenderMenuList::valueChanged, which gets called
-                // after the user makes a selection from the menu.
-                saveLastSelection();
-                if (RenderMenuList* menuList = toRenderMenuList(renderer()))
-                    menuList->showPopup();
-                handled = true;
-            }
-        } else if (renderTheme.popsMenuByArrowKeys()) {
-            if (keyCode == ' ') {
-                focus();
-
-                // Calling focus() may remove the renderer or change the
-                // renderer type.
-                if (!renderer() || !renderer()->isMenuList() || isDisabledFormControl())
-                    return;
-
-                // Save the selection so it can be compared to the new selection
-                // when dispatching change events during selectOption, which
-                // gets called from RenderMenuList::valueChanged, which gets called
-                // after the user makes a selection from the menu.
-                saveLastSelection();
-                if (RenderMenuList* menuList = toRenderMenuList(renderer()))
-                    menuList->showPopup();
-                handled = true;
-            } else if (keyCode == '\r') {
-                if (form())
-                    form()->submitImplicitly(event, false);
-                dispatchInputAndChangeEventForMenuList();
-                handled = true;
-            }
+        if (renderTheme.popsMenuByArrowKeys() && keyCode == '\r') {
+            if (form())
+                form()->submitImplicitly(event, false);
+            dispatchInputAndChangeEventForMenuList();
+            event->setDefaultHandled();
         }
-
-        if (handled)
-            event->setDefaultHandled();
     }
 
     if (event->type() == EventTypeNames::mousedown && event->isMouseEvent() && toMouseEvent(event)->button() == LeftButton) {
         focus();
         if (renderer() && renderer()->isMenuList() && !isDisabledFormControl()) {
             if (RenderMenuList* menuList = toRenderMenuList(renderer())) {
                 if (menuList->popupIsVisible())
                     menuList->hidePopup();
                 else {
                     // Save the selection so it can be compared to the new
@@ skipping 393 matching lines @@
 
 void HTMLSelectElement::trace(Visitor* visitor)
 {
 #if ENABLE(OILPAN)
     visitor->trace(m_listItems);
 #endif
     HTMLFormControlElementWithState::trace(visitor);
 }
 
 } // namespace