Get the device ID in getSettings()

third_party/WebKit/Source/platform/mediastream/MediaStreamComponent.cpp

 /*
  * Copyright (C) 2011 Ericsson AB. All rights reserved.
  * Copyright (C) 2013 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 76 matching lines @@
 
 void MediaStreamComponent::AudioSourceProviderImpl::wrap(
     WebAudioSourceProvider* provider) {
   MutexLocker locker(m_provideInputLock);
   m_webAudioSourceProvider = provider;
 }
 
 void MediaStreamComponent::getSettings(
     WebMediaStreamTrack::Settings& settings) {
   DCHECK(m_trackData);
+  m_source->getSettings(settings);
   m_trackData->getSettings(settings);

[miu, 2017/01/31 01:23:20]

There was the earlier change to WebMediaStreamTrack::ExtraData as well. Are you
planning on addressing that in a future change?

https://cs.chromium.org/chromium/src/third_party/WebKit/public/platform/WebMe...

[hta - Chromium, 2017/02/06 01:33:10]

I am staring at (in another CL) whether or not that pattern needs reversing -
there's a pattern already going "the other way" with regard to audio devices,
but utilizing that requires more thought than I thought, and the particular
change to WebMediaStreamTack::extraData -> trackData may actually still make
sense.

Let's take that in the review of that CL.

 }
 
 void MediaStreamComponent::setContentHint(
     WebMediaStreamTrack::ContentHintType hint) {
   switch (hint) {
     case WebMediaStreamTrack::ContentHintType::None:
       break;
     case WebMediaStreamTrack::ContentHintType::AudioSpeech:
     case WebMediaStreamTrack::ContentHintType::AudioMusic:
       DCHECK_EQ(MediaStreamSource::TypeAudio, source()->type());
@@ skipping 30 matching lines @@
     webAudioData[i] = bus->channel(i)->mutableData();
 
   m_webAudioSourceProvider->provideInput(webAudioData, framesToProcess);
 }
 
 DEFINE_TRACE(MediaStreamComponent) {
   visitor->trace(m_source);
 }
 
 }  // namespace blink