Let Contraints Controll Mute/Unmute Audio Local Playback For Desktop Sharing

third_party/WebKit/Source/modules/mediastream/MediaConstraintsImpl.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 73 matching lines @@
 // From content/common/media/media_stream_options.cc
 const char kMediaStreamSource[] = "chromeMediaSource";
 const char kMediaStreamSourceId[] = "chromeMediaSourceId"; // mapped to deviceId
 const char kMediaStreamSourceInfoId[] = "sourceId"; // mapped to deviceId
 const char kMediaStreamRenderToAssociatedSink[] = "chromeRenderToAssociatedSink";
 // RenderToAssociatedSink will be going away in M50-M60 some time.
 const char kMediaStreamAudioHotword[] = "googHotword";
 // TODO(hta): googHotword should go away. https://crbug.com/577627
 // From content/renderer/media/media_stream_audio_processor_options.cc
 const char kEchoCancellation[] = "echoCancellation";
+const char kGoogDisableLocalEcho[] = "googDisableLocalEcho";

[Mike West, 2016/09/27 08:03:41]

Is this web-exposed? That is, does a developer type this into JavaScript, or is
this just an internal name?

If it's web-exposed, then the name makes me sad (we should not be shipping new,
vendor-prefixed APIs), and there are no tests.

If it's not web-exposed, then I'm less concerned about the name, but I still
don't see any tests. You suggested that the implementation is in another CL: are
the tests there too?

[qiangchen, 2016/09/27 16:16:47]

It is web exposed. But it is not an API function name, just an entry in audio
constraint. 

For example:
getUserMedia({audio: {googDisableLocalEcho: true}, ......})

We want to give the developer a freedom to override default behavior and choose
whether stop local playback or not during sharing. But we did not find there is
such a parameter defined in W3C. If this is not allowed, then we have to give up
this CL.

We do have tests for the real functionality. But those tests have no idea of
googDisableLocalEcho, and in those tests we pass the bool parameter directly
into the class processing audio stream, and then we test whether it behaves as
expected. 

For here, I think the test you mean is a test that verifies that we can pass
googDisableLocalEcho from Javascript to the audio stream generating class,
right?

[Avi (use Gerrit), 2016/09/27 16:20:34]

Waaaaaait. I see lots of "googXXXX" constants in this file, around this line.

Are *all* of them web-exposed?

[qiangchen, 2016/09/27 16:32:49]

Yes. You can search them on google, and you will find the result is the
discussion of their usage. And the way to use them is much the same as 
getUserMedia({audio: {googXXXX: <value>}} ......)

 const char kGoogEchoCancellation[] = "googEchoCancellation";
 const char kGoogExperimentalEchoCancellation[] = "googEchoCancellation2";
 const char kGoogAutoGainControl[] = "googAutoGainControl";
 const char kGoogExperimentalAutoGainControl[] = "googAutoGainControl2";
 const char kGoogNoiseSuppression[] = "googNoiseSuppression";
 const char kGoogExperimentalNoiseSuppression[] = "googNoiseSuppression2";
 const char kGoogBeamforming[] = "googBeamforming";
 const char kGoogArrayGeometry[] = "googArrayGeometry";
 const char kGoogHighpassFilter[] = "googHighpassFilter";
 const char kGoogTypingNoiseDetection[] = "googTypingNoiseDetection";
@@ skipping 183 matching lines @@
             result.frameRate.setMin(atof(constraint.m_value.utf8().c_str()));
         } else if (constraint.m_name.equals(kMaxFrameRate)) {
             result.frameRate.setMax(atof(constraint.m_value.utf8().c_str()));
         } else if (constraint.m_name.equals(kEchoCancellation)) {
             result.echoCancellation.setExact(toBoolean(constraint.m_value));
         } else if (constraint.m_name.equals(kMediaStreamSource)) {
             // TODO(hta): This has only a few legal values. Should be
             // represented as an enum, and cause type errors.
             // https://crbug.com/576582
             result.mediaStreamSource.setExact(constraint.m_value);
+        } else if (constraint.m_name.equals(kGoogDisableLocalEcho)) {
+            result.googDisableLocalEcho.setExact(toBoolean(constraint.m_value));
         } else if (constraint.m_name.equals(kMediaStreamSourceId)
             || constraint.m_name.equals(kMediaStreamSourceInfoId)) {
             result.deviceId.setExact(constraint.m_value);
         } else if (constraint.m_name.equals(kMediaStreamRenderToAssociatedSink)) {
             // TODO(hta): This is a boolean represented as string.
             // Should give TypeError when it's not parseable.
             // https://crbug.com/576582
             result.renderToAssociatedSink.setExact(toBoolean(constraint.m_value));
         } else if (constraint.m_name.equals(kMediaStreamAudioHotword)) {
             result.hotwordEnabled.setExact(toBoolean(constraint.m_value));
@@ skipping 458 matching lines @@
         MediaTrackConstraintSet element;
         convertConstraintSet(it, element);
         advancedVector.append(element);
     }
     if (!advancedVector.isEmpty())
         output.setAdvanced(advancedVector);
 }
 
 } // namespace MediaConstraintsImpl
 } // namespace blink