Make SVG painting independent of GraphicsContext's alpha state

Source/platform/graphics/GraphicsContext.cpp

Index: Source/platform/graphics/GraphicsContext.cpp
diff --git a/Source/platform/graphics/GraphicsContext.cpp b/Source/platform/graphics/GraphicsContext.cpp
index bdd3e8995b9ea69ac535374486a09b4f86b29ba9..c6000772a4a81bd81c409923a8e341eee302685b 100644
--- a/Source/platform/graphics/GraphicsContext.cpp
+++ b/Source/platform/graphics/GraphicsContext.cpp
@@ -239,7 +239,7 @@ void GraphicsContext::setInDrawingRecorder(bool val)
 }
 #endif
 
-void GraphicsContext::setStrokePattern(PassRefPtr<Pattern> pattern)
+void GraphicsContext::setStrokePattern(PassRefPtr<Pattern> pattern, float additionalAlpha)
 {
     if (contextDisabled())
         return;
@@ -250,10 +250,10 @@ void GraphicsContext::setStrokePattern(PassRefPtr<Pattern> pattern)
         return;
     }
 
-    mutableState()->setStrokePattern(pattern);
+    mutableState()->setStrokePattern(pattern, additionalAlpha);
 }
 
-void GraphicsContext::setStrokeGradient(PassRefPtr<Gradient> gradient)
+void GraphicsContext::setStrokeGradient(PassRefPtr<Gradient> gradient, float additionalAlpha)
 {
     if (contextDisabled())
         return;
@@ -263,10 +263,10 @@ void GraphicsContext::setStrokeGradient(PassRefPtr<Gradient> gradient)
         setStrokeColor(Color::black);
         return;
     }
-    mutableState()->setStrokeGradient(gradient);
+    mutableState()->setStrokeGradient(gradient, additionalAlpha);
 }
 
-void GraphicsContext::setFillPattern(PassRefPtr<Pattern> pattern)
+void GraphicsContext::setFillPattern(PassRefPtr<Pattern> pattern, float additionalAlpha)
 {
     if (contextDisabled())
         return;
@@ -277,10 +277,10 @@ void GraphicsContext::setFillPattern(PassRefPtr<Pattern> pattern)
         return;
     }
 
-    mutableState()->setFillPattern(pattern);
+    mutableState()->setFillPattern(pattern, additionalAlpha);
 }
 
-void GraphicsContext::setFillGradient(PassRefPtr<Gradient> gradient)
+void GraphicsContext::setFillGradient(PassRefPtr<Gradient> gradient, float additionalAlpha)

[chrishtr, 2015/02/03 00:34:16]

Why "additional" alpha?

[pdr., 2015/02/04 04:04:31]

I thought it might disambiguate that the alpha can be combined with the alpha
already present in the gradient. Lets go with just alpha though.

 {
     if (contextDisabled())
         return;
@@ -291,7 +291,7 @@ void GraphicsContext::setFillGradient(PassRefPtr<Gradient> gradient)
         return;
     }
 
-    mutableState()->setFillGradient(gradient);
+    mutableState()->setFillGradient(gradient, additionalAlpha);
 }
 
 void GraphicsContext::setShadow(const FloatSize& offset, float blur, const Color& color,