Remove opaque frame buffer in destructor of GbmBufferBase.

Remove opaque frame buffer in destructor of GbmBufferBase.

Opaque frame buffer was introduced in http://crrev.com/2743403005. 
This causes CTS tests failures by OOM killer.

BUG=706848
Review-Url: https://codereview.chromium.org/2790553002
Cr-Commit-Position: refs/heads/master@{#460788}
Committed: https://chromium.googlesource.com/chromium/src/+/565131ae89b7f786dad9d8743975bc628ee24888

[yhanada, 2017-03-30 10:36:03]

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

[commit-bot: I haz the power, 2017-03-30 10:36:19]

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

[yhanada, 2017-03-30 10:49:57]

yhanada@chromium.org changed reviewers:
+ dcastagna@chromium.org, reveman@chromium.org

[yhanada, 2017-03-30 10:49:57]

Could you review this CL? Thanks!

[reveman, 2017-03-30 10:54:09]

lgtm

please mention in the description of the CL what change caused this leak to
appear so it's easy to tell how long it has existed and if the fix needs to be
merged to any branches.

[yhanada, 2017-03-30 10:57:10]

Description was changed from

==========
Remove opaque frame buffer in destructor of GbmBufferBase.

This causes CTS tests failures by OOM killer.

BUG=b/36630915
==========

to

==========
Remove opaque frame buffer in destructor of GbmBufferBase.

Opaque frame buffer was introduced in http://crrev.com/2743403005. 
This causes CTS tests failures by OOM killer.

BUG=b/36630915
==========

[yhanada, 2017-03-30 10:57:51]

On 2017/03/30 10:54:09, reveman wrote:
> lgtm
> 
> please mention in the description of the CL what change caused this leak to
> appear so it's easy to tell how long it has existed and if the fix needs to be
> merged to any branches.

Thank you for your quick review. Updated the description.

[yhanada, 2017-03-30 10:59:10]

yhanada@chromium.org changed reviewers:
+ dnicoara@chromium.org

[yhanada, 2017-03-30 10:59:11]

+dnicoara for owner review

[commit-bot: I haz the power, 2017-03-30 11:02:55]

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

[commit-bot: I haz the power, 2017-03-30 11:02:55]

Dry run: This issue passed the CQ dry run.

[dnicoara, 2017-03-30 14:23:47]

lgtm

[yhanada, 2017-03-30 16:07:00]

Description was changed from

==========
Remove opaque frame buffer in destructor of GbmBufferBase.

Opaque frame buffer was introduced in http://crrev.com/2743403005. 
This causes CTS tests failures by OOM killer.

BUG=b/36630915
==========

to

==========
Remove opaque frame buffer in destructor of GbmBufferBase.

Opaque frame buffer was introduced in http://crrev.com/2743403005. 
This causes CTS tests failures by OOM killer.

BUG=706848
==========

[yhanada, 2017-03-30 16:07:46]

The CQ bit was checked by yhanada@chromium.org

[commit-bot: I haz the power, 2017-03-30 16:08:30]

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, 2017-03-30 16:22:10]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490890066124630, "parent_rev":
"eff95516605a04f45c09c603fc247d0da414a003", "commit_rev":
"565131ae89b7f786dad9d8743975bc628ee24888"}

[commit-bot: I haz the power, 2017-03-30 16:23:23]

Description was changed from

==========
Remove opaque frame buffer in destructor of GbmBufferBase.

Opaque frame buffer was introduced in http://crrev.com/2743403005. 
This causes CTS tests failures by OOM killer.

BUG=706848
==========

to

==========
Remove opaque frame buffer in destructor of GbmBufferBase.

Opaque frame buffer was introduced in http://crrev.com/2743403005. 
This causes CTS tests failures by OOM killer.

BUG=706848

Review-Url: https://codereview.chromium.org/2790553002
Cr-Commit-Position: refs/heads/master@{#460788}
Committed:
https://chromium.googlesource.com/chromium/src/+/565131ae89b7f786dad9d8743975...
==========

[commit-bot: I haz the power, 2017-03-30 16:23:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/565131ae89b7f786dad9d8743975...

[Daniele Castagna, 2017-03-30 16:24:30]

Thanks for fixing this!