Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(14)

Unified Diff: chrome/browser/ui/android/javascript_app_modal_dialog_android.cc

Issue 688853005: Consolidate javascript_dialog_manager.h and javascript_dialog_manager_impl.h (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/android/javascript_app_modal_dialog_android.h ('k') | chrome/browser/ui/browser.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/android/javascript_app_modal_dialog_android.cc
diff --git a/chrome/browser/ui/android/javascript_app_modal_dialog_android.cc b/chrome/browser/ui/android/javascript_app_modal_dialog_android.cc
index ff0940986363d25ceeaf55a9830b611440747b61..34a8b99a778e1f374bb6cf5de7c44e59034bc276 100644
--- a/chrome/browser/ui/android/javascript_app_modal_dialog_android.cc
+++ b/chrome/browser/ui/android/javascript_app_modal_dialog_android.cc
@@ -23,7 +23,7 @@ using base::android::ScopedJavaLocalRef;
JavascriptAppModalDialogAndroid::JavascriptAppModalDialogAndroid(
JNIEnv* env,
- JavaScriptAppModalDialog* dialog,
+ app_modal::JavaScriptAppModalDialog* dialog,
gfx::NativeWindow parent)
: dialog_(dialog),
parent_jobject_weak_ref_(env, parent->GetJavaObject().obj()) {
@@ -131,7 +131,8 @@ const ScopedJavaGlobalRef<jobject>&
// static
jobject GetCurrentModalDialog(JNIEnv* env, jclass clazz) {
- AppModalDialog* dialog = AppModalDialogQueue::GetInstance()->active_dialog();
+ app_modal::AppModalDialog* dialog =
+ app_modal::AppModalDialogQueue::GetInstance()->active_dialog();
if (!dialog || !dialog->native_dialog())
return NULL;
@@ -159,14 +160,14 @@ JavascriptAppModalDialogAndroid::~JavascriptAppModalDialogAndroid() {
namespace {
class ChromeJavaScriptNativeDialogAndroidFactory
- : public JavaScriptNativeDialogFactory {
+ : public app_modal::JavaScriptNativeDialogFactory {
public:
ChromeJavaScriptNativeDialogAndroidFactory() {}
~ChromeJavaScriptNativeDialogAndroidFactory() override {}
private:
- NativeAppModalDialog* CreateNativeJavaScriptDialog(
- JavaScriptAppModalDialog* dialog,
+ app_modal::NativeAppModalDialog* CreateNativeJavaScriptDialog(
+ app_modal::JavaScriptAppModalDialog* dialog,
gfx::NativeWindow parent_window) override {
return new JavascriptAppModalDialogAndroid(
base::android::AttachCurrentThread(),
@@ -179,7 +180,8 @@ class ChromeJavaScriptNativeDialogAndroidFactory
} // namespace
void InstallChromeJavaScriptNativeDialogFactory() {
- SetJavaScriptNativeDialogFactory(
- make_scoped_ptr(new ChromeJavaScriptNativeDialogAndroidFactory));
+ app_modal::JavaScriptDialogManager::GetInstance()->
+ SetNativeDialogFactory(
+ make_scoped_ptr(new ChromeJavaScriptNativeDialogAndroidFactory));
}
« no previous file with comments | « chrome/browser/ui/android/javascript_app_modal_dialog_android.h ('k') | chrome/browser/ui/browser.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698