[Offline pages] Updating the UpdateCallback in OPMStoreSQL

components/offline_pages/offline_page_item.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_ITEM_H_
 #define COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_ITEM_H_
 
 #include <stdint.h>
 
 #include <string>
@@ skipping 36 matching lines @@
                   int64_t file_size);
   OfflinePageItem(const GURL& url,
                   int64_t offline_id,
                   const ClientId& client_id,
                   const base::FilePath& file_path,
                   int64_t file_size,
                   const base::Time& creation_time);
   OfflinePageItem(const OfflinePageItem& other);
   ~OfflinePageItem();
 
+  bool operator==(const OfflinePageItem& other) const;

[dewittj, 2016/09/23 17:21:35]

This should be documented; in particular, two OfflinePageItems could refer to
the same file and be !=, if one had its access_count or other fields modified
and the other didn't.

+
   // Gets a URL of the file under |file_path|.
   GURL GetOfflineURL() const;
 
   // Returns whether the offline page is expired.
   bool IsExpired() const;
 
   // The URL of the page.
   GURL url;
   // The primary key/ID for this page in offline pages internal database.
   int64_t offline_id;
@@ skipping 17 matching lines @@
   int access_count;
   // The title of the page at the time it was saved.
   base::string16 title;
   // Flags about the state and behavior of the offline page.
   Flags flags;
 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_ITEM_H_