Make ScriptPromiseResolver RefCountedWillBeRefCountedGarbageCollected.

Source/modules/crypto/CryptoResultImpl.h

 /*
  * 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.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 39 matching lines @@
 //
 //  * At creation time there must be an active ExecutionContext.
 //  * The CryptoResult interface must only be called from the origin thread.
 //  * ref(), deref(), cancelled() and cancel() can be called from any thread.
 //  * One of the completeWith***() functions must be called, or the
 //    m_resolver will be leaked until the ExecutionContext is destroyed.
 class CryptoResultImpl final : public CryptoResult {
 public:
     ~CryptoResultImpl();
 
-    static PassRefPtr<CryptoResultImpl> create(ScriptState*);
+    static PassRefPtrWillBeRawPtr<CryptoResultImpl> create(ScriptState*);
 
     virtual void completeWithError(WebCryptoErrorType, const WebString&) override;
     virtual void completeWithBuffer(const void* bytes, unsigned bytesSize) override;
     virtual void completeWithJson(const char* utf8Data, unsigned length) override;
     virtual void completeWithBoolean(bool) override;
     virtual void completeWithKey(const WebCryptoKey&) override;
     virtual void completeWithKeyPair(const WebCryptoKey& publicKey, const WebCryptoKey& privateKey) override;
     virtual bool cancelled() const override;
 
     // If called after completion (including cancellation) will return an empty
     // ScriptPromise.
     ScriptPromise promise();
 
 private:
     class Resolver;
     explicit CryptoResultImpl(ScriptState*);
 
     void clearResolver();
     void cancel();
 
     // FIXME: ScriptPromiseResolver should not be exported.
     // Instead, use ScriptPromise.
     ScriptPromiseResolver* m_resolver;

[sof, 2014/12/16 21:34:59]

Shouldn't this be trace()d? 

[tasak, 2014/12/17 08:40:08]

I discussed with haraken@ and yhiarano@.
So we concluded that we don't need to make it "OwnPtr<Persistent>".
Instead, we quickly clear m_resolver when resolving or rejecting.

     volatile int m_cancelled;
 };
 
 } // namespace blink
 
 #endif