diff options
| author | yuzo@chromium.org <yuzo@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-02 08:16:19 +0000 |
|---|---|---|
| committer | yuzo@chromium.org <yuzo@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-02 08:16:19 +0000 |
| commit | 44ce8376c42a7f7c2f3b231fef15d0fd69dfa436 (patch) | |
| tree | dd07136f805814d5fa7448c872a533851f197655 /chrome/renderer | |
| parent | fdc30615f7849eddcea3662f80074e7aba28adea (diff) | |
| download | chromium_src-44ce8376c42a7f7c2f3b231fef15d0fd69dfa436.zip chromium_src-44ce8376c42a7f7c2f3b231fef15d0fd69dfa436.tar.gz chromium_src-44ce8376c42a7f7c2f3b231fef15d0fd69dfa436.tar.bz2 | |
Fix: Off-by-one errors in loops over key codes in render_view_unittest.cc
BUG=none
TEST=n/a
Review URL: http://codereview.chromium.org/151194
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19804 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer')
| -rw-r--r-- | chrome/renderer/render_view_unittest.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/renderer/render_view_unittest.cc b/chrome/renderer/render_view_unittest.cc index 4ab5be0..ab27ec0 100644 --- a/chrome/renderer/render_view_unittest.cc +++ b/chrome/renderer/render_view_unittest.cc @@ -534,7 +534,7 @@ TEST_F(RenderViewTest, OnHandleKeyboardEvent) { }; MockKeyboard::Modifiers modifiers = kModifierData[j].modifiers; - for (size_t k = 0; k <= ARRAYSIZE_UNSAFE(kKeyCodes); ++k) { + for (size_t k = 0; k < ARRAYSIZE_UNSAFE(kKeyCodes); ++k) { // Send a keyboard event to the RenderView object. // We should test a keyboard event only when the given keyboard-layout // driver is installed in a PC and the driver can assign a Unicode @@ -759,7 +759,7 @@ TEST_F(RenderViewTest, InsertCharacters) { }; MockKeyboard::Modifiers modifiers = kModifiers[j]; - for (size_t k = 0; k <= ARRAYSIZE_UNSAFE(kKeyCodes); ++k) { + for (size_t k = 0; k < ARRAYSIZE_UNSAFE(kKeyCodes); ++k) { // Send a keyboard event to the RenderView object. // We should test a keyboard event only when the given keyboard-layout // driver is installed in a PC and the driver can assign a Unicode |