Change VideoSourceInterface::needs_denoising() to return rtc::Optional<bool>

webrtc/api/mediastreaminterface.h

Index: webrtc/api/mediastreaminterface.h
diff --git a/webrtc/api/mediastreaminterface.h b/webrtc/api/mediastreaminterface.h
index 1ea094b71454aaa2ff337b49a4da9534d3bdc1f3..d2fe9ecee43e0770109a6e7b55449fab3e448e9b 100644
--- a/webrtc/api/mediastreaminterface.h
+++ b/webrtc/api/mediastreaminterface.h
@@ -23,6 +23,7 @@
 #include "webrtc/base/basictypes.h"
 #include "webrtc/base/refcount.h"
 #include "webrtc/base/scoped_ref_ptr.h"
+#include "webrtc/base/optional.h"
 #include "webrtc/media/base/mediachannel.h"
 #include "webrtc/media/base/videosinkinterface.h"
 #include "webrtc/media/base/videosourceinterface.h"
@@ -127,7 +128,7 @@ class VideoTrackSourceInterface
   // Indicates that the encoder should denoise the video before encoding it.
   // TODO(perkj): Remove this once denoising is done by the source, and not by
   // the encoder.
-  virtual bool needs_denoising() const = 0;
+  virtual rtc::Optional<bool> needs_denoising() const = 0;

[pbos-webrtc, 2016/03/31 14:30:08]

Can we put a comment here on what unset means?

[perkj_webrtc, 2016/03/31 15:23:08]

Done.

 
  protected:
   virtual ~VideoTrackSourceInterface() {}