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

Unified Diff: chrome/browser/ui/webui/net_internals/net_internals_ui.cc

Issue 275953002: Remove HTTP pipelining support. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix line endings Created 6 years, 7 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: chrome/browser/ui/webui/net_internals/net_internals_ui.cc
diff --git a/chrome/browser/ui/webui/net_internals/net_internals_ui.cc b/chrome/browser/ui/webui/net_internals/net_internals_ui.cc
index 2bc0668cba379b1fa322afb97dfda47dc4602fc9..99a9a87504643633c712e6f5bcdb8441c9ee5fe6 100644
--- a/chrome/browser/ui/webui/net_internals/net_internals_ui.cc
+++ b/chrome/browser/ui/webui/net_internals/net_internals_ui.cc
@@ -469,7 +469,6 @@ class NetInternalsMessageHandler::IOThreadImpl
#if defined(OS_WIN)
void OnGetServiceProviders(const base::ListValue* list);
#endif
- void OnGetHttpPipeliningStatus(const base::ListValue* list);
void OnSetLogLevel(const base::ListValue* list);
// ChromeNetLog::ThreadSafeObserver implementation:
@@ -682,10 +681,6 @@ void NetInternalsMessageHandler::RegisterMessages() {
#endif
web_ui()->RegisterMessageCallback(
- "getHttpPipeliningStatus",
- base::Bind(&IOThreadImpl::CallbackHelper,
- &IOThreadImpl::OnGetHttpPipeliningStatus, proxy_));
- web_ui()->RegisterMessageCallback(
"setLogLevel",
base::Bind(&IOThreadImpl::CallbackHelper,
&IOThreadImpl::OnSetLogLevel, proxy_));
@@ -1582,63 +1577,6 @@ void NetInternalsMessageHandler::OnSetNetworkDebugModeCompleted(
}
#endif // defined(OS_CHROMEOS)
-void NetInternalsMessageHandler::IOThreadImpl::OnGetHttpPipeliningStatus(
- const base::ListValue* list) {
- DCHECK(!list);
- base::DictionaryValue* status_dict = new base::DictionaryValue();
-
- base::Value* pipelined_connection_info = NULL;
- net::HttpNetworkSession* http_network_session =
- GetHttpNetworkSession(GetMainContext());
- if (http_network_session) {
- status_dict->Set("pipelining_enabled", base::Value::CreateBooleanValue(
- http_network_session->params().http_pipelining_enabled));
-
- pipelined_connection_info =
- http_network_session->http_stream_factory()->PipelineInfoToValue();
- }
- status_dict->Set("pipelined_connection_info", pipelined_connection_info);
-
- const net::HttpServerProperties& http_server_properties =
- *GetMainContext()->http_server_properties();
-
- // TODO(simonjam): This call is slow.
- const net::PipelineCapabilityMap pipeline_capability_map =
- http_server_properties.GetPipelineCapabilityMap();
-
- base::ListValue* known_hosts_list = new base::ListValue();
- net::PipelineCapabilityMap::const_iterator it;
- for (it = pipeline_capability_map.begin();
- it != pipeline_capability_map.end(); ++it) {
- base::DictionaryValue* host_dict = new base::DictionaryValue();
- host_dict->SetString("host", it->first.ToString());
- std::string capability;
- switch (it->second) {
- case net::PIPELINE_CAPABLE:
- capability = "capable";
- break;
-
- case net::PIPELINE_PROBABLY_CAPABLE:
- capability = "probably capable";
- break;
-
- case net::PIPELINE_INCAPABLE:
- capability = "incapable";
- break;
-
- case net::PIPELINE_UNKNOWN:
- default:
- capability = "unknown";
- break;
- }
- host_dict->SetString("capability", capability);
- known_hosts_list->Append(host_dict);
- }
- status_dict->Set("pipelined_host_info", known_hosts_list);
-
- SendJavascriptCommand("receivedHttpPipeliningStatus", status_dict);
-}
-
void NetInternalsMessageHandler::IOThreadImpl::OnSetLogLevel(
const base::ListValue* list) {
int log_level;
« no previous file with comments | « chrome/browser/resources/net_internals/main.js ('k') | chrome/browser/ui/webui/net_internals/net_internals_ui_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698