Android GlTextureFrameBuffer: Re-attach texture in setSize

Android GlTextureFrameBuffer: Re-attach texture in setSize

BUG=None

Review-Url: https://codereview.webrtc.org/2571503002
Cr-Commit-Position: refs/heads/master@{#16110}
Committed: https://chromium.googlesource.com/external/webrtc/+/9b96a172ed48dbf4618dd39bcd014eebe6de0310

[magjed_webrtc, 2016-12-12 16:12:29]

The CQ bit was checked by magjed@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-12 16:12:36]

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

[commit-bot: I haz the power, 2016-12-12 16:36:53]

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

[commit-bot: I haz the power, 2016-12-12 16:36:54]

Dry run: This issue passed the CQ dry run.

[magjed_webrtc, 2017-01-16 16:03:22]

The CQ bit was checked by magjed@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-16 16:03:32]

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

[magjed_webrtc, 2017-01-16 16:03:45]

Description was changed from

==========
Android GlTextureFrameBuffer: Re-attach texture in setSize

BUG=webrtc:6470
==========

to

==========
Android GlTextureFrameBuffer: Re-attach texture in setSize

BUG=webrtc:6470
==========

[magjed_webrtc, 2017-01-16 16:03:46]

magjed@webrtc.org changed reviewers:
+ sakal@webrtc.org

[magjed_webrtc, 2017-01-16 16:04:06]

Description was changed from

==========
Android GlTextureFrameBuffer: Re-attach texture in setSize

BUG=webrtc:6470
==========

to

==========
Android GlTextureFrameBuffer: Re-attach texture in setSize

BUG=None
==========

[magjed_webrtc, 2017-01-16 16:04:15]

Sami - please take a look.

[commit-bot: I haz the power, 2017-01-16 18:02:01]

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

[commit-bot: I haz the power, 2017-01-16 18:02:02]

Dry run: This issue passed the CQ dry run.

[sakal, 2017-01-17 08:25:16]

lgtm

Can you please verify this still works on Samsung Galaxy S3 mini? I think it
should be fine since the framebuffer is only bound after changing the texture
size.

[sakal, 2017-01-17 09:08:25]

I just tested and it didn't seem to crash.

[sakal, 2017-01-17 09:08:25]

I just tested and it didn't seem to crash.

[magjed_webrtc, 2017-01-17 10:48:38]

The CQ bit was checked by magjed@webrtc.org

[commit-bot: I haz the power, 2017-01-17 10:48:50]

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

[commit-bot: I haz the power, 2017-01-17 10:51:06]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484650118137530,
"parent_rev": "1fd08c1e67b0cce45ddbadf660fc368cf3aac1ae", "commit_rev":
"9b96a172ed48dbf4618dd39bcd014eebe6de0310"}

[commit-bot: I haz the power, 2017-01-17 10:51:10]

Description was changed from

==========
Android GlTextureFrameBuffer: Re-attach texture in setSize

BUG=None
==========

to

==========
Android GlTextureFrameBuffer: Re-attach texture in setSize

BUG=None

Review-Url: https://codereview.webrtc.org/2571503002
Cr-Commit-Position: refs/heads/master@{#16110}
Committed:
https://chromium.googlesource.com/external/webrtc/+/9b96a172ed48dbf4618dd39bc...
==========

[commit-bot: I haz the power, 2017-01-17 10:51:11]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/9b96a172ed48dbf4618dd39bc...