[CSSMQ] Allow integer values for monochrome media feature.

[CSSMQ] Allow integer values for monochrome media feature.

min/max-monochrome already did.

BUG=236334
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=150415

[apavlov, 2013-05-15 07:41:48]

https://codereview.chromium.org/14203017/diff/1/LayoutTests/fast/media/media-...
File LayoutTests/fast/media/media-feature-monochrome.html (right):

https://codereview.chromium.org/14203017/diff/1/LayoutTests/fast/media/media-...
LayoutTests/fast/media/media-feature-monochrome.html:4: @media not all and
(monochrome: 0) { test::before { content: "PASS"; } }
Should the selector be actually #test::before ?

[rune, 2013-05-15 08:49:38]

On 2013/05/15 07:41:48, apavlov wrote:

> (monochrome: 0) { test::before { content: "PASS"; } }
> Should the selector be actually #test::before ?

Yes. Fixed in Patch Set 2.

[apavlov, 2013-05-15 08:54:01]

lgtm

[commit-bot: I haz the power, 2013-05-15 08:54:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rune@opera.com/14203017/5001

[commit-bot: I haz the power, 2013-05-15 08:54:10]

Failed to apply patch for Source/core/css/MediaQueryExp.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/css/MediaQueryExp.cpp
  Hunk #1 FAILED at 86.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/core/css/MediaQueryExp.cpp.rej

Patch:       Source/core/css/MediaQueryExp.cpp
Index: Source/core/css/MediaQueryExp.cpp
diff --git a/Source/core/css/MediaQueryExp.cpp
b/Source/core/css/MediaQueryExp.cpp
index
711ee9d7a79cb645ed2dc35bbd36da72967f99c6..42584f098d000b4eb9e4f8c153b38398899c3ff5
100644
--- a/Source/core/css/MediaQueryExp.cpp
+++ b/Source/core/css/MediaQueryExp.cpp
@@ -86,6 +86,7 @@ static inline bool featureWithPositiveInteger(const
AtomicString& mediaFeature,
     return mediaFeature == MediaFeatureNames::colorMediaFeature
         || mediaFeature == MediaFeatureNames::max_colorMediaFeature
         || mediaFeature == MediaFeatureNames::min_colorMediaFeature
+        || mediaFeature == MediaFeatureNames::monochromeMediaFeature
         || mediaFeature == MediaFeatureNames::min_monochromeMediaFeature
         || mediaFeature == MediaFeatureNames::max_monochromeMediaFeature;
 }

[commit-bot: I haz the power, 2013-05-15 09:21:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rune@opera.com/14203017/13001

[commit-bot: I haz the power, 2013-05-15 13:39:10]

Change committed as 150415