fix bug in navigation: allow integer default types with argType float.

[tatocaster]

Proposed Changes

changes the NavInflater to allow integer default types with argType="float".
From now on this will work:

<argument
         android:name="ratio"
         android:defaultValue="0" <-- notice the value, 0 instead of 0.0
         app:argType="float" />

before there was an error: org.xmlpull.v1.XmlPullParserException: Type is float but found integer: 0

Testing

Test: ./gradlew navigation:navigation-runtime:connectedCheck

Issues Fixed

Fixes: The bug on https://issuetracker.google.com/issues/173766247 being fixed

[tatocaster]

I do not know why those checks are failing, there are 0 changes related to them and I opened a ticket for this as well: https://issuetracker.google.com/issues/177896088

[dlam]

I'm pretty sure these are build failures on our end - I'll take a look, thanks for filing a bug!

[ianhanniballake]

Thanks for the PR!

[dlam]

The build failures fix is up here: https://android-review.googlesource.com/c/platform/frameworks/support/+/1554452

Once this change lands you'll need to rebase this PR to get these tests to pass

[tatocaster]

@dlam Hey, I rebased, the fix is kinda working, still failing for work and navigation, and the same applies to other already finished actions too.

[dlam]

Ah - sorry about this, once these two land things should be green again :)

https://android-review.googlesource.com/c/platform/frameworks/support/+/1555160
https://android-review.googlesource.com/c/platform/frameworks/support/+/1555463

I just had my wisdom teeth removed so a bit slow to reply.

[tatocaster]

@dlam finally, all checks have passed. can you merge it? Thanks for the fixes