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

Unified Diff: remoting/client/plugin/pepper_port_allocator.cc

Issue 13145003: Rewrite std::string("") to std::string(), Linux edition. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Ugh Created 7 years, 8 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 | « remoting/client/plugin/chromoting_instance.cc ('k') | remoting/host/desktop_process_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/client/plugin/pepper_port_allocator.cc
diff --git a/remoting/client/plugin/pepper_port_allocator.cc b/remoting/client/plugin/pepper_port_allocator.cc
index 62a5822e9a1e4fa90cdeb390a991bd959e0fd686..423102a0d75fb910a14aef1d7e0029a16d85364d 100644
--- a/remoting/client/plugin/pepper_port_allocator.cc
+++ b/remoting/client/plugin/pepper_port_allocator.cc
@@ -82,9 +82,15 @@ PepperPortAllocatorSession::PepperPortAllocatorSession(
const std::vector<std::string>& relay_hosts,
const std::string& relay_token,
const pp::InstanceHandle& instance)
- : HttpPortAllocatorSessionBase(
- allocator, content_name, component, ice_username_fragment, ice_password,
- stun_hosts, relay_hosts, relay_token, ""),
+ : HttpPortAllocatorSessionBase(allocator,
+ content_name,
+ component,
+ ice_username_fragment,
+ ice_password,
+ stun_hosts,
+ relay_hosts,
+ relay_token,
+ std::string()),
instance_(instance),
stun_address_resolver_(instance_),
stun_port_(0),
@@ -129,7 +135,7 @@ void PepperPortAllocatorSession::GetPortConfigurations() {
// Add an empty configuration synchronously, so a local connection
// can be started immediately.
ConfigReady(new cricket::PortConfiguration(
- talk_base::SocketAddress(), "", ""));
+ talk_base::SocketAddress(), std::string(), std::string()));
ResolveStunServerAddress();
TryCreateRelaySession();
@@ -195,7 +201,8 @@ void PepperPortAllocatorSession::OnStunAddressResolved(int32_t result) {
ReceiveSessionResponse(std::string(relay_response_body_.begin(),
relay_response_body_.end()));
} else {
- ConfigReady(new cricket::PortConfiguration(stun_address_, "", ""));
+ ConfigReady(new cricket::PortConfiguration(
+ stun_address_, std::string(), std::string()));
}
}
@@ -304,7 +311,9 @@ PepperPortAllocator::PepperPortAllocator(
const pp::InstanceHandle& instance,
scoped_ptr<talk_base::NetworkManager> network_manager,
scoped_ptr<talk_base::PacketSocketFactory> socket_factory)
- : HttpPortAllocatorBase(network_manager.get(), socket_factory.get(), ""),
+ : HttpPortAllocatorBase(network_manager.get(),
+ socket_factory.get(),
+ std::string()),
instance_(instance),
network_manager_(network_manager.Pass()),
socket_factory_(socket_factory.Pass()) {
« no previous file with comments | « remoting/client/plugin/chromoting_instance.cc ('k') | remoting/host/desktop_process_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698