Changed defaults for CreateAnswer in non-constraint mode

webrtc/api/webrtcsession_unittest.cc

 /*
  *  Copyright 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 500 matching lines @@
     if (data_channel_ && session_->data_channel_type() == cricket::DCT_RTP) {
       session_options->AddSendStream(cricket::MEDIA_TYPE_DATA,
                                      data_channel_->label(),
                                      data_channel_->label());
     }
   }
 
   void GetOptionsForOffer(
       const PeerConnectionInterface::RTCOfferAnswerOptions& rtc_options,
       cricket::MediaSessionOptions* session_options) {
-    ASSERT_TRUE(ExtractMediaSessionOptions(rtc_options, session_options));
+    ASSERT_TRUE(ExtractMediaSessionOptions(rtc_options, true, session_options));

[perkj_webrtc, 2016/03/10 15:35:24]

Hum. weird that ExtractMediaSessionOptions has been moved to peerconnection.cc
but not the tests....  Looks like all these tests are in the wrong place after a
refactoring... Anyway - not  in this cl..... 

[hta-webrtc, 2016/03/10 16:22:05]

Yep, I wondered if I could stop exporting it at all, then this happened... it
"smells" like an internal helper function over there...

 
     AddStreamsToOptions(session_options);
     if (rtc_options.offer_to_receive_audio ==
         RTCOfferAnswerOptions::kUndefined) {
       session_options->recv_audio =
           session_options->HasSendMediaStream(cricket::MEDIA_TYPE_AUDIO);
     }
     if (rtc_options.offer_to_receive_video ==
         RTCOfferAnswerOptions::kUndefined) {
       session_options->recv_video =
@@ skipping 3755 matching lines @@
 }
 
 // TODO(bemasc): Add a TestIceStatesBundle with BUNDLE enabled.  That test
 // currently fails because upon disconnection and reconnection OnIceComplete is
 // called more than once without returning to IceGatheringGathering.
 
 INSTANTIATE_TEST_CASE_P(WebRtcSessionTests,
                         WebRtcSessionTest,
                         testing::Values(ALREADY_GENERATED,
                                         DTLS_IDENTITY_STORE));