commit 391a9f64cb956cc4cd841cc3b795f25e51f36a5c
author John Stiles <johnstiles@google.com> Thu Apr 29 23:17:41 2021 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Apr 30 18:31:19 2021 +0000
tree 54a77915a152ca207657c3e2f12b531540f6e1e9
parent 768d97133da9dcd00bf2e461565502eaf08f2ba0

Fetch random numbers for TestCreate in a predictable order.

We want our randomized tests to behave predictably when given a fixed
seed, but C++ is allowed to arbitrarily reorder our calls to
random->nextXxxx() when we call it more than once on the same line.
Each random-fetching call is now performed separately.

Change-Id: I8d9c0fcd7e5c931d3a963a2e401ea8b66535fc4f
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/402716
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: John Stiles <johnstiles@google.com>

src/gpu/gradients/generated/GrRadialGradientLayout.cpp

diff --git a/src/gpu/gradients/generated/GrRadialGradientLayout.cpp b/src/gpu/gradients/generated/GrRadialGradientLayout.cpp
index be83898..1db5b06 100644
--- a/src/gpu/gradients/generated/GrRadialGradientLayout.cpp
+++ b/src/gpu/gradients/generated/GrRadialGradientLayout.cpp
@@ -63,8 +63,9 @@
     GrTest::TestAsFPArgs asFPArgs(d);
     do {
         GrGradientShader::RandomParams params(d->fRandom);
-        SkPoint center = {d->fRandom->nextRangeScalar(0.0f, scale),
-                          d->fRandom->nextRangeScalar(0.0f, scale)};
+        SkPoint center;
+        center.fX = d->fRandom->nextRangeScalar(0.0f, scale);
+        center.fY = d->fRandom->nextRangeScalar(0.0f, scale);
         SkScalar radius = d->fRandom->nextRangeScalar(0.0f, scale);
         sk_sp<SkShader> shader = params.fUseColors4f
                                          ? SkGradientShader::MakeRadial(center,