Create a private API for controlling WebRTC's AEC3

chrome/browser/resources/hangout_services/thunk.js

Index: chrome/browser/resources/hangout_services/thunk.js
diff --git a/chrome/browser/resources/hangout_services/thunk.js b/chrome/browser/resources/hangout_services/thunk.js
index 68c28bf7dc51a5a1b8a4983c1516ba7e0bb889ca..2278a65d129a8afcd13db67edb37909e53018095 100644
--- a/chrome/browser/resources/hangout_services/thunk.js
+++ b/chrome/browser/resources/hangout_services/thunk.js
@@ -202,6 +202,12 @@ chrome.runtime.onMessageExternal.addListener(
           chrome.webrtcLoggingPrivate.stopWebRtcEventLogging(
               requestInfo, origin, doSendResponse);
           return true;
+        } else if (method == 'setAudioExperiments') {
+          var experiments = message['experiments'];
+          chrome.webrtcAudioPrivate.setAudioExperiments(
+              requestInfo, origin, experiments);

[mnilsson, 2017/04/10 19:15:45]

Why not embed the doSendResponse as a callback method to setAudioExperiments
(like other methods in this file)? Wouldn't it otherwise be racy to know when
the experiments have actually been set?

[Henrik Grunell, 2017/04/11 06:49:39]

If the application needs to know exactly when it has been set then yes. (Or if
success/failure information is needed.) I don't see why that would be necessary
though. If it is, then by all means there should be a callback.

+          doSendResponse();
+          return false;
         }
 
         throw new Error('Unknown method: ' + method);