New issue
Advanced search Search tips

Issue 720533 link

Starred by 1 user
Status: WontFix
Owner:
mcasas@chromium.org
Closed: May 2017
Cc:
fbeaufort@chromium.org
mcasas@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Image Capture: do not fill in PhotoCapabilities dictionary members if not necessary

Project Member Reported by mcasas@chromium.org, May 10 2017 
From JS I might get something like: 
PhotoCapabilities
 fillLightMode : Array(0)
 imageHeight : null
 imageWidth : null
 redEyeReduction : "never"

where the first three should not be present if not adding info.

Comment 1 by mcasas@chromium.org, May 10 2017

PhotoCapabilities.PNG
6.2 KB View Download

Comment 2 by mcasas@chromium.org, May 10 2017

Status: WontFix (was: Assigned)
Sorry, my bad: PhotoCapabilities is not a dictionary, but an interface,
defined as [1]

interface PhotoCapabilities {
  readonly attribute RedEyeReduction            redEyeReduction;
  readonly attribute MediaSettingsRange         imageHeight;
  readonly attribute MediaSettingsRange         imageWidth;
  readonly attribute FrozenArray<FillLightMode> fillLightMode;
};

So, the behaviour is correct, all the member fields have to be there.



[1] https://cs.chromium.org/chromium/src/third_party/WebKit/Source/modules/imagecapture/PhotoCapabilities.idl?q=PhotoCapabilities.idl&sq=package:chromium&dr
►

Sign in to add a comment