Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(628)

Unified Diff: third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp

Issue 2815313002: Reland of Move ScriptState::GetExecutionContext (Part 5) (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp
diff --git a/third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp b/third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp
index 569b12a3a46e1a5ab6ef5f32ea4ba6537a984075..dd5215070c892e992442c229ae8a5929a953ff96 100644
--- a/third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp
+++ b/third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp
@@ -400,7 +400,7 @@
~WebRTCStatsReportCallbackResolver() override {
DCHECK(
- resolver_->GetScriptState()->GetExecutionContext()->IsContextThread());
+ ExecutionContext::From(resolver_->GetScriptState())->IsContextThread());
}
private:
@@ -409,7 +409,7 @@
void OnStatsDelivered(std::unique_ptr<WebRTCStatsReport> report) override {
DCHECK(
- resolver_->GetScriptState()->GetExecutionContext()->IsContextThread());
+ ExecutionContext::From(resolver_->GetScriptState())->IsContextThread());
resolver_->Resolve(new RTCStatsReport(std::move(report)));
}
@@ -564,7 +564,7 @@
RTCSessionDescriptionRequest* request =
RTCSessionDescriptionRequestPromiseImpl::Create(this, resolver);
if (options.hasOfferToReceiveAudio() || options.hasOfferToReceiveVideo()) {
- ExecutionContext* context = script_state->GetExecutionContext();
+ ExecutionContext* context = ExecutionContext::From(script_state);
UseCounter::Count(
context,
UseCounter::kRTCPeerConnectionCreateOfferOptionsOfferToReceive);
@@ -581,7 +581,7 @@
ExceptionState& exception_state) {
DCHECK(success_callback);
DCHECK(error_callback);
- ExecutionContext* context = script_state->GetExecutionContext();
+ ExecutionContext* context = ExecutionContext::From(script_state);
UseCounter::Count(
context, UseCounter::kRTCPeerConnectionCreateOfferLegacyFailureCallback);
if (CallErrorCallbackIfSignalingStateClosed(signaling_state_, error_callback))
@@ -654,7 +654,7 @@
const Dictionary& media_constraints) {
DCHECK(success_callback);
DCHECK(error_callback);
- ExecutionContext* context = script_state->GetExecutionContext();
+ ExecutionContext* context = ExecutionContext::From(script_state);
UseCounter::Count(
context, UseCounter::kRTCPeerConnectionCreateAnswerLegacyFailureCallback);
if (media_constraints.IsObject())
@@ -709,7 +709,7 @@
const RTCSessionDescriptionInit& session_description_init,
VoidCallback* success_callback,
RTCPeerConnectionErrorCallback* error_callback) {
- ExecutionContext* context = script_state->GetExecutionContext();
+ ExecutionContext* context = ExecutionContext::From(script_state);
if (success_callback && error_callback) {
UseCounter::Count(
context,
@@ -769,7 +769,7 @@
const RTCSessionDescriptionInit& session_description_init,
VoidCallback* success_callback,
RTCPeerConnectionErrorCallback* error_callback) {
- ExecutionContext* context = script_state->GetExecutionContext();
+ ExecutionContext* context = ExecutionContext::From(script_state);
if (success_callback && error_callback) {
UseCounter::Count(
context,
@@ -815,7 +815,7 @@
return;
WebRTCConfiguration configuration = ParseConfiguration(
- script_state->GetExecutionContext(), rtc_configuration, exception_state);
+ ExecutionContext::From(script_state), rtc_configuration, exception_state);
if (exception_state.HadException())
return;
@@ -982,7 +982,7 @@
ScriptPromise promise = resolver->Promise();
RTCVoidRequest* request = RTCVoidRequestPromiseImpl::Create(this, resolver);
WebRTCICECandidate web_candidate = ConvertToWebRTCIceCandidate(
- script_state->GetExecutionContext(), candidate);
+ ExecutionContext::From(script_state), candidate);
bool implemented = peer_handler_->AddICECandidate(request, web_candidate);
if (!implemented)
resolver->Reject(DOMException::Create(
@@ -1012,7 +1012,7 @@
RTCVoidRequest* request = RTCVoidRequestImpl::Create(
GetExecutionContext(), this, success_callback, error_callback);
WebRTCICECandidate web_candidate = ConvertToWebRTCIceCandidate(
- script_state->GetExecutionContext(), candidate);
+ ExecutionContext::From(script_state), candidate);
bool implemented = peer_handler_->AddICECandidate(request, web_candidate);
if (!implemented)
AsyncCallErrorCallback(
@@ -1098,7 +1098,7 @@
MediaErrorState media_error_state;
WebMediaConstraints constraints =
- MediaConstraintsImpl::Create(script_state->GetExecutionContext(),
+ MediaConstraintsImpl::Create(ExecutionContext::From(script_state),
media_constraints, media_error_state);
if (media_error_state.HadException()) {
media_error_state.RaiseException(exception_state);
@@ -1161,7 +1161,7 @@
ScriptPromise RTCPeerConnection::getStats(ScriptState* script_state,
RTCStatsCallback* success_callback,
MediaStreamTrack* selector) {
- ExecutionContext* context = script_state->GetExecutionContext();
+ ExecutionContext* context = ExecutionContext::From(script_state);
ScriptPromiseResolver* resolver = ScriptPromiseResolver::Create(script_state);
ScriptPromise promise = resolver->Promise();
@@ -1177,7 +1177,7 @@
}
ScriptPromise RTCPeerConnection::getStats(ScriptState* script_state) {
- ExecutionContext* context = script_state->GetExecutionContext();
+ ExecutionContext* context = ExecutionContext::From(script_state);
UseCounter::Count(context, UseCounter::kRTCPeerConnectionGetStats);
ScriptPromiseResolver* resolver = ScriptPromiseResolver::Create(script_state);
@@ -1224,7 +1224,7 @@
DictionaryHelper::Get(options, "negotiated", init.negotiated);
unsigned short value = 0;
- ExecutionContext* context = script_state->GetExecutionContext();
+ ExecutionContext* context = ExecutionContext::From(script_state);
if (DictionaryHelper::Get(options, "id", value))
init.id = value;
if (DictionaryHelper::Get(options, "maxRetransmits", value)) {

Powered by Google App Engine
This is Rietveld 408576698