OLD | NEW |
---|---|
(Empty) | |
1 // Copyright 2017 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 package org.chromium.content.browser.androidoverlay; | |
6 | |
7 import org.chromium.gfx.mojom.Rect; | |
8 import org.chromium.media.mojom.AndroidOverlay; | |
9 import org.chromium.media.mojom.AndroidOverlayClient; | |
10 import org.chromium.media.mojom.AndroidOverlayConfig; | |
11 import org.chromium.mojo.system.MojoException; | |
12 | |
13 /** | |
14 * 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
| |
15 * instance, probably via a separate object that operates only on that thread. We will post | |
16 * messages to / from that thread from the main thread. | |
17 */ | |
18 public class AndroidOverlayImpl implements AndroidOverlay { | |
19 private static final String TAG = "AndroidOverlay"; | |
20 | |
21 public AndroidOverlayImpl(AndroidOverlayClient client, AndroidOverlayConfig config) {} | |
22 | |
23 @Override | |
24 public void close() { | |
25 // Client has closed the connection. | |
26 // TODO(liberato): Allow any sync surfaceDestroyed to proceed. | |
27 // TODO(liberato): Notify our provider that we've been destroyed. | |
28 } | |
29 | |
30 @Override | |
31 public void onConnectionError(MojoException e) {} | |
32 | |
33 @Override | |
34 public void scheduleLayout(Rect rect) {} | |
35 } | |
OLD | NEW |