Vector drawable error with string resource with 2.3.0-beta2 Gradle Plugin [37133119]

Fixed

Bug

Status Update

No update yet.

Description

kk...@linkedin.com

created issue #1

Jan 17, 2017 10:24PM

In updating to support library 25.1.0, I am running into an issue that is failing some layout tests. It appears to be related to the use of string resources to contain path data in the latest support library. I have tried using the newest gradle plugin version (2.3.0-beta2) but it does not fix the issue. I also have vectorDrawables.useSupportLibrary = true. The weirdest part about this error is that only appears in our right to left (locale ar_XB) layout tests, not the left to right ones. The error it throws is caused by the unicode character &#8238, which is the "right to left override character" My guess is that because the path data is contained in a string resource instead of inline, it is having the RTL character added when before the inline value was not changed.

Here is an example StackTrace:

android.view.InflateException: Binary XML file line #57: Error inflating class android.support.design.widget.TextInputLayout at android.view.LayoutInflater.createView(LayoutInflater.java:633) at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:743) at android.view.LayoutInflater.rInflate(LayoutInflater.java:806) at android.view.LayoutInflater.rInflate(LayoutInflater.java:809) at android.view.LayoutInflater.rInflate(LayoutInflater.java:809) at android.view.LayoutInflater.parseInclude(LayoutInflater.java:916) at android.view.LayoutInflater.rInflate(LayoutInflater.java:802) at android.view.LayoutInflater.rInflate(LayoutInflater.java:809) at android.view.LayoutInflater.inflate(LayoutInflater.java:504) at android.view.LayoutInflater.inflate(LayoutInflater.java:414) at android.view.LayoutInflater.inflate(LayoutInflater.java:365) at com.linkedin.android.infra.app.LayoutTestActivity$1.run(LayoutTestActivity.java:92) at android.os.Handler.handleCallback(Handler.java:739) at android.os.Handler.dispatchMessage(Handler.java:95) at android.os.Looper.loop(Looper.java:135) at android.app.ActivityThread.main(ActivityThread.java:5254) at java.lang.reflect.Method.invoke(Native Method) at java.lang.reflect.Method.invoke(Method.java:372) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:903) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:698) Caused by: java.lang.reflect.InvocationTargetException at java.lang.reflect.Constructor.newInstance(Native Method) at java.lang.reflect.Constructor.newInstance(Constructor.java:288) at android.view.LayoutInflater.createView(LayoutInflater.java:607)
... 19 more
Caused by: android.content.res.Resources$NotFoundException: File res/drawable-v21/design_password_eye.xml from drawable resource ID #0x7f0200fe at android.content.res.Resources.loadDrawableForCookie(Resources.java:2479) at android.content.res.Resources.loadDrawable(Resources.java:2381) at android.content.res.Resources.getDrawable(Resources.java:787) at android.content.Context.getDrawable(Context.java:403) at android.support.v4.content.ContextCompatApi21.getDrawable(ContextCompatApi21.java:30) at android.support.v4.content.ContextCompat.getDrawable(ContextCompat.java:372) at android.support.v7.widget.AppCompatDrawableManager.getDrawable(AppCompatDrawableManager.java:202) at android.support.v7.widget.AppCompatDrawableManager.getDrawable(AppCompatDrawableManager.java:190) at android.support.v7.content.res.AppCompatResources.getDrawable(AppCompatResources.java:100) at android.support.v7.widget.TintTypedArray.getDrawable(TintTypedArray.java:71) at android.support.design.widget.TextInputLayout.<init>(TextInputLayout.java:239) at android.support.design.widget.TextInputLayout.<init>(TextInputLayout.java:183)
... 22 more
Caused by: android.content.res.Resources$NotFoundException: File res/drawable-anydpi-v21/avd_show_password.xml from drawable resource ID #0x7f0200c6 at android.content.res.Resources.loadDrawableForCookie(Resources.java:2479) at android.content.res.Resources.loadDrawable(Resources.java:2381) at android.content.res.TypedArray.getDrawable(TypedArray.java:749) at android.graphics.drawable.AnimatedStateListDrawable.parseTransition(AnimatedStateListDrawable.java:472) at android.graphics.drawable.AnimatedStateListDrawable.inflateChildElements(AnimatedStateListDrawable.java:454) at android.graphics.drawable.AnimatedStateListDrawable.inflate(AnimatedStateListDrawable.java:384) at android.graphics.drawable.Drawable.createFromXmlInner(Drawable.java:1128) at android.graphics.drawable.Drawable.createFromXml(Drawable.java:1032) at android.content.res.Resources.loadDrawableForCookie(Resources.java:2469)
... 33 more
Caused by: android.content.res.Resources$NotFoundException: File res/drawable-anydpi-v21/avd_show_password_1.xml from drawable resource ID #0x7f0205e6 at android.content.res.Resources.loadDrawableForCookie(Resources.java:2479) at android.content.res.Resources.loadDrawable(Resources.java:2381) at android.content.res.Resources.getDrawable(Resources.java:787) at android.graphics.drawable.AnimatedVectorDrawable.inflate(AnimatedVectorDrawable.java:286) at android.graphics.drawable.Drawable.createFromXmlInner(Drawable.java:1128) at android.graphics.drawable.Drawable.createFromXml(Drawable.java:1032) at android.content.res.Resources.loadDrawableForCookie(Resources.java:2469)
... 41 more
Caused by: java.lang.RuntimeException: error in parsing "&#8238"
at android.util.PathParser.getFloats(PathParser.java:200) at android.util.PathParser.createNodesFromPathData(PathParser.java:63) at android.graphics.drawable.VectorDrawable$VFullPath.updateStateFromTypedArray(VectorDrawable.java:1436) at android.graphics.drawable.VectorDrawable$VFullPath.inflate(VectorDrawable.java:1418) at android.graphics.drawable.VectorDrawable.inflateInternal(VectorDrawable.java:544) at android.graphics.drawable.VectorDrawable.inflate(VectorDrawable.java:460) at android.graphics.drawable.Drawable.createFromXmlInner(Drawable.java:1128) at android.graphics.drawable.Drawable.createFromXml(Drawable.java:1032) at android.content.res.Resources.loadDrawableForCookie(Resources.java:2469)
... 47 more
Caused by: java.lang.NumberFormatException: Invalid float: "&#8238"
at java.lang.StringToReal.invalidReal(StringToReal.java:63) at java.lang.StringToReal.initialParse(StringToReal.java:164) at java.lang.StringToReal.parseFloat(StringToReal.java:323) at java.lang.Float.parseFloat(Float.java:306) at android.util.PathParser.getFloats(PathParser.java:187) ... 55 more

Comments

ar...@google.com <ar...@google.com> #2Jan 18, 2017 06:23AM

Assigned to ar...@google.com.

Another workaround, getting the ActionBar size at run-time:

TypedValue typed_value = new TypedValue();
getTheme().resolveAttribute(android.support.v7.appcompat.R.attr.actionBarSize, typed_value, true);
swipe_refresh_layout.setProgressViewOffset(false, 0, getResources().getDimensionPixelSize(typed_value.resourceId));

kk...@linkedin.com <kk...@linkedin.com> #3Jan 23, 2017 05:23AM

I am experiencing this too...

kk...@linkedin.com <kk...@linkedin.com> #4Jan 23, 2017 05:54PM

Another workaround might help
handler.postDelayed(new Runnable() {
@Override
public void run() {
initiateRefresh();
}
}, 1000);

ar...@google.com <ar...@google.com> #5Jan 24, 2017 12:14PM

same problem here.

ch...@google.com <ch...@google.com> #6Jan 24, 2017 12:36PM

Status: New

Same issue here. I am calling an AsyncTask (that calls setRefreshing) from onResume.

xa...@google.com <xa...@google.com> Jan 25, 2017 08:58PM

Assigned to be...@google.com.

be...@google.com <be...@google.com> #7Jan 25, 2017 09:11PM

Same here. android.support.v4 v 21.0.0

dh...@linkedin.com <dh...@linkedin.com> #8Jan 25, 2017 09:19PM

Problem still exists with android.support.v4:21.0.3.

be...@google.com <be...@google.com> #9Jan 26, 2017 02:49PM

Reassigned to zt...@android.com.

[Comment deleted]

ji...@gmail.com <ji...@gmail.com> #10Jan 26, 2017 05:10PM

[Comment deleted]

kk...@linkedin.com <kk...@linkedin.com> #11Jan 26, 2017 05:14PM

FWIW: a workaround that seems to work (even without the delay):

final boolean refreshing = true;
swipeToRefreshLayout.post(new Runnable() {
@Override public void run() {
swipeToRefreshLayout.setRefreshing(refreshing);
}
});

zt...@google.com <zt...@google.com> #12Jan 26, 2017 10:55PM

[Comment deleted]

ji...@gmail.com <ji...@gmail.com> #13Jan 29, 2017 01:05PM

[Comment deleted]

je...@gmail.com <je...@gmail.com> #14Feb 3, 2017 03:23PM

The same problem.

je...@gmail.com <je...@gmail.com> #15Feb 3, 2017 03:25PM

The post() workaround works when loading programmatically, but the gesture down is still broken.

zt...@google.com <zt...@google.com> #16Feb 3, 2017 05:43PM

Marked as fixed.

It seems the gesture doesn't work if the view doesn't have a ScrollView widget in its hierarchy.

dn...@google.com <dn...@google.com> #17Feb 23, 2017 05:33AM

My solution is to override SwipeRefreshLayout

private boolean mMeasured = false;
private boolean mPreMeasureRefreshing = false;

@Override
public void onMeasure(int widthMeasureSpec, int heightMeasureSpec) {
super.onMeasure(widthMeasureSpec, heightMeasureSpec);
if (!mMeasured) {
mMeasured = true;
setRefreshing(mPreMeasureRefreshing);
}
}

@Override
public void setRefreshing(boolean refreshing) {
if (mMeasured) {
super.setRefreshing(refreshing);
} else {
mPreMeasureRefreshing = refreshing;
}
}

ra...@flipkart.com <ra...@flipkart.com> #18Feb 28, 2017 11:41AM

Still broken, v22.2.0

ma...@gmail.com <ma...@gmail.com> #19Mar 7, 2017 09:16AM

Fix this already!

[Deleted User] <[Deleted User]> #20Mar 24, 2017 08:39PM

Please, fix this issue.

[Deleted User] <[Deleted User]> #21Mar 29, 2017 10:09PM

Still exists with 22.2.1

on...@gmail.com <on...@gmail.com> #22Sep 21, 2019 05:25PM

Still an issue with 23.0.0

be...@google.com <be...@google.com> Sep 23, 2019 03:12PM

Reassigned to ti...@google.com.

ti...@google.com <ti...@google.com> #23Sep 25, 2019 10:49PM

Still broken in 23.0.1

Issue 37133119

Description

Issue summary

Comments

ar...@google.com <ar...@google.com> #2Jan 18, 2017 06:23AM

kk...@linkedin.com <kk...@linkedin.com> #3Jan 23, 2017 05:23AM

kk...@linkedin.com <kk...@linkedin.com> #4Jan 23, 2017 05:54PM

ar...@google.com <ar...@google.com> #5Jan 24, 2017 12:14PM

ch...@google.com <ch...@google.com> #6Jan 24, 2017 12:36PM

xa...@google.com <xa...@google.com> Jan 25, 2017 08:58PM

be...@google.com <be...@google.com> #7Jan 25, 2017 09:11PM

dh...@linkedin.com <dh...@linkedin.com> #8Jan 25, 2017 09:19PM

be...@google.com <be...@google.com> #9Jan 26, 2017 02:49PM

ji...@gmail.com <ji...@gmail.com> #10Jan 26, 2017 05:10PM

kk...@linkedin.com <kk...@linkedin.com> #11Jan 26, 2017 05:14PM

zt...@google.com <zt...@google.com> #12Jan 26, 2017 10:55PM

ji...@gmail.com <ji...@gmail.com> #13Jan 29, 2017 01:05PM

je...@gmail.com <je...@gmail.com> #14Feb 3, 2017 03:23PM

je...@gmail.com <je...@gmail.com> #15Feb 3, 2017 03:25PM

zt...@google.com <zt...@google.com> #16Feb 3, 2017 05:43PM

dn...@google.com <dn...@google.com> #17Feb 23, 2017 05:33AM

ra...@flipkart.com <ra...@flipkart.com> #18Feb 28, 2017 11:41AM

ma...@gmail.com <ma...@gmail.com> #19Mar 7, 2017 09:16AM

[Deleted User] <[Deleted User]> #20Mar 24, 2017 08:39PM

[Deleted User] <[Deleted User]> #21Mar 29, 2017 10:09PM

on...@gmail.com <on...@gmail.com> #22Sep 21, 2019 05:25PM

be...@google.com <be...@google.com> Sep 23, 2019 03:12PM

ti...@google.com <ti...@google.com> #23Sep 25, 2019 10:49PM

Add comment

Issue metadata