Move JavaScriptDialogManager, JavascriptAppModalDialogViews to components/app_modal_dialogs

chrome/browser/ui/android/chrome_javascript_native_dialog_factory_android.cc

Index: chrome/browser/ui/android/chrome_javascript_native_dialog_factory_android.cc
diff --git a/chrome/browser/ui/android/chrome_javascript_native_dialog_factory_android.cc b/chrome/browser/ui/android/chrome_javascript_native_dialog_factory_android.cc
new file mode 100644
index 0000000000000000000000000000000000000000..bc5dd6960c9fc5ca3c7934c618552fde4fedf1e7
--- /dev/null
+++ b/chrome/browser/ui/android/chrome_javascript_native_dialog_factory_android.cc
@@ -0,0 +1,38 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.

[msw, 2014/11/05 03:53:54]

Perhaps everything in this file could still live in
javascript_app_modal_dialog_android.cc?

[oshima, 2014/11/05 21:42:18]

I put this in separate file because I thought it's a bit too big to mix it into
that file, but
I don't have strong opinion.

avi@, are you ok with this?

[Avi (use Gerrit), 2014/11/05 21:50:40]

I don't have strong opinions and would be OK either way.

[msw, 2014/11/05 22:59:06]

I slightly favor combining the files, but don't have a strong opinion.
You can leave this as-is for now and consider a followup cleanup CL.

[oshima, 2014/11/05 23:22:48]

I've moved to javascript_app_modal_dialog_android.cc

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/app_modal_dialogs/chrome_javascript_native_dialog_factory.h"
+
+#include "base/android/jni_android.h"
+#include "chrome/browser/ui/android/javascript_app_modal_dialog_android.h"
+#include "components/app_modal_dialogs/javascript_dialog_manager.h"
+#include "components/app_modal_dialogs/javascript_native_dialog_factory.h"
+
+namespace {
+
+class ChromeJavaScriptNativeDialogAndroidFactory
+    : public JavaScriptNativeDialogFactory {
+ public:
+  ChromeJavaScriptNativeDialogAndroidFactory() {}
+  ~ChromeJavaScriptNativeDialogAndroidFactory() override {}
+
+ private:
+  NativeAppModalDialog* CreateNativeJavaScriptDialog(
+      JavaScriptAppModalDialog* dialog,
+      gfx::NativeWindow parent_window) override {
+    return new JavascriptAppModalDialogAndroid(
+        base::android::AttachCurrentThread(),
+        dialog, parent_window);
+  }
+
+  DISALLOW_COPY_AND_ASSIGN(ChromeJavaScriptNativeDialogAndroidFactory);
+};
+
+}  // namespace
+
+void InstallChromeJavaScriptNativeDialogFactory() {
+  SetJavaScriptNativeDialogFactory(
+      make_scoped_ptr(new ChromeJavaScriptNativeDialogAndroidFactory));
+}
+