cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mike Billau (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CB-4818) InAppBrowser addEventListener 3rd parameter?
Date Thu, 17 Oct 2013 14:35:42 GMT

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

Mike Billau commented on CB-4818:
---------------------------------

The only place I see this is: https://github.com/apache/cordova-mobile-spec/blob/master/inappbrowser/index.html#L115
It's not in the documentation or examples or any other tests.
I'm thinking that somebody just accidentally added a "false" parameter to the end, like what
we normally do for document.addEventListener.


> 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
>    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#6144)

Mime
View raw message