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

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

Issue 2724363004: Content class factory and generalized text selector (Closed)
Patch Set: Interface cleanup Created 3 years, 10 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/ContextSelectionProvider.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContextSelectionProvider.java b/content/public/android/java/src/org/chromium/content/browser/ContextSelectionProvider.java
new file mode 100644
index 0000000000000000000000000000000000000000..ce961e0c4aed6971c7bb5182885288e89af7e18f
--- /dev/null
+++ b/content/public/android/java/src/org/chromium/content/browser/ContextSelectionProvider.java
@@ -0,0 +1,86 @@
+// 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;
+
+import android.content.Intent;
+import android.graphics.drawable.Drawable;
+import android.view.View.OnClickListener;
+
+import org.chromium.base.annotations.SuppressFBWarnings;
+
+/**
+ * The interface that controls contextual text selection.
+ */
+@SuppressFBWarnings("UUF_UNUSED_PUBLIC_OR_PROTECTED_FIELD")
+public interface ContextSelectionProvider {
+ /**
+ * The result of the text analysis.
+ */
+ public static class Result {
+ /**
+ * The number of characters that the left boundary of the original
+ * selection should be moved. Negative number means moving left.
+ */
+ public int startAdjust;
+
+ /**
+ * The number of characters that the right boundary of the original
+ * selection should be moved. Negative number means moving left.
+ */
+ public int endAdjust;
+
+ /**
+ * Label for the suggested menu item.
+ */
+ public CharSequence label;
+
+ /**
+ * Icon for the suggested menu item.
+ */
+ public Drawable icon;
+
+ /**
+ * Intent for the suggested menu item.
+ */
+ public Intent intent;
+
+ /**
+ * OnClickListener for the suggested menu item.
+ */
+ public OnClickListener onClickListener;
+ }
+
+ /**
+ * The interface that returns the result of the selected text analysis.
+ */
+ public interface ResultCallback {
+ /**
+ * The result is delivered with this method.
+ */
+ void onClassified(Result result);
+ }
+
+ /**
+ * Sends asynchronous request to obtain the selection, analyze its type and suggest
+ * better selection boundaries.
+ * @param text The textual context that encloses the selected text.
+ * @param start The start index of the selected text inside the textual context.
boliu 2017/03/06 16:56:13 probably want to specify whether these indices are
Tima Vaisburd 2017/03/07 00:45:40 Rewrote the explanation. |start| is inclusive and
+ * @param end The end index of the selected text inside the textual context.
+ */
+ public void sendSuggestAndClassifyRequest(CharSequence text, int start, int end);
+
+ /**
+ * Sends asynchronous request to obtain the selection and analyze its type.
+ * @param text The textual context that encloses the selected text.
+ * @param start The start index of the selected text inside the textual context.
+ * @param end The end index of the selected text inside the textual context.
+ */
+ public void sendClassifyRequest(CharSequence text, int start, int end);
+
+ /**
+ * Cancel all asynchronous requests.
+ */
+ public void cancelAllRequests();
+}

Powered by Google App Engine
This is Rietveld 408576698