Bug 82058 - [BlackBerry] Accelerated compositing updates
: [BlackBerry] Accelerated compositing updates
Status: RESOLVED FIXED
: WebKit
WebKit BlackBerry
: 528+ (Nightly build)
: Unspecified Unspecified
: P2 Normal
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2012-03-23 09:05 PST by
Modified: 2012-03-26 10:43 PST (History)


Attachments
accel comp updates (25.19 KB, patch)
2012-03-23 09:05 PST, Filip Spacek
rwlbuis: review-
Review Patch | Details | Formatted Diff | Diff
Now with changelog (28.79 KB, patch)
2012-03-26 08:23 PST, Filip Spacek
rwlbuis: review+
webkit.review.bot: commit‑queue-
Review Patch | Details | Formatted Diff | Diff
Fixed patch (28.75 KB, patch)
2012-03-26 09:49 PST, Filip Spacek
no flags Review Patch | Details | Formatted Diff | Diff
Fixed style (28.77 KB, patch)
2012-03-26 09:59 PST, Filip Spacek
no flags Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2012-03-23 09:05:51 PST
Created an attachment (id=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 From 2012-03-26 07:51:44 PST -------
(From update of attachment 133496 [details])
Looks good, but needs ChangeLog.
------- Comment #2 From 2012-03-26 08:23:26 PST -------
Created an attachment (id=133817) [details]
Now with changelog
------- Comment #3 From 2012-03-26 08:26:54 PST -------
(From update of attachment 133817 [details])
Looks good.
------- Comment #4 From 2012-03-26 08:53:34 PST -------
(From update of attachment 133817 [details])
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 From 2012-03-26 09:06:44 PST -------
(From update of attachment 133817 [details])
Retry.
------- Comment #6 From 2012-03-26 09:08:51 PST -------
(From update of attachment 133817 [details])
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 From 2012-03-26 09:49:34 PST -------
Created an attachment (id=133838) [details]
Fixed patch
------- Comment #8 From 2012-03-26 09:56:12 PST -------
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 From 2012-03-26 09:59:37 PST -------
Created an attachment (id=133839) [details]
Fixed style
------- Comment #10 From 2012-03-26 10:28:40 PST -------
(From update of attachment 133839 [details])
Looks good.
------- Comment #11 From 2012-03-26 10:43:20 PST -------
(From update of attachment 133839 [details])
Clearing flags on attachment: 133839

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