Fix mislabeled argument comments.

Once the existing cases are fixed, we can enable bugprone-argument-
comment to allow clang-tidy to verify these for us automatically going
forward. (http://review.skia.org/306848)

Change-Id: I5c21b28bcbdebb6b74c2924cc5f19c48446ecb42
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/306847
Reviewed-by: Florin Malita <[email protected]>
Commit-Queue: John Stiles <[email protected]>

Author: johnstiles-google

gm/vertices.cpp

@@ -524,8 +524,8 @@ static sk_sp<SkVertices> make_cone(Attr::Usage u, const char* markerName) {
     // +1 for the center, +1 to repeat the first perimeter point (so we draw a complete circle)
     constexpr int kNumVerts = kPerimeterVerts + 2;
 
-    SkVertices::Builder builder(SkVertices::kTriangleFan_VertexMode, kNumVerts, /*index_count=*/ 0,
-                                &attr, /*attr_count=*/ 1);
+    SkVertices::Builder builder(SkVertices::kTriangleFan_VertexMode, kNumVerts, /*indexCount=*/0,
+                                &attr, /*attrCount=*/1);
 
     SkPoint* pos = builder.positions();
     SkPoint3* vec = static_cast<SkPoint3*>(builder.customData());

src/core/SkColorFilter_Matrix.cpp

@@ -132,15 +132,15 @@ GrFPResult SkColorFilter_Matrix::asFragmentProcessor(std::unique_ptr<GrFragmentP
     switch (fDomain) {
         case Domain::kRGBA:
             fp = GrColorMatrixFragmentProcessor::Make(std::move(fp), fMatrix,
-                                                      /* premulInput = */    true,
+                                                      /* unpremulInput = */  true,
                                                       /* clampRGBOutput = */ true,
                                                       /* premulOutput = */   true);
             break;
 
         case Domain::kHSLA:
             fp = GrRGBToHSLFilterEffect::Make(std::move(fp));
             fp = GrColorMatrixFragmentProcessor::Make(std::move(fp), fMatrix,
-                                                      /* premulInput = */    false,
+                                                      /* unpremulInput = */  false,
                                                       /* clampRGBOutput = */ false,
                                                       /* premulOutput = */   false);
             fp = GrHSLToRGBFilterEffect::Make(std::move(fp));

src/core/SkDevice.cpp

@@ -33,7 +33,7 @@
 #include "src/utils/SkPatchUtils.h"
 
 SkBaseDevice::SkBaseDevice(const SkImageInfo& info, const SkSurfaceProps& surfaceProps)
-        : SkMatrixProvider(/* fLocalToDevice = */ SkMatrix::I())
+        : SkMatrixProvider(/* localToDevice = */ SkMatrix::I())
         , fInfo(info)
         , fSurfaceProps(surfaceProps) {
     fDeviceToGlobal.reset();

tools/fm/fm.cpp

@@ -206,8 +206,8 @@ static void init(Source* source, sk_sp<skottie::Animation> animation) {
         for (int x : order) {
             SkRect dst = {x*dim, y*dim, (x+1)*dim, (y+1)*dim};
 
-            SkAutoCanvasRestore _(canvas, true/*save now*/);
-            canvas->clipRect(dst, /*aa=*/true);
+            SkAutoCanvasRestore _(canvas, /*doSave=*/true);
+            canvas->clipRect(dst, /*doAntiAlias=*/true);
             canvas->concat(SkMatrix::MakeRectToRect(SkRect::MakeSize(animation->size()),
                                                     dst,
                                                     SkMatrix::kCenter_ScaleToFit));