Rolling libjingle to 158:159

content/renderer/media/mock_media_stream_dependency_factory.cc

Index: content/renderer/media/mock_media_stream_dependency_factory.cc
===================================================================
--- content/renderer/media/mock_media_stream_dependency_factory.cc	(revision 146359)
+++ content/renderer/media/mock_media_stream_dependency_factory.cc	(working copy)
@@ -177,6 +177,10 @@
     NOTIMPLEMENTED();
     return "";
   }
+  // TODO(perkj): Implement type().

[Sergey Ulanov, 2012/07/13 01:22:48]

Many other methods in this class are marked as NOTIMPLEMETED()? Is this method
exceptional in any way so we need TODO for it but not for other methods?

[Ronghua Wu (Left Chromium), 2012/07/13 18:26:35]

Not sure about the history, those may not needed by the tests. But the one just
added should be implemented for the testing.

[Sergey Ulanov, 2012/07/13 21:06:24]

Ok, maybe need to open a bug for this?

[Ronghua Wu (Left Chromium), 2012/07/14 00:06:41]

Created crbug/137318 for Per.

+  virtual SdpType type() const OVERRIDE {
+    return kOffer;

[Sergey Ulanov, 2012/07/13 01:22:48]

add NOTIMPLEMENTED().

[Ronghua Wu (Left Chromium), 2012/07/14 00:06:41]

NOTIMPLEMENTED is already added in the last patch.

+  }
   virtual bool AddCandidate(const IceCandidateInterface* candidate) OVERRIDE {
     NOTIMPLEMENTED();
     return false;