Chromium Code Reviews| 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 |
|
boliu
2017/02/22 18:03:49
why not just use the UI thread since it's simpler.
liberato (no reviews please)
2017/02/22 19:10:50
SurfaceDestroyed is synchronous on this thread. i
boliu
2017/02/22 19:15:59
CompositorView::SurfaceDestroyed doesn't block.. B
|
| + * 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) {} |
| +} |