Add DriverBugWorkaroundsInGpuProcessPage to gpu_process_test.py

content/common/gpu/gpu_channel.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/common/gpu/gpu_channel.h"
 
 #include <utility>
 
 #if defined(OS_WIN)
 #include <windows.h>
@@ skipping 742 matching lines @@
   RemoveRouteFromStream(route_id);
 }
 
 bool GpuChannel::OnControlMessageReceived(const IPC::Message& msg) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(GpuChannel, msg)
     IPC_MESSAGE_HANDLER(GpuChannelMsg_CreateCommandBuffer,
                         OnCreateCommandBuffer)
     IPC_MESSAGE_HANDLER(GpuChannelMsg_DestroyCommandBuffer,
                         OnDestroyCommandBuffer)
+    IPC_MESSAGE_HANDLER(GpuChannelMsg_GetDriverBugWorkArounds,

[palmer, 2016/04/01 19:19:54]

Do we need this in all builds, including production? If not, maybe we can guard
it with an #ifdef or otherwise keep it out of production builds. But if you do
need it, OK. Just checking.

[piman, 2016/04/01 20:26:22]

It needs to be in the binary for any configuration we test, but it could be
behind a command-line flag. Are you specifically concerned about security, or
binary size?

+                        OnGetDriverBugWorkArounds)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
 scoped_refptr<gpu::SyncPointOrderData> GpuChannel::GetSyncPointOrderData(
     int32_t stream_id) {
   auto it = streams_.find(stream_id);
   DCHECK(it != streams_.end());
   DCHECK(it->second);
@@ skipping 198 matching lines @@
   // In case the renderer is currently blocked waiting for a sync reply from the
   // stub, we need to make sure to reschedule the correct stream here.
   if (stub && !stub->IsScheduled()) {
     // This stub won't get a chance to reschedule the stream so do that now.
     OnStreamRescheduled(stub->stream_id(), true);
   }
 
   RemoveRoute(route_id);
 }
 
+void GpuChannel::OnGetDriverBugWorkArounds(
+    std::vector<std::string>* gpu_driver_bug_workarounds) {
+  // TODO(j.isorce): Do the extraction of workarounds in GpuChannelManager's

[palmer, 2016/04/01 19:19:54]

Please attach a crbug.com link to all TODOs.

+  // constructor. Currently it is done in FeatureInfo constructor. There is no
+  // need to extract them from the command-line every time a new FeatureInfo is
+  // created (i.e. per ContextGroup) since parsing result is a constant.
+  scoped_refptr<gpu::gles2::FeatureInfo> feature_info =
+      new gpu::gles2::FeatureInfo;
+  gpu_driver_bug_workarounds->clear();
+#define GPU_OP(type, name)              \
+  if (feature_info->workarounds().name) \
+    gpu_driver_bug_workarounds->push_back(#name);
+  GPU_DRIVER_BUG_WORKAROUNDS(GPU_OP)
+#undef GPU_OP
+}
+
 void GpuChannel::CacheShader(const std::string& key,
                              const std::string& shader) {
   gpu_channel_manager_->delegate()->StoreShaderToDisk(client_id_, key, shader);
 }
 
 void GpuChannel::AddFilter(IPC::MessageFilter* filter) {
   io_task_runner_->PostTask(
       FROM_HERE, base::Bind(&GpuChannelMessageFilter::AddChannelFilter,
                             filter_, make_scoped_refptr(filter)));
 }
@@ skipping 53 matching lines @@
     }
   }
 }
 
 void GpuChannel::HandleUpdateValueState(
     unsigned int target, const gpu::ValueState& state) {
   pending_valuebuffer_state_->UpdateState(target, state);
 }
 
 }  // namespace content