Crash: InflateException: Binary XML file line #26: Error inflating class org.wordpress.android.l... #9905

sentry-io · 2019-05-22T10:43:04Z

Sentry Issue: https://sentry.io/share/issue/136cee38cb79491db456071ee0c686a0/

NotFoundException: Unable to find resource ID #0x7f0800bb
    at org.wordpress.android.util.widgets.WPTextInputLayout.<init>(WPTextInputLayout.java:20)
    at org.wordpress.android.login.widgets.WPLoginInputRow.init(WPLoginInputRow.java:67)
    at org.wordpress.android.login.widgets.WPLoginInputRow.<init>(WPLoginInputRow.java:58)
    at org.wordpress.android.login.LoginBaseFormFragment.createMainView(LoginBaseFormFragment.java:100)
    at org.wordpress.android.login.LoginBaseFormFragment.onCreateView(LoginBaseFormFragment.java:106)
...
(60 additional frame(s) were not displayed)

NotFoundException: Drawable (missing name) with resource ID #0x7f0800bb
NotFoundException: File res/drawable-v21/design_password_eye.xml from drawable resource ID #0x7f0800bd
    at org.wordpress.android.util.widgets.WPTextInputLayout.<init>(WPTextInputLayout.java:20)
    at org.wordpress.android.login.widgets.WPLoginInputRow.init(WPLoginInputRow.java:67)
    at org.wordpress.android.login.widgets.WPLoginInputRow.<init>(WPLoginInputRow.java:58)
    at org.wordpress.android.login.LoginBaseFormFragment.createMainView(LoginBaseFormFragment.java:100)
    at org.wordpress.android.login.LoginBaseFormFragment.onCreateView(LoginBaseFormFragment.java:106)
...
(49 additional frame(s) were not displayed)

NotFoundException: Drawable org.wordpress.android:drawable/design_password_eye with resource ID #0x7f0800bd
InvocationTargetException: None
    at org.wordpress.android.login.widgets.WPLoginInputRow.init(WPLoginInputRow.java:67)
    at org.wordpress.android.login.widgets.WPLoginInputRow.<init>(WPLoginInputRow.java:58)
    at org.wordpress.android.login.LoginBaseFormFragment.createMainView(LoginBaseFormFragment.java:100)
    at org.wordpress.android.login.LoginBaseFormFragment.onCreateView(LoginBaseFormFragment.java:106)
...
(37 additional frame(s) were not displayed)

InflateException: Binary XML file line #21: Error inflating class org.wordpress.android.util.widgets.WPTextInputLayout
InflateException: Binary XML file line #21: Binary XML file line #21: Error inflating class org.wordpress.android.util.widgets.WPTextInputLayout
InvocationTargetException: None
    at org.wordpress.android.login.LoginBaseFormFragment.createMainView(LoginBaseFormFragment.java:100)
    at org.wordpress.android.login.LoginBaseFormFragment.onCreateView(LoginBaseFormFragment.java:106)
...
(24 additional frame(s) were not displayed)

InflateException: Binary XML file line #26: Error inflating class org.wordpress.android.login.widgets.WPLoginInputRow
InflateException: Binary XML file line #26: Binary XML file line #26: Error inflating class org.wordpress.android.login.widgets.WPLoginInputRow

Binary XML file line #26: Binary XML file line #26: Error inflating class org.wordpress.android.login.widgets.WPLoginInputRow

The text was updated successfully, but these errors were encountered:

designsimply · 2019-07-11T22:14:27Z

30-day impact: ~73 per day
Users affected: 698 in the last 30d
Last seen in: 12.7.1 (latest release at the time of this report)

^{https://sentry.io/share/issue/9eaadccac3be4c259943d72c46427792/}

marecar3 · 2019-07-16T15:38:57Z

It seems that we are dealing with the same issue as WOOCOMMERCE is dealing with.

Thanks to @anitaa1990 and @0nko's comment: woocommerce/woocommerce-android#1141 we saved some time here.

We need to make some decision what would be next steps in solving this issue.

Suggestion from the comment is to add this: https://developer.android.com/guide/app-bundle/sideload-check
Into our application, in order to not allow users to install an app without google play.

elibud · 2019-07-16T16:34:15Z

I think @daniloercoli may have some insights too.

daniloercoli · 2019-07-16T16:48:24Z

Restrict to Google Play only devices is super sad.

Not sure if we can fix this issue here by adding run-time checks about the availability of Google Play Services, like it was done here #9831

marecar3 · 2019-07-22T22:22:31Z

Summary

This one is really difficult to be resolved as it's hard to find a scenario which will crash the application.

Based on the latest reports from the sentry I am not sure that the cause of this problem is because our users have rooted devices.

Both WPAndroid and WooComerce are using the same version of material-components -> https://github.com/material-components/material-components-android/releases/tag/1.0.0

Not sure if that is a potential problem, but as I can see the version 1.0 is using style to include drawables

And newest version: 1.1.0-alpha08 is using quite a different approach.

https://github.com/material-components/material-components-android/blob/fa0be1485f6edb6dbd3f92e604a7ff15651a3f7d/lib/java/com/google/android/material/textfield/PasswordToggleEndIconDelegate.java#L81

Not sure if it's safe to use the newest library (regarding our UI) but maybe if we update it, it will affect to crashes disappear?

@daniloercoli @hypest @maxme @mzorz Any thoughts?

mzorz · 2019-07-23T13:38:32Z

Been following this interesting issue here!

In 1.1.0-alpha-104 they've added support for a custom end icon for the TextInputLayout (material-components/material-components-android@5685941) so my reasoning is: if the cause of the crash is the resource not being available for whatever cause, we can make sure it is available by packaging the eye resource ourselves and referencing that by using the custom end icon capability.

That said, the issues described about sideloading IIUC wouldn't be totally circumvented by this, but given the crashes are not caused by missing WP resources at all I'm pretty confident packaging the resource ourselves should help alleviate the problem.

We'd be targeting an alpha product that could have other consequences, but it may be worth a try. Wdyt @marecar3 ? Would you like to try that?

marecar3 · 2019-07-23T22:28:12Z

Hey @mzorz thanks for the information. I am on it.

designsimply · 2019-09-10T01:16:46Z

90-day impact: ~113 per day
Users affected in the last 90 days: 2700
_{https://sentry.io/share/issue/9eaadccac3be4c259943d72c46427792/}

Moving this back to the Prioritized Android list in the Groundskeeping project and noting that the open PR needs a review or a nudge.

designsimply · 2019-10-10T23:14:04Z

90-day impact: ~167 per day
Users affected in the last 90 days: 3600
_{https://sentry.io/share/issue/9eaadccac3be4c259943d72c46427792/}

designsimply · 2019-11-06T23:25:01Z

90-day impact: ~189 per day
Users affected in the last 90 days: 3200
_{https://sentry.io/share/issue/9eaadccac3be4c259943d72c46427792/}

designsimply · 2019-12-04T23:29:09Z

Users affected in the last 90 days: 2400
_{https://sentry.io/share/issue/9eaadccac3be4c259943d72c46427792/}

The open PR for this needs follow-up.

designsimply · 2019-12-18T18:32:12Z

Users affected in the last 90 days: 1900
_{https://sentry.io/share/issue/9eaadccac3be4c259943d72c46427792/}

^{(Also see internal reference: paqN3M-7F-p2)}

designsimply · 2020-01-22T23:09:48Z

Users affected in the last 90 days: 1200
_{https://sentry.io/share/issue/9eaadccac3be4c259943d72c46427792/}

elibud · 2020-01-22T23:28:29Z

Users affected in the last 90 days: 1200

Not good. @oguzkocer I believe you are in groundskeeping next week? Can you please take a look at this one?

oguzkocer · 2020-01-23T01:02:43Z

@elibud I'd be happy to check it out, however there is already a PR for it with quite a bit discussion. From what I understand, it looks like there is still an issue with AppCompat and we already reported the issue. Following up on that discussion with the people that are already involved may be the faster way to resolution. Having said that, if you do want me to take over, follow up etc, I'd be happy to do that as well.

Here is the PR in question: #10279

elibud · 2020-01-23T13:18:12Z

Following up on that discussion with the people that are already involved may be the faster way to resolution.

Got it, thank you @oguzkocer. @khaykov @marecar3 any updates there?

marecar3 · 2020-01-23T13:28:42Z

Hey @elibud
I have responded to the latest question from Google and also I have asked them if they have some proposal which can be used to overcome the issue, however, we didn't get an answer from their side. (Almost week ago the last email was sent)

marecar3 · 2020-01-23T13:31:51Z

Maybe we shouldn't wait for them 😕
@oguzkocer do you maybe how some other approach on your mind?
This one isn't easy to reproduce so not sure what could be an alternative solution for it? Thanks.

oguzkocer · 2020-01-24T03:43:07Z

I have gone through the discussions across several issues and had a look at the changes in the PR. I can't help but feel like due to the confusing nature of the issue and uncertainty about the android material library update, we might have lost sight of the main issue. First of, here is my understanding of the situation:

If we use our own drawable for the password visibility, we won't have this crash in WPAndroid or WCAndroid
Updating the material library may fix the issue without having to use our own drawable, but it's not certain since we can't reproduce the crash reliably. There is also a separate issue that prevents us from merging this upgrade.

Please correct me if I am wrong about any of these assumptions and ignore the rest of the comment in that case.

With these assumptions in mind here are my suggestions:

Open a separate PR and set our own drawable and ship it as soon as possible. Our main priority should be to get rid of the crash which hopefully this change will accomplish. Even if it doesn't, it'll give us more clues about the issue. If it does fix the issue, one extra drawable is really nothing to be concerned about especially considering the time and energy spent on these issues so far.
Open a new issue to update the material library or add a new comment in the existing PR. Summarize what the situation is and refer to both this issue and the original PR. Clearly state what the blocker is and what steps should be taken once the blocker is gone. These steps should include trying to revert the separate PR from step 1. We still don't want to leave an extra drawable in the code if it's no longer necessary.
Close this issue. We can re-open it if the crash is not fixed yet.

Hope all this makes sense and let me know if you have any questions or concerns @elibud @marecar3.

marecar3 · 2020-01-24T09:49:51Z

Hey @oguzkocer,

Thanks for sharing your thoughts!

The main problem for me was how can we use our own drawable for a password as they are using their own drawable - which is crashing the app, in the constructor of the component :

https://github.com/material-components/material-components-android/blob/cd59e98f7e2185ddb075ff0fc91f29765d562968/lib/java/com/google/android/material/textfield/TextInputLayout.java#L344

@oguzkocer maybe I am missing something but I don't see which approach can be used in order to override the code from the constructor? Thanks!

oguzkocer · 2020-01-25T03:34:24Z

@marecar3 I might be missing something as well, but can we not do something like this:

<org.wordpress.android.util.widgets.WPTextInputLayout
    android:layout_centerVertical="true"
    app:theme="@style/LoginTheme.EditText"
    android:id="@+id/input_layout"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:layout_toEndOf="@+id/icon"
    app:passwordToggleDrawable="drawable/selector_password_visibility"
    android:importantForAccessibility="no">

When I check the decompiled code for TextInputLayout I don't see the same constructor, but even in the version you linked, wouldn't this just use the drawable we give bypassing the missing resource? There is not a whole lot of documentation about R.styleable but my understanding is that it'd work like this. What do you think?

marecar3 · 2020-01-27T09:40:03Z

Hey, @oguzkocer thanks for the great suggestion!

Let's do it in that way and see what will happen. Here is the new PR: #11172

cc: @elibud @designsimply

oguzkocer · 2020-02-12T17:31:44Z

Unfortunately the solution in #11172 did not work for all users. We have one user crashing multiple times in 14.1: https://sentry.io/share/issue/a7681d396e284f1990a5f3005f031b7b/

It's very early for 14.1, so we'll need to keep track to see if it's still a common issue or if the solution somewhat worked and now it's more isolated.

The device that crashed is itel it1508 with Android 5.1. I tried creating an emulator with the same resolution and Android version, but couldn't reproduce the issue.

cc @marecar3 @designsimply

marecar3 · 2020-02-13T14:25:30Z

Hey @oguzkocer,

Thanks for tracking this was. I was also wondering did we manage to solve it.

I saw this error in the stack trace: android.content.res.NotFoundException: File res/drawable/selector_password_visibility.xml from drawable resource ID #0x7f08025f

This means that the app can't find a selector that was created in a recent PR fix.
Not sure why it couldn't find the resource, I will google it.

marecar3 · 2020-02-18T11:18:45Z

It seems that users can also experience a side effect of the fix.

I assume that there is also a case when Android can't find a drawable reference and somehow it connects other drawable instead.

Here is the issue: #11307

oguzkocer · 2020-02-19T16:56:53Z

Thanks for looking into this again @marecar3! The side effect is pretty weird, I honestly don't understand what's going on with these drawables in certain phones. Also, the fact that it's happening for a password field is making me feel uneasy.

designsimply · 2020-05-01T20:16:31Z

Zero events have been tracked for this crash in 14.6.1 since it was released 9d ago on Apr 22.

Number of users affected in the last 90d: 534
_{https://sentry.io/share/issue/06479f2a38234c8abad668f173724b9f/}

Number of users affected in the last 90d from May 19: 384
_{https://sentry.io/share/issue/9eaadccac3be4c259943d72c46427792/}

@oguzkocer the latest version we're seeing this crash happen for in the last 90d is 14.0, which makes me feel it's no longer happening in current releases, but is it still worth keeping the issue open because it's related to the password field (which you mentioned in your latest comment here back in Feb)?

oguzkocer · 2020-05-02T06:01:47Z

@designsimply The issue number you used for that query is incorrect and there has been 10 events in 14.6.1. You can also see this information in the tags section of the event and verify that it has been happening in pretty much every release.

designsimply · 2020-05-19T17:24:11Z

@oguzkocer thanks for catching that! 🙇

On a closer look, I see that I got the event number correct for the 14.6.1 query but used the wrong link for the number of users affected in the last 90d.

I fixed the error in the comment above, and here's another look at the numbers drilling into the 14.7 release:

Zero events have been tracked for this crash in 14.7 since it was released 15d ago on May 4.

Events in the last 90d: 23,000
Users affected in the last 90d: 384
Limited to: WPAndroid 14.0 and below
_{https://sentry.io/share/issue/9eaadccac3be4c259943d72c46427792/}

The total number of issues is still high, but if you look at the Tags section of the Sentry issue, you can see that it's only happening to version 14.0 and below of the WP Android app. In this case do you think it's okay to close or should we look into trying to resolve this for older versions as well? Note that the number of users affected is probably in the middle for the crashes we're tracking for WPAndroid in GitHub at the moment, which would lead me to prioritize this as medium. wdyt?

oguzkocer · 2020-05-19T18:48:52Z

@designsimply That sounds reasonable and when in doubt I almost always prefer to resolve the crash in Sentry since it'll just come back as regression if it's not actually fixed. However, I believe we have several issues for this crash in Sentry, so let me look through them in my next crash monitoring session to just verify that it's not happening in the newer releases. If that's the case, I'll close this and resolve the crashes in Sentry.

oguzkocer · 2020-05-20T20:00:40Z

As mentioned in my previous comment, we have a lot of different issues in Sentry for this same crash and unfortunately some of it seems to be still happening in the latest releases. I've merged relevant issues and there seems to be a significant downward trend in the number of crashes in the latest releases, but somehow some of these crashes are ending up in the crash reported in #11966 which is a completely unrelated issue. So working on this issue right now is tricky and the impact of it is unclear. My proposed plan of action is:

Fix RuntimeException: Unable to start activity ComponentInfo{org.wordpress.android/org.wordpress.android.ui.posts.Posts... #11966 and resolve the Sentry issue for it
Resolve the Sentry issue for this issue: https://sentry.io/share/issue/136cee38cb79491db456071ee0c686a0/
When we start getting new instances for this crash, open a new issue and link it to this one, so we have a clear start with up to date information

If we are not planning to work on this issue in the next couple weeks, which I don't think we should because the numbers are kind of low in Sentry and it's hard to fix this crash, then I think this plan would work. If we want to fix the issue as soon as possible, I think whoever is working on this needs to do a little bit of investigation which I'd be happy to help with.

cc @designsimply @yaelirub (as triager for the week, I think you might want to know that the linked issue is related)

marecar3 · 2020-05-22T13:51:38Z

Hey @oguzkocer @khaykov @designsimply 👋
I have opened a new PR which we did have in the past but we didn't merge it because of the material library version. Now that we have good version of the material library, maybe we can try with this fix?

#12000

grndvl1 · 2023-07-24T23:03:31Z

I am wondering as I am getting the same crash only periodically by users that we have 100K users per day, we maybe get only 1 per day. Only thing I see is that the line number in our code for XML points to "android:importantForAccessibility="no" on the image file. Could it be something with that attribute?

loremattei added the [Type] Crash label May 22, 2019

designsimply added the Login label Jul 11, 2019

marecar3 self-assigned this Jul 15, 2019

mzorz mentioned this issue Jul 23, 2019

Groundkseeping master issue: Hack Week July 22nd-26th #10268

Closed

6 tasks

marecar3 mentioned this issue Jul 25, 2019

Fix/9905 crash inflate exception binary xml file line #10279

Closed

1 task

designsimply unassigned marecar3 Nov 7, 2019

marecar3 mentioned this issue Jan 27, 2020

Add custom selector for password button #11172

Merged

3 tasks

oguzkocer closed this as completed in #11172 Jan 27, 2020

designsimply assigned marecar3 Jan 27, 2020

oguzkocer reopened this Feb 12, 2020

khaykov mentioned this issue Feb 12, 2020

Issue/fix lifecycle crash with appcompat 1 1 0 #11292

Merged

3 tasks

yaelirub mentioned this issue May 20, 2020

RuntimeException: Unable to start activity ComponentInfo{org.wordpress.android/org.wordpress.android.ui.posts.Posts... #11966

Closed

marecar3 mentioned this issue May 22, 2020

Fix Crash InflateException Binary XML file #12000

Merged

3 tasks

oguzkocer closed this as completed in #12000 May 25, 2020

marecar3 mentioned this issue May 26, 2020

Fix password drawable on login screen wordpress-mobile/WordPress-Login-Flow-Android#40

Merged

oguzkocer mentioned this issue Jul 16, 2020

InflateException for design_password_eye in Login #12466

Closed

ParaskP7 mentioned this issue May 17, 2023

[Dependency Updates] Update googleMaterialVersion to 1.9.0 #18433

Merged

18 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Crash: InflateException: Binary XML file line #26: Error inflating class org.wordpress.android.l... #9905

Crash: InflateException: Binary XML file line #26: Error inflating class org.wordpress.android.l... #9905

sentry-io bot commented May 22, 2019 •

edited by oguzkocer

Loading

designsimply commented Jul 11, 2019

marecar3 commented Jul 16, 2019 •

edited

Loading

elibud commented Jul 16, 2019

daniloercoli commented Jul 16, 2019

marecar3 commented Jul 22, 2019

mzorz commented Jul 23, 2019 •

edited

Loading

marecar3 commented Jul 23, 2019

designsimply commented Sep 10, 2019 •

edited

Loading

designsimply commented Oct 10, 2019 •

edited

Loading

designsimply commented Nov 6, 2019 •

edited

Loading

designsimply commented Dec 4, 2019

designsimply commented Dec 18, 2019 •

edited

Loading

designsimply commented Jan 22, 2020

elibud commented Jan 22, 2020

oguzkocer commented Jan 23, 2020

elibud commented Jan 23, 2020

marecar3 commented Jan 23, 2020

marecar3 commented Jan 23, 2020

oguzkocer commented Jan 24, 2020

marecar3 commented Jan 24, 2020

oguzkocer commented Jan 25, 2020

marecar3 commented Jan 27, 2020

oguzkocer commented Feb 12, 2020

marecar3 commented Feb 13, 2020 •

edited

Loading

marecar3 commented Feb 18, 2020

oguzkocer commented Feb 19, 2020

designsimply commented May 1, 2020 •

edited

Loading

oguzkocer commented May 2, 2020

designsimply commented May 19, 2020 •

edited

Loading

oguzkocer commented May 19, 2020

oguzkocer commented May 20, 2020

marecar3 commented May 22, 2020

grndvl1 commented Jul 24, 2023 •

edited

Loading

Crash: InflateException: Binary XML file line #26: Error inflating class org.wordpress.android.l... #9905

Crash: InflateException: Binary XML file line #26: Error inflating class org.wordpress.android.l... #9905

Comments

sentry-io bot commented May 22, 2019 • edited by oguzkocer Loading

designsimply commented Jul 11, 2019

marecar3 commented Jul 16, 2019 • edited Loading

elibud commented Jul 16, 2019

daniloercoli commented Jul 16, 2019

marecar3 commented Jul 22, 2019

mzorz commented Jul 23, 2019 • edited Loading

marecar3 commented Jul 23, 2019

designsimply commented Sep 10, 2019 • edited Loading

designsimply commented Oct 10, 2019 • edited Loading

designsimply commented Nov 6, 2019 • edited Loading

designsimply commented Dec 4, 2019

designsimply commented Dec 18, 2019 • edited Loading

designsimply commented Jan 22, 2020

elibud commented Jan 22, 2020

oguzkocer commented Jan 23, 2020

elibud commented Jan 23, 2020

marecar3 commented Jan 23, 2020

marecar3 commented Jan 23, 2020

oguzkocer commented Jan 24, 2020

marecar3 commented Jan 24, 2020

oguzkocer commented Jan 25, 2020

marecar3 commented Jan 27, 2020

oguzkocer commented Feb 12, 2020

marecar3 commented Feb 13, 2020 • edited Loading

marecar3 commented Feb 18, 2020

oguzkocer commented Feb 19, 2020

designsimply commented May 1, 2020 • edited Loading

oguzkocer commented May 2, 2020

designsimply commented May 19, 2020 • edited Loading

oguzkocer commented May 19, 2020

oguzkocer commented May 20, 2020

marecar3 commented May 22, 2020

grndvl1 commented Jul 24, 2023 • edited Loading

sentry-io bot commented May 22, 2019 •

edited by oguzkocer

Loading

marecar3 commented Jul 16, 2019 •

edited

Loading

mzorz commented Jul 23, 2019 •

edited

Loading

designsimply commented Sep 10, 2019 •

edited

Loading

designsimply commented Oct 10, 2019 •

edited

Loading

designsimply commented Nov 6, 2019 •

edited

Loading

designsimply commented Dec 18, 2019 •

edited

Loading

marecar3 commented Feb 13, 2020 •

edited

Loading

designsimply commented May 1, 2020 •

edited

Loading

designsimply commented May 19, 2020 •

edited

Loading

grndvl1 commented Jul 24, 2023 •

edited

Loading