Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(140)

Unified Diff: gpu/command_buffer/service/cmd_parser_test.cc

Issue 614006: Fixes the issues with Jump, JumpRel, Call, CallRel (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « gpu/command_buffer/service/cmd_parser.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gpu/command_buffer/service/cmd_parser_test.cc
===================================================================
--- gpu/command_buffer/service/cmd_parser_test.cc (revision 39125)
+++ gpu/command_buffer/service/cmd_parser_test.cc (working copy)
@@ -225,10 +225,14 @@
// 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;
« no previous file with comments | « gpu/command_buffer/service/cmd_parser.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698