New issue
Advanced search Search tips

Issue 879388 link

Starred by 1 user
Status: Fixed
Owner:
w...@chromium.org
Closed: Aug 31
Components:
EstimatedDays: ----
NextAction: ----
OS: Fuchsia
Pri: 1
Type: Bug



Sign in to add a comment

Add 'using' declarations for dependencies to FIDL definitions

Project Member Reported by w...@chromium.org, Aug 30 
Future versions of fidlc will require explicit 'using' declarations for all dependencies, as per FIDL-111.

We will need to update e.g. the ContextProvider FIDL to comply with this.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 31 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/07fca6c693f032dd3a8cfebb4cbb89b9881b90c0

commit 07fca6c693f032dd3a8cfebb4cbb89b9881b90c0
Author: Wez <wez@chromium.org>
Date: Fri Aug 31 00:54:41 2018

Add missing 'using' declarations to fuchsia.web.Frame definition.

In future the fidlc tool will require explicit 'using' declarations for
all dependencies.

Bug:  879388 
Change-Id: Ic82e6c7338d31e045ab20f24044fb97fbc151bbf
Reviewed-on: https://chromium-review.googlesource.com/1198012
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#587880}
[modify] https://crrev.com/07fca6c693f032dd3a8cfebb4cbb89b9881b90c0/webrunner/fidl/frame.fidl

Comment 2 by w...@chromium.org, Aug 31

Status: Fixed (was: Started)
►

Sign in to add a comment