Commit c88d1ddd authored by Ian Wells's avatar Ian Wells Committed by Commit Bot

Don't call SpinnerLogger.spinnerFinished() if spinner isn't shown

Just prevents reaching the Logger.wtf(). Considering a refactoring of
the Feed library is soon to be underway, there's not much point in
investigating why the spinner isn't being shown in the first place.

Bug: 992570
Change-Id: I693bd57bb60e5396f565a3b8fd8bb2c3cc08d1e2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2006423
Commit-Queue: Ian Wells <iwells@chromium.org>
Commit-Queue: Dan H <harringtond@chromium.org>
Auto-Submit: Ian Wells <iwells@chromium.org>
Reviewed-by: default avatarDan H <harringtond@chromium.org>
Cr-Commit-Position: refs/heads/master@{#732912}
parent e39ba319
......@@ -232,7 +232,9 @@ public class ContinuationDriver extends LeafFeatureDriver
public void onError(ModelError modelError) {
mBasicLoggingApi.onTokenFailedToComplete(mModelToken.isSynthetic(), ++mFailureCount);
showErrorUi();
mSpinnerLogger.spinnerFinished();
if (mSpinnerLogger.isSpinnerActive()) {
mSpinnerLogger.spinnerFinished();
}
}
private void showErrorUi() {
......
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