Rename RTCPeerConnection.updateIce to setConfiguration and make it work.

third_party/WebKit/public/platform/WebRTCPeerConnectionHandler.h

Index: third_party/WebKit/public/platform/WebRTCPeerConnectionHandler.h
diff --git a/third_party/WebKit/public/platform/WebRTCPeerConnectionHandler.h b/third_party/WebKit/public/platform/WebRTCPeerConnectionHandler.h
index 285bcb6b657629f8942da32b6681712fb43d774a..e3408d5122dc57725ce53217fa953bdc24c7872c 100644
--- a/third_party/WebKit/public/platform/WebRTCPeerConnectionHandler.h
+++ b/third_party/WebKit/public/platform/WebRTCPeerConnectionHandler.h
@@ -41,6 +41,7 @@ class WebMediaStreamTrack;
 class WebRTCAnswerOptions;
 class WebRTCDTMFSenderHandler;
 class WebRTCDataChannelHandler;
+class WebRTCError;
 class WebRTCICECandidate;
 class WebRTCOfferOptions;
 class WebRTCSessionDescription;
@@ -72,7 +73,7 @@ class WebRTCPeerConnectionHandler {
                                     const WebRTCSessionDescription&) = 0;
   virtual WebRTCSessionDescription localDescription() = 0;
   virtual WebRTCSessionDescription remoteDescription() = 0;
-  virtual bool setConfiguration(const WebRTCConfiguration&) = 0;
+  virtual bool setConfiguration(const WebRTCConfiguration&, WebRTCError&) = 0;

[dcheng, 2017/01/13 23:10:38]

My personal feeling is it'd feel natural for this to just return WebRTCErrorType
directly, rather than having to keep the bool and error enum in sync. WDYT?

[Taylor_Brandstetter, 2017/01/14 00:08:40]

You're right. It's an output parameter in webrtc mainly for backwards
compatibility, but we don't have that restriction here. Done.

 
   // DEPRECATED
   virtual bool addICECandidate(const WebRTCICECandidate&) { return false; }