Copy float list of ruby base when merging it with a sibling

Copy float list of ruby base when merging it with a sibling

When merging two ruby bases ensure that the new base has a complete float list.

A 2nd go at https://crrev.com/6f208d569f6bd2488f2f2f6f9004e54417f78782

BUG=619380, 624028, 644605, 644803
Committed: https://crrev.com/3453e2368a018544e7143e60afbc7a1454e848ed
Cr-Commit-Position: refs/heads/master@{#424864}

[rhogan, 2016-09-11 15:12:57]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-11 15:13:02]

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

[rhogan, 2016-09-11 15:13:15]

Description was changed from

==========
Copy float list of ruby base when merging it with a sibling

When merging two ruby bases ensure that the new base has a complete float list.

BUG=619380,624028
==========

to

==========
Copy float list of ruby base when merging it with a sibling

When merging two ruby bases ensure that the new base has a complete float list.

A 2nd go at https://crrev.com/6f208d569f6bd2488f2f2f6f9004e54417f78782

BUG=619380,624028
==========

[commit-bot: I haz the power, 2016-09-11 16:36:53]

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

[commit-bot: I haz the power, 2016-09-11 16:36:53]

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

[rhogan, 2016-09-12 19:15:44]

Description was changed from

==========
Copy float list of ruby base when merging it with a sibling

When merging two ruby bases ensure that the new base has a complete float list.

A 2nd go at https://crrev.com/6f208d569f6bd2488f2f2f6f9004e54417f78782

BUG=619380,624028
==========

to

==========
Copy float list of ruby base when merging it with a sibling

When merging two ruby bases ensure that the new base has a complete float list.

A 2nd go at https://crrev.com/6f208d569f6bd2488f2f2f6f9004e54417f78782

BUG=619380,624028, 644605
==========

[rhogan, 2016-09-12 19:16:56]

Description was changed from

==========
Copy float list of ruby base when merging it with a sibling

When merging two ruby bases ensure that the new base has a complete float list.

A 2nd go at https://crrev.com/6f208d569f6bd2488f2f2f6f9004e54417f78782

BUG=619380,624028, 644605
==========

to

==========
Copy float list of ruby base when merging it with a sibling

When merging two ruby bases ensure that the new base has a complete float list.

A 2nd go at https://crrev.com/6f208d569f6bd2488f2f2f6f9004e54417f78782

BUG=619380, 624028, 644605, 644803
==========

[rhogan, 2016-10-12 18:49:18]

robhogan@gmail.com changed reviewers:
+ eae@chromium.org

[eae, 2016-10-12 18:50:20]

Nice cleanup, thank you!
LGTM

[eae, 2016-10-12 18:50:39]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2016-10-12 18:51:03]

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, 2016-10-12 18:53:21]

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

[commit-bot: I haz the power, 2016-10-12 18:53:22]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  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_...)

[rhogan, 2016-10-12 19:00:35]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-12 19:02:03]

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, 2016-10-12 20:51:37]

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

[commit-bot: I haz the power, 2016-10-12 20:51:38]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[rhogan, 2016-10-12 20:58:58]

The CQ bit was checked by robhogan@gmail.com

[rhogan, 2016-10-12 20:58:58]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org
Link to the patchset: https://codereview.chromium.org/2329863002/#ps20001
(title: "bug 619380")

[commit-bot: I haz the power, 2016-10-12 20:59:21]

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, 2016-10-12 21:30:27]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-12 21:34:19]

Description was changed from

==========
Copy float list of ruby base when merging it with a sibling

When merging two ruby bases ensure that the new base has a complete float list.

A 2nd go at https://crrev.com/6f208d569f6bd2488f2f2f6f9004e54417f78782

BUG=619380, 624028, 644605, 644803
==========

to

==========
Copy float list of ruby base when merging it with a sibling

When merging two ruby bases ensure that the new base has a complete float list.

A 2nd go at https://crrev.com/6f208d569f6bd2488f2f2f6f9004e54417f78782

BUG=619380, 624028, 644605, 644803

Committed: https://crrev.com/3453e2368a018544e7143e60afbc7a1454e848ed
Cr-Commit-Position: refs/heads/master@{#424864}
==========

[commit-bot: I haz the power, 2016-10-12 21:34:21]

Patchset 2 (id:??) landed as
https://crrev.com/3453e2368a018544e7143e60afbc7a1454e848ed
Cr-Commit-Position: refs/heads/master@{#424864}