Use move constructor for points draft
authorMiko Mynttinen <mikokm@gmail.com>
Mon, 08 Aug 2016 13:03:33 -0700
changeset 398885 2b4f56d2115e896835667f2e6b5fe78dbbe9bf83
parent 398884 299fba5dfcbfe561ef85bbe923a2917af58fb2e3
child 398886 0bc26cdc42c30be87e6c66d9a4e6414524625089
push id25668
push userbmo:mikokm@gmail.com
push dateTue, 09 Aug 2016 23:22:28 +0000
milestone51.0a1
Use move constructor for points MozReview-Commit-ID: 765UEuCJh36
gfx/2d/Polygon.h
gfx/layers/BSPTree.cpp
--- a/gfx/2d/Polygon.h
+++ b/gfx/2d/Polygon.h
@@ -20,17 +20,17 @@ template<class Units>
 class BasePolygon3D {
 public:
   explicit BasePolygon3D(const std::initializer_list<Point3DTyped<Units>>& aPoints)
     : mPoints(aPoints)
   {
     CalculateNormal();
   }
 
-  explicit BasePolygon3D(const nsTArray<Point3DTyped<Units>>& aPoints)
+  explicit BasePolygon3D(nsTArray<Point3DTyped<Units>>&& aPoints)
     : mPoints(aPoints)
   {
     CalculateNormal();
   }
 
   const Point3DTyped<Units>& GetNormal() const
   {
     return mNormal;
--- a/gfx/layers/BSPTree.cpp
+++ b/gfx/layers/BSPTree.cpp
@@ -172,14 +172,14 @@ BSPTree::SplitPolygon(const gfx::Polygon
       const gfx::Point3D p = a + (ab * t);
 
       // Add the intersection point to both polygons.
       backPoints.AppendElement(p);
       frontPoints.AppendElement(p);
     }
   }
 
-  return std::make_pair(gfx::Polygon3D(backPoints),
-                        gfx::Polygon3D(frontPoints));
+  return std::make_pair(gfx::Polygon3D(std::move(backPoints)),
+                        gfx::Polygon3D(std::move(frontPoints)));
 }
 
 } // namespace layers
 } // namespace mozilla