commit 15d9ef24aab450d03860824319ae87a3577b5384
author Herb Derby <herb@google.com> Thu Oct 18 13:41:32 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Oct 18 18:46:54 2018 +0000
tree f11c8cdd402292907f40bbc555cc39d5de387b79
parent 8f11e43f8bd05abce210f8f08101809b9edf5c63

Simplify atlas calculation for dimension only

It seems that the maximum memory size was never really respected.
Calculate everything off the max dimensions capablility of
the GPU.

Change-Id: I2f2ebe2a16220844a57f3c4d403b17af996ea1f8
Reviewed-on: https://skia-review.googlesource.com/c/163445
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Herb Derby <herb@google.com>

tests/DrawOpAtlasTest.cpp

diff --git a/tests/DrawOpAtlasTest.cpp b/tests/DrawOpAtlasTest.cpp
index 59b61f8..4e53c65 100644
--- a/tests/DrawOpAtlasTest.cpp
+++ b/tests/DrawOpAtlasTest.cpp
@@ -223,3 +223,12 @@
     flushState.setOpArgs(nullptr);
     opMemoryPool->release(std::move(op));
 }
+
+DEF_GPUTEST(GrDrawOpAtlasConfig_Basic, reporter, options) {
+    REPORTER_ASSERT(reporter, GrDrawOpAtlasConfig::PlotsPerLongDimensionForARGB(   1) == 1);
+    REPORTER_ASSERT(reporter, GrDrawOpAtlasConfig::PlotsPerLongDimensionForARGB( 256) == 1);
+    REPORTER_ASSERT(reporter, GrDrawOpAtlasConfig::PlotsPerLongDimensionForARGB( 512) == 2);
+    REPORTER_ASSERT(reporter, GrDrawOpAtlasConfig::PlotsPerLongDimensionForARGB(1024) == 4);
+    REPORTER_ASSERT(reporter, GrDrawOpAtlasConfig::PlotsPerLongDimensionForARGB(2048) == 8);
+    REPORTER_ASSERT(reporter, GrDrawOpAtlasConfig::PlotsPerLongDimensionForARGB(4096) == 8);
+}
\ No newline at end of file