[fuchsia] Convert all FIDL files to the new syntax

Bug: 1227712
Change-Id: Icdceeb69d447305ecd324b837fb9de6eb1113768
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3058964
Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Alex Zaslavsky <azaslavsky@google.com>
Cr-Commit-Position: refs/heads/master@{#907122}

base/fuchsia/test.test-fidl

@@ -3,10 +3,14 @@
 // found in the LICENSE file.
 
 // TODO(crbug.com/1227712): Migrate syntax and remove this.
-deprecated_syntax;
 library base.testfidl;
 
-[Discoverable]
+@discoverable
 protocol TestInterface {
-  Add(int32 a, int32 b) -> (int32 sum);
+    Add(struct {
+        a int32;
+        b int32;
+    }) -> (struct {
+        sum int32;
+    });
 };

fuchsia/engine/fidl/dev_tools.fidl

@@ -3,14 +3,14 @@
 // found in the LICENSE file.
 
 // TODO(crbug.com/1227712): Migrate syntax and remove this.
-deprecated_syntax;
-
 library chromium.internal;
 
 using fuchsia.web;
 
-[Discoverable]
+@discoverable
 protocol DevToolsConnector {
-  /// Connects the supplied listener to receive notifications for this instance.
-  ConnectPerContextListener(fuchsia.web.DevToolsPerContextListener listener);
+    /// Connects the supplied listener to receive notifications for this instance.
+    ConnectPerContextListener(resource struct {
+        listener client_end:fuchsia.web.DevToolsPerContextListener;
+    });
 };

fuchsia/fidl/cast/api_bindings.fidl

@@ -3,8 +3,6 @@
 // found in the LICENSE file.
 
 // TODO(crbug.com/1227712): Migrate syntax and remove this.
-deprecated_syntax;
-
 library chromium.cast;
 
 using fuchsia.mem;
@@ -12,17 +10,22 @@ using fuchsia.web;
 
 /// Provides Cast platform API binding scripts and methods for connecting those
 /// scripts with Agent backends.
-[Discoverable]
+@discoverable
 protocol ApiBindings {
     /// Gets the list of bindings to early-inject into the page at load time.
-    GetAll() -> (vector<ApiBinding> bindings);
+    GetAll() -> (resource struct {
+        bindings vector<ApiBinding>;
+    });
 
     /// Should be invoked when a connecting a named MessagePort to a native
     /// bindings backend.
-    Connect(string port_name, fuchsia.web.MessagePort message_port);
+    Connect(resource struct {
+        port_name string;
+        message_port client_end:fuchsia.web.MessagePort;
+    });
 };
 
-resource table ApiBinding {
+type ApiBinding = resource table {
     /// Script to execute before the load of a web document.
-    1: fuchsia.mem.Buffer before_load_script;
+    1: before_load_script fuchsia.mem.Buffer;
 };

fuchsia/fidl/cast/application_config.fidl

@@ -3,8 +3,6 @@
 // found in the LICENSE file.
 
 // TODO(crbug.com/1227712): Migrate syntax and remove this.
-deprecated_syntax;
-
 library chromium.cast;
 
 using fuchsia.diagnostics;
@@ -12,47 +10,50 @@ using fuchsia.ui.gfx;
 using fuchsia.web;
 
 /// Describes the configuration under which a Cast application should run.
-resource table ApplicationConfig {
-  /// Cast application Id.
-  1: string id;
+type ApplicationConfig = resource table {
+    /// Cast application Id.
+    1: id string;
 
-  /// Name to display to the user when referring to this application.
-  2: string display_name;
+    /// Name to display to the user when referring to this application.
+    2: display_name string;
 
-  /// URL from which the application should be loaded, if it has a web-based
-  /// implementation.
-  3: string web_url;
+    /// URL from which the application should be loaded, if it has a web-based
+    /// implementation.
+    3: web_url string;
 
-  4: reserved;
+    4: reserved;
 
-  // If true, enable remote debugging for this application.
-  // if false or unset, remote debugging is disabled for this application.
-  5: bool enable_remote_debugging;
+    // If true, enable remote debugging for this application.
+    // if false or unset, remote debugging is disabled for this application.
+    5: enable_remote_debugging bool;
 
-  /// List of directories whose contents are served via `fuchsia-dir://`
-  /// URLs. Networking will be disabled if any directories are set.
-  6: vector<fuchsia.web.ContentDirectoryProvider>
-      content_directories_for_isolated_application;
+    /// List of directories whose contents are served via `fuchsia-dir://`
+    /// URLs. Networking will be disabled if any directories are set.
+    6: content_directories_for_isolated_application vector<fuchsia.web.ContentDirectoryProvider>;
 
-  /// The URL of the agent to handle this application.
-  7: string agent_url;
+    /// The URL of the agent to handle this application.
+    7: agent_url string;
 
-  /// Simulates a different resolution for rendering.
-  8: fuchsia.ui.gfx.vec2 force_content_dimensions;
+    /// Simulates a different resolution for rendering.
+    8: force_content_dimensions fuchsia.ui.gfx.vec2;
 
-  /// List of the web permissions that should be granted to the application.
-  9: vector<fuchsia.web.PermissionDescriptor> permissions;
+    /// List of the web permissions that should be granted to the application.
+    9: permissions vector<fuchsia.web.PermissionDescriptor>;
 
-  /// The initial minimum severity of `console` logs that will be logged to the
-  /// [`fuchsia.logger/LogSink`] configured for this application. If not set, no
-  /// messages will be logged.
-  // TODO(crbug.com/1088094): Deprecate once there is another mechanism.
-  10: fuchsia.diagnostics.Severity initial_min_console_log_severity;
+    /// The initial minimum severity of `console` logs that will be logged to the
+    /// [`fuchsia.logger/LogSink`] configured for this application. If not set, no
+    /// messages will be logged.
+    // TODO(crbug.com/1088094): Deprecate once there is another mechanism.
+   10: initial_min_console_log_severity fuchsia.diagnostics.Severity;
 };
 
 /// Service interface for working with application configurations.
-[Discoverable]
+@discoverable
 protocol ApplicationConfigManager {
-  /// Returns the ApplicationConfig for the specified application Id.
-  GetConfig(string id) -> (ApplicationConfig config);
+    /// Returns the ApplicationConfig for the specified application Id.
+    GetConfig(struct {
+        id string;
+    }) -> (resource struct {
+        config ApplicationConfig;
+    });
 };

fuchsia/fidl/cast/application_context.fidl

@@ -3,23 +3,27 @@
 // found in the LICENSE file.
 
 // TODO(crbug.com/1227712): Migrate syntax and remove this.
-deprecated_syntax;
-
 library chromium.cast;
 
-[Discoverable]
+@discoverable
 protocol ApplicationContext {
-  /// Returns session_id to use in the AudioConsumer API.
-  GetMediaSessionId() -> (uint64 media_session_id);
+    /// Returns session_id to use in the AudioConsumer API.
+    GetMediaSessionId() -> (struct {
+        media_session_id uint64;
+    });
 
-  /// Used by the Agent to receive a controller from the Cast Runner.
-  /// Can only be called at most one time for the lifetime of the Component.
-  SetApplicationController(ApplicationController controller);
+    /// Used by the Agent to receive a controller from the Cast Runner.
+    /// Can only be called at most one time for the lifetime of the Component.
+    SetApplicationController(resource struct {
+        controller client_end:ApplicationController;
+    });
 
-  /// Called if the application component exits gracefully, with the same
-  /// `exit_code` as would be reported via `ComponentController.OnTerminated()`.
-  /// Values less than or equal to zero are `zx.status` codes.
-  /// May be called at most once, immediately prior to termination.
-  [Transitional]
-  OnApplicationExit(int64 exit_code);
+    /// Called if the application component exits gracefully, with the same
+    /// `exit_code` as would be reported via `ComponentController.OnTerminated()`.
+    /// Values less than or equal to zero are `zx.status` codes.
+    /// May be called at most once, immediately prior to termination.
+    @transitional
+    OnApplicationExit(struct {
+        exit_code int64;
+    });
 };

fuchsia/fidl/cast/application_controller.fidl

@@ -3,8 +3,6 @@
 // found in the LICENSE file.
 
 // TODO(crbug.com/1227712): Migrate syntax and remove this.
-deprecated_syntax;
-
 library chromium.cast;
 
 using fuchsia.media.sessions2;
@@ -12,17 +10,25 @@ using fuchsia.media.sessions2;
 /// Allows clients to access and modify certain aspects of the Cast receiver
 /// application runtime.
 protocol ApplicationController {
-  /// Enables or disables touch event processing.
-  SetTouchInputEnabled(bool enable);
+    /// Enables or disables touch event processing.
+    SetTouchInputEnabled(struct {
+        enable bool;
+    });
 
-  /// Sets whether to block all HTMLMediaElements in the frame from fetching and
-  /// loading media resources.
-  /// See fuchsia.web.Frame for usage details.
-  SetBlockMediaLoading(bool blocked);
+    /// Sets whether to block all HTMLMediaElements in the frame from fetching and
+    /// loading media resources.
+    /// See fuchsia.web.Frame for usage details.
+    SetBlockMediaLoading(struct {
+        blocked bool;
+    });
 
-  /// Connects to the application's media control & observation API.
-  GetMediaPlayer(request<fuchsia.media.sessions2.Player> request);
+    /// Connects to the application's media control & observation API.
+    GetMediaPlayer(resource struct {
+        request server_end:fuchsia.media.sessions2.Player;
+    });
 
-  /// Gets the private memory footprint of the application's main frame.
-  GetPrivateMemorySize() -> (uint64 size_bytes);
+    /// Gets the private memory footprint of the application's main frame.
+    GetPrivateMemorySize() -> (struct {
+        size_bytes uint64;
+    });
 };

fuchsia/fidl/cast/cors_exempt_headers.fidl

@@ -3,15 +3,15 @@
 // found in the LICENSE file.
 
 // TODO(crbug.com/1227712): Migrate syntax and remove this.
-deprecated_syntax;
-
 library chromium.cast;
 
 using fuchsia.web;
 
 /// Provides a list of header names that should be exempted from CORS preflight
 /// security checks for all Cast apps.
-[Discoverable]
+@discoverable
 protocol CorsExemptHeaderProvider {
-    GetCorsExemptHeaderNames() -> (vector<bytes:fuchsia.web.MAX_HEADERS_COUNT>:MAX header_names);
+    GetCorsExemptHeaderNames() -> (struct {
+        header_names vector<bytes:fuchsia.web.MAX_HEADERS_COUNT>:MAX;
+    });
 };

fuchsia/fidl/cast/data_reset.fidl

@@ -3,23 +3,23 @@
 // found in the LICENSE file.
 
 // TODO(crbug.com/1227712): Migrate syntax and remove this.
-deprecated_syntax;
-
 library chromium.cast;
 
 // Used to request that persistent data for components hosted by the Runner is
 // deleted.
-[Discoverable]
+@discoverable
 protocol DataReset {
-  // Deletes all persistent data stored by the Runner.
-  //
-  // Active components hosted by the Runner will be terminated
-  // before any data is removed.
-  //
-  // Returns after deletion of persistent data completes or when an error
-  // occurs. The resulting |succeeded| is true if persistent data stored
-  // on behalf of components apps was successfully deleted, and false if
-  // the request could not be completed (for example, due to an I/O error).
-  [Transitional]
-  DeletePersistentData() -> (bool succeeded);
+    // Deletes all persistent data stored by the Runner.
+    //
+    // Active components hosted by the Runner will be terminated
+    // before any data is removed.
+    //
+    // Returns after deletion of persistent data completes or when an error
+    // occurs. The resulting |succeeded| is true if persistent data stored
+    // on behalf of components apps was successfully deleted, and false if
+    // the request could not be completed (for example, due to an I/O error).
+    @transitional
+    DeletePersistentData() -> (struct {
+        succeeded bool;
+    });
 };

fuchsia/fidl/cast/url_request_rewriter.fidl

@@ -3,18 +3,17 @@
 // found in the LICENSE file.
 
 // TODO(crbug.com/1227712): Migrate syntax and remove this.
-deprecated_syntax;
-
 library chromium.cast;
 
 using fuchsia.web;
 
 /// Provides URL request rewriting rules from the Agent.
-[Discoverable]
+@discoverable
 protocol UrlRequestRewriteRulesProvider {
     /// Returns a set of URL request rewriting rules. The first call must always
     /// return immediately with a potentially empty set of rules. On subsequent
     /// calls, the callback will only be invoked when the rules have changed.
-    GetUrlRequestRewriteRules() ->
-        (vector<fuchsia.web.UrlRequestRewriteRule> rules);
+    GetUrlRequestRewriteRules() -> (struct {
+        rules vector<fuchsia.web.UrlRequestRewriteRule>;
+    });
 };