Refactoring ContentVideoViewContextDelegate.java

content/public/android/java/src/org/chromium/content/browser/ContentVideoViewClient.java

Index: content/public/android/java/src/org/chromium/content/browser/ContentVideoViewClient.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentVideoViewClient.java b/content/public/android/java/src/org/chromium/content/browser/ContentVideoViewClient.java
new file mode 100644
index 0000000000000000000000000000000000000000..3f3c00c2fb1fb8b3a00d1508373c0cd1b810f052
--- /dev/null
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentVideoViewClient.java
@@ -0,0 +1,25 @@
+// Copyright 2013 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;
+
+import android.view.View;
+
+/**
+ *  Main callback class used by ContentVideoView.
+ *
+ *  This contains the superset of callbacks must be implemented by the browser UI
+ *  and WebView API.

[joth, 2013/05/15 22:38:57]

nit: "by browser UI and WebView API." => "by the embedder"

[michaelbai, 2013/05/22 18:08:39]

Done.

+ *
+ *  onShowCustomView and onDestoryContentVideoView must be implemented,
+ *  getVideoLoadingProgressView() is optional.

[joth, 2013/05/15 22:38:57]

is optional, and may return null if not required.

[michaelbai, 2013/05/22 18:08:39]

Done.

+ *
+ *  The implementer is responsible for displaying the Android view when
+ *  {@link #onShowCustomView(View)} is called.
+ */
+public interface ContentVideoViewClient {
+    public void onShowCustomView(View view);
+    public void onDestroyContentVideoView();
+    public View getVideoLoadingProgressView();
+}