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

Unified Diff: android_webview/native/aw_browser_dependency_factory.h

Issue 10855171: Add a test runner for android_webview. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address feedback Created 8 years, 4 months 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
Index: android_webview/native/aw_browser_dependency_factory.h
diff --git a/android_webview/native/aw_browser_dependency_factory.h b/android_webview/native/aw_browser_dependency_factory.h
new file mode 100644
index 0000000000000000000000000000000000000000..ebf03fd93558ccbcfc8feace1835d4869a488c3a
--- /dev/null
+++ b/android_webview/native/aw_browser_dependency_factory.h
@@ -0,0 +1,60 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef ANDROID_WEBVIEW_AW_BROWSER_DELEGATE_H_
+#define ANDROID_WEBVIEW_AW_BROWSER_DELEGATE_H_
+
+#include "base/basictypes.h"
+
+namespace content {
+class JavaScriptDialogCreator;
+class WebContents;
+}
+
+namespace android_webview {
+
+class AwContentsContainer;
+
+// The concrete class implementing this interface is the responsible for
+// creating he browser component objects that the Android WebView depends on.
+// The motivation for this abstraction is to keep code under
+// android_webview/native from holding direct chrome/ layer dependencies.
+// Note this is a distinct role to the Webview embedder proper: this class is
+// about 'static' environmental decoupling, allowing dependency injection by the
+// top-level lib, whereas runtime behavior is controlled by propagated back to
+// the embedding application code via WebContentsDelegate and the like.
+class AwBrowserDependencyFactory {
+ public:
+ virtual ~AwBrowserDependencyFactory();
+
+ // Allows the lib executive to inject the delegate instance. Ownership of
+ // |delegate| is NOT transferred, but the object must be long-lived.
+ static void SetInstance(AwBrowserDependencyFactory* delegate);
+
+ // Returns the singleton instance. |SetInstance| must have been called.
+ static AwBrowserDependencyFactory* GetInstance();
+
+ // Constructs and returns ownership of a WebContents instance.
+ virtual content::WebContents* CreateWebContents(bool incognito) = 0;
+
+ // Creates and returns ownership of a new content container instance. That
+ // instance will take ownership of the passed |contents|.
+ virtual AwContentsContainer* CreateContentsContainer(
+ content::WebContents* contents) = 0;
+
+ // As per the WebContentsDelegate method of the same name. Ownership is NOT
+ // returned; the instance must be long-lived.
+ virtual content::JavaScriptDialogCreator* GetJavaScriptDialogCreator() = 0;
+
+ protected:
+ AwBrowserDependencyFactory();
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(AwBrowserDependencyFactory);
+};
+
+} // namespace android_webview
+
+#endif // ANDROID_WEBVIEW_AW_BROWSER_DELEGATE_H_
+
« no previous file with comments | « android_webview/native/android_webview_jni_registrar.cc ('k') | android_webview/native/aw_browser_dependency_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698