[M67 Migration] Prevent double free of MailboxManager unique pointer 59/189559/3
authorUzair <uzair.jaleel@samsung.com>
Tue, 18 Sep 2018 13:22:53 +0000 (18:52 +0530)
committerUzair <uzair.jaleel@samsung.com>
Wed, 19 Sep 2018 12:27:31 +0000 (17:57 +0530)
commit4c6eb261f1b81a5b7112e93555873734d0ac7d6e
treeb7012eaf1043ce88d60cc9f4cc259f76e92dc7fd
parent87a2f6fb6e4e3fb1cd10f70fdf9c49c39ed9c58d
[M67 Migration] Prevent double free of MailboxManager unique pointer

Crash is observed at browser shutdown as MaiboxManager is getting
deleted twice, once when GpuChannelManager is destroyed and again
in SharedMailboxManager. Since we already maintain a unique pointer
(static) for MailboxManager in SharedMailboxManager, GpuChannelManager
doesn't necessarily need to have another.

This patch also avoids creation of GLShareGroup and MailboxManager
in GpuChannelManager as they are anyway overriden in GpuChildThreadEfl.

Reference :
https://review.tizen.org/gerrit/#/c/179612

Change-Id: I3d4e7e211bb1260ccf28a03da92a94b39f6a16d0
Signed-off-by: Uzair <uzair.jaleel@samsung.com>
gpu/ipc/service/gpu_channel_manager.cc
gpu/ipc/service/gpu_channel_manager.h
tizen_src/chromium_impl/content/gpu/in_process_gpu_thread_efl.cc