"F4" doesn't close/open the "Folder" dropdown on "Bookmark added!" pop-up.

ui/views/controls/menu/menu_controller.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/views/controls/menu/menu_controller.h"
 
 #include "base/i18n/case_conversion.h"
 #include "base/i18n/rtl.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/time/time.h"
@@ skipping 1016 matching lines @@
       break;
 
     case ui::VKEY_SPACE:
       if (SendAcceleratorToHotTrackedView() == ACCELERATOR_PROCESSED_EXIT)
         return false;
       break;
 
     case ui::VKEY_F4:
       if (!is_combobox_)
         break;
       // Fallthrough to accept on F4, so combobox menus match Windows behavior.

[msw, 2014/08/20 01:05:01]

nit: "Fallthrough to accept or dismiss combobox menus on F4, like Windows."

[Deepak, 2014/08/20 03:37:33]

Done.

     case ui::VKEY_RETURN:
       if (pending_state_.item) {
         if (pending_state_.item->HasSubmenu()) {
-          OpenSubmenuChangeSelectionIfCan();
+          if (key_code == ui::VKEY_F4 &&
+              pending_state_.item->GetSubmenu()->IsShowing())

[sky, 2014/08/20 02:45:39]

Why does it matter whether the submenu is showing?

[Deepak, 2014/08/20 03:37:33]

As When Submneu is shown, and we select F4, we wanted to dismiss submenu and
don't want to show the selection in the submenu.

and if submenu is not shown and we select F4 then we want submenu to be shown,
To get the same windows behavior.

[sky, 2014/08/20 15:21:39]

Comboboxs don't have submenus, right?

+            return false;
+          else
+            OpenSubmenuChangeSelectionIfCan();
         } else {
           SendAcceleratorResultType result = SendAcceleratorToHotTrackedView();
           if (result == ACCELERATOR_NOT_PROCESSED &&
               pending_state_.item->enabled()) {
             Accept(pending_state_.item, 0);
             return false;
           } else if (result == ACCELERATOR_PROCESSED_EXIT) {
             return false;
           }
         }
@@ skipping 1253 matching lines @@
   }
 }
 
 gfx::Screen* MenuController::GetScreen() {
   Widget* root = owner_ ? owner_->GetTopLevelWidget() : NULL;
   return root ? gfx::Screen::GetScreenFor(root->GetNativeView())
               : gfx::Screen::GetNativeScreen();
 }
 
 }  // namespace views