Add use counter for offerToReceiveAudio/offerToReceiveVideo

third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.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 482 matching lines @@
 }
 
 ScriptPromise RTCPeerConnection::createOffer(ScriptState* scriptState, const RTCOfferOptions& options)
 {
     if (m_signalingState == SignalingStateClosed)
         return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(InvalidStateError, kSignalingStateClosedMessage));
 
     ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
     ScriptPromise promise = resolver->promise();
     RTCSessionDescriptionRequest* request = RTCSessionDescriptionRequestPromiseImpl::create(this, resolver);
+    if (options.hasOfferToReceiveAudio())
+        UseCounter::count(scriptState->getExecutionContext(), UseCounter::RTCPeerConnectionCreateOfferOptionsOfferToReceiveAudio);

[Guido Urdaneta, 2016/09/09 12:07:48]

Would it be better to have a single counter and count requests where any of the
two options are used?

[foolip, 2016/09/09 12:17:00]

Done.

+    if (options.hasOfferToReceiveVideo())
+        UseCounter::count(scriptState->getExecutionContext(), UseCounter::RTCPeerConnectionCreateOfferOptionsOfferToReceiveVideo);
     m_peerHandler->createOffer(request, convertToWebRTCOfferOptions(options));
     return promise;
 }
 
 ScriptPromise RTCPeerConnection::createOffer(ScriptState* scriptState, RTCSessionDescriptionCallback* successCallback, RTCPeerConnectionErrorCallback* errorCallback, const Dictionary& rtcOfferOptions)
 {
     DCHECK(successCallback);
     DCHECK(errorCallback);
     ExecutionContext* context = scriptState->getExecutionContext();
     UseCounter::count(context, UseCounter::RTCPeerConnectionCreateOfferLegacyFailureCallback);
@@ skipping 762 matching lines @@
 {
     visitor->trace(m_localStreams);
     visitor->trace(m_remoteStreams);
     visitor->trace(m_dispatchScheduledEventRunner);
     visitor->trace(m_scheduledEvents);
     EventTargetWithInlineData::trace(visitor);
     ActiveDOMObject::trace(visitor);
 }
 
 } // namespace blink