Storage Service: Add use counters for deprecated FileSystem read/write.

Bug: 1000932
Change-Id: I1c5286092a79e9a5014c77d3a3ba0526280add0f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786923
Commit-Queue: Staphany Park <staphany@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Auto-Submit: Staphany Park <staphany@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693908}

third_party/blink/public/mojom/web_feature/web_feature.mojom

@@ -2405,6 +2405,8 @@ enum WebFeature {
   kCacheStorageWrite = 3022,
   kIndexedDBRead = 3023,
   kIndexedDBWrite = 3024,
+  kDeprecatedFileSystemRead = 3025,
+  kDeprecatedFileSystemWrite = 3026,
 
   // Add new features immediately above this line. Don't change assigned
   // numbers of any item, and don't reuse removed slots.

third_party/blink/renderer/modules/filesystem/directory_entry.idl

@@ -33,12 +33,23 @@
   NoInterfaceObject
 ] interface DirectoryEntry : Entry {
   DirectoryReader createReader();
+
   // TODO(crbug.com/841185): |successCallback| and |errorCallback| are not
   // nullable in the spec.
-    void getFile(DOMString? path, optional FileSystemFlags options, optional EntryCallback? successCallback, optional ErrorCallback? errorCallback);
+  void getFile(DOMString? path,
+               optional FileSystemFlags options,
+               optional EntryCallback? successCallback,
+               optional ErrorCallback? errorCallback);
+
   // TODO(crbug.com/841185): |successCallback| and |errorCallback| are not
   // nullable in the spec.
-    void getDirectory(DOMString? path, optional FileSystemFlags options, optional EntryCallback? successCallback, optional ErrorCallback? errorCallback);
+  void getDirectory(DOMString? path,
+                    optional FileSystemFlags options,
+                    optional EntryCallback? successCallback,
+                    optional ErrorCallback? errorCallback);
+
   // TODO(crbug.com/841185): |errorCallback| is not nullable in the spec.
-    void removeRecursively(VoidCallback successCallback, optional ErrorCallback? errorCallback);
+  [MeasureAs=DeprecatedFileSystemWrite]
+  void removeRecursively(VoidCallback successCallback,
+                         optional ErrorCallback? errorCallback);
 };

third_party/blink/renderer/modules/filesystem/directory_reader.idl

@@ -33,5 +33,7 @@
   NoInterfaceObject
 ] interface DirectoryReader {
   // TODO(crbug.com/841185): |errorCallback| is not nullable in the spec.
-    void readEntries(EntriesCallback successCallback, optional ErrorCallback? errorCallback);
+  [MeasureAs=DeprecatedFileSystemRead]
+  void readEntries(EntriesCallback successCallback,
+                   optional ErrorCallback? errorCallback);
 };

third_party/blink/renderer/modules/filesystem/file_entry.idl

@@ -33,7 +33,11 @@
   NoInterfaceObject
 ] interface FileEntry : Entry {
   // TODO(crbug.com/841185): |errorCallback| is not nullable in the spec.
-    void createWriter(FileWriterCallback successCallback, optional ErrorCallback? errorCallback);
+  void createWriter(FileWriterCallback successCallback,
+                    optional ErrorCallback? errorCallback);
+
   // TODO(crbug.com/841185): |errorCallback| is not nullable in the spec.
-    void file(FileCallback successCallback, optional ErrorCallback? errorCallback);
+  [MeasureAs=DeprecatedFileSystemRead]
+  void file(FileCallback successCallback,
+            optional ErrorCallback? errorCallback);
 };

third_party/blink/renderer/modules/filesystem/file_writer.idl

@@ -41,9 +41,13 @@
   [ImplementedAs=getReadyState] readonly attribute unsigned short readyState;
 
   // async write/modify methods
-    [RaisesException] void write(Blob data);
+  [MeasureAs=DeprecatedFileSystemWrite, RaisesException]
+  void write(Blob data);
+
   [RaisesException] void seek(long long position);
-    [RaisesException] void truncate(long long size);
+
+  [MeasureAs=DeprecatedFileSystemWrite, RaisesException]
+  void truncate(long long size);
 
   [RaisesException] void abort();
 

tools/metrics/histograms/enums.xml

@@ -24592,6 +24592,8 @@ Called by update_net_error_codes.py.-->
   <int value="3022" label="CacheStorageWrite"/>
   <int value="3023" label="IndexedDBRead"/>
   <int value="3024" label="IndexedDBWrite"/>
+  <int value="3025" label="DeprecatedFileSystemRead"/>
+  <int value="3026" label="DeprecatedFileSystemWrite"/>
 </enum>
 
 <enum name="FeaturePolicyAllowlistType">