Fix assert in GetOverlayRect when transforming video with Ozone overlay

cc/output/gl_renderer.cc

 // Copyright 2010 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 "cc/output/gl_renderer.h"
 
 #include <algorithm>
 #include <limits>
 #include <set>
 #include <string>
@@ skipping 3416 matching lines @@
   return output_surface_->context_provider()->IsContextLost();
 }
 
 void GLRenderer::ScheduleOverlays(DrawingFrame* frame) {
   if (!frame->overlay_list.size())
     return;
 
   ResourceProvider::ResourceIdArray resources;
   OverlayCandidateList& overlays = frame->overlay_list;
   OverlayCandidateList::iterator it;
   for (it = overlays.begin(); it != overlays.end(); ++it) {

[danakj, 2015/05/04 19:56:41]

while you're here.. you could change this to for (const OC& overlay :
overlays).. if so inclined.

[halliwell, 2015/05/04 21:38:16]

Done.

     const OverlayCandidate& overlay = *it;
     // Skip primary plane.
     if (overlay.plane_z_order == 0)
       continue;
 
     pending_overlay_resources_.push_back(
         make_scoped_ptr(new ResourceProvider::ScopedReadLockGL(
             resource_provider_, overlay.resource_id)));
 
+    gfx::Rect display_rect(ToNearestRect(overlay.display_rect));

[danakj, 2015/05/04 19:56:41]

i don't think the temp var is buying much here it's not adding any information,
just call ToNearestRect below instead?

[halliwell, 2015/05/04 21:38:16]

yep, done.

     context_support_->ScheduleOverlayPlane(
         overlay.plane_z_order,
         overlay.transform,
         pending_overlay_resources_.back()->texture_id(),
-        overlay.display_rect,
+        display_rect,
         overlay.uv_rect);
   }
 }
 
 }  // namespace cc