Bug 59628 - Make start parameter of Blob.slice optional
Summary: Make start parameter of Blob.slice optional
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Jian Li
Depends on:
Reported: 2011-04-27 13:35 PDT by Jian Li
Modified: 2011-05-24 11:58 PDT (History)
6 users (show)

See Also:

Proposed Patch (4.06 KB, patch)
2011-04-27 13:54 PDT, Jian Li
kbr: review+
jianli: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jian Li 2011-04-27 13:35:15 PDT
We need to make start parameter of Blob.slice optional, per the latest File API spec: http://dev.w3.org/2006/webapi/FileAPI/#dfn-Blob
Comment 1 Jian Li 2011-04-27 13:54:00 PDT
Created attachment 91339 [details]
Proposed Patch
Comment 2 Eric Seidel (no email) 2011-05-01 12:05:35 PDT
There was a gigantic thread about this on webkit-dev.  I'm surprised more of those players aren't cc'd here.
Comment 3 Eric Seidel (no email) 2011-05-23 15:50:28 PDT
Jian:  Could you please CC the relevant people and get this resolved?  it's been in the queue for almost a month.
Comment 4 Jian Li 2011-05-23 16:04:29 PDT
Per the discussion, we decide to do no change to how code generator handles optional parameter since it is the right behavior as in ECMA-262. The spec author Arun also updated the File API spec to explicitly illustrate the case for optional end parameter.

The only change in this patch to make start parameter as optional. It does not change any other existing behaviors.

CC peoples that were in the discussion if they have more to day.
Comment 5 Kenneth Russell 2011-05-23 16:09:36 PDT
Comment on attachment 91339 [details]
Proposed Patch

Looks fine.
Comment 6 Jian Li 2011-05-24 11:58:39 PDT
Committed as https://trac.webkit.org/changeset/87181.