Do not show pinned tabs on multiple instances

Do not show pinned tabs on multiple instances

BUG=691781
Review-Url: https://codereview.chromium.org/2709393006
Cr-Commit-Position: refs/heads/master@{#453294}
Committed: https://chromium.googlesource.com/chromium/src/+/4349089ebb28a0509a0fc20af83c9a2c0912c3ed

[tmartino, 2017-02-24 18:03:46]

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

[commit-bot: I haz the power, 2017-02-24 18:04:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tmartino, 2017-02-24 18:44:38]

tmartino@chromium.org changed reviewers:
+ pkasting@chromium.org

[commit-bot: I haz the power, 2017-02-24 18:46:28]

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

[commit-bot: I haz the power, 2017-02-24 18:46:29]

Dry run: This issue passed the CQ dry run.

[Peter Kasting, 2017-02-25 06:07:07]

LGTM

https://codereview.chromium.org/2709393006/diff/1/chrome/browser/ui/startup/s...
File chrome/browser/ui/startup/startup_tab_provider.cc (right):

https://codereview.chromium.org/2709393006/diff/1/chrome/browser/ui/startup/s...
chrome/browser/ui/startup/startup_tab_provider.cc:213:
profile_has_other_tabbed_browser) {
Nit: {} unnecessary

COuld continue to use ?: as before

[tmartino, 2017-02-27 18:26:19]

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

[commit-bot: I haz the power, 2017-02-27 18:26:52]

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-02-27 19:20:53]

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

[commit-bot: I haz the power, 2017-02-27 19:20:54]

Dry run: This issue passed the CQ dry run.

[tmartino, 2017-02-27 19:22:23]

https://codereview.chromium.org/2709393006/diff/1/chrome/browser/ui/startup/s...
File chrome/browser/ui/startup/startup_tab_provider.cc (right):

https://codereview.chromium.org/2709393006/diff/1/chrome/browser/ui/startup/s...
chrome/browser/ui/startup/startup_tab_provider.cc:213:
profile_has_other_tabbed_browser) {
On 2017/02/25 at 06:07:07, Peter Kasting wrote:
> Nit: {} unnecessary
> 
> COuld continue to use ?: as before

Removed {}. Tried it with ?: but the condition is complex enough that I found it
hard to read.

[tmartino, 2017-02-27 19:22:24]

The CQ bit was checked by tmartino@chromium.org

[tmartino, 2017-02-27 19:22:24]

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

[commit-bot: I haz the power, 2017-02-27 19:22:57]

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-02-27 19:28:28]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488223344357900,
"parent_rev": "52d9483cb732e314990f93bc7291927b6369c4f0", "commit_rev":
"4349089ebb28a0509a0fc20af83c9a2c0912c3ed"}

[commit-bot: I haz the power, 2017-02-27 19:29:12]

Description was changed from

==========
Do not show pinned tabs on multiple instances

BUG=691781
==========

to

==========
Do not show pinned tabs on multiple instances

BUG=691781

Review-Url: https://codereview.chromium.org/2709393006
Cr-Commit-Position: refs/heads/master@{#453294}
Committed:
https://chromium.googlesource.com/chromium/src/+/4349089ebb28a0509a0fc20af83c...
==========

[commit-bot: I haz the power, 2017-02-27 19:29:13]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/4349089ebb28a0509a0fc20af83c...