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

Unified Diff: blimp/client/session/blimp_client_session.cc

Issue 1962393004: Added a debug info UI for Blimp (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added empty implementation for linux client Created 4 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
« no previous file with comments | « blimp/client/session/blimp_client_session.h ('k') | blimp/engine/session/blimp_engine_session.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/client/session/blimp_client_session.cc
diff --git a/blimp/client/session/blimp_client_session.cc b/blimp/client/session/blimp_client_session.cc
index f4a0f4c3d9ebc4e67b311b95ef0ae8fdd08acee7..ca75ba435715c43a267aa67cc6557ae9a56b2f0c 100644
--- a/blimp/client/session/blimp_client_session.cc
+++ b/blimp/client/session/blimp_client_session.cc
@@ -72,8 +72,9 @@ class ClientNetworkComponents : public ConnectionHandler,
public ConnectionErrorObserver {
public:
// Can be created on any thread.
- explicit ClientNetworkComponents(
- std::unique_ptr<NetworkEventObserver> observer);
+ ClientNetworkComponents(
+ std::unique_ptr<NetworkEventObserver> observer,
+ std::unique_ptr<BlimpConnectionStatistics> blimp_connection_statistics);
~ClientNetworkComponents() override;
// Sets up network components.
@@ -95,14 +96,19 @@ class ClientNetworkComponents : public ConnectionHandler,
std::unique_ptr<BrowserConnectionHandler> connection_handler_;
std::unique_ptr<ClientConnectionManager> connection_manager_;
std::unique_ptr<NetworkEventObserver> network_observer_;
+ std::unique_ptr<BlimpConnectionStatistics> connection_statistics_;
DISALLOW_COPY_AND_ASSIGN(ClientNetworkComponents);
};
ClientNetworkComponents::ClientNetworkComponents(
- std::unique_ptr<NetworkEventObserver> network_observer)
+ std::unique_ptr<NetworkEventObserver> network_observer,
+ std::unique_ptr<BlimpConnectionStatistics> statistics)
: connection_handler_(new BrowserConnectionHandler),
- network_observer_(std::move(network_observer)) {}
+ network_observer_(std::move(network_observer)),
+ connection_statistics_(std::move(statistics)) {
+ DCHECK(connection_statistics_);
+}
ClientNetworkComponents::~ClientNetworkComponents() {}
@@ -121,12 +127,13 @@ void ClientNetworkComponents::ConnectWithAssignment(
case Assignment::SSL:
DCHECK(assignment.cert);
connection_manager_->AddTransport(base::WrapUnique(new SSLClientTransport(
- assignment.engine_endpoint, std::move(assignment.cert), nullptr)));
+ assignment.engine_endpoint, std::move(assignment.cert),
+ connection_statistics_.get(), nullptr)));
transport_type = "SSL";
break;
case Assignment::TCP:
- connection_manager_->AddTransport(base::WrapUnique(
- new TCPClientTransport(assignment.engine_endpoint, nullptr)));
+ connection_manager_->AddTransport(base::WrapUnique(new TCPClientTransport(
+ assignment.engine_endpoint, connection_statistics_.get(), nullptr)));
transport_type = "TCP";
break;
case Assignment::UNKNOWN:
@@ -166,13 +173,14 @@ BlimpClientSession::BlimpClientSession(const GURL& assigner_endpoint)
render_widget_feature_(new RenderWidgetFeature),
settings_feature_(new SettingsFeature),
weak_factory_(this) {
- net_components_.reset(new ClientNetworkComponents(
- base::WrapUnique(new CrossThreadNetworkEventObserver(
- weak_factory_.GetWeakPtr(),
- base::SequencedTaskRunnerHandle::Get()))));
base::Thread::Options options;
options.message_loop_type = base::MessageLoop::TYPE_IO;
io_thread_.StartWithOptions(options);
+ blimp_connection_statistics_ = new BlimpConnectionStatistics();
+ net_components_.reset(new ClientNetworkComponents(
+ base::WrapUnique(new CrossThreadNetworkEventObserver(
+ weak_factory_.GetWeakPtr(), base::SequencedTaskRunnerHandle::Get())),
+ base::WrapUnique(blimp_connection_statistics_)));
assignment_source_.reset(new AssignmentSource(
assigner_endpoint, io_thread_.task_runner(), io_thread_.task_runner()));
@@ -278,5 +286,10 @@ SettingsFeature* BlimpClientSession::GetSettingsFeature() const {
return settings_feature_.get();
}
+BlimpConnectionStatistics* BlimpClientSession::GetBlimpConnectionStatistics()
+ const {
+ return blimp_connection_statistics_;
+}
+
} // namespace client
} // namespace blimp
« no previous file with comments | « blimp/client/session/blimp_client_session.h ('k') | blimp/engine/session/blimp_engine_session.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698