Fix dashing bug where hwaa was unintentionally disabled

include/gpu/GrDrawContext.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrDrawContext_DEFINED
 #define GrDrawContext_DEFINED
 
 #include "GrColor.h"
+#include "GrContext.h"
 #include "GrRenderTarget.h"
 #include "SkRefCnt.h"
 #include "SkRegion.h"
 #include "SkSurfaceProps.h"
 #include "../private/GrSingleOwner.h"
 
 class GrAtlasTextContext;
 class GrAuditTrail;
 class GrClip;
 class GrContext;
@@ skipping 235 matching lines @@
      * @param batch    the batch to draw
      */
     void drawBatch(const GrClip&, const GrPaint&, GrDrawBatch*);
 
     bool isStencilBufferMultisampled() const {
         return fRenderTarget->isStencilBufferMultisampled();
     }
     bool isUnifiedMultisampled() const { return fRenderTarget->isUnifiedMultisampled(); }
     bool hasMixedSamples() const { return fRenderTarget->hasMixedSamples(); }
 
+    const GrCaps* caps() const { return fContext->caps(); }

[bsalomon, 2016/06/07 19:32:09]

IIRC, Rob is fairly adamant that we don't expose caps through here, that we
should see drawContext as complete drawing API that doesn't require the client
to make decisions about how to implement drawing. I pretty much agree with that
(and think isUnifiedMultisampled() and hasMixedSamples() should be removed for
similar reasons).

However, the path renderers are sort of violating the layering. They're
effectively implementing some of the GrDC's draw methods, yet they do it by
going back through the public GrDC API (drawBatch). The long term plan here is
to not install pipelines in batches before they're recorded, allowing multipass
PRs to produce a single GrBatch for their passes. Then, PRs become batch
factories and don't go through the GrDC interface (or even see a GrDC), but
rather are just invoked by GrDC to produce a batch that GrDC enqueues.

Shorter term, we could pass GrCaps in the args structs that GrPR see. I don't
want to ask you to do that work now, though, just to unblock the work you're
trying to do around instancing. So, alternatively, we could just file a bug
about this (not using the disable msaa cap to avoid forcing aa for dashed lines)
and fix it later. I'm going to be messing around with the interface between GrDC
and GrPR and can tackle this over the next few weeks.

     const GrSurfaceDesc& desc() const { return fRenderTarget->desc(); }
     int width() const { return fRenderTarget->width(); }
     int height() const { return fRenderTarget->height(); }
     GrPixelConfig config() const { return fRenderTarget->config(); }
     int numColorSamples() const { return fRenderTarget->numColorSamples(); }
     bool isGammaCorrect() const { return fSurfaceProps.isGammaCorrect(); }
     const SkSurfaceProps& surfaceProps() const { return fSurfaceProps; }
 
     bool wasAbandoned() const;
 
@@ skipping 73 matching lines @@
     GrContext*                        fContext;
 
     SkSurfaceProps                    fSurfaceProps;
     GrAuditTrail*                     fAuditTrail;
 
     // In debug builds we guard against improper thread handling
     SkDEBUGCODE(mutable GrSingleOwner* fSingleOwner;)
 };
 
 #endif