Commit 18d4c919 authored by Sky Malice's avatar Sky Malice Committed by Commit Bot

[Feed] Call simple triggerRefresh method.

Bug: 933395
Change-Id: I2b76b43f15c695449c6af30db0992b386fd02222
Reviewed-on: https://chromium-review.googlesource.com/c/1476827
Commit-Queue: Sky Malice <skym@chromium.org>
Reviewed-by: default avatarGang Wu <gangwu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#633351}
parent 9ec76df9
......@@ -147,8 +147,7 @@ public class FeedProcessScopeFactory {
.setContentStorage(contentStorage)
.setJournalStorage(journalStorage)
.build();
schedulerBridge.initializeFeedDependencies(
sFeedProcessScope.getRequestManager(), sFeedProcessScope.getSessionManager());
schedulerBridge.initializeFeedDependencies(sFeedProcessScope.getRequestManager());
sFeedOfflineIndicator =
new FeedOfflineBridge(profile, sFeedProcessScope.getKnownContentApi());
......
......@@ -6,9 +6,7 @@ package org.chromium.chrome.browser.feed;
import android.support.annotation.NonNull;
import com.google.android.libraries.feed.api.common.MutationContext;
import com.google.android.libraries.feed.api.requestmanager.RequestManager;
import com.google.android.libraries.feed.api.sessionmanager.SessionManager;
import com.google.android.libraries.feed.host.scheduler.SchedulerApi;
import com.google.search.now.wire.feed.FeedQueryProto.FeedQuery.RequestReason;
......@@ -24,7 +22,6 @@ import org.chromium.components.feed.NativeRequestBehavior;
public class FeedSchedulerBridge implements FeedScheduler {
private long mNativeBridge;
private RequestManager mRequestManager;
private SessionManager mSessionManager;
/**
* Creates a FeedSchedulerBridge for accessing native scheduling logic.
......@@ -46,17 +43,13 @@ public class FeedSchedulerBridge implements FeedScheduler {
* Sets our copies for various interfaces provided by the Feed library. Should be done as early
* as possible, as the scheduler will be unable to trigger refreshes until after it has the
* mechanisms to correctly do so. When this is called, it is assumed that the given
* RequestManager and SessionManager are initialized and can be used immediately.
* RequestManager is initialized and can be used immediately.
*
* @param requestManager The interface that allows us make refresh requests.
* @param sessionManager The interface that provides correct consumtion of refresh results.
*/
public void initializeFeedDependencies(
@NonNull RequestManager requestManager, @NonNull SessionManager sessionManager) {
public void initializeFeedDependencies(@NonNull RequestManager requestManager) {
assert mRequestManager == null;
assert mSessionManager == null;
mRequestManager = requestManager;
mSessionManager = sessionManager;
}
@Override
......@@ -130,9 +123,8 @@ public class FeedSchedulerBridge implements FeedScheduler {
@CalledByNative
private boolean triggerRefresh() {
if (mRequestManager != null && mSessionManager != null) {
mRequestManager.triggerRefresh(RequestReason.SCHEDULED_REFRESH,
mSessionManager.getUpdateConsumer(MutationContext.EMPTY_CONTEXT));
if (mRequestManager != null) {
mRequestManager.triggerRefresh(RequestReason.SCHEDULED_REFRESH);
return true;
}
return false;
......
......@@ -8,7 +8,6 @@ import android.support.test.filters.SmallTest;
import android.support.test.rule.UiThreadTestRule;
import com.google.android.libraries.feed.api.requestmanager.RequestManager;
import com.google.android.libraries.feed.api.sessionmanager.SessionManager;
import com.google.android.libraries.feed.testing.conformance.scheduler.SchedulerConformanceTest;
import org.junit.After;
......@@ -58,8 +57,6 @@ public final class FeedSchedulerBridgeConformanceTest extends SchedulerConforman
@Mock
private RequestManager mRequestManager;
@Mock
private SessionManager mSessionManager;
private boolean mUseRequestManager;
public FeedSchedulerBridgeConformanceTest(boolean useRequestManager) {
......@@ -71,8 +68,7 @@ public final class FeedSchedulerBridgeConformanceTest extends SchedulerConforman
// The scheduler is declared and tested in SchedulerConformanceTest.
scheduler = new FeedSchedulerBridge(Profile.getLastUsedProfile());
if (mUseRequestManager) {
((FeedSchedulerBridge) scheduler)
.initializeFeedDependencies(mRequestManager, mSessionManager);
((FeedSchedulerBridge) scheduler).initializeFeedDependencies(mRequestManager);
}
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment