Don't refcount tracking id -> slot id mapping.

ui/views/controls/menu/menu_event_dispatcher_linux.cc

 // Copyright 2014 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_event_dispatcher_linux.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "ui/aura/window.h"
 #include "ui/events/event_utils.h"
 #include "ui/events/keycodes/keyboard_code_conversion.h"
@@ skipping 57 matching lines @@
           }
         }
         should_quit = false;
         should_perform_default = false;
         break;
       }
       case ui::ET_KEY_RELEASED:
         should_quit = false;
         should_perform_default = false;
         break;
+      case ui::ET_TOUCH_RELEASED:
+      case ui::ET_TOUCH_CANCELLED:
+        static_cast<ui::TouchEvent*>(ui_event.get())
+            ->set_should_remove_native_touch_id_mapping(false);

[sadrul, 2015/01/14 20:21:35]

+ break;

Add a comment explaining why this is necessary.

Add a test for this in menu_controller_unittest.cc

[tdresser, 2015/01/23 18:12:34]

Done (eek, thanks)
Done
Done.

       default:
         break;
     }
   }
 
   if (should_quit || menu_controller_->exit_type() != MenuController::EXIT_NONE)
     menu_controller_->TerminateNestedMessageLoop();
 
   return should_perform_default ? ui::POST_DISPATCH_PERFORM_DEFAULT
                                 : ui::POST_DISPATCH_NONE;
 }
 
 }  // namespace internal
 }  // namespace views