Fix controlling volume slider with keyboard

ui/views/controls/slider_unittest.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/slider.h"
 
 #include <memory>
 #include <string>
 
 #include "base/i18n/rtl.h"
 #include "base/macros.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/time/time.h"
 #include "build/build_config.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/events/event.h"
 #include "ui/events/gesture_event_details.h"
+#include "ui/events/keycodes/keyboard_codes.h"
 #include "ui/events/test/event_generator.h"
 #include "ui/views/test/slider_test_api.h"
 #include "ui/views/test/views_test_base.h"
 #include "ui/views/view.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/widget/widget_delegate.h"
 
 namespace {
 
 // A views::SliderListener that tracks simple event call history.
@@ skipping 266 matching lines @@
   // Scroll somewhere in the middle.
   slider()->SetValue(0.25);
   event_generator()->GestureScrollSequence(
     gfx::Point(0.25 * max_x(), 0.25 * max_y()),
     gfx::Point(0.75 * max_x(), 0.75 * max_y()),
     base::TimeDelta::FromMilliseconds(10),
     5 /* steps */);
   EXPECT_NEAR(0.75, slider()->value(), 0.03);
 }
 
+// Test the slider location by adjusting it using keyboard.
+TEST_F(HorizontalSliderTest, SliderValueForKeyboard) {
+  float value =0.5;
+  slider()->SetValue(value);
+  slider()->RequestFocus();
+  event_generator()->PressKey(ui::VKEY_RIGHT, 0);
+  EXPECT_GT(slider()->value(), value);
+
+  slider()->SetValue(value);
+  event_generator()->PressKey(ui::VKEY_LEFT, 0);
+  EXPECT_LT(slider()->value(), value);
+}
+
 // Verifies the correct SliderListener events are raised for a tap gesture.
 TEST_F(HorizontalSliderTest, SliderListenerEventsForTapGesture) {
   test::SliderTestApi slider_test_api(slider());
   slider_test_api.SetListener(&slider_listener());
 
   event_generator()->GestureTapAt(gfx::Point(0, 0));
   EXPECT_EQ(1, slider_listener().last_drag_started_epoch());
   EXPECT_EQ(2, slider_listener().last_drag_ended_epoch());
   EXPECT_EQ(slider(), slider_listener().last_drag_started_sender());
   EXPECT_EQ(slider(), slider_listener().last_drag_ended_sender());
@@ skipping 32 matching lines @@
   EXPECT_EQ(1, slider_listener().last_drag_started_epoch());
   EXPECT_GT(slider_listener().last_drag_ended_epoch(),
             slider_listener().last_drag_started_epoch());
   EXPECT_EQ(slider(), slider_listener().last_drag_started_sender());
   EXPECT_EQ(slider(), slider_listener().last_drag_ended_sender());
 }
 
 #endif  // !defined(OS_MACOSX) || defined(USE_AURA)
 
 }  // namespace views