Clear shader disk cache after glProgramBinary failure.

services/ui/gpu/gpu_main.cc

Index: services/ui/gpu/gpu_main.cc
diff --git a/services/ui/gpu/gpu_main.cc b/services/ui/gpu/gpu_main.cc
index 3e4015a3d26bbd92dfe7fa699d08f24dfe962bfd..0bb995d1c628c53ef293bc0194fffbdd145ce2ce 100644
--- a/services/ui/gpu/gpu_main.cc
+++ b/services/ui/gpu/gpu_main.cc
@@ -7,6 +7,7 @@
 #include "base/command_line.h"
 #include "base/memory/ptr_util.h"
 #include "base/message_loop/message_loop.h"
+#include "gpu/command_buffer/common/activity_flags.h"
 #include "gpu/ipc/common/gpu_memory_buffer_support.h"
 #include "gpu/ipc/gpu_in_process_thread_service.h"
 #include "gpu/ipc/service/gpu_memory_buffer_factory.h"
@@ -118,7 +119,10 @@ void GpuMain::OnStart() {
 
 void GpuMain::CreateGpuService(mojom::GpuServiceRequest request,
                                mojom::GpuHostPtr gpu_host,
-                               const gpu::GpuPreferences& preferences) {
+                               const gpu::GpuPreferences& preferences,
+                               mojo::ScopedSharedBufferHandle activity_flags) {
+  gpu::GpuProcessActivityFlags::InitializeInstance(std::move(activity_flags));

[sadrul, 2017/03/16 04:40:32]

Note that GpuChannelManager is not created in the current thread. Instead, it
gets created by GpuService, in GpuMain::CreateGpuServiceOnGpuThread(). Should
this be initialized in that thread too?

Maybe it'd be simplest to move this GpuProcessActivityFlags initialization into
GpuService::Bind(), or GpuService::InitializeWithHost()? (since GpuService is
used by both this GpuMain, and content::GpuChildThread)

[ericrk, 2017/03/16 21:25:23]

Seems like it shouldn't matter where this is initialized, it's not doing
anything in init other than moving the ScopedSharedBufferHandle and mapping it.

I've re-factored things so we don't use a singleton any more, but instead pass
the GpuProcessActivityFlags through to the GpuChannelManager.

I still create the GpuProcessActivityFlags here, as it seems more
self-documenting to pass that object around rather than a
ScopedSharedBufferHandle... WDYT?

+
   // |this| will outlive the gpu thread and so it's safe to use
   // base::Unretained here.
   gpu_thread_task_runner_->PostTask(