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-14349: Implementing setCacheMode for web view #4510

Merged
merged 3 commits into from Aug 19, 2013

Conversation

hieupham007
Copy link
Contributor

testing steps in JIRA

@ghost ghost assigned pingwang2011 Jul 29, 2013
@pingwang2011
Copy link
Contributor

No test steps in JIRA.

if (Build.VERSION.SDK_INT >= 9) {
nativeView.setOverScrollMode(TiConvert.toInt(newValue, View.OVER_SCROLL_ALWAYS));
}
} else if (TiC.PROPERTY_CACHE_MODE.equals(key)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you need to add TiC.PROPERTY_CACHE_MODE to propertyAccessors in WebViewProxy.java. Otherwise, propertyChanged() may not work.

Copy link
Contributor

Choose a reason for hiding this comment

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

never mind.

@pingwang2011
Copy link
Contributor

The PR is not merge-able now. Please merge with master.

…e into timob-14349

Conflicts:
	android/modules/ui/src/java/ti/modules/titanium/ui/WebViewProxy.java
	android/modules/ui/src/java/ti/modules/titanium/ui/widget/webview/TiUIWebView.java
	apidoc/Titanium/UI/WebView.yml
@hieupham007
Copy link
Contributor Author

Merged.

@pingwang2011
Copy link
Contributor

Code reviewed and functionally tested. Accepted

pingwang2011 added a commit that referenced this pull request Aug 19, 2013
timob-14349: Implementing setCacheMode for web view
@pingwang2011 pingwang2011 merged commit aabadf5 into tidev:master Aug 19, 2013
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

2 participants