Bug 82058

Summary: [BlackBerry] Accelerated compositing updates
Product: WebKit Reporter: Filip Spacek <fspacek>
Component: WebKit BlackBerryAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
accel comp updates
rwlbuis: review-
Now with changelog
rwlbuis: review+, webkit.review.bot: commit-queue-
Fixed patch
none
Fixed style none

Description Filip Spacek 2012-03-23 09:05:51 PDT
Created attachment 133496 [details]
accel comp updates

Attached patch includes updates to the accelerated compositing code for BlackBerry port. It reflects changes in skia portion of webkit as well as skia itself.
Comment 1 Rob Buis 2012-03-26 07:51:44 PDT
Comment on attachment 133496 [details]
accel comp updates

Looks good, but needs ChangeLog.
Comment 2 Filip Spacek 2012-03-26 08:23:26 PDT
Created attachment 133817 [details]
Now with changelog
Comment 3 Rob Buis 2012-03-26 08:26:54 PDT
Comment on attachment 133817 [details]
Now with changelog

Looks good.
Comment 4 WebKit Review Bot 2012-03-26 08:53:34 PDT
Comment on attachment 133817 [details]
Now with changelog

Rejecting attachment 133817 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
ead.cpp
patching file Source/WebCore/platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp
patching file Source/WebCore/platform/graphics/blackberry/skia/ImageBufferData.h
rm 'Source/WebCore/platform/graphics/blackberry/skia/ImageBufferData.h'
patching file Source/WebCore/platform/graphics/blackberry/skia/ImageBufferDataSkia.h

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Rob Buis']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue/

Full output: http://queues.webkit.org/results/12132846
Comment 5 Rob Buis 2012-03-26 09:06:44 PDT
Comment on attachment 133817 [details]
Now with changelog

Retry.
Comment 6 WebKit Review Bot 2012-03-26 09:08:51 PDT
Comment on attachment 133817 [details]
Now with changelog

Rejecting attachment 133817 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
ead.cpp
patching file Source/WebCore/platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp
patching file Source/WebCore/platform/graphics/blackberry/skia/ImageBufferData.h
rm 'Source/WebCore/platform/graphics/blackberry/skia/ImageBufferData.h'
patching file Source/WebCore/platform/graphics/blackberry/skia/ImageBufferDataSkia.h

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Rob Buis']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue/

Full output: http://queues.webkit.org/results/12134883
Comment 7 Filip Spacek 2012-03-26 09:49:34 PDT
Created attachment 133838 [details]
Fixed patch
Comment 8 WebKit Review Bot 2012-03-26 09:56:12 PDT
Attachment 133838 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1
Source/WebCore/platform/graphics/blackberry/skia/ImageBufferDataSkia.h:32:  #ifndef header guard has wrong style, please use: ImageBufferDataSkia_h  [build/header_guard] [5]
Total errors found: 1 in 10 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 9 Filip Spacek 2012-03-26 09:59:37 PDT
Created attachment 133839 [details]
Fixed style
Comment 10 Rob Buis 2012-03-26 10:28:40 PDT
Comment on attachment 133839 [details]
Fixed style

Looks good.
Comment 11 WebKit Review Bot 2012-03-26 10:43:20 PDT
Comment on attachment 133839 [details]
Fixed style

Clearing flags on attachment: 133839

Committed r112124: <http://trac.webkit.org/changeset/112124>
Comment 12 WebKit Review Bot 2012-03-26 10:43:25 PDT
All reviewed patches have been landed.  Closing bug.