cc: Make OutputSurfaceFrame::sub_buffer_rect optional

services/ui/surfaces/display_output_surface_ozone.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/surfaces/display_output_surface_ozone.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
@@ skipping 65 matching lines @@
 }
 
 void DisplayOutputSurfaceOzone::SwapBuffers(cc::OutputSurfaceFrame frame) {
   DCHECK(buffer_queue_);
 
   // TODO(rjkroege): What if swap happens again before DidReceiveSwapBuffersAck
   // then it would see the wrong size?
   DCHECK(reshape_size_ == frame.size);
   swap_size_ = reshape_size_;
 
-  buffer_queue_->SwapBuffers(frame.sub_buffer_rect);
+  gfx::Rect damage(swap_size_);
+  if (frame.sub_buffer_rect) {

[danakj, 2017/02/22 00:36:57]

no {}

+    damage = *frame.sub_buffer_rect;
+  }
+  buffer_queue_->SwapBuffers(damage);

[danakj, 2017/02/22 00:36:57]

?: maybe here also?

[halliwell, 2017/02/22 14:19:28]

Done.

   DisplayOutputSurface::SwapBuffers(std::move(frame));
 }
 
 uint32_t DisplayOutputSurfaceOzone::GetFramebufferCopyTextureFormat() {
   return buffer_queue_->internal_format();
 }
 
 bool DisplayOutputSurfaceOzone::IsDisplayedAsOverlayPlane() const {
   // TODO(rjkroege): implement remaining overlay functionality.
   return true;
@@ skipping 15 matching lines @@
   }
 
   buffer_queue_->PageFlipComplete();
   client()->DidReceiveSwapBuffersAck();
 
   if (force_swap)
     client()->SetNeedsRedrawRect(gfx::Rect(swap_size_));
 }
 
 }  // namespace ui