Add a ctor for InterfaceRequest<I> that takes a ScopedMessagePipeHandle, and remove MakeRequest<I>().

InterfaceRequest is basically analogous to InterfaceHandle (and vice
versa), and the latter has such a ctor. Moreover, MakeRequest doesn't
really add anything and is mostly just confusing.

R=vardhan@google.com

Review URL: https://codereview.chromium.org/1926123002 .
diff --git a/shell/android/android_handler.cc b/shell/android/android_handler.cc
index b2c1118..c52b340 100644
--- a/shell/android/android_handler.cc
+++ b/shell/android/android_handler.cc
@@ -44,7 +44,7 @@
                            const base::FilePath& app_path,
                            jint j_handle) {
   mojo::InterfaceRequest<mojo::Application> application_request =
-      mojo::MakeRequest<mojo::Application>(
+      mojo::InterfaceRequest<mojo::Application>(
           mojo::MakeScopedHandle(mojo::MessagePipeHandle(j_handle)));
 
   // Load the library, so that we can set the application context there if