Plumb in flag to enable antialiased clipping in HTML 5 Canvas

Source/platform/graphics/GraphicsContext.cpp

 /*
  * Copyright (C) 2003, 2004, 2005, 2006, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 1543 matching lines @@
 }
 
 void GraphicsContext::clipOutRoundedRect(const RoundedRect& rect)
 {
     if (contextDisabled())
         return;
 
     clipRoundedRect(rect, SkRegion::kDifference_Op);
 }
 
-void GraphicsContext::canvasClip(const Path& pathToClip, WindRule clipRule)
+void GraphicsContext::canvasClip(const Path& pathToClip, WindRule clipRule, bool antialias)
 {
     if (contextDisabled())
         return;
 
     // Use const_cast and temporarily modify the fill type instead of copying the path.
     SkPath& path = const_cast<SkPath&>(pathToClip.skPath());
     SkPath::FillType previousFillType = path.getFillType();
 
     SkPath::FillType temporaryFillType = WebCoreWindRuleToSkFillType(clipRule);
     path.setFillType(temporaryFillType);
-    clipPath(path);
+    clipPath(path, antialias ? AntiAliased : NotAntiAliased);

[Justin Novosad, 2014/10/14 14:46:43]

Since we already have this enumr this, we should use it rather than bool for the
antialias argument to this function. Same for
GraphicsRenderingContext2D::m_antialiasClips, etc. 
The rule in blink is that we use enum rather than bool whenever the argument
would be specified as a literal, which is not the case in any of the new code,
but since we have to convert here, we may as well convert to enum earlier.

[robertphillips, 2014/10/16 13:24:18]

Done.

 
     path.setFillType(previousFillType);
 }
 
 void GraphicsContext::clipRect(const SkRect& rect, AntiAliasingMode aa, SkRegion::Op op)
 {
     ASSERT(m_canvas);
     if (contextDisabled())
         return;
 
@@ skipping 427 matching lines @@
         // FIXME: This is to not break tests (it results in the filter bitmap flag
         // being set to true). We need to decide if we respect InterpolationNone
         // being returned from computeInterpolationQuality.
         resampling = InterpolationLow;
     }
     resampling = limitInterpolationQuality(this, resampling);
     paint->setFilterLevel(static_cast<SkPaint::FilterLevel>(resampling));
 }
 
 } // namespace blink