Disable testIsTracingRunning for guest login.

Disable testIsTracingRunning for guest login.

BUG=chromium:598731
TEST=manual

Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/fb92d6bfba353bb1608486a6630e523c1de72994

[achuithb, 2016-04-01 23:38:38]

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

[commit-bot: I haz the power, 2016-04-01 23:38:43]

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

[achuithb, 2016-04-01 23:43:35]

achuith@chromium.org changed reviewers:
+ khmel@chromium.org

[achuithb, 2016-04-01 23:43:35]

Yury, this should fix the bot failures.

[commit-bot: I haz the power, 2016-04-01 23:51:52]

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

[commit-bot: I haz the power, 2016-04-01 23:51:53]

Dry run: This issue passed the CQ dry run.

[khmel, 2016-04-02 00:46:14]

https://codereview.chromium.org/1852823003/diff/1/telemetry/telemetry/interna...
File telemetry/telemetry/internal/browser/browser_unittest.py (right):

https://codereview.chromium.org/1852823003/diff/1/telemetry/telemetry/interna...
telemetry/telemetry/internal/browser/browser_unittest.py:116:
@decorators.Disabled('cros-chrome-guest', 'system-guest')  # chromeos guest
As I understand, this disables test for guest, right? Is it expected not to pass
it for guest or this is temp solution.

[achuithb, 2016-04-02 00:48:04]

On 2016/04/02 00:46:14, khmel wrote:
>
https://codereview.chromium.org/1852823003/diff/1/telemetry/telemetry/interna...
> File telemetry/telemetry/internal/browser/browser_unittest.py (right):
> 
>
https://codereview.chromium.org/1852823003/diff/1/telemetry/telemetry/interna...
> telemetry/telemetry/internal/browser/browser_unittest.py:116:
> @decorators.Disabled('cros-chrome-guest', 'system-guest')  # chromeos guest
> As I understand, this disables test for guest, right? Is it expected not to
pass
> it for guest or this is temp solution.

This is a fine permanent solution. We don't run any performance tests in guest
mode - incognito is only needed for integration tests which do not using
tracing.

It's like a P3 to figure out why tracing doesn't work in incognito.

[khmel, 2016-04-02 00:48:53]

lgtm

[achuithb, 2016-04-02 00:49:47]

The CQ bit was checked by achuith@chromium.org

[commit-bot: I haz the power, 2016-04-02 00:49:54]

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

[achuithb, 2016-04-02 00:49:55]

On 2016/04/02 00:48:53, khmel wrote:
> lgtm

ty

[commit-bot: I haz the power, 2016-04-02 00:51:31]

Description was changed from

==========
Disable testIsTracingRunning for guest login.

BUG=chromium:598731
TEST=manual
==========

to

==========
Disable testIsTracingRunning for guest login.

BUG=chromium:598731
TEST=manual

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-04-02 00:51:33]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...