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

Issue 2736603002: mediaview: Support watchers in ArcFileSystemOperationRunner. (Closed)

Created:
3 years, 9 months ago by Shuhei Takahashi
Modified:
3 years, 9 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/3029
Project:
chromium
Visibility:
Public.

Description

mediaview: Support watchers in ArcFileSystemOperationRunner. AddWatcher() and RemoveWatcher() are added. Also, ArcFileSystemOperationRunner implements FileSystemHost to receive watcher callbacks from the Android container. BUG=chromium:684233 TEST=trybot Review-Url: https://codereview.chromium.org/2715493002 Cr-Commit-Position: refs/heads/master@{#454589} (cherry picked from commit 1cf2cd5fd31329f2cb7de2cd2df919648b5e331b) Review-Url: https://codereview.chromium.org/2736603002 . Cr-Commit-Position: refs/branch-heads/3029@{#12} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} Committed: https://chromium.googlesource.com/chromium/src/+/38e21206c01bdbb584ba2deb8dc23d5774755ccc

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+426 lines, -113 lines) Patch
M chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner.h View 9 chunks +49 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner.cc View 5 chunks +148 lines, -13 lines 0 comments Download
M chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner_unittest.cc View 4 chunks +50 lines, -82 lines 0 comments Download
M chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner_util.h View 2 chunks +54 lines, -6 lines 0 comments Download
M chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner_util.cc View 6 chunks +125 lines, -9 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
Shuhei Takahashi
3 years, 9 months ago (2017-03-06 03:31:58 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
38e21206c01bdbb584ba2deb8dc23d5774755ccc.

Powered by Google App Engine
This is Rietveld 408576698