Remove deprecated MessageLoop::Type

MessagePumpType should be used instead.

Bug: 891670
Change-Id: I6e73c7d3197beca3885aefbf71a7a64a26e42efb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1777868
Auto-Submit: Carlos Caballero <carlscab@google.com>
Commit-Queue: Carlos Caballero <carlscab@google.com>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693067}

base/message_loop/message_loop.h

@@ -85,18 +85,6 @@ class SequenceManagerImpl;
 // for tests. TODO(https://crbug.com/891670/) remove this class.
 class BASE_EXPORT MessageLoop {
  public:
-  // DEPRECATED: Use MessagePumpType instead
-  using Type = MessagePumpType;
-
-  // DEPRECATED: Use MessagePumpType::* instead
-  static constexpr Type TYPE_DEFAULT = Type::DEFAULT;
-  static constexpr Type TYPE_UI = Type::UI;
-  static constexpr Type TYPE_CUSTOM = Type::CUSTOM;
-  static constexpr Type TYPE_IO = Type::IO;
-#if defined(OS_ANDROID)
-  static constexpr Type TYPE_JAVA = Type::JAVA;
-#endif  // defined(OS_ANDROID)
-
   // Normally, it is not necessary to instantiate a MessageLoop.  Instead, it
   // is typical to make use of the current thread's MessageLoop instance.
   explicit MessageLoop(MessagePumpType type = MessagePumpType::DEFAULT);

base/message_loop/message_pump.h

@@ -21,7 +21,7 @@ class BASE_EXPORT MessagePump {
  public:
   using MessagePumpFactory = std::unique_ptr<MessagePump>();
   // Uses the given base::MessagePumpFactory to override the default MessagePump
-  // implementation for 'Type::UI'. May only be called once.
+  // implementation for 'MessagePumpType::UI'. May only be called once.
   static void OverrideMessagePumpForUIFactory(MessagePumpFactory* factory);
 
   // Returns true if the MessagePumpForUI has been overidden.