[Chromecast] Support software rendering when disable-gpu used

ui/ozone/platform/cast/ozone_platform_cast.cc

Index: ui/ozone/platform/cast/ozone_platform_cast.cc
diff --git a/ui/ozone/platform/cast/ozone_platform_cast.cc b/ui/ozone/platform/cast/ozone_platform_cast.cc
index bd4053e9ae93765a4a5ab7c37bf41195e2f241cf..6f2675a9d1ad297672162ec2bf781b82541fd87b 100644
--- a/ui/ozone/platform/cast/ozone_platform_cast.cc
+++ b/ui/ozone/platform/cast/ozone_platform_cast.cc
@@ -80,6 +80,10 @@ class OzonePlatformCast : public OzonePlatform {
     cursor_factory_.reset(new CursorFactoryOzone());
     input_controller_ = CreateStubInputController();
     gpu_platform_support_host_.reset(CreateStubGpuPlatformSupportHost());
+
+    // Enable dummy software rendering support if GPU process disabled
+    if (base::CommandLine::ForCurrentProcess()->HasSwitch("disable-gpu"))

[derekjchow1, 2015/12/08 22:03:15]

define kDisableGpu

[slan, 2015/12/08 22:40:25]

+1

Although I'm afraid we might be overloading the switch. Do you intend to use the
switch declared here:

src/content/public/common/content_switches.h

If not, we should change this string. If so, I wish we could create a dependency
here, but that is almost certainly a layering violation...

[halliwell, 2015/12/08 23:27:56]

Indeed, I'm using the switch from content, but can't include the header :(

[derekjchow1, 2015/12/09 21:19:34]

Please redefine in a local anonymous namespace.

+      surface_factory_.reset(new SurfaceFactoryCast());
   }
   void InitializeGPU() override {
     surface_factory_.reset(new SurfaceFactoryCast(egl_platform_.Pass()));