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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/omaha/OmahaDelegate.java

Issue 2686783003: Revert of [Omaha] Make the whole pipeline run each time (patchset #4 id:60001 of https://codereview… (Closed)
Patch Set: 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: chrome/android/java/src/org/chromium/chrome/browser/omaha/OmahaDelegate.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/omaha/OmahaDelegate.java b/chrome/android/java/src/org/chromium/chrome/browser/omaha/OmahaDelegate.java
deleted file mode 100644
index 95091743eb3d258b80b8b5415b10ed3713522891..0000000000000000000000000000000000000000
--- a/chrome/android/java/src/org/chromium/chrome/browser/omaha/OmahaDelegate.java
+++ /dev/null
@@ -1,80 +0,0 @@
-// 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.chrome.browser.omaha;
-
-import android.app.Service;
-import android.content.Context;
-
-import org.chromium.chrome.browser.omaha.OmahaClient.PostResult;
-
-/** Delegates calls out from the OmahaClient. */
-public abstract class OmahaDelegate {
- protected final Context mContext;
- private RequestGenerator mRequestGenerator;
-
- OmahaDelegate(Context context) {
- mContext = context;
- }
-
- /** @return Context that is used to interact with the system. */
- Context getContext() {
- return mContext;
- }
-
- /** @return Whether Chrome is installed as part of the system image. */
- abstract boolean isInSystemImage();
-
- /** @return The scheduler used to trigger jobs. */
- abstract ExponentialBackoffScheduler getScheduler();
-
- /** @return The {@link RequestGenerator} used to create Omaha XML. */
- final RequestGenerator getRequestGenerator() {
- if (mRequestGenerator == null) mRequestGenerator = createRequestGenerator(getContext());
- return mRequestGenerator;
- }
-
- /** @return A UUID that can be used to identify particular requests. */
- abstract String generateUUID();
-
- /** Determine whether or not Chrome is currently being used actively. */
- abstract boolean isChromeBeingUsed();
-
- /**
- * Schedules the {@link Service} to run again at the given time.
- * @param service Service that is doing the scheduling.
- * @param nextTimestampMs When the service should be run again.
- */
- abstract void scheduleService(Service service, long nextTimestampMs);
-
- /** Creates a {@link RequestGenerator}. */
- abstract RequestGenerator createRequestGenerator(Context context);
-
- /**
- * Called when {@link OmahaClient#registerNewRequest} finishes.
- * @param timestampRequestMs When the next active user request should be generated.
- * @param timestampPostMs Earliest time the next POST should be allowed.
- */
- void onRegisterNewRequestDone(long timestampRequestMs, long timestampPostMs) {}
-
- /**
- * Called when {@link OmahaClient#handlePostRequest} finishes.
- * @param result See {@link PostResult}.
- * @param installEventWasSent Whether or not an install event was sent.
- */
- void onHandlePostRequestDone(@PostResult int result, boolean installEventWasSent) {}
-
- /**
- * Called when {@link OmahaClient#generateAndPostRequest} finishes.
- * @param succeeded Whether or not the post was successfully received by the server.
- */
- void onGenerateAndPostRequestDone(boolean succeeded) {}
-
- /**
- * Called when {@link OmahaClient#saveState} finishes.
- * @param timestampRequestMs When the next active user request should be generated.
- * @param timestampPostMs Earliest time the next POST should be allowed.
- */
- void onSaveStateDone(long timestampRequestMs, long timestampPostMs) {}
-}

Powered by Google App Engine
This is Rietveld 408576698