Move TODO(philipj) to TODO(foolip)

Move TODO(philipj) to TODO(foolip)

R=davve@opera.com

Committed: https://crrev.com/08fe647b17456aca2bc951942ce27bda036ca090
Cr-Commit-Position: refs/heads/master@{#395890}

[foolip, 2016-05-25 09:41:00]

First CL, PTAL?

[davve, 2016-05-25 09:43:53]

Seems mechanical enough from a rough glance through the patch. rs lgtm.

[foolip, 2016-05-25 09:50:59]

The CQ bit was checked by foolip@chromium.org

[commit-bot: I haz the power, 2016-05-25 09:51:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2015503002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2015503002/1

[commit-bot: I haz the power, 2016-05-25 09:57:04]

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

[commit-bot: I haz the power, 2016-05-25 09:57:07]

Try jobs failed on following builders:
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[foolip, 2016-05-25 10:38:16]

The CQ bit was checked by foolip@chromium.org

[foolip, 2016-05-25 10:38:18]

The patchset sent to the CQ was uploaded after l-g-t-m from davve@opera.com
Link to the patchset: https://codereview.chromium.org/2015503002/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2016-05-25 10:38:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2015503002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2015503002/20001

[commit-bot: I haz the power, 2016-05-25 10:53:22]

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

[commit-bot: I haz the power, 2016-05-25 10:53:25]

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

[foolip, 2016-05-25 11:28:29]

foolip@chromium.org changed reviewers:
+ rbyers@chromium.org

[foolip, 2016-05-25 11:28:31]

Oops, this touched modules/ and platform/ too. Rick, PTAL?

[Rick Byers, 2016-05-25 13:47:22]

On 2016/05/25 11:28:31, Philip Jägenstedt wrote:
> Oops, this touched modules/ and platform/ too. Rick, PTAL?

Wow, you have a lot of work to do ;-)

LGTM

[foolip, 2016-05-25 14:30:13]

On 2016/05/25 13:47:22, Rick Byers wrote:
> On 2016/05/25 11:28:31, Philip Jägenstedt wrote:
> > Oops, this touched modules/ and platform/ too. Rick, PTAL?
> 
> Wow, you have a lot of work to do ;-)
> 
> LGTM

I was assured that a TODO was not a promise to actually do it, so I went crazy
:)

[foolip, 2016-05-25 14:30:25]

The CQ bit was checked by foolip@chromium.org

[commit-bot: I haz the power, 2016-05-25 14:30:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2015503002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2015503002/20001

[dtapuska, 2016-05-25 14:33:53]

On 2016/05/25 14:30:13, Philip Jägenstedt wrote:
> On 2016/05/25 13:47:22, Rick Byers wrote:
> > On 2016/05/25 11:28:31, Philip Jägenstedt wrote:
> > > Oops, this touched modules/ and platform/ too. Rick, PTAL?
> > 
> > Wow, you have a lot of work to do ;-)
> > 
> > LGTM
> 
> I was assured that a TODO was not a promise to actually do it, so I went crazy
> :)

Expectations always change :-)

[commit-bot: I haz the power, 2016-05-25 14:45:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-25 14:46:16]

Description was changed from

==========
Move TODO(philipj) to TODO(foolip)

R=davve@opera.com
==========

to

==========
Move TODO(philipj) to TODO(foolip)

R=davve@opera.com

Committed: https://crrev.com/08fe647b17456aca2bc951942ce27bda036ca090
Cr-Commit-Position: refs/heads/master@{#395890}
==========

[commit-bot: I haz the power, 2016-05-25 14:46:19]

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