commit 4c56b03cfdc1cffaf1b6d70b3864ae142d97b235
author Chris Dalton <csmartdalton@google.com> Mon Mar 04 12:28:42 2019 -0700
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Mar 05 11:35:21 2019 +0000
tree 07a13b32bb243c16e6af604c01b321d07b1fc147
parent e1d3a28ed285824d7ea0de2a0dd9c69c9033be4b

Drop support for GL_EXT_raster_multisample

We had only used this extension in conjunction with
GL_NV_framebuffer_mixed_samples to draw directly to the framebuffer.
However, it's faster and higher quality to just use our own analytic
AA if we can draw directly to the framebuffer.

It might have been interesting to fake mixed samples using
GL_EXT_raster_multisample and PLS together, but those two extensions
don't appear to ever exist together.

Bug: skia:
Change-Id: Iea489cb0e03fc55aa64d3ea7fccc61b539cc461e
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/197082
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>

src/gpu/gl/GrGLGpu.h

diff --git a/src/gpu/gl/GrGLGpu.h b/src/gpu/gl/GrGLGpu.h
index e5d0db4..5c23924 100644
--- a/src/gpu/gl/GrGLGpu.h
+++ b/src/gpu/gl/GrGLGpu.h
@@ -377,7 +377,7 @@
     void disableStencil();
 
     // rt is used only if useHWAA is true.
-    void flushHWAAState(GrRenderTarget* rt, bool useHWAA, bool stencilEnabled);
+    void flushHWAAState(GrRenderTarget* rt, bool useHWAA);
 
     void flushFramebufferSRGB(bool enable);
 
@@ -624,11 +624,6 @@
 
     GrGLuint fBoundDrawFramebuffer = 0;
 
-    // EXT_raster_multisample.
-    TriState                                fHWRasterMultisampleEnabled;
-    int                                     fHWNumRasterSamples;
-    ///@}
-
     /** IDs for copy surface program. (3 sampler types) */
     struct {
         GrGLuint    fProgram = 0;