Rename OnError to OnErrorAdapter and OnSuccess to OnSucessAdapter.

third_party/WebKit/Source/bindings/core/v8/CallbackPromiseAdapter.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 76 matching lines @@
 //         new CallbackPromiseAdapter<MyClass, MyErrorClass>(resolver));
 // ...
 //
 // std::unique_ptr<WebCallbacks<bool, const WebMyErrorClass&>> callbacks2 =
 //     WTF::wrapUnique(
 //         new CallbackPromiseAdapter<bool, MyErrorClass>(resolver));
 // ...
 //
 //
 // In order to implement the above exceptions, we have template classes below.
-// OnSuccess and OnError provide onSuccess and onError implementation, and there
-// are utility templates that provide the trivial WebType holder.
+// OnSuccessAdapter and OnErrorAdapter provide onSuccess and onError
+// implementation, and there are utility templates that provide the trivial
+// WebType holder.
 
 namespace internal {
 
 // This template is placed outside of CallbackPromiseAdapterInternal because
 // explicit specialization is forbidden in a class scope.
 template <typename T>
 struct CallbackPromiseAdapterTrivialWebTypeHolder {
   using WebType = T;
   static T take(ScriptPromiseResolver*, const T& x) { return x; }
 };
@@ skipping 17 matching lines @@
   class Base : public WebCallbacks<typename S::WebType, typename T::WebType> {
    public:
     explicit Base(ScriptPromiseResolver* resolver) : m_resolver(resolver) {}
     ScriptPromiseResolver* resolver() { return m_resolver; }
 
    private:
     Persistent<ScriptPromiseResolver> m_resolver;
   };
 
   template <typename S, typename T>
-  class OnSuccess : public Base<S, T> {
+  class OnSuccessAdapter : public Base<S, T> {
    public:
-    explicit OnSuccess(ScriptPromiseResolver* resolver)
+    explicit OnSuccessAdapter(ScriptPromiseResolver* resolver)
         : Base<S, T>(resolver) {}
     void onSuccess(typename S::WebType result) override {
       ScriptPromiseResolver* resolver = this->resolver();
       if (!resolver->getExecutionContext() ||
           resolver->getExecutionContext()->isContextDestroyed())
         return;
       resolver->resolve(S::take(resolver, std::move(result)));
     }
   };
   template <typename T>
-  class OnSuccess<CallbackPromiseAdapterTrivialWebTypeHolder<void>, T>
+  class OnSuccessAdapter<CallbackPromiseAdapterTrivialWebTypeHolder<void>, T>
       : public Base<CallbackPromiseAdapterTrivialWebTypeHolder<void>, T> {
    public:
-    explicit OnSuccess(ScriptPromiseResolver* resolver)
+    explicit OnSuccessAdapter(ScriptPromiseResolver* resolver)
         : Base<CallbackPromiseAdapterTrivialWebTypeHolder<void>, T>(resolver) {}
     void onSuccess() override {
       ScriptPromiseResolver* resolver = this->resolver();
       if (!resolver->getExecutionContext() ||
           resolver->getExecutionContext()->isContextDestroyed())
         return;
       resolver->resolve();
     }
   };
   template <typename S, typename T>
-  class OnError : public OnSuccess<S, T> {
+  class OnErrorAdapter : public OnSuccessAdapter<S, T> {
    public:
-    explicit OnError(ScriptPromiseResolver* resolver)
-        : OnSuccess<S, T>(resolver) {}
+    explicit OnErrorAdapter(ScriptPromiseResolver* resolver)
+        : OnSuccessAdapter<S, T>(resolver) {}
     void onError(typename T::WebType e) override {
       ScriptPromiseResolver* resolver = this->resolver();
       if (!resolver->getExecutionContext() ||
           resolver->getExecutionContext()->isContextDestroyed())
         return;
       ScriptState::Scope scope(resolver->getScriptState());
       resolver->reject(T::take(resolver, std::move(e)));
     }
   };
   template <typename S>
-  class OnError<S, CallbackPromiseAdapterTrivialWebTypeHolder<void>>
-      : public OnSuccess<S, CallbackPromiseAdapterTrivialWebTypeHolder<void>> {
+  class OnErrorAdapter<S, CallbackPromiseAdapterTrivialWebTypeHolder<void>>
+      : public OnSuccessAdapter<
+            S,
+            CallbackPromiseAdapterTrivialWebTypeHolder<void>> {
    public:
-    explicit OnError(ScriptPromiseResolver* resolver)
-        : OnSuccess<S, CallbackPromiseAdapterTrivialWebTypeHolder<void>>(
+    explicit OnErrorAdapter(ScriptPromiseResolver* resolver)
+        : OnSuccessAdapter<S, CallbackPromiseAdapterTrivialWebTypeHolder<void>>(
               resolver) {}
     void onError() override {
       ScriptPromiseResolver* resolver = this->resolver();
       if (!resolver->getExecutionContext() ||
           resolver->getExecutionContext()->isContextDestroyed())
         return;
       resolver->reject();
     }
   };
 
  public:
   template <typename S, typename T>
   class CallbackPromiseAdapter final
-      : public OnError<WebTypeHolder<S>, WebTypeHolder<T>> {
+      : public OnErrorAdapter<WebTypeHolder<S>, WebTypeHolder<T>> {
     WTF_MAKE_NONCOPYABLE(CallbackPromiseAdapter);
 
    public:
     explicit CallbackPromiseAdapter(ScriptPromiseResolver* resolver)
-        : OnError<WebTypeHolder<S>, WebTypeHolder<T>>(resolver) {}
+        : OnErrorAdapter<WebTypeHolder<S>, WebTypeHolder<T>>(resolver) {}
   };
 };
 
 }  // namespace internal
 
 template <typename S, typename T>
 using CallbackPromiseAdapter =
     internal::CallbackPromiseAdapterInternal::CallbackPromiseAdapter<S, T>;
 
 }  // namespace blink
 
 #endif