modify getWebViewClient behavior

modify getWebViewClient behavior

getWebViewClient returns the default client instead of null when the client is
not yet set.

BUG=627248
Implement getWebViewClient and getWebChromeClient

Implement getWebViewClient and getWebChromeClient for new APIs in Android

Committed: https://crrev.com/8e4aa5f84630cfa5934bec31f74a00dd0145f9d3
Cr-Commit-Position: refs/heads/master@{#422547}

[Nate Fischer, 2016-10-03 18:42:46]

ntfschr@chromium.org changed reviewers:
+ tobiasjs@chromium.org

[Nate Fischer, 2016-10-03 18:42:46]

This is the change as discussed on the android side (let me know if I'm
misunderstanding). I can also revert the previous CL and upload this one, if
that's preferred.

[Tobias Sargeant, 2016-10-03 20:53:00]

On 2016/10/03 18:42:46, Nate Fischer wrote:
> This is the change as discussed on the android side (let me know if I'm
> misunderstanding). I can also revert the previous CL and upload this one, if
> that's preferred.

LGTM. Thanks. Selim is right; it would be nicer to not have to do this, but we
kind of have to.

[Nate Fischer, 2016-10-03 21:13:11]

The CQ bit was checked by ntfschr@chromium.org

[Nate Fischer, 2016-10-03 21:13:56]

On 2016/10/03 at 20:53:00, tobiasjs wrote:
> On 2016/10/03 18:42:46, Nate Fischer wrote:
> > This is the change as discussed on the android side (let me know if I'm
> > misunderstanding). I can also revert the previous CL and upload this one, if
> > that's preferred.
> 
> LGTM. Thanks. Selim is right; it would be nicer to not have to do this, but we
kind of have to.

Makes sense now that you and Selim explained the justification behind it.

[commit-bot: I haz the power, 2016-10-03 21:14:13]

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-10-03 21:49:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-03 21:52:17]

Description was changed from

==========
modify getWebViewClient behavior

getWebViewClient returns the default client instead of null when the client is
not yet set.

BUG=627248

Implement getWebViewClient and getWebChromeClient

Implement getWebViewClient and getWebChromeClient for new APIs in Android
==========

to

==========
modify getWebViewClient behavior

getWebViewClient returns the default client instead of null when the client is
not yet set.

BUG=627248

Implement getWebViewClient and getWebChromeClient

Implement getWebViewClient and getWebChromeClient for new APIs in Android

Committed: https://crrev.com/8e4aa5f84630cfa5934bec31f74a00dd0145f9d3
Cr-Commit-Position: refs/heads/master@{#422547}
==========

[commit-bot: I haz the power, 2016-10-03 21:52:18]

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