[Contextual Search] Fixes flaky test.

[Contextual Search] Fixes and re-enables
ContextualSearchManagerTest#testChainedSearchLoadsCorrectSearchTerm.

Sometimes, the test tries to check the search term before
the URL has started loading, which ends up resulting in
failing to match the search term. The empty URL could also
cause a crash when running the test, so this CL fixes the
problem by waiting for the URL to match the search term,
and also checks if the URL is null to prevent crashes.

BUG=551711
Committed: https://crrev.com/938110aa31eef055669b539990d7f87336b57633
Cr-Commit-Position: refs/heads/master@{#373889}

[pedro (no code reviews), 2016-02-05 18:34:55]

Description was changed from

==========
[Contextual Search] Fixes flaky test.

BUG=
==========

to

==========
[Contextual Search] Fixes flaky test.

BUG=551711
==========

[pedro (no code reviews), 2016-02-05 18:34:55]

pedrosimonetti@chromium.org changed reviewers:
+ donnd@chromium.org, twellington@chromium.org

[pedro (no code reviews), 2016-02-05 18:35:28]

Theresa and Donn, please take a look at this change.

[Theresa, 2016-02-05 18:47:42]

lgtm

Can you make the commit message more descriptive, e.g. "Fixes and re-enables
ContextualSearchManagerTest#testChainedSearchLoadsCorrectSearchTerm"

[Donn Denman, 2016-02-05 19:05:23]

lgtm

[Donn Denman, 2016-02-05 19:05:26]

lgtm

[pedro (no code reviews), 2016-02-05 19:26:41]

Description was changed from

==========
[Contextual Search] Fixes flaky test.

BUG=551711
==========

to

==========
[Contextual Search] Fixes and re-enables
ContextualSearchManagerTest#testChainedSearchLoadsCorrectSearchTerm.

Sometimes, the test tries to check the search term before the URL has started
loading, which ends up resulting in failing to match the search term. The empty
URL could also cause a crash when running the test, so this CL fixes the problem
by waiting for the URL to match the search term, and also checks if the URL is
null to prevent crashes.

BUG=551711
==========

[pedro (no code reviews), 2016-02-05 19:28:07]

Description was changed from

==========
[Contextual Search] Fixes and re-enables
ContextualSearchManagerTest#testChainedSearchLoadsCorrectSearchTerm.

Sometimes, the test tries to check the search term before the URL has started
loading, which ends up resulting in failing to match the search term. The empty
URL could also cause a crash when running the test, so this CL fixes the problem
by waiting for the URL to match the search term, and also checks if the URL is
null to prevent crashes.

BUG=551711
==========

to

==========
[Contextual Search] Fixes and re-enables 
ContextualSearchManagerTest#testChainedSearchLoadsCorrectSearchTerm.

Sometimes, the test tries to check the search term before 
the URL has started loading, which ends up resulting in 
failing to match the search term. The empty URL could also 
cause a crash when running the test, so this CL fixes the 
problem by waiting for the URL to match the search term, 
and also checks if the URL is null to prevent crashes.

BUG=551711
==========

[pedro (no code reviews), 2016-02-05 19:28:15]

The CQ bit was checked by pedrosimonetti@chromium.org

[pedro (no code reviews), 2016-02-05 19:28:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
twellington@chromium.org, donnd@chromium.org
Link to the patchset: https://codereview.chromium.org/1669983003/#ps20001
(title: "Sync & rebase")

[commit-bot: I haz the power, 2016-02-05 19:30:00]

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

[commit-bot: I haz the power, 2016-02-05 20:10:10]

Description was changed from

==========
[Contextual Search] Fixes and re-enables 
ContextualSearchManagerTest#testChainedSearchLoadsCorrectSearchTerm.

Sometimes, the test tries to check the search term before 
the URL has started loading, which ends up resulting in 
failing to match the search term. The empty URL could also 
cause a crash when running the test, so this CL fixes the 
problem by waiting for the URL to match the search term, 
and also checks if the URL is null to prevent crashes.

BUG=551711
==========

to

==========
[Contextual Search] Fixes and re-enables 
ContextualSearchManagerTest#testChainedSearchLoadsCorrectSearchTerm.

Sometimes, the test tries to check the search term before 
the URL has started loading, which ends up resulting in 
failing to match the search term. The empty URL could also 
cause a crash when running the test, so this CL fixes the 
problem by waiting for the URL to match the search term, 
and also checks if the URL is null to prevent crashes.

BUG=551711
==========

[commit-bot: I haz the power, 2016-02-05 20:10:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-05 20:24:18]

Description was changed from

==========
[Contextual Search] Fixes and re-enables 
ContextualSearchManagerTest#testChainedSearchLoadsCorrectSearchTerm.

Sometimes, the test tries to check the search term before 
the URL has started loading, which ends up resulting in 
failing to match the search term. The empty URL could also 
cause a crash when running the test, so this CL fixes the 
problem by waiting for the URL to match the search term, 
and also checks if the URL is null to prevent crashes.

BUG=551711
==========

to

==========
[Contextual Search] Fixes and re-enables
ContextualSearchManagerTest#testChainedSearchLoadsCorrectSearchTerm.

Sometimes, the test tries to check the search term before
the URL has started loading, which ends up resulting in
failing to match the search term. The empty URL could also
cause a crash when running the test, so this CL fixes the
problem by waiting for the URL to match the search term,
and also checks if the URL is null to prevent crashes.

BUG=551711

Committed: https://crrev.com/938110aa31eef055669b539990d7f87336b57633
Cr-Commit-Position: refs/heads/master@{#373889}
==========

[commit-bot: I haz the power, 2016-02-05 20:24:19]

Patchset 2 (id:??) landed as
https://crrev.com/938110aa31eef055669b539990d7f87336b57633
Cr-Commit-Position: refs/heads/master@{#373889}

[pedro (no code reviews), 2016-02-05 23:45:36]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1678573002/ by pedrosimonetti@chromium.org.

The reason for reverting is: Reverting it, since it's making some bots to
crash..