From c2cc73e3e696a43d5febd35f7c42e8ce9c5d276d Mon Sep 17 00:00:00 2001 From: gaaclarke <30870216+gaaclarke@users.noreply.github.com> Date: Wed, 25 Oct 2023 11:26:11 -0700 Subject: [PATCH] Missed linter argument comment violations (flutter/engine#47310) ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. - [ ] I listed at least one issue that this PR fixes in the description above. - [x] I added new tests to check the change I am making or feature I am adding, or the PR is [test-exempt]. See [testing the engine] for instructions on writing and running engine tests. - [x] I updated/added relevant documentation (doc comments with `///`). - [x] I signed the [CLA]. - [x] All existing and new tests are passing. If you need help, consider asking for advice on the #hackers-new channel on [Discord]. [Contributor Guide]: https://github.com/flutter/flutter/wiki/Tree-hygiene#overview [Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene [test-exempt]: https://github.com/flutter/flutter/wiki/Tree-hygiene#tests [Flutter Style Guide]: https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style [testing the engine]: https://github.com/flutter/flutter/wiki/Testing-the-engine [CLA]: https://cla.developers.google.com/ [flutter/tests]: https://github.com/flutter/tests [breaking change policy]: https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes [Discord]: https://github.com/flutter/flutter/wiki/Chat --- .../src/flutter/impeller/display_list/dl_unittests.cc | 10 +++++----- .../external_view_embedder/external_view_embedder.cc | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/engine/src/flutter/impeller/display_list/dl_unittests.cc b/engine/src/flutter/impeller/display_list/dl_unittests.cc index c59ca6b0105..9e0456d50db 100644 --- a/engine/src/flutter/impeller/display_list/dl_unittests.cc +++ b/engine/src/flutter/impeller/display_list/dl_unittests.cc @@ -1390,7 +1390,7 @@ TEST_P(DisplayListTest, DrawVerticesSolidColorTrianglesWithoutIndices) { auto vertices = flutter::DlVertices::Make( flutter::DlVertexMode::kTriangles, 3, positions.data(), - /*texture_coorindates=*/nullptr, colors.data()); + /*texture_coordinates=*/nullptr, colors.data()); flutter::DisplayListBuilder builder; flutter::DlPaint paint; @@ -1409,7 +1409,7 @@ TEST_P(DisplayListTest, DrawVerticesLinearGradientWithoutIndices) { auto vertices = flutter::DlVertices::Make( flutter::DlVertexMode::kTriangles, 3, positions.data(), - /*texture_coorindates=*/nullptr, /*colors=*/nullptr); + /*texture_coordinates=*/nullptr, /*colors=*/nullptr); std::vector colors = {flutter::DlColor::kBlue(), flutter::DlColor::kRed()}; @@ -1519,7 +1519,7 @@ TEST_P(DisplayListTest, DrawVerticesSolidColorTrianglesWithIndices) { auto vertices = flutter::DlVertices::Make( flutter::DlVertexMode::kTriangles, 6, positions.data(), - /*texture_coorindates=*/nullptr, /*colors=*/nullptr, 6, indices.data()); + /*texture_coordinates=*/nullptr, /*colors=*/nullptr, 6, indices.data()); flutter::DisplayListBuilder builder; flutter::DlPaint paint; @@ -1540,7 +1540,7 @@ TEST_P(DisplayListTest, DrawVerticesPremultipliesColors) { auto vertices = flutter::DlVertices::Make( flutter::DlVertexMode::kTriangles, 6, positions.data(), - /*texture_coorindates=*/nullptr, colors.data(), 6, indices.data()); + /*texture_coordinates=*/nullptr, colors.data(), 6, indices.data()); flutter::DisplayListBuilder builder; flutter::DlPaint paint; @@ -1668,7 +1668,7 @@ TEST_P(DisplayListTest, DrawVerticesBlendModes) { auto vertices = flutter::DlVertices::Make( flutter::DlVertexMode::kTriangles, 3, positions.data(), - /*texture_coorindates=*/nullptr, colors.data()); + /*texture_coordinates=*/nullptr, colors.data()); flutter::DisplayListBuilder builder; flutter::DlPaint paint; diff --git a/engine/src/flutter/shell/platform/android/external_view_embedder/external_view_embedder.cc b/engine/src/flutter/shell/platform/android/external_view_embedder/external_view_embedder.cc index d29a80eb16a..a0e516e4c10 100644 --- a/engine/src/flutter/shell/platform/android/external_view_embedder/external_view_embedder.cc +++ b/engine/src/flutter/shell/platform/android/external_view_embedder/external_view_embedder.cc @@ -79,7 +79,7 @@ void AndroidExternalViewEmbedder::SubmitFrame( // Restore the clip context after exiting this method since it's changed // below. - DlAutoCanvasRestore save(background_canvas, /*doSave=*/true); + DlAutoCanvasRestore save(background_canvas, /*do_save=*/true); for (size_t i = 0; i < current_frame_view_count; i++) { int64_t view_id = composition_order_[i];