Fixes the issues with Jump, JumpRel, Call, CallRel

gpu/command_buffer/service/cmd_parser_test.cc

 // Copyright (c) 2009 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.
 
 // Tests for the command parser.
 
 #include "gpu/command_buffer/service/precompile.h"
 
 #include "base/scoped_ptr.h"
 #include "gpu/command_buffer/service/cmd_parser.h"
@@ skipping 207 matching lines @@
   AddDoCommandExpect(error::kNoError, 4, 1, &param);
   parser->set_put(put);
   EXPECT_EQ(put, parser->put());
   EXPECT_EQ(error::kNoError, parser->ProcessAllCommands());
   EXPECT_EQ(put, parser->get());
   Mock::VerifyAndClearExpectations(api_mock());
 }
 
 // Tests error conditions.
 TEST_F(CommandParserTest, TestError) {
-  scoped_ptr<CommandParser> parser(MakeParser(5));
+  const unsigned int kNumEntries = 5;
+  scoped_ptr<CommandParser> parser(MakeParser(kNumEntries));
   CommandBufferOffset put = parser->put();
   CommandHeader header;
 
+  EXPECT_FALSE(parser->set_get(-1));
+  EXPECT_FALSE(parser->set_get(kNumEntries));
+
   // Generate a command with size 0.
   header.size = 0;
   header.command = 3;
   buffer()[put++].value_header = header;
 
   parser->set_put(put);
   EXPECT_EQ(put, parser->put());
   EXPECT_EQ(error::kInvalidSize,
             parser->ProcessAllCommands());
   // check that no DoCommand call was made.
@@ skipping 37 matching lines @@
   EXPECT_EQ(put_post_fail, parser->get());
   Mock::VerifyAndClearExpectations(api_mock());
   // make the second one succeed, and check that the parser recovered fine.
   AddDoCommandExpect(error::kNoError, 4, 0, NULL);
   EXPECT_EQ(error::kNoError, parser->ProcessAllCommands());
   EXPECT_EQ(put, parser->get());
   Mock::VerifyAndClearExpectations(api_mock());
 }
 
 }  // namespace gpu