Bug 79291 - [Chromium][V8] Support Uint8ClampedArray in postMessage
Summary: [Chromium][V8] Support Uint8ClampedArray in postMessage
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dmitry Lomov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-22 15:51 PST by Dmitry Lomov
Modified: 2012-02-22 23:04 PST (History)
4 users (show)

See Also:


Attachments
Fix (34.83 KB, patch)
2012-02-22 18:04 PST, Dmitry Lomov
no flags Details | Formatted Diff | Diff
Rebaselined (34.99 KB, patch)
2012-02-22 19:34 PST, Dmitry Lomov
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitry Lomov 2012-02-22 15:51:02 PST
Support Uint8ClampedArray in postMessage - currently they deserialzie as Uint8Arrays.
Comment 1 Dmitry Lomov 2012-02-22 18:04:29 PST
Created attachment 128355 [details]
Fix

Can only be landed after the fix for https://bugs.webkit.org/show_bug.cgi?id=79294 lands - the latter has updated tests for Uint8ClampedArray
Comment 2 Dmitry Lomov 2012-02-22 19:20:41 PST
Comment on attachment 128355 [details]
Fix

Fix for 79294 landed
Comment 3 Dmitry Lomov 2012-02-22 19:34:00 PST
Created attachment 128372 [details]
Rebaselined
Comment 4 Kenneth Russell 2012-02-22 19:38:56 PST
Comment on attachment 128372 [details]
Rebaselined

Big thumbs up. r=me

Could you cq+ this yourself once it's passed the EWS bots?
Comment 5 Dmitry Lomov 2012-02-22 19:47:39 PST
(In reply to comment #4)
> (From update of attachment 128372 [details])
> Big thumbs up. r=me

Thumbs up not accepted. I shouldn't have missed clamped array in the test.
> 
> Could you cq+ this yourself once it's passed the EWS bots?

Sure.
Comment 6 WebKit Review Bot 2012-02-22 23:04:39 PST
Comment on attachment 128372 [details]
Rebaselined

Clearing flags on attachment: 128372

Committed r108613: <http://trac.webkit.org/changeset/108613>
Comment 7 WebKit Review Bot 2012-02-22 23:04:44 PST
All reviewed patches have been landed.  Closing bug.