Properly set the BufferToTextureTargetMap in Blimp

Properly set the BufferToTextureTargetMap in Blimp

Make sure to set the texture target map properly.  This is temporary as
this code will be going away shortly.  Android (and the Linux client)
look like they only use GL_TEXTURE_2D for the map entries, so hard
coding that for the standalone app.

BUG=632418
Committed: https://crrev.com/5ea3a8291f91fcddc9e408b50b736f0d5ea929a1
Cr-Commit-Position: refs/heads/master@{#409213}

[David Trainor- moved to gerrit, 2016-08-02 02:17:30]

dtrainor@chromium.org changed reviewers:
+ khushalsagar@chromium.org

[David Trainor- moved to gerrit, 2016-08-02 02:17:31]

ptal thanks!

[David Trainor- moved to gerrit, 2016-08-02 02:19:23]

The CQ bit was checked by dtrainor@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-02 02:19:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-02 02:49:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-02 02:50:00]

Dry run: This issue passed the CQ dry run.

[Khushal, 2016-08-02 06:39:01]

lgtm. Thanks!

https://codereview.chromium.org/2207453002/diff/1/blimp/client/feature/compos...
File blimp/client/feature/compositor/blimp_gpu_memory_buffer_manager.cc (right):

https://codereview.chromium.org/2207453002/diff/1/blimp/client/feature/compos...
blimp/client/feature/compositor/blimp_gpu_memory_buffer_manager.cc:20: namespace
{
Doesn't look like GetDefaultBufferToTextureTargetMap is using this code?

[David Trainor- moved to gerrit, 2016-08-02 15:37:14]

The CQ bit was checked by dtrainor@chromium.org to run a CQ dry run

[David Trainor- moved to gerrit, 2016-08-02 15:37:28]

https://codereview.chromium.org/2207453002/diff/1/blimp/client/feature/compos...
File blimp/client/feature/compositor/blimp_gpu_memory_buffer_manager.cc (right):

https://codereview.chromium.org/2207453002/diff/1/blimp/client/feature/compos...
blimp/client/feature/compositor/blimp_gpu_memory_buffer_manager.cc:20: namespace
{
On 2016/08/02 06:39:01, Khushal wrote:
> Doesn't look like GetDefaultBufferToTextureTargetMap is using this code?

Crap.  done.

[commit-bot: I haz the power, 2016-08-02 15:37:33]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-02 16:40:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-02 16:40:26]

Dry run: This issue passed the CQ dry run.

[David Trainor- moved to gerrit, 2016-08-02 17:05:35]

The CQ bit was checked by dtrainor@chromium.org

[David Trainor- moved to gerrit, 2016-08-02 17:05:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
khushalsagar@chromium.org
Link to the patchset: https://codereview.chromium.org/2207453002/#ps20001
(title: "Removed unused code.")

[commit-bot: I haz the power, 2016-08-02 17:06:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-02 17:10:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-02 17:13:58]

Description was changed from

==========
Properly set the BufferToTextureTargetMap in Blimp

Make sure to set the texture target map properly.  This is temporary as
this code will be going away shortly.  Android (and the Linux client)
look like they only use GL_TEXTURE_2D for the map entries, so hard
coding that for the standalone app.

BUG=632418
==========

to

==========
Properly set the BufferToTextureTargetMap in Blimp

Make sure to set the texture target map properly.  This is temporary as
this code will be going away shortly.  Android (and the Linux client)
look like they only use GL_TEXTURE_2D for the map entries, so hard
coding that for the standalone app.

BUG=632418

Committed: https://crrev.com/5ea3a8291f91fcddc9e408b50b736f0d5ea929a1
Cr-Commit-Position: refs/heads/master@{#409213}
==========

[commit-bot: I haz the power, 2016-08-02 17:13:59]

Patchset 2 (id:??) landed as
https://crrev.com/5ea3a8291f91fcddc9e408b50b736f0d5ea929a1
Cr-Commit-Position: refs/heads/master@{#409213}