Allow GrReducedClip to take non-integer query bounds

src/utils/SkLua.cpp

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkLua.h"
 
 #if SK_SUPPORT_GPU
@@ skipping 623 matching lines @@
     SkLua(L).pushClipStack(*get_ref<SkCanvas>(L, 1)->getClipStack());
     return 1;
 }
 
 int SkLua::lcanvas_getReducedClipStack(lua_State* L) {
 #if SK_SUPPORT_GPU
     const SkCanvas* canvas = get_ref<SkCanvas>(L, 1);
     SkIRect queryBounds = canvas->getTopLayerBounds();
 
     GrReducedClip::ElementList elements;
-    GrReducedClip::InitialState initialState;
     int32_t genID;
     SkIRect resultBounds;
+    bool requiresAA;
 
     const SkClipStack& stack = *canvas->getClipStack();
 
-    GrReducedClip::ReduceClipStack(stack,
-                                   queryBounds,
-                                   &elements,
-                                   &genID,
-                                   &initialState,
-                                   &resultBounds,
-                                   nullptr);
+    GrReducedClip::InitialState initialState SK_UNUSED =
+        GrReducedClip::ReduceClipStack(stack,
+                                       SkRect::Make(queryBounds),
+                                       &elements,
+                                       &genID,
+                                       &resultBounds,
+                                       &requiresAA);
 
     GrReducedClip::ElementList::Iter iter(elements);
     int i = 0;
     lua_newtable(L);
     while(iter.get()) {
         SkLua(L).pushClipStackElement(*iter.get());
         iter.next();
         lua_rawseti(L, -2, ++i);
     }
     // Currently this only returns the element list to lua, not the initial state or result bounds.
-    // It could return these as additional items on the lua stack.
+    // It absolutely must return these as additional items on the lua stack in order to be correct.

[bsalomon, 2016/07/19 13:33:12]

This was used to gather information about multi-element clips. It isn't used to
actually implement anything by lua other than dumping the element list. I don't
know that anything is necessarily incorrect here.

[csmartdalton, 2016/07/19 15:59:01]

Done.

     return 1;
 #else
     return 0;
 #endif
 }
 
 static int lcanvas_save(lua_State* L) {
     lua_pushinteger(L, get_ref<SkCanvas>(L, 1)->save());
     return 1;
 }
@@ skipping 1458 matching lines @@
     REG_CLASS(L, SkTextBlob);
     REG_CLASS(L, SkTypeface);
     REG_CLASS(L, SkXfermode);
 }
 
 extern "C" int luaopen_skia(lua_State* L);
 extern "C" int luaopen_skia(lua_State* L) {
     SkLua::Load(L);
     return 0;
 }