Hey
I've thrown together a quick 4 line fix to fix it for now, but we need
to make sure that we actually test things when we change them. This
change totally broke the Media API on Android, and could have been
easily avoided. We shouldn't have to wait for the RC1 to find these
bugs, and people should run the tests now, since about half of the
issues that come through JIRA could be tested with mobile-spec.
On Fri, May 24, 2013 at 10:59 AM, Joe Bowser <bows...@gmail.com> wrote:
> OK, I was wrong. It turns out the DataResource code that was added in
> doesn't have support for streaming audio. We need to get this fixed
> ASAP. Seriously, we have these tests for a reason. Every time we
> start changing how an API works, we should run the Mobile-Spec tests
> to make sure that it's covered. If it's not covered by mobile-spec,
> we should create a ticket to write a test for the Android Test Suite.
>
> The culprit appears to be on line 60 of AudioHandler.java:
>
> public String getFilePath(String url, String source){
> DataResource dataResource =
> DataResource.initiateNewDataRequestForUri(url,
> this.webView.pluginManager, cordova, source);
> return dataResource.getRealFile().getPath();
> }
>
> If it's an web URI, we shouldn't screw with it. Seriously!
>
> Joe
>
> On Fri, May 24, 2013 at 10:36 AM, Joe Bowser <bows...@gmail.com> wrote:
>> Hey
>>
>> After looking into the bug further, for some reason URIs such as
>> http://cordova.apache.org/downloads/BlueZedEx.mp3 are losing slashes
>> after the protocol. (i.e.
>> http:/cordova.apache.org/downloads/BlueZedEx.mp3). This is causing
>> Android to epic fail the media tests and I suspect that it would cause
>> the same problem in other platforms. Did anything change that would
>> cause the URIs to be malformed? This is a pretty important thing to
>> get fixed for 2.8.0rc1, and I don't want to tag until this is
>> resolved.
>>
>> Joe
