Move out OnceCallback and RepeatingCallback from internal namespace

BUG=554299

Review-Url: https://codereview.chromium.org/2394023003
Cr-Commit-Position: refs/heads/master@{#424694}

base/bind.h

@@ -23,7 +23,6 @@
 // terms and concepts.
 
 namespace base {
-namespace internal {
 
 // Bind as OnceCallback.
 template <typename Functor, typename... Args>
@@ -69,16 +68,14 @@ BindRepeating(Functor&& functor, Args&&... args) {
       std::forward<Args>(args)...));
 }
 
-}  // namespace internal
-
 // Unannotated Bind.
 // TODO(tzik): Deprecate this and migrate to OnceCallback and
 // RepeatingCallback, once they get ready.
 template <typename Functor, typename... Args>
 inline Callback<MakeUnboundRunType<Functor, Args...>>
 Bind(Functor&& functor, Args&&... args) {
-  return internal::BindRepeating(std::forward<Functor>(functor),
-                                 std::forward<Args>(args)...);
+  return BindRepeating(std::forward<Functor>(functor),
+                       std::forward<Args>(args)...);
 }
 
 }  // namespace base

base/bind_unittest.cc

@@ -25,14 +25,6 @@ using ::testing::Return;
 using ::testing::StrictMock;
 
 namespace base {
-
-using internal::OnceCallback;
-using internal::RepeatingCallback;
-using internal::OnceClosure;
-using internal::RepeatingClosure;
-using internal::BindOnce;
-using internal::BindRepeating;
-
 namespace {
 
 class IncompleteType;

base/callback_forward.h

@@ -32,8 +32,6 @@ class Callback;
 // will be used in a lot of APIs with delayed execution.
 using Closure = Callback<void()>;
 
-namespace internal {
-
 template <typename Signature>
 using OnceCallback = Callback<Signature,
                               internal::CopyMode::MoveOnly,
@@ -45,7 +43,6 @@ using RepeatingCallback = Callback<Signature,
 using OnceClosure = OnceCallback<void()>;
 using RepeatingClosure = RepeatingCallback<void()>;
 
-}  // namespace internal
 }  // namespace base
 
 #endif  // BASE_CALLBACK_FORWARD_H_