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

Shazron Abdullah commented on CB-7287:
--------------------------------------

One solution:
Each bridge should clean itself up, or we add a new function to cordova.js to 
set and cleanup bridges (ie each bridge has a deInit() function, and 
cordova.changeBridge(bridge) would call it, or something).

But for now, to be backwards compatible, we could do:
{code}
if (cordova.exec !== iOSExec) { // where iOSExec is our exec in our scope
    console.log("The bridge has changed, do something, pass remnant commands to 
new bridge, cleanup, stop any timers etc");
    // TODO: For each command in the commandQueue, we have to "unwrap" the 
callbackId to get the 
    // success and failure callbacks, and send to the new cordova.exec.
}
{code}

NOTE: this problem only exists because of that timer to catch commands that may 
have been missed because of an iframe change

> [WKWebView] Failing tests
> -------------------------
>
>                 Key: CB-7287
>                 URL: https://issues.apache.org/jira/browse/CB-7287
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: iOS
>         Environment: iOS 8
>            Reporter: Shazron Abdullah
>            Assignee: Anis Kadri
>              Labels: cordova-ios-4.0.x
>
> This is only for the wkwebview branch, and using these plugins, and 3.5.0 
> mobile-spec:
> {code}
>     "org.apache.cordova.battery-status": "0.2.10",
>     "org.apache.cordova.camera": "0.3.1",
>     "org.apache.cordova.console": "0.2.10",
>     "org.apache.cordova.contacts": "0.2.12",
>     "org.apache.cordova.device": "0.2.11",
>     "org.apache.cordova.device-motion": "0.2.9",
>     "org.apache.cordova.device-orientation": "0.3.8",
>     "org.apache.cordova.dialogs": "0.2.9",
>     "org.apache.cordova.file": "1.3.0",
>     "org.apache.cordova.file-transfer": "0.4.5",
>     "org.apache.cordova.geolocation": "0.3.9",
>     "org.apache.cordova.globalization": "0.3.0",
>     "org.apache.cordova.inappbrowser": "0.5.1",
>     "org.apache.cordova.media": "0.2.12",
>     "org.apache.cordova.media-capture": "0.3.2",
>     "org.apache.cordova.network-information": "0.2.11",
>     "org.apache.cordova.splashscreen": "0.3.2",
>     "org.apache.cordova.vibration": "0.3.10"
> {code}
> The 3 failing tests:
> {code}
> FileTransfer upload method filetransfer.spec.18 should be able to upload a 
> file.
> Expected 
> '{"fields":{"value2":"param","value1":"test"},"files":{"file":{"size":24,"path":"/opt/run/snapshot/.tmp/ed60ec667664b454d72b193bf8841e5c","name":"upload.txt","type":"text/plain","mtime":"2014-08-12T21:53:28.862Z"}}}
>  ' to match /fields:\s*{\s*value1.*/.
> FileTransfer upload method filetransfer.spec.19 should be able to upload a 
> file with http basic auth.
> Expected 
> '{"fields":{"value2":"param","value1":"test"},"files":{"file":{"size":24,"path":"/opt/run/snapshot/.tmp/94aa407aca7bd9c53081f0da5ab33d54","name":"upload.txt","type":"text/plain","mtime":"2014-08-12T21:53:29.322Z"}}}
>  ' to match /fields:\s*{\s*value1.*/.
> FileTransfer Backwards compatibility filetransfer.spec.29 should be able to 
> upload a file using local paths.
> Expected 
> '{"fields":{"value2":"param","value1":"test"},"files":{"file":{"size":24,"path":"/opt/run/snapshot/.tmp/9bf857a3d64ef9beaf4eae4e22876ecc","name":"upload.txt","type":"text/plain","mtime":"2014-08-12T21:53:31.307Z"}}}
>  ' to match /fields:\s*{\s*value1.*/.
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org

Reply via email to