diff options
| author | piman@chromium.org <piman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-15 14:22:37 +0000 |
|---|---|---|
| committer | piman@chromium.org <piman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-15 14:22:37 +0000 |
| commit | 64ba52f03b2936af0e6b924417ca8f29958b473e (patch) | |
| tree | 62d4f2e60903550e97c618f9074ec92340bfe3f2 /mojo/gles2 | |
| parent | b5641b965afa45a0c0a1e85a669935e9150ec9f0 (diff) | |
| download | chromium_src-64ba52f03b2936af0e6b924417ca8f29958b473e.zip chromium_src-64ba52f03b2936af0e6b924417ca8f29958b473e.tar.gz chromium_src-64ba52f03b2936af0e6b924417ca8f29958b473e.tar.bz2 | |
gpu: Generate mailboxes on client side
Because mailboxes are now just a random number, we don't need to round-trip to
the gpu process to generate them, which lets us get rid of IPCs, layers, etc.
It also means generating a mailbox can't fail any more (even on a lost context),
so, removing some code paths and associated tests.
I'm adding a debug-only verification to ensure the mailboxes are generated from
the crypto-random function (in debug, we burn a byte to compute a mini XOR-check
as a sentinel for "we went through the Generate function"). It's not a secure
check, but should hit incorrect/unsafe usage.
BUG=None
Review URL: https://codereview.chromium.org/165393003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251570 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'mojo/gles2')
| -rw-r--r-- | mojo/gles2/command_buffer_client_impl.cc | 8 | ||||
| -rw-r--r-- | mojo/gles2/command_buffer_client_impl.h | 2 |
2 files changed, 0 insertions, 10 deletions
diff --git a/mojo/gles2/command_buffer_client_impl.cc b/mojo/gles2/command_buffer_client_impl.cc index e4d0357..60fa366 100644 --- a/mojo/gles2/command_buffer_client_impl.cc +++ b/mojo/gles2/command_buffer_client_impl.cc @@ -186,14 +186,6 @@ void CommandBufferClientImpl::DestroyGpuMemoryBuffer(int32 id) { NOTIMPLEMENTED(); } -bool CommandBufferClientImpl::GenerateMailboxNames( - unsigned num, - std::vector<gpu::Mailbox>* names) { - // TODO(piman) - NOTIMPLEMENTED(); - return false; -} - uint32 CommandBufferClientImpl::InsertSyncPoint() { // TODO(piman) NOTIMPLEMENTED(); diff --git a/mojo/gles2/command_buffer_client_impl.h b/mojo/gles2/command_buffer_client_impl.h index fa1d1a8..ab5fed3 100644 --- a/mojo/gles2/command_buffer_client_impl.h +++ b/mojo/gles2/command_buffer_client_impl.h @@ -72,8 +72,6 @@ class CommandBufferClientImpl : public CommandBufferClient, unsigned internalformat, int32* id) OVERRIDE; virtual void DestroyGpuMemoryBuffer(int32 id) OVERRIDE; - virtual bool GenerateMailboxNames(unsigned num, - std::vector<gpu::Mailbox>* names) OVERRIDE; virtual uint32 InsertSyncPoint() OVERRIDE; virtual void SignalSyncPoint(uint32 sync_point, const base::Closure& callback) OVERRIDE; |