[Offline pages] Updating the UpdateCallback in OPMStoreSQL

components/offline_pages/offline_page_item.cc

Index: components/offline_pages/offline_page_item.cc
diff --git a/components/offline_pages/offline_page_item.cc b/components/offline_pages/offline_page_item.cc
index cd3a5da538d2012c9ec5887edf27c605bfb63d79..11e4685078b35787d9a0e15c69220a3de6f8ac12 100644
--- a/components/offline_pages/offline_page_item.cc
+++ b/components/offline_pages/offline_page_item.cc
@@ -61,6 +61,16 @@ OfflinePageItem::OfflinePageItem(const OfflinePageItem& other) = default;
 OfflinePageItem::~OfflinePageItem() {
 }
 
+bool OfflinePageItem::operator==(const OfflinePageItem& other) const {
+  return url == other.url && offline_id == other.offline_id &&

[Dmitry Titov, 2016/09/20 20:43:37]

It'd be more human-readable if it was just a single column. A bit more lines but
less strain on eyes.

[fgorski, 2016/09/20 22:15:02]

Done. The problem is caused by "git cl format" and I absolutely agree with you.
I made a manual update at the end to prevent automatic formatting.

+         client_id == other.client_id && file_path == other.file_path &&
+         creation_time == other.creation_time &&
+         last_access_time == other.last_access_time &&
+         expiration_time == other.expiration_time &&
+         access_count == other.access_count && title == other.title &&
+         flags == other.flags;
+}
+
 GURL OfflinePageItem::GetOfflineURL() const {
   return net::FilePathToFileURL(file_path);
 }