Fixed flaky CronetSampleTest#testLoadUrl test

Fixed flaky CronetSampleTest#testLoadUrl test

The test was occasionally failing because of the race condition. There
are two scenarios how the test can be executed:
1. The URL can be retrieved before addTextChangedListener() is called.
   As the result onTextChanged() listener will never be called by the
   system. To address it, we call textWatcher.onTextChanged() explicitly
   at the end of the test.
2. The second scenario is when the URL is retrieved after the
   done.block() is reached. In that case the system calls
   onTextChanged() method. The problem is that the argument passed to
   the method is of type android.text.SpannableStringBuilder(). This
   object cannot be compared with the String directly. Instead,
   SpannableStringBuilder() must be converted to String and then
   compared.

BUG=635021
Committed: https://crrev.com/a41926a9b613da8641639cb0b4f3a3e54dfe5f46
Cr-Commit-Position: refs/heads/master@{#411191}

[kapishnikov, 2016-08-10 22:02:38]

kapishnikov@chromium.org changed reviewers:
+ xunjieli@chromium.org

[xunjieli, 2016-08-10 22:30:26]

LGTM! Awesome work. This is very smart :)

[xunjieli, 2016-08-10 22:32:59]

The CQ bit was checked by xunjieli@chromium.org

[commit-bot: I haz the power, 2016-08-10 22:34:10]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-10 23:53:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-10 23:56:26]

Description was changed from

==========
Fixed flaky CronetSampleTest#testLoadUrl test

The test was occasionally failing because of the race condition. There
are two scenarios how the test can be executed:
1. The URL can be retrieved before addTextChangedListener() is called.
   As the result onTextChanged() listener will never be called by the
   system. To address it, we call textWatcher.onTextChanged() explicitly
   at the end of the test.
2. The second scenario is when the URL is retrieved after the
   done.block() is reached. In that case the system calls
   onTextChanged() method. The problem is that the argument passed to
   the method is of type android.text.SpannableStringBuilder(). This
   object cannot be compared with the String directly. Instead,
   SpannableStringBuilder() must be converted to String and then
   compared.

BUG=635021
==========

to

==========
Fixed flaky CronetSampleTest#testLoadUrl test

The test was occasionally failing because of the race condition. There
are two scenarios how the test can be executed:
1. The URL can be retrieved before addTextChangedListener() is called.
   As the result onTextChanged() listener will never be called by the
   system. To address it, we call textWatcher.onTextChanged() explicitly
   at the end of the test.
2. The second scenario is when the URL is retrieved after the
   done.block() is reached. In that case the system calls
   onTextChanged() method. The problem is that the argument passed to
   the method is of type android.text.SpannableStringBuilder(). This
   object cannot be compared with the String directly. Instead,
   SpannableStringBuilder() must be converted to String and then
   compared.

BUG=635021

Committed: https://crrev.com/a41926a9b613da8641639cb0b4f3a3e54dfe5f46
Cr-Commit-Position: refs/heads/master@{#411191}
==========

[commit-bot: I haz the power, 2016-08-10 23:56:27]

Patchset 1 (id:??) landed as
https://crrev.com/a41926a9b613da8641639cb0b4f3a3e54dfe5f46
Cr-Commit-Position: refs/heads/master@{#411191}