Removing the JumpRelative, Call, CallRelative and Return commands.

gpu/command_buffer/client/cmd_buffer_helper.h

Index: gpu/command_buffer/client/cmd_buffer_helper.h
diff --git a/gpu/command_buffer/client/cmd_buffer_helper.h b/gpu/command_buffer/client/cmd_buffer_helper.h
index 8e7cd78caf70eb6f3c943ee28ea9b156d9394227..7e44de9b4d0c16fda6eabdfc7e85b822fb257198 100644
--- a/gpu/command_buffer/client/cmd_buffer_helper.h
+++ b/gpu/command_buffer/client/cmd_buffer_helper.h
@@ -149,41 +149,6 @@ class GPU_EXPORT CommandBufferHelper {
     }
   }
 
-  void Jump(uint32 offset) {

[greggman, 2012/12/19 03:32:18]

Isn't Jump still needed? You seem to be keeping Jump everywhere else.

[dsinclair, 2012/12/19 03:41:07]

The code that uses Jump in cmd_buffer_helper.cc uses cmd::Jump::Set directly. It
doesn't use the Jump method for some reason. I figured taking the method out was
safe (everything is compiling and most of the try-bots are happy), and removes
the client side API if not the struct and service side handling.

-    cmd::Jump* cmd = GetCmdSpace<cmd::Jump>();
-    if (cmd) {
-      cmd->Init(offset);
-    }
-  }
-
-  void JumpRelative(int32 offset) {
-    cmd::JumpRelative* cmd = GetCmdSpace<cmd::JumpRelative>();
-    if (cmd) {
-      cmd->Init(offset);
-    }
-  }
-
-  void Call(uint32 offset) {
-    cmd::Call* cmd = GetCmdSpace<cmd::Call>();
-    if (cmd) {
-      cmd->Init(offset);
-    }
-  }
-
-  void CallRelative(int32 offset) {
-    cmd::CallRelative* cmd = GetCmdSpace<cmd::CallRelative>();
-    if (cmd) {
-      cmd->Init(offset);
-    }
-  }
-
-  void Return() {
-    cmd::Return* cmd = GetCmdSpace<cmd::Return>();
-    if (cmd) {
-      cmd->Init();
-    }
-  }
-
   void SetBucketSize(uint32 bucket_id, uint32 size) {
     cmd::SetBucketSize* cmd = GetCmdSpace<cmd::SetBucketSize>();
     if (cmd) {