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 15244: Android: expose the cookie management methods for HTTPClient and WebView #4804
Conversation
return; | ||
} | ||
|
||
super.onPropertyChanged(name, value); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be called in the else case at the end of this method
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It can not be in the else case. Because in the KrollProxy.onPropertyChanged(), it calls properties.put(name, newValue). That's where the property value get updated.
Reviewed. Ran validate and deploy scripts. Left a few comments. |
public CookieProxy[] getHTTPCookiesForDomain(String domain) | ||
{ | ||
if (domain == null || domain.length() == 0) { | ||
Log.e(TAG, "Unable to get the HTTP cookies. Need to provide a valid domain."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably only show in debug mode.
Code reviewed. Please address comments. |
Updated. Ready for review. |
Code reviewed and functionally tested. Ran all test cases in ticket + anvil. Request accepted |
Timob 15244: Android: expose the cookie management methods for HTTPClient and WebView
https://jira.appcelerator.org/browse/TIMOB-15244