commit bd0881cb58ebff7ab605be6d11dd7b0ce29baa85
author Jim Van Verth <jvanverth@google.com> Mon Oct 05 16:20:30 2020 +0000
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Oct 05 16:34:15 2020 +0000
tree 0eb3967600188e104bdaee0a80399580e46810e5
parent c1916c34febe0c6f9805b5d9fc18349ec8b74008

Revert "Remove use of legacy display globals."

This reverts commit c1916c34febe0c6f9805b5d9fc18349ec8b74008.

Reason for revert: Bots unhappy

Original change's description:
> Remove use of legacy display globals.
>
> In the ongoing effort to remove the display globals from Skia, allow
> their use only if SK_LEGACY_SURFACE_PROPS is defined. Do not define this
> in a normal Skia build and remove all use from Skia code.
>
> Change-Id: I9ff550f5db246b9024aac687a1bc01321f1be4c8
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/319343
> Reviewed-by: Herb Derby <herb@google.com>
> Commit-Queue: Ben Wagner <bungeman@google.com>

TBR=djsollen@google.com,bungeman@google.com,herb@google.com,reed@google.com

Change-Id: I365d2b1d19241a90130bc1b59663651817966f63
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/322400
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>

tools/viewer/Viewer.cpp

diff --git a/tools/viewer/Viewer.cpp b/tools/viewer/Viewer.cpp
index 61c68a0..daafba4 100644
--- a/tools/viewer/Viewer.cpp
+++ b/tools/viewer/Viewer.cpp
@@ -369,7 +369,6 @@
     displayParams.fGrContextOptions.fShaderErrorHandler = &gShaderErrorHandler;
     displayParams.fGrContextOptions.fSuppressPrints = true;
     fWindow->setRequestedDisplayParams(displayParams);
-    fDisplay = fWindow->getRequestedDisplayParams();
     fRefresh = FLAGS_redraw;
 
     // Configure timers
@@ -494,9 +493,8 @@
     fCommands.addCommand('G', "Modes", "Geometry", [this]() {
         DisplayParams params = fWindow->getRequestedDisplayParams();
         uint32_t flags = params.fSurfaceProps.flags();
-        SkPixelGeometry defaultPixelGeometry = fDisplay.fSurfaceProps.pixelGeometry();
-        if (!fDisplayOverrides.fSurfaceProps.fPixelGeometry) {
-            fDisplayOverrides.fSurfaceProps.fPixelGeometry = true;
+        if (!fPixelGeometryOverrides) {
+            fPixelGeometryOverrides = true;
             params.fSurfaceProps = SkSurfaceProps(flags, kUnknown_SkPixelGeometry);
         } else {
             switch (params.fSurfaceProps.pixelGeometry()) {
@@ -513,8 +511,8 @@
                     params.fSurfaceProps = SkSurfaceProps(flags, kBGR_V_SkPixelGeometry);
                     break;
                 case kBGR_V_SkPixelGeometry:
-                    params.fSurfaceProps = SkSurfaceProps(flags, defaultPixelGeometry);
-                    fDisplayOverrides.fSurfaceProps.fPixelGeometry = false;
+                    params.fSurfaceProps = SkSurfaceProps(flags, SkSurfaceProps::kLegacyFontHost_InitType);
+                    fPixelGeometryOverrides = false;
                     break;
             }
         }
@@ -1024,7 +1022,7 @@
     }
 
     const DisplayParams& params = fWindow->getRequestedDisplayParams();
-    if (fDisplayOverrides.fSurfaceProps.fPixelGeometry) {
+    if (fPixelGeometryOverrides) {
         switch (params.fSurfaceProps.pixelGeometry()) {
             case kUnknown_SkPixelGeometry:
                 title.append( " Flat");
@@ -1412,7 +1410,7 @@
     }
 
     auto make_surface = [=](int w, int h) {
-        SkSurfaceProps props(fWindow->getRequestedDisplayParams().fSurfaceProps);
+        SkSurfaceProps props(SkSurfaceProps::kLegacyFontHost_InitType);
         slideCanvas->getProps(&props);
 
         SkImageInfo info = SkImageInfo::Make(w, h, colorType, kPremul_SkAlphaType, colorSpace);
@@ -1830,7 +1828,7 @@
                 }
 
                 int pixelGeometryIdx = 0;
-                if (fDisplayOverrides.fSurfaceProps.fPixelGeometry) {
+                if (fPixelGeometryOverrides) {
                     pixelGeometryIdx = params.fSurfaceProps.pixelGeometry() + 1;
                 }
                 if (ImGui::Combo("Pixel Geometry", &pixelGeometryIdx,
@@ -1838,11 +1836,10 @@
                 {
                     uint32_t flags = params.fSurfaceProps.flags();
                     if (pixelGeometryIdx == 0) {
-                        fDisplayOverrides.fSurfaceProps.fPixelGeometry = false;
-                        SkPixelGeometry pixelGeometry = fDisplay.fSurfaceProps.pixelGeometry();
-                        params.fSurfaceProps = SkSurfaceProps(flags, pixelGeometry);
+                        fPixelGeometryOverrides = false;
+                        params.fSurfaceProps = SkSurfaceProps(flags, SkSurfaceProps::kLegacyFontHost_InitType);
                     } else {
-                        fDisplayOverrides.fSurfaceProps.fPixelGeometry = true;
+                        fPixelGeometryOverrides = true;
                         SkPixelGeometry pixelGeometry = SkTo<SkPixelGeometry>(pixelGeometryIdx - 1);
                         params.fSurfaceProps = SkSurfaceProps(flags, pixelGeometry);
                     }