Remove is_debug from enable_webvr flag (N SDK is now available!)

Remove is_debug from enable_webvr flag (N SDK is now available!)

Also add comment about why is_component_build check is necessary

BUG=389343
Committed: https://crrev.com/ffb586400bdf276af3a94bf95b5790a0afdc8f31
Cr-Commit-Position: refs/heads/master@{#430844}

[bshe, 2016-10-14 23:25:09]

Description was changed from

==========
Remove is_debug for enable_webvr flag (N SDK is now available!)

Also add comment about why is_component_build check is necessary


BUG=389343
==========

to

==========
Remove is_debug from enable_webvr flag (N SDK is now available!)

Also add comment about why is_component_build check is necessary


BUG=389343
==========

[bshe, 2016-10-14 23:28:26]

bshe@chromium.org changed reviewers:
+ brettw@chromium.org

[bshe, 2016-10-14 23:28:27]

Hi Brett.

Do you mind to take a look at this CL? I have verified locally that we no longer
see the proguard compile errors with the new N sdk.

Thanks!

[brettw, 2016-10-15 05:29:16]

lgtm

[bshe, 2016-10-17 12:40:35]

On 2016/10/15 05:29:16, brettw (ping on IM after 24h) wrote:
> lgtm

Thanks!
It looks like the SDK roll reverted over weekend. So I will wait for
it to land again and then try to land this one once I am sure SDK that
SDK won't be rolled back.

[bshe, 2016-11-03 21:09:48]

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

[commit-bot: I haz the power, 2016-11-03 21:10: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, 2016-11-03 22:56:39]

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

[commit-bot: I haz the power, 2016-11-03 22:56:40]

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

[bshe, 2016-11-09 01:06:23]

The CQ bit was checked by bshe@chromium.org

[commit-bot: I haz the power, 2016-11-09 01:07:06]

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-11-09 03:25:48]

Description was changed from

==========
Remove is_debug from enable_webvr flag (N SDK is now available!)

Also add comment about why is_component_build check is necessary


BUG=389343
==========

to

==========
Remove is_debug from enable_webvr flag (N SDK is now available!)

Also add comment about why is_component_build check is necessary


BUG=389343
==========

[commit-bot: I haz the power, 2016-11-09 03:25:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-09 03:28:57]

Description was changed from

==========
Remove is_debug from enable_webvr flag (N SDK is now available!)

Also add comment about why is_component_build check is necessary


BUG=389343
==========

to

==========
Remove is_debug from enable_webvr flag (N SDK is now available!)

Also add comment about why is_component_build check is necessary

BUG=389343

Committed: https://crrev.com/ffb586400bdf276af3a94bf95b5790a0afdc8f31
Cr-Commit-Position: refs/heads/master@{#430844}
==========

[commit-bot: I haz the power, 2016-11-09 03:28:57]

Patchset 1 (id:??) landed as
https://crrev.com/ffb586400bdf276af3a94bf95b5790a0afdc8f31
Cr-Commit-Position: refs/heads/master@{#430844}