Add FrameGenerator Unit Tests

services/ui/ws/frame_generator.cc

 // Copyright 2016 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 "services/ui/ws/frame_generator.h"
 
 #include <utility>
 #include <vector>
 
 #include "cc/output/compositor_frame.h"
@@ skipping 21 matching lines @@
 }
 
 FrameGenerator::~FrameGenerator() {
   compositor_frame_sink_->DetachFromClient();
 }
 
 void FrameGenerator::SetDeviceScaleFactor(float device_scale_factor) {
   if (device_scale_factor_ == device_scale_factor)
     return;
   device_scale_factor_ = device_scale_factor;
-  if (window_manager_surface_info_.is_valid())
+  if (window_manager_surface_info_.is_valid()) {

[Fady Samuel, 2017/03/15 17:47:00]

nit: this change is unnecessary.

[Alex Z., 2017/03/15 20:37:12]

Done.

     SetNeedsBeginFrame(true);
+  }
 }
 
 void FrameGenerator::OnSurfaceCreated(const cc::SurfaceInfo& surface_info) {
   DCHECK(surface_info.is_valid());
 
   // Only handle embedded surfaces changing here. The display root surface
   // changing is handled immediately after the CompositorFrame is submitted.
   if (surface_info != window_manager_surface_info_) {
     window_manager_surface_info_ = surface_info;
     SetNeedsBeginFrame(true);
@@ skipping 138 matching lines @@
     return;
   }
 
   begin_frame_source_->RemoveObserver(this);
   observing_begin_frames_ = false;
 }
 
 }  // namespace ws
 
 }  // namespace ui