Change e local space to clamp to texture space

Author: DD Liu

src/Drawable.js

@@ -37,10 +37,6 @@ const getLocalPosition = (drawable, vec) => {
     // localPosition matches that transformation.
     localPosition[0] = 0.5 - (((v0 * m[0]) + (v1 * m[4]) + m[12]) / d);
     localPosition[1] = (((v0 * m[1]) + (v1 * m[5]) + m[13]) / d) + 0.5;
-    // Fix floating point issues near 0.
-    // TODO: Check if this can be removed after render pull 479 is merged
-    if (Math.abs(localPosition[0] < 1e-8)) localPosition[0] = 0;
-    if (Math.abs(localPosition[1] < 1e-8)) localPosition[1] = 0;
     // Apply texture effect transform if the localPosition is within the drawable's space,
     // and any effects are currently active.
     if (drawable.enabledEffects !== 0 &&
@@ -716,21 +712,25 @@ class Drawable {
      * Sample a color from a drawable's texture.
      * The caller is responsible for ensuring this drawable's inverse matrix & its skin's silhouette are up-to-date.
      * @see updateCPURenderAttributes
-     * @param {twgl.v3} vec The scratch space [x,y] vector
+     * @param {twgl.v3} vec The scratch space [x,y] vector. This will be clamped to the texture space, to match the
+     *     GL code (See https://github.com/LLK/scratch-render/blob/develop/src/BitmapSkin.js#L88)
      * @param {Drawable} drawable The drawable to sample the texture from
      * @param {Uint8ClampedArray} dst The "color4b" representation of the texture at point.
      * @param {number} [effectMask] A bitmask for which effects to use. Optional.
      * @returns {Uint8ClampedArray} The dst object filled with the color4b
      */
     static sampleColor4b (vec, drawable, dst, effectMask) {
         const localPosition = getLocalPosition(drawable, vec);
-        if (localPosition[0] < 0 || localPosition[1] < 0 ||
-            localPosition[0] > 1 || localPosition[1] > 1) {
-            dst[0] = 0;
-            dst[1] = 0;
-            dst[2] = 0;
-            dst[3] = 0;
-            return dst;
+        if (localPosition[0] < 0) {
+            localPosition[0] = 0;
+        } else if (localPosition[0] > 1) {
+            localPosition[0] = 1;
+        }
+
+        if (localPosition[1] < 0) {
+            localPosition[1] = 0;
+        } else if (localPosition[1] > 1) {
+            localPosition[1] = 1;
         }
         const textColor =
         // commenting out to only use nearest for now