[CSS Typed OM] Delete obsolete number and length classes from Typed OM

Delete obsolete number and length classes from Typed OM

This breaks most of our layout tests, since most things
depend on numbers and lengths, but it's better to be in
a single half state than have two half states hanging
around.

BUG=545318
Review-Url: https://codereview.chromium.org/2867883003
Cr-Commit-Position: refs/heads/master@{#474925}
Committed: https://chromium.googlesource.com/chromium/src/+/42110b22278552b70e9b5156148df454e77f6be5

[meade_UTC10, 2017-05-09 07:05:25]

Description was changed from

==========
Delete obsolete number and length classes from Typed OM

This CL is dependent on rjwright's CL:
https://codereview.chromium.org/2791193004
==========

to

==========
Delete obsolete number and length classes from Typed OM

This CL is dependent on rjwright's CL:
https://codereview.chromium.org/2791193004
==========

[meade_UTC10, 2017-05-09 07:05:31]

The CQ bit was checked by meade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-09 07:05:47]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-09 07:23:08]

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

[commit-bot: I haz the power, 2017-05-09 07:23:09]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[meade_UTC10, 2017-05-11 07:35:26]

The CQ bit was checked by meade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-11 07:36:10]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[meade_UTC10, 2017-05-11 07:41:06]

The CQ bit was checked by meade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-11 07:41:44]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-11 07:46:52]

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

[commit-bot: I haz the power, 2017-05-11 07:46:53]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[meade_UTC10, 2017-05-11 08:23:29]

meade@chromium.org changed reviewers:
+ rjwright@chromium.org

[meade_UTC10, 2017-05-11 08:23:30]

The bots are failing because of that silly bug where the bot_update step fails
if you modify a file in both the depended-upon patch and delete it in the patch.
The typed OM layout tests are all running as expected locally. I'll try it again
once the other patch is submitted.

cc'ing shans and ikilpatrick as this is a big change that basically breaks
everything, but I still think it's better to have a single half state, than two
concurrent half states.

[meade_UTC10, 2017-05-11 08:27:10]

Description was changed from

==========
Delete obsolete number and length classes from Typed OM

This CL is dependent on rjwright's CL:
https://codereview.chromium.org/2791193004
==========

to

==========
Delete obsolete number and length classes from Typed OM

This breaks most of our layout tests, since most things
depend on numbers and lengths, but it's better to be in
a single half state than have two half states hanging
around.

BUG=545318
==========

[meade_UTC10, 2017-05-11 08:27:10]

meade@chromium.org changed reviewers:
+ nainar@chromium.org
- rjwright@chromium.org

[meade_UTC10, 2017-05-11 08:27:32]

nainar: Since you volunteered, PTAL!

[meade_UTC10, 2017-05-11 08:32:17]

Description was changed from

==========
Delete obsolete number and length classes from Typed OM

This breaks most of our layout tests, since most things
depend on numbers and lengths, but it's better to be in
a single half state than have two half states hanging
around.

BUG=545318
==========

to

==========
Delete obsolete number and length classes from Typed OM

This breaks most of our layout tests, since most things
depend on numbers and lengths, but it's better to be in
a single half state than have two half states hanging
around.

This CL is dependent on rjwright's patch:
https://codereview.chromium.org/2791193004/

BUG=545318
==========

[nainar, 2017-05-12 06:15:58]

nainar@chromium.org changed reviewers:
- nainar@chromium.org

[meade_UTC10, 2017-05-15 05:52:08]

Description was changed from

==========
Delete obsolete number and length classes from Typed OM

This breaks most of our layout tests, since most things
depend on numbers and lengths, but it's better to be in
a single half state than have two half states hanging
around.

This CL is dependent on rjwright's patch:
https://codereview.chromium.org/2791193004/

BUG=545318
==========

to

==========
Delete obsolete number and length classes from Typed OM

This breaks most of our layout tests, since most things
depend on numbers and lengths, but it's better to be in
a single half state than have two half states hanging
around.

BUG=545318
==========

[meade_UTC10, 2017-05-15 06:08:23]

meade@chromium.org changed reviewers:
+ nainar@chromium.org

[meade_UTC10, 2017-05-15 06:08:24]

Ok, I've rebased this now. PTAL

[meade_UTC10, 2017-05-15 06:08:47]

The CQ bit was checked by meade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-15 06:08:59]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-15 08:03:53]

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

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

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[nainar, 2017-05-15 08:20:31]

On 2017/05/15 at 06:08:24, meade wrote:
> Ok, I've rebased this now. PTAL

This l*tm but you have a bunch more failing tests:
csspaint/registered-properties-in-custom-paint.html
+csspaint/style-first-letter-pseudo.html
external/wpt/css-typed-om/styleMap-update-function.html
http/tests/worklet/webexposed/global-interface-listing-paint-worklet.html
typedcssom/inlinestyle/numbers.html
webexposed/global-interface-listing.html

[meade_UTC10, 2017-05-15 09:11:19]

Ah oops, I missed some. Fixed (I think.. will check once the bots run again)

[meade_UTC10, 2017-05-15 09:11:39]

The CQ bit was checked by meade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-15 09:11:53]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-15 10:46:26]

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

[commit-bot: I haz the power, 2017-05-15 10:46:27]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[nainar1, 2017-05-16 00:20:46]

nainar@google.com changed reviewers:
+ nainar@google.com

[nainar1, 2017-05-16 00:20:48]

lgtm once you can fix the failing tests

https://codereview.chromium.org/2867883003/diff/140001/third_party/WebKit/Lay...
File
third_party/WebKit/LayoutTests/external/wpt/css-typed-om/styleMap-update-function.html
(right):

https://codereview.chromium.org/2867883003/diff/140001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/external/wpt/css-typed-om/styleMap-update-function.html:12:
element.styleMap.set('width', new CSSUnitValue(42, 'px'));
This test is failing

https://codereview.chromium.org/2867883003/diff/140001/third_party/WebKit/Lay...
File
third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
(left):

https://codereview.chromium.org/2867883003/diff/140001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt:506:
interface CSSAngleValue : CSSStyleValue
Change expectation for: 
http/tests/worklet/webexposed/global-interface-listing-paint-worklet.html
virtual/mojo-loading/http/tests/worklet/webexposed/global-interface-listing-paint-worklet.html
virtual/mojo-loading/webexposed/global-interface-listing.html	virtual/service-worker-navigation-preload-disabled/webexposed/global-interface-listing.html
virtual/threaded/http/tests/worklet/webexposed/global-interface-listing-paint-worklet.html
webexposed/global-interface-listing.html

[nainar1, 2017-05-16 00:20:49]

lgtm once you can fix the failing tests

[nainar, 2017-05-17 01:34:08]

lgtm

[meade_UTC10, 2017-05-18 01:08:34]

The CQ bit was checked by meade@chromium.org

[meade_UTC10, 2017-05-18 01:08:35]

The patchset sent to the CQ was uploaded after l-g-t-m from nainar@chromium.org,
nainar@google.com
Link to the patchset: https://codereview.chromium.org/2867883003/#ps160001
(title: "Fix missed layout tests")

[commit-bot: I haz the power, 2017-05-18 01:09:43]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-18 03:01:42]

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

[commit-bot: I haz the power, 2017-05-18 03:01:44]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[meade_UTC10, 2017-05-18 03:21:58]

The CQ bit was checked by meade@chromium.org

[commit-bot: I haz the power, 2017-05-18 03:24:05]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-18 04:21:06]

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

[commit-bot: I haz the power, 2017-05-18 04:21:07]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[meade_UTC10, 2017-05-23 08:31:12]

The CQ bit was checked by meade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-23 08:31:52]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-23 10:04:58]

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

[commit-bot: I haz the power, 2017-05-23 10:04:59]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[meade_UTC10, 2017-05-24 00:27:42]

The CQ bit was checked by meade@chromium.org

[meade_UTC10, 2017-05-24 00:27:43]

The patchset sent to the CQ was uploaded after l-g-t-m from nainar@chromium.org,
nainar@google.com
Link to the patchset: https://codereview.chromium.org/2867883003/#ps240001
(title: "Fix more missed test expectations")

[commit-bot: I haz the power, 2017-05-24 00:28:34]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-24 00:34:32]

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

[commit-bot: I haz the power, 2017-05-24 00:34:34]

Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  android_cronet on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[meade_UTC10, 2017-05-25 05:09:06]

The CQ bit was checked by meade@chromium.org

[meade_UTC10, 2017-05-25 05:09:07]

The patchset sent to the CQ was uploaded after l-g-t-m from nainar@chromium.org,
nainar@google.com
Link to the patchset: https://codereview.chromium.org/2867883003/#ps260001
(title: "rebase")

[commit-bot: I haz the power, 2017-05-25 05:09:29]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-25 06:39:26]

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

[commit-bot: I haz the power, 2017-05-25 06:39:28]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[meade_UTC10, 2017-05-25 06:48:13]

The CQ bit was checked by meade@chromium.org

[meade_UTC10, 2017-05-25 06:48:14]

The patchset sent to the CQ was uploaded after l-g-t-m from nainar@chromium.org,
nainar@google.com
Link to the patchset: https://codereview.chromium.org/2867883003/#ps280001
(title: "Fix typo")

[commit-bot: I haz the power, 2017-05-25 06:48:31]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-25 08:31:28]

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

[commit-bot: I haz the power, 2017-05-25 08:31:30]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[meade_UTC10, 2017-05-26 01:20:42]

The CQ bit was checked by meade@chromium.org

[meade_UTC10, 2017-05-26 01:20:43]

The patchset sent to the CQ was uploaded after l-g-t-m from nainar@chromium.org,
nainar@google.com
Link to the patchset: https://codereview.chromium.org/2867883003/#ps320001
(title: "rebase")

[commit-bot: I haz the power, 2017-05-26 01:22:10]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-26 05:37:32]

CQ is committing da patch.

Bot data: {"patchset_id": 320001, "attempt_start_ts": 1495761642662090,
"parent_rev": "ea2ef980b38e4a7f741f9f8220f48841ddd71df7", "commit_rev":
"42110b22278552b70e9b5156148df454e77f6be5"}

[commit-bot: I haz the power, 2017-05-26 05:37:54]

Description was changed from

==========
Delete obsolete number and length classes from Typed OM

This breaks most of our layout tests, since most things
depend on numbers and lengths, but it's better to be in
a single half state than have two half states hanging
around.

BUG=545318
==========

to

==========
Delete obsolete number and length classes from Typed OM

This breaks most of our layout tests, since most things
depend on numbers and lengths, but it's better to be in
a single half state than have two half states hanging
around.

BUG=545318

Review-Url: https://codereview.chromium.org/2867883003
Cr-Commit-Position: refs/heads/master@{#474925}
Committed:
https://chromium.googlesource.com/chromium/src/+/42110b22278552b70e9b5156148d...
==========

[commit-bot: I haz the power, 2017-05-26 05:37:56]

Committed patchset #17 (id:320001) as
https://chromium.googlesource.com/chromium/src/+/42110b22278552b70e9b5156148d...