Reduce APK size by disabling WebRTC.

remoting/client/chromoting_client.cc

Index: remoting/client/chromoting_client.cc
diff --git a/remoting/client/chromoting_client.cc b/remoting/client/chromoting_client.cc
index 1f7165ab5db91f8a8a5b2f59b467e9bf7f02e8c3..3aefc9fe6988d0484cfae9850631228b798453a9 100644
--- a/remoting/client/chromoting_client.cc
+++ b/remoting/client/chromoting_client.cc
@@ -72,11 +72,11 @@ void ChromotingClient::Start(
   if (!connection_) {
     if (protocol_config_->webrtc_supported()) {
       DCHECK(!protocol_config_->ice_supported());
-#if defined(OS_NACL)
-      LOG(FATAL) << "WebRTC is not supported in webapp.";
-#else   // defined(OS_NACL)
+#if !defined(ENABLE_WEBRTC_REMOTING_CLIENT)   // !defined(ENABLE_WEBRTC_REMOTING_CLIENT)

[Sergey Ulanov, 2016/03/16 23:15:07]

Don't need the comment.

[Yuwei, 2016/03/16 23:31:55]

Acknowledged.

[Yuwei, 2016/03/16 23:58:18]

Done.

+      LOG(FATAL) << "WebRTC is not supported";

[Sergey Ulanov, 2016/03/16 23:15:07]

nit: add . at the end of the message.

[Yuwei, 2016/03/16 23:31:55]

Acknowledged.

[Yuwei, 2016/03/16 23:58:18]

Done.

+#else
       connection_.reset(new protocol::WebrtcConnectionToHost());
-#endif  // !defined(OS_NACL)
+#endif  // defined(ENABLE_WEBRTC_REMOTING_CLIENT)

[Sergey Ulanov, 2016/03/16 23:15:07]

nit: add comment for the #else statement or remove the comment from here. The
comments for short #ifdef blocks like this aren't required (because it's clear
what's going on without them), but we want to be consistent, i.e. you should
either have comments for both #else and #endif or neither.

[Yuwei, 2016/03/16 23:31:55]

Acknowledged.

[Yuwei, 2016/03/16 23:58:18]

Done.

     } else {
       DCHECK(protocol_config_->ice_supported());
       connection_.reset(new protocol::IceConnectionToHost());