Bug 23191

Summary: Add the first pieces of platform/graphics/skia
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
Add the first few files from platform/graphics/skia
none
Add two more files for platform/graphics/skia.
none
Add a cleaned-up GraphicsContextSkia implementation to WebKit
none
Add ImageBufferSkia and ImageSkia files. none

Eric Seidel (no email)
Reported 2009-01-08 10:52:04 PST
Add the first pieces of platform/graphics/skia In order to expedite the process of up-streaming, Dimitri and I are pair-programming today. We're scrubbing all the platform/graphics/skia files locally, committing them to the Chrome tree, and then posting them here and landing them in the WebKit tree. I'm not requesting a second review, since I've already reviewed them as Dimitri has been typing, but I'm posting the patches here for completeness.
Attachments
Add the first few files from platform/graphics/skia (24.29 KB, patch)
2009-01-08 10:56 PST, Eric Seidel (no email)
no flags
Add two more files for platform/graphics/skia. (9.93 KB, patch)
2009-01-08 13:46 PST, Eric Seidel (no email)
no flags
Add a cleaned-up GraphicsContextSkia implementation to WebKit (38.04 KB, patch)
2009-01-08 14:05 PST, Eric Seidel (no email)
no flags
Add ImageBufferSkia and ImageSkia files. (28.56 KB, patch)
2009-01-08 16:40 PST, Eric Seidel (no email)
no flags
Eric Seidel (no email)
Comment 1 2009-01-08 10:56:54 PST
Created attachment 26528 [details] Add the first few files from platform/graphics/skia WebCore/ChangeLog | 48 ++++ .../graphics/skia/BitmapImageSingleFrameSkia.h | 62 +++++ WebCore/platform/graphics/skia/FloatPointSkia.cpp | 47 ++++ WebCore/platform/graphics/skia/FloatRectSkia.cpp | 46 ++++ WebCore/platform/graphics/skia/IntPointSkia.cpp | 50 ++++ WebCore/platform/graphics/skia/IntRectSkia.cpp | 54 +++++ WebCore/platform/graphics/skia/SkiaUtils.cpp | 240 ++++++++++++++++++++ WebCore/platform/graphics/skia/SkiaUtils.h | 64 ++++++ 8 files changed, 611 insertions(+), 0 deletions(-)
Eric Seidel (no email)
Comment 2 2009-01-08 10:59:02 PST
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog A WebCore/platform/graphics/skia/BitmapImageSingleFrameSkia.h A WebCore/platform/graphics/skia/FloatPointSkia.cpp A WebCore/platform/graphics/skia/FloatRectSkia.cpp A WebCore/platform/graphics/skia/IntPointSkia.cpp A WebCore/platform/graphics/skia/IntRectSkia.cpp A WebCore/platform/graphics/skia/SkiaUtils.cpp A WebCore/platform/graphics/skia/SkiaUtils.h Committed r39712
Eric Seidel (no email)
Comment 3 2009-01-08 11:46:19 PST
Commit the right set of files the second time! Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/platform/graphics/skia/BitmapImageSingleFrameSkia.h M WebCore/platform/graphics/skia/FloatPointSkia.cpp M WebCore/platform/graphics/skia/FloatRectSkia.cpp M WebCore/platform/graphics/skia/IntPointSkia.cpp M WebCore/platform/graphics/skia/IntRectSkia.cpp M WebCore/platform/graphics/skia/SkiaUtils.cpp M WebCore/platform/graphics/skia/SkiaUtils.h Committed r39713
Eric Seidel (no email)
Comment 4 2009-01-08 13:46:05 PST
Created attachment 26532 [details] Add two more files for platform/graphics/skia. WebCore/ChangeLog | 20 +++ WebCore/platform/graphics/skia/GradientSkia.cpp | 164 ++++++++++++++++++++ .../graphics/skia/GraphicsContextPlatformPrivate.h | 55 +++++++ 3 files changed, 239 insertions(+), 0 deletions(-)
Eric Seidel (no email)
Comment 5 2009-01-08 13:51:51 PST
Second round: Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog A WebCore/platform/graphics/skia/GradientSkia.cpp A WebCore/platform/graphics/skia/GraphicsContextPlatformPrivate.h Committed r39718 (This time I even committed the right files!)
Eric Seidel (no email)
Comment 6 2009-01-08 14:05:43 PST
Created attachment 26533 [details] Add a cleaned-up GraphicsContextSkia implementation to WebKit WebCore/ChangeLog | 61 ++ .../platform/graphics/skia/GraphicsContextSkia.cpp | 1122 ++++++++++++++++++++ 2 files changed, 1183 insertions(+), 0 deletions(-)
Eric Seidel (no email)
Comment 7 2009-01-08 14:06:56 PST
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog A WebCore/platform/graphics/skia/GraphicsContextSkia.cpp Committed r39719 I think it's time to retire this bug. I'll open a new one for the next set of upstream changes.
Eric Seidel (no email)
Comment 8 2009-01-08 16:40:18 PST
Created attachment 26545 [details] Add ImageBufferSkia and ImageSkia files. 2009-01-08 Dimitri Glazkov <dglazkov@chromium.org> Reviewed by Eric Seidel. Add ImageBufferSkia and ImageSkia files. https://bugs.webkit.org/show_bug.cgi?id=23200 For now we've left BitmapImage and BitmapImageSingleFrameSkia implementations in the ImageSkia file (like ImageCG has), but we intend to break them out into their own files. * platform/graphics/skia/ImageBufferSkia.cpp: Added. (WebCore::ImageBufferData::ImageBufferData): (WebCore::ImageBuffer::ImageBuffer): (WebCore::ImageBuffer::~ImageBuffer): (WebCore::ImageBuffer::context): (WebCore::ImageBuffer::image): (WebCore::ImageBuffer::getImageData): (WebCore::ImageBuffer::putImageData): (WebCore::ImageBuffer::toDataURL): * platform/graphics/skia/ImageSkia.cpp: Added. (WebCore::): (WebCore::FrameData::clear): (WebCore::Image::loadPlatformResource): (WebCore::Image::drawPattern): (WebCore::BitmapImage::initPlatformData): (WebCore::BitmapImage::invalidatePlatformData): (WebCore::BitmapImage::checkForSolidColor): (WebCore::BitmapImage::draw): (WebCore::BitmapImageSingleFrameSkia::draw): (WebCore::BitmapImageSingleFrameSkia::create): --- WebCore/ChangeLog | 31 ++ WebCore/platform/graphics/skia/ImageBufferSkia.cpp | 210 +++++++++ WebCore/platform/graphics/skia/ImageSkia.cpp | 458 ++++++++++++++++++++ 3 files changed, 699 insertions(+), 0 deletions(-)
Note You need to log in before you can comment on or make changes to this bug.