Pyauto perf test testNewTabDocs now waits for a different page element.

Pyauto perf test testNewTabDocs now waits for a different page element.

testNewTabDocs loads Google Docs and then waits until an expected
substring appears on the page before continuing.  Previously, the test
waited for a string that only appears when viewing the "my drive" folder,
because it was assumed that we'd always see that folder when opening google
docs.

However, sometimes the "shared with me" folder appears on page load.  This
appears to be because google docs opens the last folder opened by the
account.  Since we have another perf test that clicks on the "shared with me"
folder, then the current test will sometimes fail to realize the page loaded,
since an unexpected screen is displayed after page load.

The fix is to have this test wait for a different substring on the page,
which appears regardless of which folder shows up when loading google docs.

BUG=None
TEST=Verified the test runs for 10 iterations whereas previously it failed
after only 1 or 2 iterations.
NOTRY=true


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=149777

[dennis_jeffrey, 2012-08-03 01:11:35]

big description for a small change

[Nirnimesh, 2012-08-03 01:21:44]

LGTM

[asharif1, 2012-08-03 01:28:46]

On 2012/08/03 01:21:44, Nirnimesh wrote:
> LGTM

lgtm.

Thanks for this fix.

[commit-bot: I haz the power, 2012-08-03 01:30:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dennisjeffrey@chromium.org/10832127/1

[commit-bot: I haz the power, 2012-08-03 01:55:47]

Change committed as 149777

[asharif1, 2012-08-03 17:51:40]

On 2012/08/03 01:55:47, I haz the power (commit-bot) wrote:
> Change committed as 149777

The test will show some variability since the pages loaded may be different
every time.

[dennisjeffrey, 2012-08-03 17:54:23]

Yes.  That should be fixed soon once we convert the test to use a recording
of the site (with Web Page Replay), rather than connecting to the live site
every time.

-Dennis


On Fri, Aug 3, 2012 at 10:51 AM, <asharif@chromium.org> wrote:

> On 2012/08/03 01:55:47, I haz the power (commit-bot) wrote:
>
>> Change committed as 149777
>>
>
> The test will show some variability since the pages loaded may be different
> every time.
>
>
https://chromiumcodereview.**appspot.com/10832127/<https://chromiumcodereview...
>