Buildbot spec and gyp defs for SK_USE_DISCARDABLE_SCALEDIMAGECACHE bot

Buildbot spec and gyp defs for SK_USE_DISCARDABLE_SCALEDIMAGECACHE bot

BUG=skia:4354
Committed: https://skia.googlesource.com/skia/+/9063e427d209d703bf9326f186f72061928c59c2

[borenet, 2015-09-18 12:38:47]

borenet@google.com changed reviewers:
+ reed@google.com

[borenet, 2015-09-18 12:38:47]

I think this is all that's needed for the new bot.

[borenet, 2015-09-18 12:39:05]

On 2015/09/18 12:38:47, borenet wrote:
> I think this is all that's needed for the new bot.

(once the trybot shows up I'll want to run it with this patch and verify)

[mtklein, 2015-09-18 12:42:20]

lgtm

Here I was thinking you'd set up the bots to parse the bot extra_config and just
turn it into `export CPPFLAGS=-D%s` when unknown.  (This would be pretty neat.)

> (once the trybot shows up I'll want to run it with this patch and verify)
Damn that's cool.

[borenet, 2015-09-18 12:52:57]

On 2015/09/18 12:42:20, mtklein wrote:
> lgtm
> 
> Here I was thinking you'd set up the bots to parse the bot extra_config and
just
> turn it into `export CPPFLAGS=-D%s` when unknown.  (This would be pretty
neat.)
> 

Oooh, I like this.  Can probably just detect the case where the extra_config is
all caps and underscores and starts with "SK_"...

> > (once the trybot shows up I'll want to run it with this patch and verify)
> Damn that's cool.

[mtklein, 2015-09-18 12:54:00]

On 2015/09/18 12:52:57, borenet wrote:
> On 2015/09/18 12:42:20, mtklein wrote:
> > lgtm
> > 
> > Here I was thinking you'd set up the bots to parse the bot extra_config and
> just
> > turn it into `export CPPFLAGS=-D%s` when unknown.  (This would be pretty
> neat.)
> > 
> 
> Oooh, I like this.  Can probably just detect the case where the extra_config
is
> all caps and underscores and starts with "SK_"...

sgtm!

[borenet, 2015-09-18 13:22:59]

This CL is unnecessary now, due to Mike's
https://codereview.chromium.org/1346833004.  I changed it in patch set 3 to add
the new bot to the buildbot_spec test, just so I could see it defining the
variable as expected.

[borenet, 2015-09-18 13:26:27]

The CQ bit was checked by borenet@google.com

[borenet, 2015-09-18 13:26:28]

The patchset sent to the CQ was uploaded after l-g-t-m from mtklein@google.com
Link to the patchset: https://codereview.chromium.org/1350283003/#ps40001
(title: "Backpedal")

[commit-bot: I haz the power, 2015-09-18 13:26:33]

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

[commit-bot: I haz the power, 2015-09-18 13:27:35]

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

[commit-bot: I haz the power, 2015-09-18 13:27:35]

Try jobs failed on following builders:
  skia_presubmit-Trybot on client.skia.fyi (JOB_FAILED,
http://build.chromium.org/p/client.skia.fyi/builders/skia_presubmit-Trybot/bu...)

[mtklein, 2015-09-18 13:28:23]

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

[mtklein, 2015-09-18 13:28:23]

lgtm

[borenet, 2015-09-18 13:31:15]

On 2015/09/18 13:27:35, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   skia_presubmit-Trybot on client.skia.fyi (JOB_FAILED,
>
http://build.chromium.org/p/client.skia.fyi/builders/skia_presubmit-Trybot/bu...)

Long name :(

[borenet, 2015-09-18 13:31:17]

The CQ bit was checked by borenet@google.com

[borenet, 2015-09-18 13:31:17]

The patchset sent to the CQ was uploaded after l-g-t-m from mtklein@google.com
Link to the patchset: https://codereview.chromium.org/1350283003/#ps60001
(title: "line break")

[commit-bot: I haz the power, 2015-09-18 13:31:20]

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

[commit-bot: I haz the power, 2015-09-18 13:37:17]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/9063e427d209d703bf9326f186f72061928c59c2