Wrap GCed raw pointer parameters of WTF::bind with Persistent

third_party/WebKit/Source/modules/imagecapture/ImageCapture.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/imagecapture/ImageCapture.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMException.h"
 #include "core/dom/ExceptionCode.h"
@@ skipping 74 matching lines @@
     if (!m_service) {
         resolver->reject(DOMException::create(NotFoundError, kNoServiceError));
         return promise;
     }
 
     m_serviceRequests.add(resolver);
 
     // m_streamTrack->component()->source()->id() is the renderer "name" of the camera;
     // TODO(mcasas) consider sending the security origin as well:
     // scriptState->getExecutionContext()->getSecurityOrigin()->toString()
-    m_service->GetCapabilities(m_streamTrack->component()->source()->id(), createBaseCallback(bind(&ImageCapture::onCapabilities, this, resolver)));
+    m_service->GetCapabilities(m_streamTrack->component()->source()->id(), createBaseCallback(bind(&ImageCapture::onCapabilities, wrapPersistent(this), wrapPersistent(resolver))));
     return promise;
 }
 
 ScriptPromise ImageCapture::setOptions(ScriptState* scriptState, const PhotoSettings& photoSettings, ExceptionState& exceptionState)
 {
     ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
     ScriptPromise promise = resolver->promise();
 
     if (trackIsInactive(*m_streamTrack)) {
         resolver->reject(DOMException::create(InvalidStateError, "The associated Track is in an invalid state."));
         return promise;
     }
 
     if (!m_service) {
         resolver->reject(DOMException::create(NotFoundError, kNoServiceError));
         return promise;
     }
 
     m_serviceRequests.add(resolver);
 
     // TODO(mcasas): should be using a mojo::StructTraits instead.
     media::mojom::blink::PhotoSettingsPtr settings = media::mojom::blink::PhotoSettings::New();
     settings->has_zoom = photoSettings.hasZoom();
     if (settings->has_zoom)
         settings->zoom = photoSettings.zoom();
 
-    m_service->SetOptions(m_streamTrack->component()->source()->id(), std::move(settings), createBaseCallback(bind(&ImageCapture::onSetOptions, this, resolver)));
+    m_service->SetOptions(m_streamTrack->component()->source()->id(), std::move(settings), createBaseCallback(bind(&ImageCapture::onSetOptions, wrapPersistent(this), wrapPersistent(resolver))));
     return promise;
 }
 
 
 ScriptPromise ImageCapture::takePhoto(ScriptState* scriptState, ExceptionState& exceptionState)
 {
     ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
     ScriptPromise promise = resolver->promise();
 
     if (trackIsInactive(*m_streamTrack)) {
         resolver->reject(DOMException::create(InvalidStateError, "The associated Track is in an invalid state."));
         return promise;
     }
 
     if (!m_service) {
         resolver->reject(DOMException::create(NotFoundError, kNoServiceError));
         return promise;
     }
 
     m_serviceRequests.add(resolver);
 
     // m_streamTrack->component()->source()->id() is the renderer "name" of the camera;
     // TODO(mcasas) consider sending the security origin as well:
     // scriptState->getExecutionContext()->getSecurityOrigin()->toString()
-    m_service->TakePhoto(m_streamTrack->component()->source()->id(), createBaseCallback(bind(&ImageCapture::onTakePhoto, this, resolver)));
+    m_service->TakePhoto(m_streamTrack->component()->source()->id(), createBaseCallback(bind(&ImageCapture::onTakePhoto, wrapPersistent(this), wrapPersistent(resolver))));
     return promise;
 }
 
 ScriptPromise ImageCapture::grabFrame(ScriptState* scriptState, ExceptionState& exceptionState)
 {
     ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
     ScriptPromise promise = resolver->promise();
 
     if (trackIsInactive(*m_streamTrack)) {
         resolver->reject(DOMException::create(InvalidStateError, "The associated Track is in an invalid state."));
@@ skipping 83 matching lines @@
 
 DEFINE_TRACE(ImageCapture)
 {
     visitor->trace(m_streamTrack);
     visitor->trace(m_serviceRequests);
     EventTargetWithInlineData::trace(visitor);
     ContextLifecycleObserver::trace(visitor);
 }
 
 } // namespace blink