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

Unified Diff: components/dom_distiller/android/java/src/org/chromium/components/dom_distiller/content/DistillablePageUtils.java

Issue 2252963004: Remove dom_distiller core dependency on content (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix test deps Created 4 years, 4 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: components/dom_distiller/android/java/src/org/chromium/components/dom_distiller/content/DistillablePageUtils.java
diff --git a/components/dom_distiller/android/java/src/org/chromium/components/dom_distiller/content/DistillablePageUtils.java b/components/dom_distiller/android/java/src/org/chromium/components/dom_distiller/content/DistillablePageUtils.java
deleted file mode 100644
index ad1d87fc19f40214a1da6086109c4b930f0338c8..0000000000000000000000000000000000000000
--- a/components/dom_distiller/android/java/src/org/chromium/components/dom_distiller/content/DistillablePageUtils.java
+++ /dev/null
@@ -1,64 +0,0 @@
-// Copyright 2015 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.components.dom_distiller.content;
-
-import org.chromium.base.annotations.CalledByNative;
-import org.chromium.base.annotations.JNINamespace;
-import org.chromium.content_public.browser.WebContents;
-
-/**
- * Provides access to the native dom_distiller::IsPageDistillable function.
- */
-@JNINamespace("dom_distiller::android")
-public final class DistillablePageUtils {
- /**
- * Callback for handling the result of isPageDistillable.
- */
- public static interface PageDistillableCallback {
- public void onIsPageDistillableResult(boolean isDistillable);
- }
-
- public static void isPageDistillable(WebContents webContents, boolean isMobileOptimized,
- PageDistillableCallback callback) {
- nativeIsPageDistillable(webContents, isMobileOptimized, callback);
- }
-
- @CalledByNative
- private static void callOnIsPageDistillableResult(
- PageDistillableCallback callback, boolean isDistillable) {
- callback.onIsPageDistillableResult(isDistillable);
- }
-
- private static native void nativeIsPageDistillable(
- WebContents webContents, boolean isMobileOptimized, PageDistillableCallback callback);
-
- /**
- * Delegate to receive distillability updates.
- */
- public static interface PageDistillableDelegate {
- /**
- * Called when the distillability status changes.
- * @param isDistillable Whether the page is distillable.
- * @param isLast Whether the update is the last one for this page.
- */
- public void onIsPageDistillableResult(boolean isDistillable, boolean isLast);
- }
-
- public static void setDelegate(WebContents webContents,
- PageDistillableDelegate delegate) {
- nativeSetDelegate(webContents, delegate);
- }
-
- @CalledByNative
- private static void callOnIsPageDistillableUpdate(
- PageDistillableDelegate delegate, boolean isDistillable, boolean isLast) {
- if (delegate != null) {
- delegate.onIsPageDistillableResult(isDistillable, isLast);
- }
- }
-
- private static native void nativeSetDelegate(
- WebContents webContents, PageDistillableDelegate delegate);
-}

Powered by Google App Engine
This is Rietveld 408576698