Change WebRTC IP handling policy from multiple booleans to an enum.

content/public/common/webrtc_ip_handling_policy.h

+// Copyright (c) 2015 The Chromium Authors. All rights reserved.

[nasko, 2015/11/03 18:54:01]

No (c)

[guoweis_webrtc, 2015/11/03 23:19:22]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CONTENT_PUBLIC_RENDERER_WEBRTC_IP_HANDLING_POLICY_H_
+#define CONTENT_PUBLIC_RENDERER_WEBRTC_IP_HANDLING_POLICY_H_
+
+#include "content/common/content_export.h"
+
+namespace content {
+
+// This is the default behavior of Chrome. Currently, WebRTC has the right to
+// enumerate all interfaces and bind them to discover public interfaces.
+CONTENT_EXPORT extern const char kWebRTCIPHandlingDefault[];
+
+// WebRTC should only use the default route used by http. This also exposes the
+// associated default private address.
+CONTENT_EXPORT extern const char
+    kWebRTCIPHandlingDefaultPublicAndPrivateInterfaces[];

[nasko, 2015/11/03 18:54:01]

Does the "Default" part add meaning to the flag? Is
kWebRTCIPHandlingPublicAndPrivateInterfaces not sufficient enough? The same
question about the flag below.

[guoweis_webrtc, 2015/11/03 23:19:22]

Yes, default here is the one interface chosen by OS for http traffic. There
could be multiple interfaces all with internet access on a multi-homed machine.

+
+// WebRTC should only use the default route used by http. This doesn't expose
+// any local addresses.
+CONTENT_EXPORT extern const char kWebRTCIPHandlingDefaultPublicInterfaceOnly[];
+
+// WebRTC should only use TCP to contact peers or servers unless the proxy
+// server supports UDP. This doesn't expose any local addresses either.
+CONTENT_EXPORT extern const char kWebRTCIPHandlingDisableNonProxiedUdp[];
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_RENDERER_WEBRTC_IP_HANDLING_POLICY_H_