Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TIMOB-4357: Webview fatal exception / crash on re-open #594

Merged
merged 4 commits into from Nov 11, 2011

Conversation

ayeung
Copy link
Contributor

@ayeung ayeung commented Oct 26, 2011

http://jira.appcelerator.org/browse/TIMOB-4357

Test case should be here

This ticket should be tested as well: timob-1936

webView.changeProxyUrl(url);
if (DEFAULT_PAGE_FINISH_URL.equals(url)) {
url = "";
webView.getProxy().getProperties().remove("url");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should actually check for parity with iOS in this case. What do they do with the "url" property of the event that fires when a page is loaded, in the case where the page is loaded via setHtml rather than a true url? Also, check to see what they do with the proxy "url" proerty in this case. We're removing it here (my suggestion) but maybe they do something diff.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just tested in iOS and the url is set to "app://com.iphone/(null)" when we use setHtml rather than a true url.

@billdawson
Copy link
Contributor

Functional test passed. Waiting on parity issue check for code review.

@marshall
Copy link
Contributor

marshall commented Nov 9, 2011

Code review accepted

@billdawson
Copy link
Contributor

Code review and functional test accepted. Did functional both for this ticket and timob-1936, both in rhino and v8.

billdawson added a commit that referenced this pull request Nov 11, 2011
TIMOB-4357: Webview fatal exception / crash on re-open
@billdawson billdawson merged commit 97862cb into tidev:master Nov 11, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants