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-14746-added support for appCache #4661

Merged
merged 1 commit into from Oct 4, 2013

Conversation

salachi
Copy link
Contributor

@salachi salachi commented Sep 9, 2013

@@ -147,6 +147,11 @@ public TiUIWebView(TiViewProxy proxy)
settings.setDatabaseEnabled(true);
}

File cacheDir = TiApplication.getInstance().getCacheDir();
if (cacheDir != null) {
settings.setAppCacheEnabled(true);
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure if this addresses the issue mentioned. The ticket specifically ask for caching using a custom manifest, i.e: only cache the files declared in the manifest. Essentially we would need to implement our own custom cache.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The manifest file is deployed in the server describing which files can be cached and is referred in the html file using 'manifest' attribute in the html tag. The client webview will only cache those files, I don't think we need to do anything else. I tested with 'http://www.w3schools.com/html/tryhtml5_html_manifest.htm' and seems working.

@pingwang2011
Copy link
Contributor

Ran the test case. When I went to step 3, the webview shows "Webpage not available: the webpage at [my local host] might be temporarily down ....". Can you confirm the testing steps in the ticket?

@salachi
Copy link
Contributor Author

salachi commented Sep 29, 2013

Did you set up the server with right html and the manifest files?
I tried with http://www.w3schools.com/html/tryhtml5_html_manifest.htm and it seems to be working.

@pingwang2011
Copy link
Contributor

I followed "Test case" and "Test code" in the ticket description. If you used some other test case, can you put all your code and testing steps in the ticket?

@ghost ghost assigned pingwang2011 Oct 1, 2013
@salachi
Copy link
Contributor Author

salachi commented Oct 3, 2013

Added comment with test case to use an existing server url that is already setup for html5 caching.

@pingwang2011
Copy link
Contributor

Code reviewed and functionally tested. Accepted

pingwang2011 added a commit that referenced this pull request Oct 4, 2013
TIMOB-14746-added support for appCache
@pingwang2011 pingwang2011 merged commit ca70d4a into tidev:master Oct 4, 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

3 participants