You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2014/02/09 02:35:19 UTC

[jira] [Commented] (CB-4818) InAppBrowser addEventListener 3rd parameter?

    [ https://issues.apache.org/jira/browse/CB-4818?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13895799#comment-13895799 ] 

ASF subversion and git services commented on CB-4818:
-----------------------------------------------------

Commit c33d6c675b07cabfc5501ffeda550f13a87aedb2 in branch refs/heads/master from [~BBosman]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-mobile-spec.git;h=c33d6c6 ]

CB-4818 Remove bogus third parameter


> InAppBrowser addEventListener 3rd parameter?
> --------------------------------------------
>
>                 Key: CB-4818
>                 URL: https://issues.apache.org/jira/browse/CB-4818
>             Project: Apache Cordova
>          Issue Type: Test
>          Components: Docs, mobile-spec, Plugin InAppBrowser
>    Affects Versions: 2.9.0
>            Reporter: Peter
>            Assignee: Michael Brooks
>            Priority: Minor
>
> The mobile-spec InAppBrowser tests addEventListener:
> {code}
>         iab.addEventListener('loaderror', logEvent, false);
>         iab.addEventListener('loadstart', logEvent, false);
>         iab.addEventListener('loadstop', logEvent, false);
>         iab.addEventListener('exit', logEvent, false);
> {code}
> But that test code above is calling addEventListener with 3 parameters, and the 3rd parameter is neither documented
> http://cordova.apache.org/docs/en/3.0.0/cordova_inappbrowser_inappbrowser.md.html#addEventListener, nor (IIUC) is it implemented in the cordova.js



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)