colorDepth/pixelDepth should not include alpha channel in Android |
|||||
Issue descriptionChrome Canary 64.0.3274.0 Android 8.1.0; Pixel Build/Opp5.170921.005 What steps will reproduce the problem? 1. Open DevTools and check `screen.colorDepth` value What is the expected result? It should be 24 What happens instead of that? It is 32 According to the spec https://drafts.csswg.org/cssom-view/#screen, The colorDepth and pixelDepth attributes should return the number of bits allocated to colors for a pixel in the output device, excluding the alpha channel.
,
Nov 24 2017
,
Nov 24 2017
,
Nov 27 2017
,
Nov 28 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/64f45fc4f73a556b64e8254ffa5b1dc040f5b3f8 commit 64f45fc4f73a556b64e8254ffa5b1dc040f5b3f8 Author: Mounir Lamouri <mlamouri@chromium.org> Date: Tue Nov 28 17:04:47 2017 Do not use alpha channel for colorDepth/pixelDepth on Android. Android is the only Chrome platform that takes into account the alpha channel. This value is used for CSS Media Queries and Screen object on the Web Platform. Bug: 788329 Change-Id: I9f3f20db1b00fcf8fbf2588f23785b695deb6546 Reviewed-on: https://chromium-review.googlesource.com/789333 Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Reviewed-by: Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#519710} [modify] https://crrev.com/64f45fc4f73a556b64e8254ffa5b1dc040f5b3f8/ui/android/java/src/org/chromium/ui/display/PhysicalDisplayAndroid.java
,
Nov 28 2017
,
Nov 29 2017
Verified at https://beaufortfrancois.github.io/sandbox/media-capabilities/screen.html in Chromium 64.0.3281.0. Thanks! |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by fbeaufort@chromium.org
, Nov 24 2017