Rename SameThreadClosure to Closure

third_party/WebKit/Source/wtf/Functional.h

 /*
  * Copyright (C) 2011 Apple 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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 41 matching lines @@
 
 }
 
 namespace WTF {
 
 // Functional.h provides a very simple way to bind a function pointer and arguments together into a function object
 // that can be stored, copied and invoked, similar to how boost::bind and std::bind in C++11.
 
 // Thread Safety:
 //
-// WTF::bind() and SameThreadClosure should be used for same-thread closures
+// WTF::bind() and Closure should be used for same-thread closures
 // only, i.e. the closures must be created, executed and destructed on
 // the same thread.
 // Use crossThreadBind() and CrossThreadClosure if the function/task is called
 // or destructed on a (potentially) different thread from the current thread.
 
 // WTF::bind() and move semantics
 // ==============================
 //
 // For unbound parameters (arguments supplied later on the bound functor directly), there are two ways to pass movable
 // arguments:
@@ skipping 182 matching lines @@
     using UnboundRunType = base::MakeUnboundRunType<FunctionType, BoundParameters...>;
     return wrapUnique(new Function<UnboundRunType, threadAffinity>(base::Bind(function, typename ParamStorageTraits<typename std::decay<BoundParameters>::type>::StorageType(std::forward<BoundParameters>(boundParameters))...)));
 }
 
 template <typename FunctionType, typename... BoundParameters>
 std::unique_ptr<Function<base::MakeUnboundRunType<FunctionType, BoundParameters...>, SameThreadAffinity>> bind(FunctionType function, BoundParameters&&... boundParameters)
 {
     return bindInternal<SameThreadAffinity>(function, std::forward<BoundParameters>(boundParameters)...);
 }
 
-typedef Function<void(), SameThreadAffinity> SameThreadClosure;
+typedef Function<void(), SameThreadAffinity> Closure;
 typedef Function<void(), CrossThreadAffinity> CrossThreadClosure;
 
 } // namespace WTF
 
 using WTF::passed;
 using WTF::unretained;
 using WTF::crossThreadUnretained;
 
 using WTF::Function;
-using WTF::SameThreadClosure;
+using WTF::Closure;

[tzik, 2016/06/30 06:40:32]

Can we remove `using WTF::Closure;` here and replace SameThreadClosure with
WTF::Closure instead of bare Closure?
Since there is base::Closure already, the fully qualified one looks less
confusing to me.

[hiroshige, 2016/06/30 11:38:43]

Done.

 using WTF::CrossThreadClosure;
 
 #endif // WTF_Functional_h