Remove custom left overs from object grouping

Remove custom left overs from object grouping

BUG=chromium:682688
Review-Url: https://codereview.chromium.org/2675943003
Cr-Commit-Position: refs/heads/master@{#448163}
Committed: https://chromium.googlesource.com/chromium/src/+/2d54dc83d5083e0494f0f642b5d01cfaff39a3b6

[Michael Lippautz, 2017-02-04 02:33:02]

Description was changed from

==========
Remove custom left overs from object grouping.

BUG=chromium:682688
==========

to

==========
Remove custom left overs from object grouping.

BUG=chromium:682688
==========

[Michael Lippautz, 2017-02-04 02:33:03]

mlippautz@chromium.org changed reviewers:
+ haraken@chromium.org, jochen@chromium.org

[Michael Lippautz, 2017-02-04 02:34:01]

ptal

if I interpret CS correctly, then this should be the last round of removals

[Michael Lippautz, 2017-02-04 02:34:20]

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

[commit-bot: I haz the power, 2017-02-04 02:34:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Michael Lippautz, 2017-02-04 02:46:54]

Description was changed from

==========
Remove custom left overs from object grouping.

BUG=chromium:682688
==========

to

==========
Remove custom left overs from object grouping

BUG=chromium:682688
==========

[commit-bot: I haz the power, 2017-02-04 05:37:18]

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

[commit-bot: I haz the power, 2017-02-04 05:37:19]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_TIMED_OUT,
no build URL)
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_TIMED_OUT, no build
URL)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_TIMED_OUT, no
build URL)

[Michael Lippautz, 2017-02-04 05:58:46]

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

[commit-bot: I haz the power, 2017-02-04 05:58:55]

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-02-04 07:59:29]

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

[commit-bot: I haz the power, 2017-02-04 07:59:30]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_TIMED_OUT,
no build URL)
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_TIMED_OUT, no build
URL)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_TIMED_OUT, no
build URL)
  win_clang on master.tryserver.chromium.win (JOB_TIMED_OUT, no build URL)

[haraken, 2017-02-04 09:27:13]

LGTM

[Michael Lippautz, 2017-02-04 16:59:09]

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

[commit-bot: I haz the power, 2017-02-04 16:59:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jochen (gone - plz use gerrit), 2017-02-04 18:12:57]

lgtm

[commit-bot: I haz the power, 2017-02-04 18:56:30]

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

[commit-bot: I haz the power, 2017-02-04 18:56:31]

Dry run: Try jobs failed on following builders:
  win_clang on master.tryserver.chromium.win (JOB_TIMED_OUT, no build URL)

[Michael Lippautz, 2017-02-04 22:11:33]

The CQ bit was checked by mlippautz@chromium.org

[Michael Lippautz, 2017-02-04 22:11:34]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2675943003/#ps20001
(title: "Rebase (0 changes)")

[commit-bot: I haz the power, 2017-02-04 22:11:41]

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-02-05 00:12:06]

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

[commit-bot: I haz the power, 2017-02-05 00:12:08]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)

[Michael Lippautz, 2017-02-05 01:38:08]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2017-02-05 01:38:15]

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-02-05 01:58:15]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486258688822680,
"parent_rev": "a343404060a2ec085df8f42cd9566655a64291d1", "commit_rev":
"2d54dc83d5083e0494f0f642b5d01cfaff39a3b6"}

[commit-bot: I haz the power, 2017-02-05 02:00:04]

Description was changed from

==========
Remove custom left overs from object grouping

BUG=chromium:682688
==========

to

==========
Remove custom left overs from object grouping

BUG=chromium:682688

Review-Url: https://codereview.chromium.org/2675943003
Cr-Commit-Position: refs/heads/master@{#448163}
Committed:
https://chromium.googlesource.com/chromium/src/+/2d54dc83d5083e0494f0f642b5d0...
==========

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

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/2d54dc83d5083e0494f0f642b5d0...