Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(173)

Unified Diff: webkit/appcache/appcache.cc

Issue 10575044: AppCache - Don't ignore network namespace entries when '*' is in the list. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webkit/appcache/appcache.cc
===================================================================
--- webkit/appcache/appcache.cc (revision 143213)
+++ webkit/appcache/appcache.cc (working copy)
@@ -152,11 +152,8 @@
std::sort(fallback_namespaces_.begin(), fallback_namespaces_.end(),
SortNamespacesByLength);
- if (!online_whitelist_all_) {
- for (size_t i = 0; i < whitelists.size(); ++i) {
- online_whitelist_namespaces_.push_back(whitelists.at(i).namespace_url);
- }
- }
+ for (size_t i = 0; i < whitelists.size(); ++i)
+ online_whitelist_namespaces_.push_back(whitelists.at(i).namespace_url);
}
void AppCache::ToDatabaseRecords(
@@ -209,13 +206,11 @@
record.target_url = fallback_namespaces_[i].target_url;
}
- if (!online_whitelist_all_) {
- for (size_t i = 0; i < online_whitelist_namespaces_.size(); ++i) {
- whitelists->push_back(AppCacheDatabase::OnlineWhiteListRecord());
- AppCacheDatabase::OnlineWhiteListRecord& record = whitelists->back();
- record.cache_id = cache_id_;
- record.namespace_url = online_whitelist_namespaces_[i];
- }
+ for (size_t i = 0; i < online_whitelist_namespaces_.size(); ++i) {
+ whitelists->push_back(AppCacheDatabase::OnlineWhiteListRecord());
+ AppCacheDatabase::OnlineWhiteListRecord& record = whitelists->back();
+ record.cache_id = cache_id_;
+ record.namespace_url = online_whitelist_namespaces_[i];
}
}

Powered by Google App Engine
This is Rietveld 408576698