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

Unified Diff: chrome/browser/intents/web_intents_registry.cc

Issue 10914278: Fix for non-sticky defaults bug. Added a bunch of tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 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: chrome/browser/intents/web_intents_registry.cc
diff --git a/chrome/browser/intents/web_intents_registry.cc b/chrome/browser/intents/web_intents_registry.cc
index 62d0789ad404cab524d70358bdabbc5322e9423d..c557e3b19960e9f365d0c10dfe2ecdae19f8769d 100644
--- a/chrome/browser/intents/web_intents_registry.cc
+++ b/chrome/browser/intents/web_intents_registry.cc
@@ -296,6 +296,8 @@ void WebIntentsRegistry::OnWebIntentsDefaultsResultReceived(
default_service = *iter;
} else if (iter->url_pattern < default_service.url_pattern) {
default_service = *iter;
+ } else if (default_service.user_date < iter->user_date) {
+ default_service = *iter;
}
}

Powered by Google App Engine
This is Rietveld 408576698