Enable GN check for gpu and rlz

Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a .h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on the NaCl IRT so people will get an error if they accidentally add this Skia dependency (this has come up before, it's easy to add a //ui/gfx dependency in the wrong place). It also fixes the Skia build in NaCl by updating the condition for extra warning to match build config. Without this update, the assert_no_deps won't be reached before an error running the skia build file, which won't make any sense in the context of addnig a random dependency.

Minor cleanup of the use_libpci conditionals that was exposed by this (the define that controlled the header didn't exactly match the corresponding dependencies).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
TBR=jschuh (gpu/ipc build changes)

Committed: https://crrev.com/09039c14ad6710587d13e64e3ddf8cad1d292347
Cr-Commit-Position: refs/heads/master@{#381881}

[brettw, 2016-03-16 00:05:39]

Description was changed from

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.
==========

to

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[brettw, 2016-03-16 00:10:47]

Description was changed from

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included ui/gfx/gpu_memory_buffer.h but adding a
dependency on //ui/gfx pulls Skia into the NaCl IRT and everything goes badly.
It just needed a forward declaration plus some enums, so I split out
ui/gfx/buffer_types.h (which actually wasn't mentioned anywhere in the build!)
into a separate target (it doesn't depend on anything) so we can include it with
no other dependencies.

In support of the above problem, I added an assert_no_deps annotation on the
NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[brettw, 2016-03-16 00:10:47]

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

[brettw, 2016-03-16 00:11:54]

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

[commit-bot: I haz the power, 2016-03-16 00:12:19]

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

[brettw, 2016-03-16 00:12:26]

Please check the GPU dependencies carefully; I don't know what is allowed to
depend on what. I'm actually pretty surprised I don't see any circular
dependency problems which gives me hope that this is correct.

[commit-bot: I haz the power, 2016-03-16 00:16:55]

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

[commit-bot: I haz the power, 2016-03-16 00:16:56]

Dry run: Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[piman, 2016-03-17 00:20:51]

LGTM, this looks right. Thanks!

[brettw, 2016-03-17 17:54:55]

The CQ bit was checked by brettw@chromium.org

[brettw, 2016-03-17 17:54:55]

The patchset sent to the CQ was uploaded after l-g-t-m from piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1800343002/#ps40001
(title: "Circular deps")

[commit-bot: I haz the power, 2016-03-17 17:55:13]

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

[commit-bot: I haz the power, 2016-03-17 18:00:06]

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

[commit-bot: I haz the power, 2016-03-17 18:00:06]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[brettw, 2016-03-17 19:37:22]

Description was changed from

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included ui/gfx/gpu_memory_buffer.h but adding a
dependency on //ui/gfx pulls Skia into the NaCl IRT and everything goes badly.
It just needed a forward declaration plus some enums, so I split out
ui/gfx/buffer_types.h (which actually wasn't mentioned anywhere in the build!)
into a separate target (it doesn't depend on anything) so we can include it with
no other dependencies.

In support of the above problem, I added an assert_no_deps annotation on the
NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

In support of the above problem, I added an assert_no_deps annotation on the
NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[brettw, 2016-03-17 19:39:15]

Description was changed from

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

In support of the above problem, I added an assert_no_deps annotation on the
NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a
.h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on
the NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

[brettw, 2016-03-17 19:39:43]

The CQ bit was checked by brettw@chromium.org

[brettw, 2016-03-17 19:39:44]

The patchset sent to the CQ was uploaded after l-g-t-m from piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1800343002/#ps80001
(title: " ")

[commit-bot: I haz the power, 2016-03-17 19:40:06]

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

[commit-bot: I haz the power, 2016-03-17 19:44:31]

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

[commit-bot: I haz the power, 2016-03-17 19:44:32]

Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[brettw, 2016-03-17 19:49:35]

Description was changed from

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a
.h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on
the NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
==========

to

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a
.h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on
the NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
TBR=jschuh (gpu/ipc build changes)
==========

[brettw, 2016-03-17 19:49:56]

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

[brettw, 2016-03-17 19:49:56]

Justing: gpu/ipc

[jschuh, 2016-03-17 19:51:51]

lgtm for ipc: mechanical change

[brettw, 2016-03-17 20:22:16]

Description was changed from

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a
.h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on
the NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
TBR=jschuh (gpu/ipc build changes)
==========

to

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a
.h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on
the NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

Minor cleanup of the use_libpci conditionals that was exposed by this (the
define that controlled the header didn't exactly match the corresponding
dependencies).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
TBR=jschuh (gpu/ipc build changes)
==========

[brettw, 2016-03-17 20:22:30]

The CQ bit was checked by brettw@chromium.org

[brettw, 2016-03-17 20:22:30]

The patchset sent to the CQ was uploaded after l-g-t-m from jschuh@chromium.org,
piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1800343002/#ps100001
(title: " ")

[commit-bot: I haz the power, 2016-03-17 20:22:55]

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

[commit-bot: I haz the power, 2016-03-17 23:56:25]

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

[commit-bot: I haz the power, 2016-03-17 23:56:27]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[brettw, 2016-03-18 01:10:29]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2016-03-18 01:10:45]

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

[commit-bot: I haz the power, 2016-03-18 03:22:54]

Description was changed from

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a
.h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on
the NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

Minor cleanup of the use_libpci conditionals that was exposed by this (the
define that controlled the header didn't exactly match the corresponding
dependencies).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
TBR=jschuh (gpu/ipc build changes)
==========

to

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a
.h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on
the NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

Minor cleanup of the use_libpci conditionals that was exposed by this (the
define that controlled the header didn't exactly match the corresponding
dependencies).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
TBR=jschuh (gpu/ipc build changes)
==========

[commit-bot: I haz the power, 2016-03-18 03:22:55]

Committed patchset #6 (id:100001)

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

Description was changed from

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a
.h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on
the NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

Minor cleanup of the use_libpci conditionals that was exposed by this (the
define that controlled the header didn't exactly match the corresponding
dependencies).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
TBR=jschuh (gpu/ipc build changes)
==========

to

==========
Enable GN check for gpu and rlz

Rlz already passed on Windows.

GPU had a bunch of errors which are fixed here by adding the necessary
dependencies and in a few cases removing unnecessary headers.

gpu_memory_buffer_manager.h included stuff from ui/gfx but adding a dependency
on //ui/gfx pulls Skia into the NaCl IRT (and mesa into Android) and everything
goes badly. The GPU memory buffer stuff turns out to be separable from gfx so I
created a new target for this.

The same is true for ui/gfx/native_widget_types.h (this is just some enums in a
.h file) so I made a separate target for this as well.

In support of the Skia-in-NaCl problem, I added an assert_no_deps annotation on
the NaCl IRT so people will get an error if they accidentally add this Skia
dependency (this has come up before, it's easy to add a //ui/gfx dependency in
the wrong place). It also fixes the Skia build in NaCl by updating the condition
for extra warning to match build config. Without this update, the assert_no_deps
won't be reached before an error running the skia build file, which won't make
any sense in the context of addnig a random dependency.

Minor cleanup of the use_libpci conditionals that was exposed by this (the
define that controlled the header didn't exactly match the corresponding
dependencies).

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
TBR=jschuh (gpu/ipc build changes)

Committed: https://crrev.com/09039c14ad6710587d13e64e3ddf8cad1d292347
Cr-Commit-Position: refs/heads/master@{#381881}
==========

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

Patchset 6 (id:??) landed as
https://crrev.com/09039c14ad6710587d13e64e3ddf8cad1d292347
Cr-Commit-Position: refs/heads/master@{#381881}

[oetuaho-nv, 2016-03-21 09:32:41]

On 2016/03/18 03:24:12, commit-bot: I haz the power wrote:
> Patchset 6 (id:??) landed as
> https://crrev.com/09039c14ad6710587d13e64e3ddf8cad1d292347
> Cr-Commit-Position: refs/heads/master@{#381881}

It seems like this change may have broken some of the Linux GPU bots:
https://build.chromium.org/p/chromium.gpu.fyi/waterfall?builder=GPU%20Linux%2...

This failure is blocking submissions of ANGLE patches.

[brettw, 2016-03-21 20:48:16]

Will look at the GPU issue.

[brettw, 2016-03-21 21:45:54]

https://codereview.chromium.org/1818143003/