https://bugs.webkit.org/show_bug.cgi?id=136416
Reviewed by Dan Bernstein.
* WebCore.exp.in:
* platform/graphics/GraphicsContext.h:
* platform/graphics/cg/GraphicsContextCG.cpp:
(WebCore::setStrokeAndFillColor): Deleted.
Remove.
* WebView/WebFrameView.mm:
(-[WebFrameView drawRect:]):
We only end up filling, so only set the fill color.
* WebView/WebPDFViewIOS.mm:
(-[WebPDFView drawPage:]):
(-[WebPDFView drawRect:]):
* WebView/WebPlainWhiteView.mm:
We only end up filling, so only set the fill color.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@173150
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2014-08-31 Tim Horton <timothy_horton@apple.com>
+
+ Remove unnecessary (and unnecessarily iOS-specific) setStrokeAndFillColor
+ https://bugs.webkit.org/show_bug.cgi?id=136416
+
+ Reviewed by Dan Bernstein.
+
+ * WebCore.exp.in:
+ * platform/graphics/GraphicsContext.h:
+ * platform/graphics/cg/GraphicsContextCG.cpp:
+ (WebCore::setStrokeAndFillColor): Deleted.
+ Remove.
+
2014-08-31 Tim Horton <timothy_horton@apple.com>
Remove duplicate implementation of drawEllipse and some related PLATFORM(IOS) ifdefs
__ZN7WebCore21SQLiteDatabaseTracker9setClientEPNS_27SQLiteDatabaseTrackerClientE
__ZN7WebCore21applicationIsFacebookEv
__ZN7WebCore21nextParagraphPositionERKNS_15VisiblePositionEi
-__ZN7WebCore21setStrokeAndFillColorEP9CGContextP7CGColor
__ZN7WebCore21wordRangeFromPositionERKNS_15VisiblePositionE
__ZN7WebCore22startOfEditableContentERKNS_15VisiblePositionE
__ZN7WebCore23applicationIsMobileMailEv
bool drawLuminanceMask : 1;
};
-#if PLATFORM(IOS)
- WEBCORE_EXPORT void setStrokeAndFillColor(PlatformGraphicsContext*, CGColorRef);
-#endif
-
struct ImagePaintingOptions {
ImagePaintingOptions(CompositeOperator compositeOperator = CompositeSourceOver, BlendMode blendMode = BlendModeNormal, ImageOrientationDescription orientationDescription = ImageOrientationDescription(), bool useLowQualityScale = false)
: m_compositeOperator(compositeOperator)
#endif // PLATFORM(IOS)
}
-#if PLATFORM(IOS)
-void setStrokeAndFillColor(CGContextRef context, CGColorRef color)
-{
- CGContextSetStrokeColorWithColor(context, color);
- CGContextSetFillColorWithColor(context, color);
-}
-#endif // PLATFORM(IOS)
-
#if PLATFORM(WIN) || PLATFORM(IOS)
CGColorSpaceRef linearRGBColorSpaceRef()
{
+2014-08-31 Tim Horton <timothy_horton@apple.com>
+
+ Remove unnecessary (and unnecessarily iOS-specific) setStrokeAndFillColor
+ https://bugs.webkit.org/show_bug.cgi?id=136416
+
+ Reviewed by Dan Bernstein.
+
+ * WebView/WebPDFViewIOS.mm:
+ (-[WebPDFView drawPage:]):
+ (-[WebPDFView drawRect:]):
+ * WebView/WebPlainWhiteView.mm:
+ We only end up filling, so only set the fill color.
+
2014-08-22 Simon Fraser <simon.fraser@apple.com>
Implement paint flashing in the WK1 InspectorOverlay page
CGContextSaveGState(context);
CGFloat height = WebKitLinkedOnOrAfter(WEBKIT_FIRST_VERSION_WITH_FLIPPED_SHADOWS) ? 2.0f : -2.0f;
CGContextSetShadowWithColor(context, CGSizeMake(0.0f, height), 3.0f, [[self class] shadowColor]);
- setStrokeAndFillColor(context, cachedCGColor(Color::white, ColorSpaceDeviceRGB));
+ CGContextSetFillColorWithColor(context, cachedCGColor(Color::white, ColorSpaceDeviceRGB));
CGContextFillRect(context, pageRect);
CGContextRestoreGState(context);
// Draw Background.
CGContextSaveGState(context);
- setStrokeAndFillColor(context, [[self class] backgroundColor]);
+ CGContextSetFillColorWithColor(context, [[self class] backgroundColor]);
CGContextFillRect(context, aRect);
CGContextRestoreGState(context);
- (void)drawRect:(NSRect)rect
{
CGContextRef context = WKGetCurrentGraphicsContext();
- setStrokeAndFillColor(context, cachedCGColor(Color::white, ColorSpaceDeviceRGB));
+ CGContextSetFillColorWithColor(context, cachedCGColor(Color::white, ColorSpaceDeviceRGB));
WKRectFill(context, [self bounds]);
}
+2014-08-31 Tim Horton <timothy_horton@apple.com>
+
+ Remove unnecessary (and unnecessarily iOS-specific) setStrokeAndFillColor
+ https://bugs.webkit.org/show_bug.cgi?id=136416
+
+ Reviewed by Dan Bernstein.
+
+ * WebView/WebFrameView.mm:
+ (-[WebFrameView drawRect:]):
+ We only end up filling, so only set the fill color.
+
2014-08-26 Maciej Stachowiak <mjs@apple.com>
Use RetainPtr::autorelease in some places where it seems appropriate
NSRectFill(rect);
#else
CGContextRef cgContext = WKGetCurrentGraphicsContext();
- setStrokeAndFillColor(cgContext, cachedCGColor(Color::white, ColorSpaceDeviceRGB));
+ CGContextSetFillColorWithColor(cgContext, cachedCGColor(Color::white, ColorSpaceDeviceRGB));
WKRectFill(cgContext, rect);
#endif
}
NSRectFill(rect);
#else
CGContextRef cgContext = WKGetCurrentGraphicsContext();
- setStrokeAndFillColor(cgContext, cachedCGColor(Color::cyan, ColorSpaceDeviceRGB));
+ CGContextSetFillColorWithColor(cgContext, cachedCGColor(Color::cyan, ColorSpaceDeviceRGB));
WKRectFill(cgContext, rect);
#endif
}