[
https://issues.apache.org/jira/browse/CB-10451?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15121800#comment-15121800
]
ASF GitHub Bot commented on CB-10451:
-------------------------------------
Github user sgrebnov commented on the pull request:
https://github.com/apache/cordova-plugin-inappbrowser/pull/145#issuecomment-176253919
Use the following url to review proposed changes w/o whitespace:
https://github.com/apache/cordova-plugin-inappbrowser/pull/145/files?w=1
> InAppBrowser: loadstart event is not triggered on Windows
> ---------------------------------------------------------
>
> Key: CB-10451
> URL: https://issues.apache.org/jira/browse/CB-10451
> Project: Apache Cordova
> Issue Type: Bug
> Components: Plugin InAppBrowser
> Reporter: Sergey Grebnov
> Assignee: Sergey Grebnov
> Labels: Windows, triaged
>
> The following unit test fails as 'loadstart' callback is not called on
> Windows. Main reason is that it is called internally when we do open before
> we attach event handler. But this is what docs recommend and what other
> people uses. We should make open method async (Windows proxy or
> inappbrowser.js) so that handler could be added this way
> {code}
> it("inappbrowser.spec.4 should support loadstart and loadstop events",
> function (done) {
> var onLoadStart = jasmine.createSpy('loadstart event
> callback').and.callFake(function (evt) {
> verifyEvent(evt, 'loadstart');
> });
> iabInsance = cordova.InAppBrowser.open(url, '_blank');
> iabInsance.addEventListener('loadstart', onLoadStart);
> iabInsance.addEventListener('loadstop', function (evt) {
> verifyEvent(evt, 'loadstop');
> expect(onLoadStart).toHaveBeenCalled();
> done();
> });
> });
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
