samli@ It's not obvious to me how to make the 4/8-digit support in devTools conditional on a RuntimeEnabledFeature (or if that's even strictly necessary).  Can you do that or revert your patch?

I don't see any necessity to revert devtools side support.

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

commit b7bc5783027ae86ff6eb6912d0a6112cda6a8814
Author: rbyers <rbyers@chromium.org>
Date: Thu Jun 09 04:57:08 2016

Disable 4 and 8 digit hex CSS colors

New compat concerns have been raised with 4 and 8 digit HEX color
values in CSS.  In particular, a targetSdk quirk needs to be added
for Android WebView to avoid breaking Android apps.

Demote this feature to "experimental" status until an owner can be found.

BUG= 618518 ,  76362 

Review-Url: https://codereview.chromium.org/2047413003
Cr-Commit-Position: refs/heads/master@{#398789}

[modify] https://crrev.com/b7bc5783027ae86ff6eb6912d0a6112cda6a8814/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in
[modify] https://crrev.com/b7bc5783027ae86ff6eb6912d0a6112cda6a8814/third_party/WebKit/Source/platform/graphics/Color.cpp

rbyers@, can this be tested manually so that we can verify it on Canary before it gets merged in to M52 branch ?

Confirmed the issues is resolved on 53.0.2768.0

Looks like OS-All, so tagging as such, re-tag if needed.

In the future please make sure all ReleaseBlock-* issues have an OS set, that's how release teams manage blockers.

A friendly reminder that M52 Stable is launching soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch by July 12. All changes MUST be merged into the release branch by 5pm on July 15 to make into the desktop Stable final build cut. Thank you!

rbyers@, As per comment #8 could you please let us know if this can be closed.

Sorry for the delay - BlinkOn.  This has stuck in canary and seems like the lowest-risk thing we can merge to avoid the problem discussed in  issue 618472 .  Requesting merge of b7bc5783027ae86ff6eb6912d0a6112cda6a8814 for 52.

Filed  issue 621538  to track DevTools support separately.

Re #7: simple test page: http://jsbin.com/moyobo
The box will be transparent (grey) when the feature is disabled, and blue when supported.

Your change meets the bar and is auto-approved for M52 (branch: 2743)

I see the transparent box on [Version 53.0.2773.0 canary (64-bit)] - Win, Mac OSX & Linux/Ubuntu 14.04 . Seems like this is the intended behavior as the feature is disabled.

rbyers@, 
Please also have the CL merged in to M52 branch by EOD so that it gets picked up for beta promotion scheduled this week.

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

commit 1c24aa54c19c702703d68bc0bc62b7a5449d3755
Author: Rick Byers <rbyers@chromium.org>
Date: Mon Jun 20 21:20:27 2016

Disable 4 and 8 digit hex CSS colors

New compat concerns have been raised with 4 and 8 digit HEX color
values in CSS.  In particular, a targetSdk quirk needs to be added
for Android WebView to avoid breaking Android apps.

Demote this feature to "experimental" status until an owner can be found.

BUG= 618518 ,  76362 

Review-Url: https://codereview.chromium.org/2047413003
Cr-Commit-Position: refs/heads/master@{#398789}
(cherry picked from commit b7bc5783027ae86ff6eb6912d0a6112cda6a8814)

Review URL: https://codereview.chromium.org/2086433003 .

Cr-Commit-Position: refs/branch-heads/2743@{#413}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/1c24aa54c19c702703d68bc0bc62b7a5449d3755/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in
[modify] https://crrev.com/1c24aa54c19c702703d68bc0bc62b7a5449d3755/third_party/WebKit/Source/platform/graphics/Color.cpp

Tested the issue on windows 7, Linux Ubuntu 14.04 and Mac 10.11.5 using chrome version 52.0.2743.49.Observed the transparent box.Please find the attached screen shot for the same.

Adding TE-Verified label.

Thanks,

Deleted: 618518.png 71.2 KB

	618518.png 71.2 KB View Download