Buildfix for Ozone Linux build: guard build/linux/system.gyp:atk dep.

Buildfix for Ozone Linux build: guard build/linux/system.gyp:atk dep.

R=dmazzoni@chromium.org
BUG=None

Committed: https://crrev.com/1f11e922c34e28786c16436839ca9146df10fd74
Cr-Commit-Position: refs/heads/master@{#320829}

[dmazzoni, 2015-03-16 21:36:00]

https://codereview.chromium.org/1014663002/diff/1/ui/views/views.gyp
File ui/views/views.gyp (right):

https://codereview.chromium.org/1014663002/diff/1/ui/views/views.gyp#newcode638
ui/views/views.gyp:638: ['OS=="linux" and chromeos==0 and use_ozone==0', {
My change only added the dependency on system.gyp:atk, below. Your patch will
also affect the dependency on shell_dialogs and on
window_button_order_provider.cc, I'm not sure those are right.

To be safe, maybe add a new conditional for 'OS=="linux" and chromeos==0 and
use_ozone==0 just for atk?

[gunsch, 2015-03-16 21:39:03]

gunsch@chromium.org changed reviewers:
+ sky@chromium.org

[gunsch, 2015-03-16 21:39:04]

https://codereview.chromium.org/1014663002/diff/1/ui/views/views.gyp
File ui/views/views.gyp (right):

https://codereview.chromium.org/1014663002/diff/1/ui/views/views.gyp#newcode638
ui/views/views.gyp:638: ['OS=="linux" and chromeos==0 and use_ozone==0', {
On 2015/03/16 21:36:00, dmazzoni wrote:
> My change only added the dependency on system.gyp:atk, below. Your patch will
> also affect the dependency on shell_dialogs and on
> window_button_order_provider.cc, I'm not sure those are right.
> 
> To be safe, maybe add a new conditional for 'OS=="linux" and chromeos==0 and
> use_ozone==0 just for atk?

I wasn't sure either, figured I'd mail and hope for guidance from someone who
knew which was more appropriate :) changed to guarding just this one separately.

[dmazzoni, 2015-03-16 21:43:38]

lgtm

[sky, 2015-03-16 23:09:55]

LGTM

[gunsch, 2015-03-16 23:10:05]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2015-03-16 23:10:40]

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

[commit-bot: I haz the power, 2015-03-17 00:22:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-17 00:23:24]

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