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

Unified Diff: chrome/browser/data_use_measurement/chrome_data_use_ascriber.h

Issue 2829163004: Remove uses of base::hash_map from //chrome (Closed)
Patch Set: Downloads back Created 3 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
« no previous file with comments | « chrome/browser/command_updater.h ('k') | chrome/browser/download/chrome_download_manager_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/data_use_measurement/chrome_data_use_ascriber.h
diff --git a/chrome/browser/data_use_measurement/chrome_data_use_ascriber.h b/chrome/browser/data_use_measurement/chrome_data_use_ascriber.h
index 9e2c30bf71ad56e98b1a8c32d1545f70f950c8f7..9dc15307cad8b3a37ae8921eb5511b90ffc0120f 100644
--- a/chrome/browser/data_use_measurement/chrome_data_use_ascriber.h
+++ b/chrome/browser/data_use_measurement/chrome_data_use_ascriber.h
@@ -6,13 +6,12 @@
#define CHROME_BROWSER_DATA_USE_MEASUREMENT_CHROME_DATA_USE_ASCRIBER_H_
#include <list>
+#include <map>
#include <memory>
#include <string>
#include <tuple>
-#include <unordered_map>
#include <utility>
-#include "base/containers/hash_tables.h"
#include "base/hash.h"
#include "base/macros.h"
#include "base/supports_user_data.h"
@@ -119,13 +118,6 @@ class ChromeDataUseAscriber : public DataUseAscriber {
typedef std::list<ChromeDataUseRecorder> DataUseRecorderList;
typedef DataUseRecorderList::iterator DataUseRecorderEntry;
- struct GlobalRequestIDHash {
- public:
- std::size_t operator()(const content::GlobalRequestID& x) const {
- return base::HashInts(x.child_id, x.request_id);
- }
- };
-
class DataUseRecorderEntryAsUserData : public base::SupportsUserData::Data {
public:
explicit DataUseRecorderEntryAsUserData(DataUseRecorderEntry entry);
@@ -192,20 +184,17 @@ class ChromeDataUseAscriber : public DataUseAscriber {
// Map from RenderFrameHost to the MainRenderFrameEntry which contains all
// details of the main frame. New entry is added on main render frame creation
// and removed on its deletion.
- base::hash_map<RenderFrameHostID, MainRenderFrameEntry>
+ std::map<RenderFrameHostID, MainRenderFrameEntry>
main_render_frame_entry_map_;
// Maps subframe IDs to the mainframe ID, so the mainframe lifetime can have
// ownership over the lifetime of entries in |data_use_recorders_|. Mainframes
// are mapped to themselves.
- base::hash_map<RenderFrameHostID, RenderFrameHostID>
- subframe_to_mainframe_map_;
+ std::map<RenderFrameHostID, RenderFrameHostID> subframe_to_mainframe_map_;
// Map from pending navigations to the DataUseRecorderEntry in
// |data_use_recorders_| that the navigation ascribes data use to.
- std::unordered_map<content::GlobalRequestID,
- DataUseRecorderEntry,
- GlobalRequestIDHash>
+ std::map<content::GlobalRequestID, DataUseRecorderEntry>
pending_navigation_data_use_map_;
DISALLOW_COPY_AND_ASSIGN(ChromeDataUseAscriber);
« no previous file with comments | « chrome/browser/command_updater.h ('k') | chrome/browser/download/chrome_download_manager_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698