backend_pdf: optional rgbFace arg in fillp replaces code in draw_markers

Author: efiring

lib/matplotlib/backends/backend_pdf.py

@@ -1556,10 +1556,7 @@ def draw_markers(self, gc, marker_path, marker_trans, path, trans, rgbFace=None)
             return
 
         self.check_gc(gc, rgbFace)
-        if rgbFace:
-            fillp = gc.fillp()
-        else:
-            fillp = None
+        fillp = gc.fillp(rgbFace)
         strokep = gc.strokep()
 
         output = self.file.output
@@ -2005,13 +2002,20 @@ def strokep(self):
         return (self._linewidth > 0 and self._alpha > 0 and
                 (len(self._rgb) <= 3 or self._rgb[3] != 0.0))
 
-    def fillp(self):
+    def fillp(self, *args):
         """
         Predicate: does the path need to be filled?
+
+        An optional argument can be used to specify an alternative
+        _fillcolor, as needed by RendererPdf.draw_markers.
         """
-        return self._hatch or \
-            (self._fillcolor is not None and
-             (len(self._fillcolor) <= 3 or self._fillcolor[3] != 0.0))
+        if len(args):
+            _fillcolor = args[0]
+        else:
+            _fillcolor = self._fillcolor
+        return (self._hatch or
+                (_fillcolor is not None and
+                 (len(_fillcolor) <= 3 or _fillcolor[3] != 0.0)))
 
     def close_and_paint(self):
         """