Remove CSS Exclusions leftovers

Remove CSS Exclusions leftovers

https://codereview.chromium.org/60013004 started this but must have missed these.

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

[Bem Jones-Bey (adobe), 2014-06-27 18:25:59]

https://codereview.chromium.org/359023002/diff/1/Source/core/css/CSSValueKeyw...
File Source/core/css/CSSValueKeywords.in (right):

https://codereview.chromium.org/359023002/diff/1/Source/core/css/CSSValueKeyw...
Source/core/css/CSSValueKeywords.in:889: wrap
Why is this left here? Is it used by something else?

https://codereview.chromium.org/359023002/diff/1/Source/core/frame/UseCounter...
File Source/core/frame/UseCounter.cpp (left):

https://codereview.chromium.org/359023002/diff/1/Source/core/frame/UseCounter...
Source/core/frame/UseCounter.cpp:402: case CSSPropertyWebkitWrapThrough: return
351;
Shouldn't these be commented just like the other entries that have been removed?

[rwlbuis, 2014-06-27 18:28:35]

On 2014/06/27 18:25:59, Bem Jones-Bey (adobe) wrote:
>
https://codereview.chromium.org/359023002/diff/1/Source/core/css/CSSValueKeyw...
> File Source/core/css/CSSValueKeywords.in (right):
> 
>
https://codereview.chromium.org/359023002/diff/1/Source/core/css/CSSValueKeyw...
> Source/core/css/CSSValueKeywords.in:889: wrap
> Why is this left here? Is it used by something else?

I removed it at first, only to get compile errors :) Yep flex-wrap uses it.

>
https://codereview.chromium.org/359023002/diff/1/Source/core/frame/UseCounter...
> File Source/core/frame/UseCounter.cpp (left):
> 
>
https://codereview.chromium.org/359023002/diff/1/Source/core/frame/UseCounter...
> Source/core/frame/UseCounter.cpp:402: case CSSPropertyWebkitWrapThrough:
return
> 351;
> Shouldn't these be commented just like the other entries that have been
removed?

Maybe, I am not very familiar with the usecounter stuff, will have a look.

[Bem Jones-Bey (adobe), 2014-06-27 20:34:37]

lgtm

[rwlbuis, 2014-06-27 20:54:31]

PTAL.

[abarth-chromium, 2014-06-27 21:55:23]

lgtm

[rwlbuis, 2014-06-27 21:57:31]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-06-27 21:58:35]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rob.buis@samsung.com/359023002/60001

[commit-bot: I haz the power, 2014-06-27 23:07:20]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_db...)
  android_blink_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_re...)
  android_chromium_gn_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_chromium_gn_comp...)
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/8783)
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  linux_chromium_gn_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/bu...)
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/13218)
  win_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_compile_dbg/bu...)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/14199)
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/22740)

[commit-bot: I haz the power, 2014-06-27 23:08:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-27 23:08:49]

Try jobs failed on following builders:
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/13226)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/22769)

[rwlbuis, 2014-06-28 02:39:29]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-06-28 02:40:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rob.buis@samsung.com/359023002/60001

[commit-bot: I haz the power, 2014-06-28 02:45:16]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_db...)
  android_blink_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_re...)
  android_chromium_gn_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_chromium_gn_comp...)
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/8799)
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  linux_chromium_gn_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/bu...)
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/13238)
  win_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_compile_dbg/bu...)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/14221)
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/22824)

[commit-bot: I haz the power, 2014-06-28 02:46:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-28 02:46:38]

Try jobs failed on following builders:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/22826)

[rwlbuis, 2014-06-28 20:24:34]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-06-28 20:25:33]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rob.buis@samsung.com/359023002/80001

[commit-bot: I haz the power, 2014-06-28 21:32:25]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_db...)
  android_blink_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_re...)

[commit-bot: I haz the power, 2014-06-28 22:44:10]

Change committed as 177167