Support providers with two namespaces

components/offline_items_collection/core/offline_item.cc

Index: components/offline_items_collection/core/offline_item.cc
diff --git a/components/offline_items_collection/core/offline_item.cc b/components/offline_items_collection/core/offline_item.cc
index 81d47ea07d08fef31b693fd5285d5a10bc7ac188..63ec30a36bc1e2077a28a2ceb926d10f83267f0e 100644
--- a/components/offline_items_collection/core/offline_item.cc
+++ b/components/offline_items_collection/core/offline_item.cc
@@ -61,4 +61,8 @@ bool OfflineItem::operator==(const OfflineItem& offline_item) const {
          time_remaining_ms == offline_item.time_remaining_ms;
 }
 
+bool OfflineItem::operator<(const OfflineItem& offline_item) const {
+  return id < offline_item.id;

[fgorski, 2017/03/17 04:15:16]

this is a little dangerous as (!(a < b) && !(b < a)) should imply equality,
which will be inconsistent with implementation of operator== . At least
documenting that would be useful.

[David Trainor- moved to gerrit, 2017/03/22 00:12:45]

Yeah that's a good point.  I didn't like this either.  I only needed it for a
unit test so I just wrote a custom comparator there for std::map.

+}
+
 }  // namespace offline_items_collection