Remove differences in detecting logged in state between guest and regular user.

Remove differences in detecting logged in state between guest and regular user.

BUG=
TEST=trybot, manual.
NOTRY=True

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271530

[achuithb, 2014-05-16 23:34:19]

please review, Tim

[achuithb, 2014-05-19 18:34:15]

On 2014/05/16 23:34:19, achuithb wrote:
> please review, Tim

ping, in case you missed it

[Tim Song, 2014-05-19 19:07:38]

https://codereview.chromium.org/282303013/diff/1/tools/telemetry/telemetry/co...
File tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py
(right):

https://codereview.chromium.org/282303013/diff/1/tools/telemetry/telemetry/co...
tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py:245: #
Workaround for crbug.com/374462.
Why isn't this workaround needed for guest mode?

[achuithb, 2014-05-19 19:17:02]

https://codereview.chromium.org/282303013/diff/1/tools/telemetry/telemetry/co...
File tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py
(right):

https://codereview.chromium.org/282303013/diff/1/tools/telemetry/telemetry/co...
tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py:245: #
Workaround for crbug.com/374462.
On 2014/05/19 19:07:39, Tim Song wrote:
> Why isn't this workaround needed for guest mode?

Guest mode starts with an new tab, so we don't have to create the tab. 

We've never seen this connection problem with guest mode sessions. Guest mode
sessions are only used for one autotest (login_CryptohomeIncognito), so I
suppose it's theoretically possible that we've just gotten lucky because we run
this mode far less frequently. But I suspect that the chrome bug that this WAR
is for is probably caused by profile-related code.

The short answer is that guest sessions work properly without this WAR.

[Tim Song, 2014-05-19 19:51:53]

Okay that makes sense. Please add a comment explaining why the WAR isn't needed
in guest mode. LGTM.

[achuithb, 2014-05-20 00:44:37]

On 2014/05/19 19:51:53, Tim Song wrote:
> Okay that makes sense. Please add a comment explaining why the WAR isn't
needed
> in guest mode. LGTM.

ty, Done

[achuithb, 2014-05-20 00:44:55]

The CQ bit was checked by achuith@chromium.org

[achuithb, 2014-05-20 00:44:57]

The CQ bit was unchecked by achuith@chromium.org

[achuithb, 2014-05-20 01:34:59]

The CQ bit was checked by achuith@chromium.org

[commit-bot: I haz the power, 2014-05-20 01:36:07]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/achuith@chromium.org/282303013/20001

[commit-bot: I haz the power, 2014-05-20 01:37:48]

Change committed as 271530