ui::Compositor needs a valid FrameSinkId even when it doesn't have ContextFactoryPrivate.

ui/compositor/compositor.cc

Index: ui/compositor/compositor.cc
diff --git a/ui/compositor/compositor.cc b/ui/compositor/compositor.cc
index f2d08cde512820fb5ea9af028f7e25811b276e4d..809dc28a4eb0a44401be9a0356246577445db97b 100644
--- a/ui/compositor/compositor.cc
+++ b/ui/compositor/compositor.cc
@@ -46,6 +46,13 @@ namespace {
 const double kDefaultRefreshRate = 60.0;
 const double kTestRefreshRate = 200.0;
 
+// TODO(mfomitchev, fsamuel): Looks at removing this when transition from

[sadrul, 2016/12/16 20:17:28]

*Look

[mfomitchev, 2016/12/16 20:49:45]

Done.

+// SurfaceSequence to SurfaceReference is fully complete.
+// Used when ui::Compositor is created with null ContextFactoryPrivate.
+// Compositor needs a valid FrameSinkId, so that cc::SurfaceLayer can create
+// valid SurfaceSequences.
+constexpr cc::FrameSinkId kDefaultFrameSinkId(1, 1);
+
 }  // namespace
 
 namespace ui {
@@ -85,7 +92,7 @@ Compositor::Compositor(ui::ContextFactory* context_factory,
       compositor_frame_sink_requested_(false),
       frame_sink_id_(context_factory_private
                          ? context_factory_private->AllocateFrameSinkId()
-                         : cc::FrameSinkId()),
+                         : kDefaultFrameSinkId),
       task_runner_(task_runner),
       vsync_manager_(new CompositorVSyncManager()),
       device_scale_factor_(0.0f),