Diff
Modified: trunk/Source/WebCore/ChangeLog (206719 => 206720)
--- trunk/Source/WebCore/ChangeLog 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebCore/ChangeLog 2016-10-02 19:19:29 UTC (rev 206720)
@@ -1,3 +1,17 @@
+2016-10-02 Ryan Haddad <[email protected]>
+
+ Unreviewed, rolling out r206683.
+
+ This change caused API test WebKit2.FindMatches to fail on
+ Mac.
+
+ Reverted changeset:
+
+ "The dragged image should be the current frame only of the
+ animated image"
+ https://bugs.webkit.org/show_bug.cgi?id=162109
+ http://trac.webkit.org/changeset/206683
+
2016-10-02 Chris Dumez <[email protected]>
Unreviewed, rolling out r206692 and r206718.
Modified: trunk/Source/WebCore/dom/DataTransferMac.mm (206719 => 206720)
--- trunk/Source/WebCore/dom/DataTransferMac.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebCore/dom/DataTransferMac.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -49,7 +49,7 @@
location.setY(imageRect.height() - (elementRect.y() - imageRect.y() + m_dragLocation.y()));
}
} else if (m_dragImage) {
- result = m_dragImage->image()->currentFrameNSImage();
+ result = m_dragImage->image()->getNSImage();
location = m_dragLocation;
location.setY([result size].height - location.y());
Modified: trunk/Source/WebCore/editing/cocoa/HTMLConverter.mm (206719 => 206720)
--- trunk/Source/WebCore/editing/cocoa/HTMLConverter.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebCore/editing/cocoa/HTMLConverter.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -2464,7 +2464,7 @@
if (is<RenderImage>(renderer)) {
auto* image = downcast<RenderImage>(*renderer).cachedImage();
if (image && !image->errorOccurred()) {
- RetainPtr<NSFileWrapper> wrapper = adoptNS([[NSFileWrapper alloc] initRegularFileWithContents:(NSData *)image->imageForRenderer(renderer)->tiffRepresentation()]);
+ RetainPtr<NSFileWrapper> wrapper = adoptNS([[NSFileWrapper alloc] initRegularFileWithContents:(NSData *)image->imageForRenderer(renderer)->getTIFFRepresentation()]);
[wrapper setPreferredFilename:@"image.tiff"];
return wrapper;
}
Modified: trunk/Source/WebCore/platform/graphics/BitmapImage.h (206719 => 206720)
--- trunk/Source/WebCore/platform/graphics/BitmapImage.h 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebCore/platform/graphics/BitmapImage.h 2016-10-02 19:19:29 UTC (rev 206720)
@@ -96,12 +96,11 @@
// Accessors for native image formats.
#if USE(APPKIT)
- NSImage *nsImage() override;
- RetainPtr<NSImage> currentFrameNSImage() override;
+ NSImage* getNSImage() override;
#endif
#if PLATFORM(COCOA)
- CFDataRef tiffRepresentation() override;
+ CFDataRef getTIFFRepresentation() override;
#endif
#if PLATFORM(WIN)
@@ -192,10 +191,10 @@
bool m_animationFinishedWhenCatchingUp { false };
#if USE(APPKIT)
- mutable RetainPtr<NSImage> m_nsImage; // A cached NSImage of all the frames. Only built lazily if someone actually queries for one.
+ mutable RetainPtr<NSImage> m_nsImage; // A cached NSImage of frame 0. Only built lazily if someone actually queries for one.
#endif
#if USE(CG)
- mutable RetainPtr<CFDataRef> m_tiffRep; // Cached TIFF rep for all the frames. Only built lazily if someone queries for one.
+ mutable RetainPtr<CFDataRef> m_tiffRep; // Cached TIFF rep for frame 0. Only built lazily if someone queries for one.
#endif
RefPtr<Image> m_cachedImage;
};
Modified: trunk/Source/WebCore/platform/graphics/Image.h (206719 => 206720)
--- trunk/Source/WebCore/platform/graphics/Image.h 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebCore/platform/graphics/Image.h 2016-10-02 19:19:29 UTC (rev 206720)
@@ -146,12 +146,11 @@
// Accessors for native image formats.
#if USE(APPKIT)
- virtual NSImage *nsImage() { return nullptr; }
- virtual RetainPtr<NSImage> currentFrameNSImage() { return nullptr; }
+ virtual NSImage* getNSImage() { return nullptr; }
#endif
#if PLATFORM(COCOA)
- virtual CFDataRef tiffRepresentation() { return nullptr; }
+ virtual CFDataRef getTIFFRepresentation() { return nullptr; }
#endif
#if PLATFORM(WIN)
Modified: trunk/Source/WebCore/platform/graphics/mac/ImageMac.mm (206719 => 206720)
--- trunk/Source/WebCore/platform/graphics/mac/ImageMac.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebCore/platform/graphics/mac/ImageMac.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -74,14 +74,14 @@
return Image::nullImage();
}
-CFDataRef BitmapImage::tiffRepresentation()
+CFDataRef BitmapImage::getTIFFRepresentation()
{
if (m_tiffRep)
return m_tiffRep.get();
- auto nativeImages = framesNativeImages();
+ auto nativeImages = this->framesNativeImages();
- // If nativeImages.size() is zero, we know for certain this image doesn't have valid data
+ // If framesImages.size() is zero, we know for certain this image doesn't have valid data
// Even though the call to CGImageDestinationCreateWithData will fail and we'll handle it gracefully,
// in certain circumstances that call will spam the console with an error message
if (!nativeImages.size())
@@ -103,27 +103,18 @@
}
#if USE(APPKIT)
-NSImage* BitmapImage::nsImage()
+NSImage* BitmapImage::getNSImage()
{
if (m_nsImage)
return m_nsImage.get();
- CFDataRef data = ""
+ CFDataRef data = ""
if (!data)
- return nullptr;
+ return 0;
m_nsImage = adoptNS([[NSImage alloc] initWithData:(NSData*)data]);
return m_nsImage.get();
}
-
-RetainPtr<NSImage> BitmapImage::currentFrameNSImage()
-{
- auto nativeImage = this->nativeImageForCurrentFrame();
- if (!nativeImage)
- return nullptr;
-
- return adoptNS([[NSImage alloc] initWithCGImage:nativeImage.get() size:NSZeroSize]);
-}
#endif
}
Modified: trunk/Source/WebCore/platform/mac/CursorMac.mm (206719 => 206720)
--- trunk/Source/WebCore/platform/mac/CursorMac.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebCore/platform/mac/CursorMac.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -48,9 +48,9 @@
#endif
{
// FIXME: The cursor won't animate. Not sure if that's a big deal.
- auto nsImage = image->currentFrameNSImage();
+ NSImage* nsImage = image->getNSImage();
if (!nsImage)
- return nullptr;
+ return 0;
BEGIN_BLOCK_OBJC_EXCEPTIONS;
#if ENABLE(MOUSE_CURSOR_SCALE)
@@ -78,7 +78,7 @@
[[[nsImage representations] objectAtIndex:0] setSize:expandedSize];
#endif
- return adoptNS([[NSCursor alloc] initWithImage:nsImage.get() hotSpot:hotSpot]);
+ return adoptNS([[NSCursor alloc] initWithImage:nsImage hotSpot:hotSpot]);
END_BLOCK_OBJC_EXCEPTIONS;
return nullptr;
}
Modified: trunk/Source/WebCore/platform/mac/DragImageMac.mm (206719 => 206720)
--- trunk/Source/WebCore/platform/mac/DragImageMac.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebCore/platform/mac/DragImageMac.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -121,7 +121,7 @@
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wdeprecated-declarations"
- [image->currentFrameNSImage() drawInRect:destRect fromRect:NSMakeRect(0, 0, size.width(), size.height()) operation:NSCompositeSourceOver fraction:1.0];
+ [image->getNSImage() drawInRect:destRect fromRect:NSMakeRect(0, 0, size.width(), size.height()) operation:NSCompositeSourceOver fraction:1.0];
#pragma clang diagnostic pop
[rotatedDragImage.get() unlockFocus];
@@ -129,7 +129,7 @@
}
}
- auto dragImage = image->currentFrameNSImage();
+ RetainPtr<NSImage> dragImage = adoptNS([image->getNSImage() copy]);
[dragImage.get() setSize:(NSSize)size];
return dragImage;
}
Modified: trunk/Source/WebCore/platform/mac/PasteboardMac.mm (206719 => 206720)
--- trunk/Source/WebCore/platform/mac/PasteboardMac.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebCore/platform/mac/PasteboardMac.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -255,7 +255,7 @@
void Pasteboard::write(const PasteboardImage& pasteboardImage)
{
- CFDataRef imageData = pasteboardImage.image->tiffRepresentation();
+ CFDataRef imageData = pasteboardImage.image->getTIFFRepresentation();
if (!imageData)
return;
Modified: trunk/Source/WebKit/mac/ChangeLog (206719 => 206720)
--- trunk/Source/WebKit/mac/ChangeLog 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebKit/mac/ChangeLog 2016-10-02 19:19:29 UTC (rev 206720)
@@ -1,3 +1,17 @@
+2016-10-02 Ryan Haddad <[email protected]>
+
+ Unreviewed, rolling out r206683.
+
+ This change caused API test WebKit2.FindMatches to fail on
+ Mac.
+
+ Reverted changeset:
+
+ "The dragged image should be the current frame only of the
+ animated image"
+ https://bugs.webkit.org/show_bug.cgi?id=162109
+ http://trac.webkit.org/changeset/206683
+
2016-09-30 Said Abou-Hallawa <[email protected]>
The dragged image should be the current frame only of the animated image
Modified: trunk/Source/WebKit/mac/DOM/DOM.mm (206719 => 206720)
--- trunk/Source/WebKit/mac/DOM/DOM.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebKit/mac/DOM/DOM.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -670,7 +670,7 @@
auto* cachedImage = downcast<RenderImage>(*renderer).cachedImage();
if (!cachedImage || cachedImage->errorOccurred())
return nil;
- return cachedImage->imageForRenderer(renderer)->nsImage();
+ return cachedImage->imageForRenderer(renderer)->getNSImage();
}
#endif
@@ -698,7 +698,7 @@
auto* cachedImage = downcast<RenderImage>(*renderer).cachedImage();
if (!cachedImage || cachedImage->errorOccurred())
return nil;
- return (NSData *)cachedImage->imageForRenderer(renderer)->tiffRepresentation();
+ return (NSData *)cachedImage->imageForRenderer(renderer)->getTIFFRepresentation();
}
#endif
Modified: trunk/Source/WebKit/mac/Misc/WebElementDictionary.mm (206719 => 206720)
--- trunk/Source/WebKit/mac/Misc/WebElementDictionary.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebKit/mac/Misc/WebElementDictionary.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -204,7 +204,7 @@
- (NSImage *)_image
{
Image* image = _result->image();
- return image ? image->nsImage() : nil;
+ return image ? image->getNSImage() : nil;
}
- (NSValue *)_imageRect
Modified: trunk/Source/WebKit/mac/Misc/WebIconDatabase.mm (206719 => 206720)
--- trunk/Source/WebKit/mac/Misc/WebIconDatabase.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebKit/mac/Misc/WebIconDatabase.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -163,7 +163,7 @@
ASSERT(size.height);
Image* image = iconDatabase().defaultIcon(IntSize(size));
- return image ? image->currentFrameNSImage().autorelease() : nil;
+ return image ? image->getNSImage() : nil;
}
- (NSImage *)defaultIconForURL:(NSString *)URL withSize:(NSSize)size
@@ -470,7 +470,7 @@
// to WebCore::Image at some point.
if (!image)
return nil;
- NSImage* nsImage = image->nsImage();
+ NSImage* nsImage = image->getNSImage();
if (!nsImage)
return nil;
if (!NSEqualSizes([nsImage size], size)) {
Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebContextMenuClient.mm (206719 => 206720)
--- trunk/Source/WebKit/mac/WebCoreSupport/WebContextMenuClient.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebContextMenuClient.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -226,7 +226,7 @@
if (!image)
return nil;
- return image->currentFrameNSImage();
+ return [[image->getNSImage() retain] autorelease];
}
#endif
@@ -242,7 +242,7 @@
if (Image* image = page->contextMenuController().context().controlledImage()) {
ASSERT(page->contextMenuController().context().hitTestResult().innerNode());
- RetainPtr<NSItemProvider> itemProvider = adoptNS([[NSItemProvider alloc] initWithItem:image->currentFrameNSImage().autorelease() typeIdentifier:@"public.image"]);
+ RetainPtr<NSItemProvider> itemProvider = adoptNS([[NSItemProvider alloc] initWithItem:image->getNSImage() typeIdentifier:@"public.image"]);
bool isContentEditable = page->contextMenuController().context().hitTestResult().innerNode()->isContentEditable();
m_sharingServicePickerController = adoptNS([[WebSharingServicePickerController alloc] initWithItems:@[ itemProvider.get() ] includeEditorServices:isContentEditable client:this style:NSSharingServicePickerStyleRollover]);
Modified: trunk/Source/WebKit/mac/WebView/WebHTMLView.mm (206719 => 206720)
--- trunk/Source/WebKit/mac/WebView/WebHTMLView.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebKit/mac/WebView/WebHTMLView.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -2210,7 +2210,7 @@
[archive release];
} else if ([type isEqual:NSTIFFPboardType] && [self promisedDragTIFFDataSource]) {
if (Image* image = [self promisedDragTIFFDataSource]->image())
- [pasteboard setData:(NSData *)image->tiffRepresentation() forType:NSTIFFPboardType];
+ [pasteboard setData:(NSData *)image->getTIFFRepresentation() forType:NSTIFFPboardType];
[self setPromisedDragTIFFDataSource:0];
}
}
Modified: trunk/Source/WebKit2/ChangeLog (206719 => 206720)
--- trunk/Source/WebKit2/ChangeLog 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebKit2/ChangeLog 2016-10-02 19:19:29 UTC (rev 206720)
@@ -1,3 +1,17 @@
+2016-10-02 Ryan Haddad <[email protected]>
+
+ Unreviewed, rolling out r206683.
+
+ This change caused API test WebKit2.FindMatches to fail on
+ Mac.
+
+ Reverted changeset:
+
+ "The dragged image should be the current frame only of the
+ animated image"
+ https://bugs.webkit.org/show_bug.cgi?id=162109
+ http://trac.webkit.org/changeset/206683
+
2016-09-30 Myles C. Maxfield <[email protected]>
Follow-up patch after r206701
Modified: trunk/Source/WebKit2/UIProcess/Cocoa/WebViewImpl.mm (206719 => 206720)
--- trunk/Source/WebKit2/UIProcess/Cocoa/WebViewImpl.mm 2016-10-02 18:19:05 UTC (rev 206719)
+++ trunk/Source/WebKit2/UIProcess/Cocoa/WebViewImpl.mm 2016-10-02 19:19:29 UTC (rev 206720)
@@ -3038,7 +3038,7 @@
// FIXME: need to support NSRTFDPboardType
if ([type isEqual:NSTIFFPboardType] && m_promisedImage) {
- [pasteboard setData:(NSData *)m_promisedImage->tiffRepresentation() forType:NSTIFFPboardType];
+ [pasteboard setData:(NSData *)m_promisedImage->getTIFFRepresentation() forType:NSTIFFPboardType];
m_promisedImage = nullptr;
}
}
