diff --git a/PRESUBMIT.py b/PRESUBMIT.py index be2626963a..9747d9160b 100755 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -18,6 +18,7 @@ import sys CPPLINT_DIRS = [ 'webrtc/audio', 'webrtc/call', + 'webrtc/examples', 'webrtc/modules/video_processing', 'webrtc/video', ] diff --git a/webrtc/examples/peerconnection/client/conductor.cc b/webrtc/examples/peerconnection/client/conductor.cc index e3def9955f..c675026bc7 100644 --- a/webrtc/examples/peerconnection/client/conductor.cc +++ b/webrtc/examples/peerconnection/client/conductor.cc @@ -122,9 +122,7 @@ bool Conductor::CreatePeerConnection(bool dtls) { if (dtls) { constraints.AddOptional(webrtc::MediaConstraintsInterface::kEnableDtlsSrtp, "true"); - } - else - { + } else { constraints.AddOptional(webrtc::MediaConstraintsInterface::kEnableDtlsSrtp, "false"); } diff --git a/webrtc/examples/peerconnection/client/conductor.h b/webrtc/examples/peerconnection/client/conductor.h index f5f16a3d10..e5ee170299 100644 --- a/webrtc/examples/peerconnection/client/conductor.h +++ b/webrtc/examples/peerconnection/client/conductor.h @@ -8,8 +8,8 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef TALK_EXAMPLES_PEERCONNECTION_CLIENT_CONDUCTOR_H_ -#define TALK_EXAMPLES_PEERCONNECTION_CLIENT_CONDUCTOR_H_ +#ifndef WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_CONDUCTOR_H_ +#define WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_CONDUCTOR_H_ #pragma once #include @@ -126,4 +126,4 @@ class Conductor std::string server_; }; -#endif // TALK_EXAMPLES_PEERCONNECTION_CLIENT_CONDUCTOR_H_ +#endif // WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_CONDUCTOR_H_ diff --git a/webrtc/examples/peerconnection/client/defaults.cc b/webrtc/examples/peerconnection/client/defaults.cc index f70aa72e3d..a2501c718f 100644 --- a/webrtc/examples/peerconnection/client/defaults.cc +++ b/webrtc/examples/peerconnection/client/defaults.cc @@ -50,10 +50,12 @@ std::string GetDefaultServerName() { std::string GetPeerName() { char computer_name[256]; - if (gethostname(computer_name, arraysize(computer_name)) != 0) - strcpy(computer_name, "host"); std::string ret(GetEnvVarOrDefault("USERNAME", "user")); ret += '@'; - ret += computer_name; + if (gethostname(computer_name, arraysize(computer_name)) == 0) { + ret += computer_name; + } else { + ret += "host"; + } return ret; } diff --git a/webrtc/examples/peerconnection/client/defaults.h b/webrtc/examples/peerconnection/client/defaults.h index 7b503974e5..f4d3bf52f7 100644 --- a/webrtc/examples/peerconnection/client/defaults.h +++ b/webrtc/examples/peerconnection/client/defaults.h @@ -8,8 +8,8 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef PEERCONNECTION_SAMPLES_CLIENT_DEFAULTS_H_ -#define PEERCONNECTION_SAMPLES_CLIENT_DEFAULTS_H_ +#ifndef WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_DEFAULTS_H_ +#define WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_DEFAULTS_H_ #pragma once #include @@ -27,4 +27,4 @@ std::string GetPeerConnectionString(); std::string GetDefaultServerName(); std::string GetPeerName(); -#endif // PEERCONNECTION_SAMPLES_CLIENT_DEFAULTS_H_ +#endif // WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_DEFAULTS_H_ diff --git a/webrtc/examples/peerconnection/client/flagdefs.h b/webrtc/examples/peerconnection/client/flagdefs.h index 0cffffb135..92e2773166 100644 --- a/webrtc/examples/peerconnection/client/flagdefs.h +++ b/webrtc/examples/peerconnection/client/flagdefs.h @@ -8,8 +8,8 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef TALK_EXAMPLES_PEERCONNECTION_CLIENT_FLAGDEFS_H_ -#define TALK_EXAMPLES_PEERCONNECTION_CLIENT_FLAGDEFS_H_ +#ifndef WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_FLAGDEFS_H_ +#define WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_FLAGDEFS_H_ #pragma once #include "webrtc/base/flags.h" @@ -30,4 +30,4 @@ DEFINE_bool(autocall, false, "Call the first available other client on " "the server without user intervention. Note: this flag should only be set " "to true on one of the two clients."); -#endif // TALK_EXAMPLES_PEERCONNECTION_CLIENT_FLAGDEFS_H_ +#endif // WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_FLAGDEFS_H_ diff --git a/webrtc/examples/peerconnection/client/linux/main.cc b/webrtc/examples/peerconnection/client/linux/main.cc index cf88c36fbb..4db929c82e 100644 --- a/webrtc/examples/peerconnection/client/linux/main.cc +++ b/webrtc/examples/peerconnection/client/linux/main.cc @@ -30,7 +30,7 @@ class CustomSocketServer : public rtc::PhysicalSocketServer { // Override so that we can also pump the GTK message loop. virtual bool Wait(int cms, bool process_io) { // Pump GTK events. - // TODO: We really should move either the socket server or UI to a + // TODO(henrike): We really should move either the socket server or UI to a // different thread. Alternatively we could look at merging the two loops // by implementing a dispatcher for the socket server and/or use // g_main_context_set_poll_func. @@ -96,10 +96,12 @@ int main(int argc, char* argv[]) { wnd.Destroy(); thread->set_socketserver(NULL); - // TODO: Run the Gtk main loop to tear down the connection. - //while (gtk_events_pending()) { - // gtk_main_iteration(); - //} + // TODO(henrike): Run the Gtk main loop to tear down the connection. + /* + while (gtk_events_pending()) { + gtk_main_iteration(); + } + */ rtc::CleanupSSL(); return 0; } diff --git a/webrtc/examples/peerconnection/client/linux/main_wnd.cc b/webrtc/examples/peerconnection/client/linux/main_wnd.cc index 254fb946f9..cf98c1cac7 100644 --- a/webrtc/examples/peerconnection/client/linux/main_wnd.cc +++ b/webrtc/examples/peerconnection/client/linux/main_wnd.cc @@ -116,7 +116,8 @@ gboolean Redraw(gpointer data) { wnd->OnRedraw(); return false; } -} // end anonymous + +} // namespace // // GtkMainWnd implementation. @@ -174,7 +175,8 @@ void GtkMainWnd::StopLocalRenderer() { local_renderer_.reset(); } -void GtkMainWnd::StartRemoteRenderer(webrtc::VideoTrackInterface* remote_video) { +void GtkMainWnd::StartRemoteRenderer( + webrtc::VideoTrackInterface* remote_video) { remote_renderer_.reset(new VideoRenderer(this, remote_video)); } @@ -488,7 +490,7 @@ void GtkMainWnd::VideoRenderer::RenderFrame( static_cast(frame->GetHeight())); int size = width_ * height_ * 4; - // TODO: Convert directly to RGBA + // TODO(henrike): Convert directly to RGBA frame->ConvertToRgbBuffer(cricket::FOURCC_ARGB, image_.get(), size, diff --git a/webrtc/examples/peerconnection/client/linux/main_wnd.h b/webrtc/examples/peerconnection/client/linux/main_wnd.h index 1a91082768..e35d4dd8fa 100644 --- a/webrtc/examples/peerconnection/client/linux/main_wnd.h +++ b/webrtc/examples/peerconnection/client/linux/main_wnd.h @@ -8,8 +8,10 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef PEERCONNECTION_SAMPLES_CLIENT_LINUX_MAIN_WND_H_ -#define PEERCONNECTION_SAMPLES_CLIENT_LINUX_MAIN_WND_H_ +#ifndef WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_LINUX_MAIN_WND_H_ +#define WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_LINUX_MAIN_WND_H_ + +#include #include "webrtc/examples/peerconnection/client/main_wnd.h" #include "webrtc/examples/peerconnection/client/peer_connection_client.h" @@ -115,4 +117,4 @@ class GtkMainWnd : public MainWindow { int draw_buffer_size_; }; -#endif // PEERCONNECTION_SAMPLES_CLIENT_LINUX_MAIN_WND_H_ +#endif // WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_LINUX_MAIN_WND_H_ diff --git a/webrtc/examples/peerconnection/client/main_wnd.h b/webrtc/examples/peerconnection/client/main_wnd.h index 9f61a568fd..ac4fd8a9b9 100644 --- a/webrtc/examples/peerconnection/client/main_wnd.h +++ b/webrtc/examples/peerconnection/client/main_wnd.h @@ -8,8 +8,8 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef PEERCONNECTION_SAMPLES_CLIENT_MAIN_WND_H_ -#define PEERCONNECTION_SAMPLES_CLIENT_MAIN_WND_H_ +#ifndef WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_MAIN_WND_H_ +#define WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_MAIN_WND_H_ #pragma once #include @@ -60,7 +60,8 @@ class MainWindow { virtual void StartLocalRenderer(webrtc::VideoTrackInterface* local_video) = 0; virtual void StopLocalRenderer() = 0; - virtual void StartRemoteRenderer(webrtc::VideoTrackInterface* remote_video) = 0; + virtual void StartRemoteRenderer( + webrtc::VideoTrackInterface* remote_video) = 0; virtual void StopRemoteRenderer() = 0; virtual void QueueUIThreadCallback(int msg_id, void* data) = 0; @@ -197,4 +198,4 @@ class MainWnd : public MainWindow { }; #endif // WIN32 -#endif // PEERCONNECTION_SAMPLES_CLIENT_MAIN_WND_H_ +#endif // WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_MAIN_WND_H_ diff --git a/webrtc/examples/peerconnection/client/peer_connection_client.cc b/webrtc/examples/peerconnection/client/peer_connection_client.cc index ab6f6abdf6..9875115c4b 100644 --- a/webrtc/examples/peerconnection/client/peer_connection_client.cc +++ b/webrtc/examples/peerconnection/client/peer_connection_client.cc @@ -43,7 +43,7 @@ rtc::AsyncSocket* CreateClientSocket(int family) { #endif } -} +} // namespace PeerConnectionClient::PeerConnectionClient() : callback_(NULL), diff --git a/webrtc/examples/peerconnection/client/peer_connection_client.h b/webrtc/examples/peerconnection/client/peer_connection_client.h index 5b5787bc14..b7abfdfe18 100644 --- a/webrtc/examples/peerconnection/client/peer_connection_client.h +++ b/webrtc/examples/peerconnection/client/peer_connection_client.h @@ -8,8 +8,8 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef PEERCONNECTION_SAMPLES_CLIENT_PEER_CONNECTION_CLIENT_H_ -#define PEERCONNECTION_SAMPLES_CLIENT_PEER_CONNECTION_CLIENT_H_ +#ifndef WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_PEER_CONNECTION_CLIENT_H_ +#define WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_PEER_CONNECTION_CLIENT_H_ #pragma once #include @@ -120,4 +120,4 @@ class PeerConnectionClient : public sigslot::has_slots<>, int my_id_; }; -#endif // PEERCONNECTION_SAMPLES_CLIENT_PEER_CONNECTION_CLIENT_H_ +#endif // WEBRTC_EXAMPLES_PEERCONNECTION_CLIENT_PEER_CONNECTION_CLIENT_H_ diff --git a/webrtc/examples/peerconnection/server/data_socket.h b/webrtc/examples/peerconnection/server/data_socket.h index 454ad3978a..0ef61ea6aa 100644 --- a/webrtc/examples/peerconnection/server/data_socket.h +++ b/webrtc/examples/peerconnection/server/data_socket.h @@ -8,8 +8,8 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef TALK_EXAMPLES_PEERCONNECTION_SERVER_DATA_SOCKET_H_ -#define TALK_EXAMPLES_PEERCONNECTION_SERVER_DATA_SOCKET_H_ +#ifndef WEBRTC_EXAMPLES_PEERCONNECTION_SERVER_DATA_SOCKET_H_ +#define WEBRTC_EXAMPLES_PEERCONNECTION_SERVER_DATA_SOCKET_H_ #pragma once #ifdef WIN32 @@ -150,4 +150,4 @@ class ListeningSocket : public SocketBase { DataSocket* Accept() const; }; -#endif // TALK_EXAMPLES_PEERCONNECTION_SERVER_DATA_SOCKET_H_ +#endif // WEBRTC_EXAMPLES_PEERCONNECTION_SERVER_DATA_SOCKET_H_ diff --git a/webrtc/examples/peerconnection/server/peer_channel.cc b/webrtc/examples/peerconnection/server/peer_channel.cc index 150e5dec97..5e173cd460 100644 --- a/webrtc/examples/peerconnection/server/peer_channel.cc +++ b/webrtc/examples/peerconnection/server/peer_channel.cc @@ -19,6 +19,7 @@ #include "webrtc/examples/peerconnection/server/data_socket.h" #include "webrtc/examples/peerconnection/server/utils.h" #include "webrtc/base/stringutils.h" +#include "webrtc/base/urlencode.h" using rtc::sprintfn; @@ -59,7 +60,7 @@ ChannelMember::ChannelMember(DataSocket* socket) assert(socket); assert(socket->method() == DataSocket::GET); assert(socket->PathEquals("/sign_in")); - name_ = socket->request_arguments(); // TODO: urldecode + name_ = rtc::UrlDecodeString(socket->request_arguments()); if (name_.empty()) name_ = "peer_" + int2str(id_); else if (name_.length() > kMaxNameLength) diff --git a/webrtc/examples/peerconnection/server/peer_channel.h b/webrtc/examples/peerconnection/server/peer_channel.h index 263f17dfa8..6fd740d2f9 100644 --- a/webrtc/examples/peerconnection/server/peer_channel.h +++ b/webrtc/examples/peerconnection/server/peer_channel.h @@ -8,8 +8,8 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef TALK_EXAMPLES_PEERCONNECTION_SERVER_PEER_CHANNEL_H_ -#define TALK_EXAMPLES_PEERCONNECTION_SERVER_PEER_CHANNEL_H_ +#ifndef WEBRTC_EXAMPLES_PEERCONNECTION_SERVER_PEER_CHANNEL_H_ +#define WEBRTC_EXAMPLES_PEERCONNECTION_SERVER_PEER_CHANNEL_H_ #pragma once #include @@ -117,4 +117,4 @@ class PeerChannel { Members members_; }; -#endif // TALK_EXAMPLES_PEERCONNECTION_SERVER_PEER_CHANNEL_H_ +#endif // WEBRTC_EXAMPLES_PEERCONNECTION_SERVER_PEER_CHANNEL_H_ diff --git a/webrtc/examples/peerconnection/server/utils.h b/webrtc/examples/peerconnection/server/utils.h index e70968b875..e1c8729c0b 100644 --- a/webrtc/examples/peerconnection/server/utils.h +++ b/webrtc/examples/peerconnection/server/utils.h @@ -8,8 +8,8 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef TALK_EXAMPLES_PEERCONNECTION_SERVER_UTILS_H_ -#define TALK_EXAMPLES_PEERCONNECTION_SERVER_UTILS_H_ +#ifndef WEBRTC_EXAMPLES_PEERCONNECTION_SERVER_UTILS_H_ +#define WEBRTC_EXAMPLES_PEERCONNECTION_SERVER_UTILS_H_ #pragma once #include @@ -22,4 +22,4 @@ std::string int2str(int i); std::string size_t2str(size_t i); -#endif // TALK_EXAMPLES_PEERCONNECTION_SERVER_UTILS_H_ +#endif // WEBRTC_EXAMPLES_PEERCONNECTION_SERVER_UTILS_H_ diff --git a/webrtc/examples/stunserver/stunserver_main.cc b/webrtc/examples/stunserver/stunserver_main.cc index 9cbd6156da..9bdf58ac3f 100644 --- a/webrtc/examples/stunserver/stunserver_main.cc +++ b/webrtc/examples/stunserver/stunserver_main.cc @@ -17,7 +17,7 @@ #include "webrtc/p2p/base/stunserver.h" #include "webrtc/base/thread.h" -using namespace cricket; +using cricket::StunServer; int main(int argc, char* argv[]) { if (argc != 2) {