bindings: Add missing default values to FileSystem APIs

Bug: 839389
Change-Id: I8aed766371f6422df1ac671e8a6a0d12fdea8a34
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2079729Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Yuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#745446}

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

@@ -32,9 +32,9 @@ partial interface DedicatedWorkerGlobalScope {
 
     // TODO(crbug.com/841185): |successCallback| and |errorCallback| are not
     // nullable in the spec.
-    [RuntimeEnabled=FileSystem, MeasureAs=RequestFileSystemWorker] void webkitRequestFileSystem(unsigned short type, long long size, optional FileSystemCallback? successCallback, optional ErrorCallback? errorCallback);
+    [RuntimeEnabled=FileSystem, MeasureAs=RequestFileSystemWorker] void webkitRequestFileSystem(unsigned short type, long long size, optional FileSystemCallback? successCallback = null, optional ErrorCallback? errorCallback = null);
     [RuntimeEnabled=FileSystem, RaisesException, MeasureAs=RequestFileSystemSyncWorker] DOMFileSystemSync webkitRequestFileSystemSync(unsigned short type, long long size);
     // TODO(crbug.com/841185): |errorCallback| is not nullable in the spec.
-    [RuntimeEnabled=FileSystem] void webkitResolveLocalFileSystemURL(DOMString url, EntryCallback successCallback, optional ErrorCallback? errorCallback);
+    [RuntimeEnabled=FileSystem] void webkitResolveLocalFileSystemURL(DOMString url, EntryCallback successCallback, optional ErrorCallback? errorCallback = null);
     [RuntimeEnabled=FileSystem, RaisesException] EntrySync webkitResolveLocalFileSystemSyncURL(DOMString url);
 };

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

@@ -32,9 +32,9 @@ partial interface SharedWorkerGlobalScope {
 
     // TODO(crbug.com/841185): |successCallback| and |errorCallback| are not
     // nullable in the spec.
-    [RuntimeEnabled=FileSystem, MeasureAs=RequestFileSystemWorker] void webkitRequestFileSystem(unsigned short type, long long size, optional FileSystemCallback? successCallback, optional ErrorCallback? errorCallback);
+    [RuntimeEnabled=FileSystem, MeasureAs=RequestFileSystemWorker] void webkitRequestFileSystem(unsigned short type, long long size, optional FileSystemCallback? successCallback = null, optional ErrorCallback? errorCallback = null);
     [RuntimeEnabled=FileSystem, RaisesException, MeasureAs=RequestFileSystemSyncWorker] DOMFileSystemSync webkitRequestFileSystemSync(unsigned short type, long long size);
     // TODO(crbug.com/841185): |errorCallback| is not nullable in the spec.
-    [RuntimeEnabled=FileSystem] void webkitResolveLocalFileSystemURL(DOMString url, EntryCallback successCallback, optional ErrorCallback? errorCallback);
+    [RuntimeEnabled=FileSystem] void webkitResolveLocalFileSystemURL(DOMString url, EntryCallback successCallback, optional ErrorCallback? errorCallback = null);
     [RuntimeEnabled=FileSystem, RaisesException] EntrySync webkitResolveLocalFileSystemSyncURL(DOMString url);
 };

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

@@ -33,8 +33,8 @@
 
     // TODO(crbug.com/841185): |errorCallback| is not nullable in the spec.
     [RuntimeEnabled=FileSystem, MeasureAs=RequestFileSystem] void webkitRequestFileSystem(unsigned short type, long long size,
-            FileSystemCallback successCallback, optional ErrorCallback? errorCallback);
+            FileSystemCallback successCallback, optional ErrorCallback? errorCallback = null);
     // TODO(crbug.com/841185): |errorCallback| is not nullable in the spec.
     [RuntimeEnabled=FileSystem] void webkitResolveLocalFileSystemURL(DOMString url,
-            EntryCallback successCallback, optional ErrorCallback? errorCallback);
+            EntryCallback successCallback, optional ErrorCallback? errorCallback = null);
 };