Mojo: Change the blink variant from "wtf" to "blink".

Mojo: Change the blink variant from "wtf" to "blink".

BUG=596202
Committed: https://crrev.com/0c8396083e862032a031f843fd9087eb751f3227
Cr-Commit-Position: refs/heads/master@{#389704}

[Sam McNally, 2016-04-20 00:06:05]

sammc@chromium.org changed reviewers:
+ esprehn@chromium.org, haraken@chromium.org, yzshen@chromium.org

[haraken, 2016-04-20 00:12:03]

LGTM

[yzshen1, 2016-04-20 17:47:34]

https://codereview.chromium.org/1902263004/diff/1/mojo/mojo_public.gyp
File mojo/mojo_public.gyp (right):

https://codereview.chromium.org/1902263004/diff/1/mojo/mojo_public.gyp#newcod...
mojo/mojo_public.gyp:422: 'mojom_variant': 'blink',
Now that we use a fixed variant in GN, maybe we should let 'for_blink' implies
'blink' variant and save one line?

[Sam McNally, 2016-04-21 04:04:20]

Patchset #3 (id:40001) has been deleted

[Sam McNally, 2016-04-21 04:04:28]

Patchset #3 (id:60001) has been deleted

[Sam McNally, 2016-04-21 06:05:20]

https://codereview.chromium.org/1902263004/diff/1/mojo/mojo_public.gyp
File mojo/mojo_public.gyp (right):

https://codereview.chromium.org/1902263004/diff/1/mojo/mojo_public.gyp#newcod...
mojo/mojo_public.gyp:422: 'mojom_variant': 'blink',
On 2016/04/20 17:47:33, yzshen1 wrote:
> Now that we use a fixed variant in GN, maybe we should let 'for_blink' implies
> 'blink' variant and save one line?

Done.

[yzshen1, 2016-04-21 15:38:20]

lgtm

[Sam McNally, 2016-04-21 23:46:57]

sammc@chromium.org changed reviewers:
+ rockot@chromium.org

[Sam McNally, 2016-04-21 23:46:57]

+rockot for mojo/ and device/

[Ken Rockot(use gerrit already), 2016-04-22 18:26:59]

lgtm

[Sam McNally, 2016-04-26 00:50:10]

esprehn: ping

[esprehn, 2016-04-26 00:55:41]

lgtm

[Sam McNally, 2016-04-26 01:44:16]

The CQ bit was checked by sammc@chromium.org

[Sam McNally, 2016-04-26 01:44:17]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, rockot@chromium.org, esprehn@chromium.org,
yzshen@chromium.org
Link to the patchset: https://codereview.chromium.org/1902263004/#ps100001
(title: "rebase")

[commit-bot: I haz the power, 2016-04-26 01:44:53]

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

[commit-bot: I haz the power, 2016-04-26 05:24:28]

Committed patchset #4 (id:100001)

[commit-bot: I haz the power, 2016-04-26 05:25:29]

Description was changed from

==========
Mojo: Change the blink variant from "wtf" to "blink".

BUG=596202
==========

to

==========
Mojo: Change the blink variant from "wtf" to "blink".

BUG=596202

Committed: https://crrev.com/0c8396083e862032a031f843fd9087eb751f3227
Cr-Commit-Position: refs/heads/master@{#389704}
==========

[commit-bot: I haz the power, 2016-04-26 05:25:30]

Patchset 4 (id:??) landed as
https://crrev.com/0c8396083e862032a031f843fd9087eb751f3227
Cr-Commit-Position: refs/heads/master@{#389704}