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

Unified Diff: components/network_hints/renderer/renderer_preconnect.h

Issue 922533003: Eliminated the logic that accumulated multiple preconnect requests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use existing switches and fix runloop Created 5 years, 10 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: components/network_hints/renderer/renderer_preconnect.h
diff --git a/components/network_hints/renderer/renderer_preconnect.h b/components/network_hints/renderer/renderer_preconnect.h
index ae7d3d111a9ab3191c864ecf181570f9224c6e94..adc014339fadd5f1fb3c196819c42ea0f8aced19 100644
--- a/components/network_hints/renderer/renderer_preconnect.h
+++ b/components/network_hints/renderer/renderer_preconnect.h
@@ -17,10 +17,6 @@
#ifndef COMPONENTS_NETWORK_HINTS_RENDERER_RENDERER_PRECONNECT_H_
#define COMPONENTS_NETWORK_HINTS_RENDERER_RENDERER_PRECONNECT_H_
-#include <map>
-
-#include "base/basictypes.h"
-#include "base/memory/weak_ptr.h"
#include "url/gurl.h"
namespace network_hints {
@@ -32,21 +28,10 @@ class RendererPreconnect {
RendererPreconnect();
~RendererPreconnect();
- // Push a name into the queue to be preconnected.
+ // Submit a preconnect request for a single connection.
void Preconnect(const GURL &url);
- // SubmitPreconnect processes the buffered URLs, and submits them for
- // preconnecting. Multiple requests for the same URL will increase the number
- // of connections requested.
- void SubmitPreconnect();
-
private:
- // url_request_count_map_ contains (for each URL) a count for the number of
- // times it was requested.
- typedef std::map<GURL, int> UrlRequestCountMap;
- UrlRequestCountMap url_request_count_map_;
-
- base::WeakPtrFactory<RendererPreconnect> weak_factory_;
DISALLOW_COPY_AND_ASSIGN(RendererPreconnect);
}; // class RendererPreconnect
« no previous file with comments | « chrome/browser/net/predictor_browsertest.cc ('k') | components/network_hints/renderer/renderer_preconnect.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698