Issue metadata
Sign in to add a comment
|
Chrome_Android: Crash Report - [Android Java Exception] android.view.InflateException at org.chromium.chrome.browser.page_info.PageInfoPopup.<init>(PageInfoPopup.java) |
||||||||||||||||||
Issue descriptionProduct name: Chrome_Android Magic Signature: [Android Java Exception] android.view.InflateException at org.chromium.chrome.browser.page_info.PageInfoPopup.<init>(PageInfoPopup.java) Current link: https://crash.corp.google.com/browse?q=ReportID%3D'98a2f23480000000'%20AND%20custom_data.ChromeCrashProto.magic_signature_1.name%3D'%5BAndroid%20Java%20Exception%5D%20android.view.InflateException%20at%20org.chromium.chrome.browser.page_info.PageInfoPopup.%3Cinit%3E(PageInfoPopup.java)'&ignore_case=false&enable_rewrite=true&omit_field_name=&omit_field_value=&omit_field_opt=%3D#3 Search properties: reportid: 98a2f23480000000 Metadata : Product Name: Chrome_Android Product Version: 59.0.3053.0 Report ID: 98a2f23480000000 Report Time: Mon, 27 Mar 2017 05:52:18 GMT Uptime: 224602 ms Cumulative Uptime: 0 ms User Email: OS Name: Android OS Version: 0.0.0 Linux 3.0.31-2833503 #1 SMP PREEMPT Thu May 21 22:05:42 KST 2015 armv7l CPU Architecture: arm CPU Info: ARMv7 ARM Cortex-A9 features: swp,half,thumb,fastmult,vfpv2,edsp,neon,vfpv3,tls
,
Mar 27 2017
Please find logs and Video @ http://go/chrome-androidlogs1/7/705355 Bisect range: https://chromium.googlesource.com/chromium/src/+log/59.0.3050.0..59.0.3051.3?pretty=fuller&n=10000 Good commit: 459561 Bad commit: 459562 Culprit CL: https://chromium.googlesource.com/chromium/src/+/617a30f35e5cec5df6c15a8d7fbb222cdbabcccc @lgarron could you please look into this issue, thanks!
,
Mar 27 2017
,
Mar 27 2017
I can verify that this crashes in the release build. That CL is almost certainly the culprit, but I didn't have trouble in a debug build. Looking into it.
,
Mar 27 2017
Thanks for looking into the crash. Please note that this is blocking our M59 Dev push, so it will need to be fixed today. Can the CL be reverted?
,
Mar 27 2017
I would prefer to avoid reverting the CL if the fix is not large, since that would otherwise cause a bunch of rebasing for me. And if the issue is really only in the release build (and not debug), then I might not have a better chance of fixing it until relanding. Could you give me 3 hours to investigate?
,
Mar 27 2017
I think the fix is straightforward: replace WebsiteSettingsPopup with PageInfoPopup. https://cs.chromium.org/chromium/src/chrome/android/java/res/layout/page_info.xml?rcl=f9df31e030a462ef2421b188c7aa5129c7b6044c&l=25 I'm on the last stages of verifying.
,
Mar 27 2017
Verified that the fix prevents the crash. CL is up at https://codereview.chromium.org/2778023002
,
Mar 27 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bdab57c60efc34aedfb2055b1296008ba9015414 commit bdab57c60efc34aedfb2055b1296008ba9015414 Author: lgarron <lgarron@chromium.org> Date: Mon Mar 27 20:48:14 2017 Fix class reference in page_info.xml BUG= 705355 TEST=Visit https://google.com, click the lock icon in the URL bar, and check that the Page Info popup appears (it contains a heading stating "Secure connection"). Review-Url: https://codereview.chromium.org/2778023002 Cr-Commit-Position: refs/heads/master@{#459869} [modify] https://crrev.com/bdab57c60efc34aedfb2055b1296008ba9015414/chrome/android/java/res/layout/page_info.xml
,
Mar 27 2017
,
Mar 27 2017
,
Mar 28 2017
This issue is fixed in latest M59 build 59.0.3054.0 |
|||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||
Comment 1 by arunshan@chromium.org
, Mar 27 2017