Skip to content

Commit

Permalink
Fixed a strict mode violation in BackgroundTaskScheduler
Browse files Browse the repository at this point in the history
Bug: 1061957
Change-Id: I5ef5b4f26b1ee2b02ff2f6d221bc32efad23c124
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2158938
Reviewed-by: Min Qin <qinmin@chromium.org>
Commit-Queue: Shakti Sahu <shaktisahu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#761206}
  • Loading branch information
Shakti Sahu authored and Commit Bot committed Apr 21, 2020
1 parent e420577 commit 20768a5
Showing 1 changed file with 9 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@

import org.chromium.base.ContextUtils;
import org.chromium.base.Log;
import org.chromium.base.StrictModeContext;
import org.chromium.base.TraceEvent;
import org.chromium.components.background_task_scheduler.TaskInfo;

Expand Down Expand Up @@ -143,12 +144,14 @@ String getSerializedScheduledTask() {

@Override
public void visit(TaskInfo.OneOffInfo oneOffInfo) {
ScheduledTaskProto.ScheduledTask scheduledTask =
ScheduledTaskProto.ScheduledTask.newBuilder()
.setType(ScheduledTaskProto.ScheduledTask.Type.ONE_OFF)
.build();
mSerializedScheduledTask =
Base64.encodeToString(scheduledTask.toByteArray(), Base64.DEFAULT);
try (StrictModeContext ignored = StrictModeContext.allowDiskReads()) {
ScheduledTaskProto.ScheduledTask scheduledTask =
ScheduledTaskProto.ScheduledTask.newBuilder()
.setType(ScheduledTaskProto.ScheduledTask.Type.ONE_OFF)
.build();
mSerializedScheduledTask =
Base64.encodeToString(scheduledTask.toByteArray(), Base64.DEFAULT);
}
}

@Override
Expand Down

0 comments on commit 20768a5

Please sign in to comment.