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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/androidoverlay/AndroidOverlayImpl.java

Issue 2688193002: Mojo framework for AndroidOverlay. (Closed)
Patch Set: fixed mojo manifest bug introduced by rebase Created 3 years, 9 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: content/public/android/java/src/org/chromium/content/browser/androidoverlay/AndroidOverlayImpl.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/androidoverlay/AndroidOverlayImpl.java b/content/public/android/java/src/org/chromium/content/browser/androidoverlay/AndroidOverlayImpl.java
new file mode 100644
index 0000000000000000000000000000000000000000..7861b50bbe9e7e64d3fc7a80ef9b121b457c6d35
--- /dev/null
+++ b/content/public/android/java/src/org/chromium/content/browser/androidoverlay/AndroidOverlayImpl.java
@@ -0,0 +1,35 @@
+// Copyright 2017 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.
+
+package org.chromium.content.browser.androidoverlay;
+
+import org.chromium.gfx.mojom.Rect;
+import org.chromium.media.mojom.AndroidOverlay;
+import org.chromium.media.mojom.AndroidOverlayClient;
+import org.chromium.media.mojom.AndroidOverlayConfig;
+import org.chromium.mojo.system.MojoException;
+
+/**
+ * Default AndroidOverlay impl. Will use a separate (shared) overlay-ui thread to own a Dialog
+ * instance, probably via a separate object that operates only on that thread. We will post
+ * messages to / from that thread from the main thread.
+ */
+public class AndroidOverlayImpl implements AndroidOverlay {
+ private static final String TAG = "AndroidOverlay";
+
+ public AndroidOverlayImpl(AndroidOverlayClient client, AndroidOverlayConfig config) {}
+
+ @Override
+ public void close() {
+ // Client has closed the connection.
+ // TODO(liberato): Allow any sync surfaceDestroyed to proceed.
+ // TODO(liberato): Notify our provider that we've been destroyed.
+ }
+
+ @Override
+ public void onConnectionError(MojoException e) {}
+
+ @Override
+ public void scheduleLayout(Rect rect) {}
+}

Powered by Google App Engine
This is Rietveld 408576698