New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 621616 link

Starred by 1 user
Status: Fixed
Owner:
tedc...@chromium.org
Closed: Jun 2016
Cc:
█ ian...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: ----



Sign in to add a comment

Chrome crashes if you unbind CCT service w/o first accepting FRE

Project Member Reported by tedc...@chromium.org, Jun 20 2016 
1.) Clear chrome's data and do NOT launch
2.) Build and install custom_tabs_client_example_apk
3.) Launch CustomTabsClientExample and bind to your Chrome instance.  Hit the back button.

Expected:
Nothing

Observerd:
Chrome crashes:
Caused by: java.lang.NullPointerException
E/AndroidRuntime( 8358):        at org.chromium.chrome.browser.customtabs.CustomTabsConnectionService.onUnbind(CustomTabsConnectionService.java:37)

Comment 1 by tedc...@chromium.org, Jun 20 2016 

Fixed here but did not add bug label:
https://codereview.chromium.org/2075893003/

Comment 2 by tedc...@chromium.org, Jun 20 2016

Labels: Merge-Request-52

Comment 3 by tin...@google.com, Jun 21 2016

Labels: -Merge-Request-52 Merge-Approved-52 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M52 (branch: 2743)

Comment 4 by tedc...@chromium.org, Jun 21 2016

Labels: -M-52 -Merge-Request-52
Status: Fixed (was: Started)
I apologize, turns out it was broken only on 53 and we requested merge too eagerly.  Removing the bits and marking this fixed.
►

Sign in to add a comment