Assigning to Sasha as the P.O.C. for the camera integration. 

Friendly ping.

Sorry, I've been on vacation :)

Let me take a look at this today.

Sorry. I didn't set the priority correctly.

Hi Sheng-Hao,

Thanks for flagging this.

With more investigation, I don't think we actually support adding per-file watchers in the Files app. We originally wanted this functionality for Recents, but in this case we scan at a timed interval to check for File changes. You my have to implement something similar for M69, and we can look into this again for M70+.

Sasha

Hi Sasha,
We are not requesting for per-file watchers. We want to use the existing addFileWatch() for a directory. The problem is that it does not enter the if clause but enter the else clause (https://cs.chromium.org/chromium/src/ui/file_manager/file_manager/foreground/js/directory_model.js?type=cs&q=watcher-directory-changed+file:js&sq=package:chromium&g=0&l=284), which does not provide the specific changed files in callback.

Could this be possibly addressed in M69?

To quote mtomasz@ from the e-mail thread:

"IIRC we only pass list of changed entries for Drive and FSP providers which support it. For Downloads we only know that something in the directory has changed. Maybe this is something which can be fixed, as according to [1] inotify should be able to provide us a list of changes. We use inotify for local files, including Downloads."

[1] https://docs.google.com/document/d/1XNdO-oKq86NqDdlu4z94yx9OVwDFO0AsQAsUDGg25B0/edit#heading=h.2fupsrf8lwh6

AFAIK, all the infrustructure/APIs are already there, we'd just need to plumb it through. But the hard part might be investigating why this hasn't been done already for e.g. performance reasons?

Sorry, but I'd say out of scope for M69. Can you still get the functionality you need for Camera app without it?

Moving to P2 for M69.