services/ui: Move cc::SoftwareOutputDevice implementation into the client lib.

services/ui/public/cpp/BUILD.gn

Index: services/ui/public/cpp/BUILD.gn
diff --git a/services/ui/public/cpp/BUILD.gn b/services/ui/public/cpp/BUILD.gn
index a4a14d7ec47e85b94104b15c592307c315a011c7..b8be781253674e7ba2961215da0bca58e15f181a 100644
--- a/services/ui/public/cpp/BUILD.gn
+++ b/services/ui/public/cpp/BUILD.gn
@@ -6,12 +6,12 @@
 # implementation (and private haders) are in 'internal'.
 source_set("cpp") {
   sources = [
-    "bitmap_uploader.h",
     "input_event_handler.h",
     "output_surface.h",
     "property_type_converters.h",
     "raster_thread_helper.h",
     "scoped_window_ptr.h",
+    "software_output_device.h",
     "window.h",
     "window_manager_delegate.h",
     "window_observer.h",
@@ -85,10 +85,14 @@ source_set("internal_or_test") {
 source_set("internal") {
   # This target is an implementation detail and is intended only to be used by
   # the 'cpp' target.
-  visibility = [ ":cpp" ]
+  visibility = [
+    ":cpp",
+    "//services/ui/demo:mus_demo_library",

[sky, 2016/08/10 15:10:39]

Why do we want to move bitmap_uploader to internal, but expose it to the demo?
It seems like either we should go with bitmap_uploader be public as we had it,
or change mus_demo_library.

[sadrul, 2016/08/10 15:44:01]

I have been thinking about it myself since I sent it out for review. I think
clients should normally use OutputSurface/SoftwareOutputDevice for sending
frames to mus, instead of directly using BitmapUploader. mus_demo is a bit of an
exception since it exists more as a mus test than as a proper mus client. I feel
like having all of OutputSurface, SoftwareOutputDevice, BitmapUploader in the
API may be a bit confusing, and it'd be better to restrict the API the clients
should use. But maybe it does make sense to allow clients to directly use
BitmapUploader too?

rjkroege@ what do you think?

+  ]
 
   sources = [
     "bitmap_uploader.cc",
+    "bitmap_uploader.h",
     "command_buffer_client_impl.cc",
     "command_buffer_client_impl.h",
     "context_provider.cc",
@@ -101,6 +105,7 @@ source_set("internal") {
     "property_type_converters.cc",
     "raster_thread_helper.cc",
     "scoped_window_ptr.cc",
+    "software_output_device.cc",
     "window.cc",
     "window_manager_delegate.cc",
     "window_observer.cc",