MacViews: Fix mac_views_browser compilation due to unused variable on Mac.

MacViews: Fix mac_views_browser compilation due to unused variable on Mac.

AvatarButton declares a variable that is only used on Win/Linux. This breaks the
MacViews browser compile (mac_views_browser=true in gn), so fix it by declaring
the same variable inside both #ifdef branches in which it's used.

BUG=425229
Review-Url: https://codereview.chromium.org/2913663004
Cr-Commit-Position: refs/heads/master@{#476120}
Committed: https://chromium.googlesource.com/chromium/src/+/83ad1eda722e5e35b9d0a943617036958c6b4b4e

[Patti Lor, 2017-05-31 06:04:14]

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

[commit-bot: I haz the power, 2017-05-31 06:04:32]

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, 2017-05-31 06:49:20]

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

[commit-bot: I haz the power, 2017-05-31 06:49:21]

Dry run: This issue passed the CQ dry run.

[Patti Lor, 2017-05-31 06:55:54]

Description was changed from

==========
MacViews: Fix compilation due to unused variable on Mac.

AvatarButton declares a variable that is only used on Win/Linux. This breaks the
MacViews browser compile (mac_views_browser=true in gn), so fix it by declaring
the same variable inside both #ifdef branches in which it's used.

BUG=425229
==========

to

==========
MacViews: Fix mac_views_browser compilation due to unused variable on Mac.

AvatarButton declares a variable that is only used on Win/Linux. This breaks the
MacViews browser compile (mac_views_browser=true in gn), so fix it by declaring
the same variable inside both #ifdef branches in which it's used.

BUG=425229
==========

[Patti Lor, 2017-05-31 06:56:06]

Description was changed from

==========
MacViews: Fix mac_views_browser compilation due to unused variable on Mac.

AvatarButton declares a variable that is only used on Win/Linux. This breaks the
MacViews browser compile (mac_views_browser=true in gn), so fix it by declaring
the same variable inside both #ifdef branches in which it's used.

BUG=425229
==========

to

==========
MacViews: Fix mac_views_browser compilation due to unused variable on Mac.

AvatarButton declares a variable that is only used on Win/Linux. This breaks the
MacViews browser compile (mac_views_browser=true in gn), so fix it by declaring
the same variable inside both #ifdef branches in which it's used.

BUG=425229
==========

[Patti Lor, 2017-05-31 06:56:48]

patricialor@chromium.org changed reviewers:
+ tapted@chromium.org

[Patti Lor, 2017-05-31 06:56:49]

Hey Trent, is this fix OK?

Thanks!

[tapted, 2017-05-31 07:01:32]

lgtm

[Patti Lor, 2017-05-31 07:07:07]

patricialor@chromium.org changed reviewers:
+ msw@chromium.org

[Patti Lor, 2017-05-31 07:07:07]

Hi msw@, PTAL for owner's review? Thanks.

[msw, 2017-05-31 14:19:53]

lgtm

[Patti Lor, 2017-06-01 01:00:19]

The CQ bit was checked by patricialor@chromium.org

[commit-bot: I haz the power, 2017-06-01 01:01:21]

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, 2017-06-01 01:06:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496278819058570, "parent_rev":
"2d60f034473f06fd040407a2945f5b4a35b07df4", "commit_rev":
"83ad1eda722e5e35b9d0a943617036958c6b4b4e"}

[commit-bot: I haz the power, 2017-06-01 01:06:13]

Description was changed from

==========
MacViews: Fix mac_views_browser compilation due to unused variable on Mac.

AvatarButton declares a variable that is only used on Win/Linux. This breaks the
MacViews browser compile (mac_views_browser=true in gn), so fix it by declaring
the same variable inside both #ifdef branches in which it's used.

BUG=425229
==========

to

==========
MacViews: Fix mac_views_browser compilation due to unused variable on Mac.

AvatarButton declares a variable that is only used on Win/Linux. This breaks the
MacViews browser compile (mac_views_browser=true in gn), so fix it by declaring
the same variable inside both #ifdef branches in which it's used.

BUG=425229

Review-Url: https://codereview.chromium.org/2913663004
Cr-Commit-Position: refs/heads/master@{#476120}
Committed:
https://chromium.googlesource.com/chromium/src/+/83ad1eda722e5e35b9d0a9436170...
==========

[commit-bot: I haz the power, 2017-06-01 01:06:15]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/83ad1eda722e5e35b9d0a9436170...