Changed to not use NetworkInfo class for Lollipop and above

[chaesung]

This PR introduces the following update:

Was concerned about issue #318

However I couldn't completely remove the NetworkInfo class in Connectivity class

I just changed it so that it would use the network callback api from Lollipop and above.

If you don't have time please give me some tips on how I should implement it or just dump in ideas in this PR.

I will try to work on it.

Hope to hear from you soon.

[chaesung]

Any feedback on this??

[pwittchen]

Hi @chaesung.

Thanks for your PR. I didn't have time to perform the review of your code. I'll do it soon.

Regards,
Piotr

[pwittchen]

Code looks quite fine. Maybe small adjustments can be done after merge. I'll test it later and if everything will be ok, I'll merge it.

[chaesung]

@pwittchen Thanks for the approval. Would it be possible to know when it would be merged?

[chaesung]

@pwittchen any feedback on this?

[henryglendening-hh]

Has this been merged?

[jortas]

Please merge it

[pwittchen]

I checked out this branch after fixing test configuration and I see, several tests are not passing.

Unfortunately, we cannot see them here, because Travis CI were changed and this CI doesn't work anymore with old configuration. I need to fix this CI or migrate do GitHub actions...

From what I see, MarshmallowNetowrkObservingStrategy#propagateAnyConnectedState() method, needs and update to consider NetworkState besides NetworkInfo class to make it work properly and to make tests pass. You can try to fix it by yourself or I can take a look at it later.

[pwittchen]

I took your PR and added some adjustments in PR #469 regarding library logic and tests. I need to verify that and maybe add some more tests for the newer Android API. If everything will be working fine, I'll merge it.