Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of agrieve@google.com designates
 209.85.210.175 as permitted sender)
MIME-Version: 1.0
Sender: agrieve@google.com
In-Reply-To: <CC6BB981.CC26%fil@adobe.com>
References: 
 <CAKP6k2ufh6h7FA8NYDNFOg2=1CmUrJjP4aBhjM3zCGgQBnDBfQ@mail.gmail.com>
 <CC6BB981.CC26%fil@adobe.com>
From: Andrew Grieve <agrieve@chromium.org>
Date: Wed, 5 Sep 2012 00:20:24 -0400
Message-ID: 
 <CABiQX1Upmw1r4KQOt8+bt42Yp53JUm2ze7w8627qDH9UHLNgHw@mail.gmail.com>
Subject: Re: The spec defines the docs which defines the tests which defines
 the implementation, and other dependency issues
To: callback-dev@incubator.apache.org
Content-Type: multipart/alternative; boundary=bcaec54d47a4cb28d604c8ecb24e

--bcaec54d47a4cb28d604c8ecb24e
Content-Type: text/plain; charset=ISO-8859-1

Maybe just add MEDIA_ERR_SRC_NOT_SUPPORTED alongside MEDIA_ERR_NONE_
SUPPORTED?


On Tue, Sep 4, 2012 at 5:01 PM, Filip Maj <fil@adobe.com> wrote:

> This API was written against an earlier revision of this spec so the
> codes/labels have likely changed.
>
> Changing stuff requires changes to reflect across all platforms. That's my
> only concern at this point.
>
> I would shy away from changing anything. It's make-work until we review
> the API(s) in the coming months anyways.
>
> As for this specific scenario.. Fuck. I dunno. I would update the docs to
> reflect the JS + tests?
>
> On 9/4/12 1:08 PM, "Jesse" <purplecabbage@gmail.com> wrote:
>
> >I am trying to chase down a few failing tests, here is one of the
> >simpler cases I am having to deal with.
> >I need to know which of the following is correct.
> >( note that there are numerous other similar cases like this
> >throughout our code base )
> >
> >The W3 Spec for MediaError is [1] :
> >
> >interface MediaError {
> >  const unsigned short MEDIA_ERR_ABORTED = 1;
> >  const unsigned short MEDIA_ERR_NETWORK = 2;
> >  const unsigned short MEDIA_ERR_DECODE = 3;
> >  const unsigned short MEDIA_ERR_SRC_NOT_SUPPORTED = 4;
> >  readonly attribute unsigned short code;
> >};
> >
> >The phonegap edge docs state [2]:
> >
> >MediaError.MEDIA_ERR_ABORTED
> >MediaError.MEDIA_ERR_NETWORK
> >MediaError.MEDIA_ERR_DECODE
> >MediaError.MEDIA_ERR_NONE_SUPPORTED
> >
> >The tests are testing for [3] :
> >
> >expect(MediaError.MEDIA_ERR_NONE_ACTIVE).toBe(0);
> >expect(MediaError.MEDIA_ERR_ABORTED).toBe(1);
> >expect(MediaError.MEDIA_ERR_NETWORK).toBe(2);
> >expect(MediaError.MEDIA_ERR_DECODE).toBe(3);
> >expect(MediaError.MEDIA_ERR_NONE_SUPPORTED).toBe(4);
> >
> >The cordova js repo defines this [4] :
> >
> >MediaError.MEDIA_ERR_NONE_ACTIVE    = 0;
> >MediaError.MEDIA_ERR_ABORTED        = 1;
> >MediaError.MEDIA_ERR_NETWORK        = 2;
> >MediaError.MEDIA_ERR_DECODE         = 3;
> >MediaError.MEDIA_ERR_NONE_SUPPORTED = 4;
> >
> >How should I resolve this?
> >Can we remove MEDIA_ERR_NONE_ACTIVE ?
> >Can we re-define 4 as MEDIA_ERR_SRC_NOT_SUPPORTED instead of
> >MEDIA_ERR_NONE_SUPPORTED ?
> >Can we change our js code to only write the MediaError interface if it
> >does not exist?
> >
> >[1] http://dev.w3.org/html5/spec-author-view/video.html#error-codes
> >[2]
> >http://docs.phonegap.com/en/edge/cordova_media_media.md.html#mediaError
> >[3]
> >
> https://github.com/apache/incubator-cordova-mobile-spec/blob/master/autote
> >st/tests/media.tests.js
> >[4]
> >
> https://github.com/apache/incubator-cordova-js/blob/master/lib/common/plug
> >in/MediaError.js
> >--
> >@purplecabbage
> >risingj.com
>
>

--bcaec54d47a4cb28d604c8ecb24e--