New issue
Advanced search Search tips

Issue 772004 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

IDL annotation for limiting the length of string arguments and properties

Project Member Reported by peter@chromium.org, Oct 5 2017

Issue description

Certain arguments and properties in the extension IDL bindings may be length limited. A concrete example of this is the `notificationId` argument for notifications:

https://cs.chromium.org/chromium/src/chrome/common/extensions/api/notifications.idl?q=notifications.idl&sq=package:chromium&dr&l=150

This is implemented by checking the length in C++ and throwing an error message manually. Because this is implicit, it needs to be duplicated in the API's documentation.

It would be convenient if it could be defined as:

    [MaxLength=500] optional DOMString notificationId

Which could automatically generate the limitation code, provide consistent error messages and power the documentation view.
 
Project Member

Comment 1 by sheriffbot@chromium.org, Oct 5

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Available (was: Untriaged)

Sign in to add a comment