Linux UI: Add IsVisibleOnAllWorkspaces()

Linux UI: Add IsVisibleOnAllWorkspaces()

CL404544 gave precedence to the browsers on the current workspace when
opening a tab in an existing browser session, however it did not take
into account that windows may be on all workspaces.

BUG=619673
Committed: https://crrev.com/dbf499f5aa00782185d392848204284bde71b448
Cr-Commit-Position: refs/heads/master@{#406200}

[Tom (Use chromium acct), 2016-07-16 01:48:15]

The CQ bit was checked by thomasanderson@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-16 01:48: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, 2016-07-16 02:02:43]

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

[commit-bot: I haz the power, 2016-07-16 02:02:44]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Tom (Use chromium acct), 2016-07-18 17:58:28]

thomasanderson@google.com changed reviewers:
+ sky@chromium.org

[Tom (Use chromium acct), 2016-07-18 17:58:29]

sky@ PTAL
Not sure if we need the complexity introduced by IsVisibleOnAllWorkspaces or if
"const std::string kAllWorkspaces" would be better.

[Tom (Use chromium acct), 2016-07-18 17:58:36]

The CQ bit was checked by thomasanderson@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-18 17:59:20]

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-07-18 18:27:56]

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

[commit-bot: I haz the power, 2016-07-18 18:27:58]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[sky, 2016-07-18 19:59:25]

LGTM

[Tom (Use chromium acct), 2016-07-19 01:45:08]

The CQ bit was checked by thomasanderson@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-19 01:46:05]

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-07-19 02:59:17]

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

[commit-bot: I haz the power, 2016-07-19 02:59:20]

Dry run: This issue passed the CQ dry run.

[Tom (Use chromium acct), 2016-07-19 03:10:47]

The CQ bit was checked by thomasanderson@google.com

[Tom (Use chromium acct), 2016-07-19 03:10:48]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2158653002/#ps60001
(title: "Add more mac impls")

[commit-bot: I haz the power, 2016-07-19 03:11:30]

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-07-19 03:19:04]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-19 03:19:08]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-19 03:21:35]

Description was changed from

==========
Linux UI: Add IsVisibleOnAllWorkspaces()

CL404544 gave precedence to the browsers on the current workspace when
opening a tab in an existing browser session, however it did not take
into account that windows may be on all workspaces.

BUG=619673
==========

to

==========
Linux UI: Add IsVisibleOnAllWorkspaces()

CL404544 gave precedence to the browsers on the current workspace when
opening a tab in an existing browser session, however it did not take
into account that windows may be on all workspaces.

BUG=619673

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

[commit-bot: I haz the power, 2016-07-19 03:21:37]

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