Remove dependency on Earl Grey from WebViewInteractionTestUtil

Remove dependency on Earl Grey from WebViewInteractionTestUtil

WebViewInteractionTestUtil should not depend on Earl Grey.
This CL removes this dependency and also removes the Earl Grey timeout.
The current methods using this function should add an assertion on the return
value.

BUG=none

[gambard, 2016-08-26 14:44:29]

Description was changed from

==========
Remove dependency on Earl Grey from WebViewInteractionTestUtil

WebViewInteractionTestUtil should not depend on Earl Grey.
This CL removes this dependency.

BUG=none
==========

to

==========
Remove dependency on Earl Grey from WebViewInteractionTestUtil

WebViewInteractionTestUtil should not depend on Earl Grey.
This CL removes this dependency and also removes the Earl Grey timeout.
The current methods using this function should add an assertion on the return
value.

BUG=none
==========

[gambard, 2016-08-26 14:45:48]

gambard@chromium.org changed reviewers:
+ baxley@chromium.org

[gambard, 2016-08-26 14:45:48]

WDYT?
I don't think we should be depend on Earl Grey as this already caused some
problems.

[baxley, 2016-08-26 15:02:33]

lgtm

[gambard, 2016-08-26 15:03:17]

I think we should add a Grey Assertion for every use of this function before
landing it.
WDYT?

[baxley, 2016-08-26 15:06:39]

On 2016/08/26 15:03:17, gambard wrote:
> I think we should add a Grey Assertion for every use of this function before
> landing it.
> WDYT?

agreed, I (or someone in MTV) can take that on before landing your CL.

[gambard, 2016-08-26 15:26:39]

+bzanotti in cc as he was interested in it.
PTAL for BUILD.gn changes.
I will be OOO for the next two weeks so I cannot do the downstream changes. The
changes will need to wait the autoroller for
https://codereview.chromium.org/2268863004/

[bzanotti, 2016-08-26 15:45:44]

On 2016/08/26 15:26:39, gambard (OOO until 12 Sept) wrote:
> +bzanotti in cc as he was interested in it.
> PTAL for BUILD.gn changes.
> I will be OOO for the next two weeks so I cannot do the downstream changes.
The
> changes will need to wait the autoroller for
> https://codereview.chromium.org/2268863004/

Note that this doesn't change a thing for the downstream change. This file is
still compiled alongside ios/web/public/test/earl_grey/web_view_matchers.(h|mm)
in the earl_grey_test_support target.
The file should probably be moved in the test_support target in
ios/web/BUILD.gn.