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 3426 matching lines @@
   for (it = overlays.begin(); it != overlays.end(); ++it) {
     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));
     context_support_->ScheduleOverlayPlane(
         overlay.plane_z_order,
         overlay.transform,
         pending_overlay_resources_.back()->texture_id(),
-        overlay.display_rect,
+        display_rect,

[achaulk, 2015/04/29 21:37:03]

You'll still hit the same problem here, no? On your implementation you'll accept
it and it will get here

[halliwell, 2015/04/29 21:42:54]

Yep, unless we modify the transform when accepting, but that's maybe not the
intended use of the API?

What did you have in mind - should we push the float rect right through to
ScheduleOverlayPlane?

[achaulk, 2015/04/29 21:45:00]

Probably change this to accept a float as well I would say

         overlay.uv_rect);
   }
 }
 
 }  // namespace cc