New issue
Advanced search Search tips

Issue 716421 link

Starred by 2 users
Status: Duplicate
Owner:
jianli@chromium.org
Closed: Apr 2017
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Crash when clicking on other tab header (after reopening browser)

Project Member Reported by mar...@mwiacek.com, Apr 28 2017 
Steps to reproduce the problem:
1. open few tabs
2. switch from Chrome to other app
3. kill Chrome
4. open Chrome
5. go into tab list
6. click on other tab header

What is the expected behavior?
Switching to clicked tab

What went wrong?

crash, see log.txt

Crashed report ID: 25b1d06b-bcb0-8504-658666cc-388d3626 ?

How much crashed? Whole browser

Is it a problem with a plugin? No 

Did this work before? Yes builds from this week

Chrome version: 60.0.3082.0  Channel: canary
OS Version: 7
Flash Version: 

1. If you will make it internal, I will probably not have access.

2. Galaxy S7 / Android 7
log.txt
105 KB View Download

Comment 1 by amp@chromium.org, Apr 28 2017

Labels: -Pri-2 ReleaseBlock-Dev Pri-1
I see this as well.  I bisected the change to the following, but I'm not sure why this change causes it.

commit f5fbad7f77e6e39ee8c5d3ec7516447b72128ddc (HEAD)
Author: jianli <jianli@chromium.org>
Date:   Tue Apr 25 15:35:47 2017 -0700

    Move offline related functions from Tab to OfflinePageUtils
    
    BUG= 715260 
    
    Review-Url: https://codereview.chromium.org/2838773002
    Cr-Commit-Position: refs/heads/master@{#467143}

Comment 2 by amp@chromium.org, Apr 28 2017

Owner: jianli@chromium.org
Status: Assigned (was: Unconfirmed)
Assigning to jianli to take a look.  I'm not sure what the best components would be here.

The stack trace I get is (this is after syncing to f5fbad7f77e6e39ee8c5d3ec7516447b72128ddc so line numbers should correspond to the files at that time):
signal 11 (SIGSEGV), code 1, fault addr 0x0 in tid 5510 (chromium.chrome)
pid: 5510, tid: 5510, name: chromium.chrome  >>> org.chromium.chrome <<<
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x0
     r0 00000001  r1 00008001  r2 00000000  r3 00001586
     r4 00000000  r5 00000000  r6 cadad294  r7 ff89d538
     r8 00000001  r9 e765a000  sl 00000000  fp ff89d6a4
     ip cadab578  sp ff89d498  lr cad1e447  pc c84669b8

Stack Trace:
  RELADDR   FUNCTION                                                                                                                                                       FILE:LINE
  000069b8  BrowserContextKeyedServiceFactory::GetBrowserContextToUse(content::BrowserContext*) const                                                                      /usr/local/google/home/amp/workspaces/chromium/src/components/keyed_service/content/browser_context_keyed_service_factory.cc:55
  00006ee9  KeyedServiceFactory::GetServiceForContext(base::SupportsUserData*, bool)                                                                                       /usr/local/google/home/amp/workspaces/chromium/src/components/keyed_service/core/keyed_service_factory.cc:63
  0022cd1b  offline_pages::OfflinePageModelFactory::GetForBrowserContext(content::BrowserContext*)                                                                         /usr/local/google/home/amp/workspaces/chromium/src/chrome/browser/android/offline_pages/offline_page_model_factory.cc:38
  v------>  offline_pages::android::GetOfflinePageBridgeForProfile(_JNIEnv*, base::android::JavaParamRef<_jclass*> const&, base::android::JavaParamRef<_jobject*> const&)  /usr/local/google/home/amp/workspaces/chromium/src/chrome/browser/android/offline_pages/offline_page_bridge.cc:236
  00271cdb  Java_org_chromium_chrome_browser_offlinepages_OfflinePageBridge_nativeGetOfflinePageBridgeForProfile                                                           /usr/local/google/home/amp/workspaces/chromium/src/out/Default/gen/chrome/browser/jni_headers/chrome/jni/OfflinePageBridge_jni.h:92
  00068ad1  offset 0x4b000                                                                                                                                                 /data/app/org.chromium.chrome-m0r8pz4pHPfN3UuVpcUT4A==/oat/arm/base.odex

Comment 3 by amp@chromium.org, Apr 28 2017 

Hmm.  Appears to be fixed as of today (after syncing to 4e899699e945d5b065580d2bd42f59e6ea09879e)

Comment 4 by amp@chromium.org, Apr 28 2017

Mergedinto: 715751
Status: Duplicate (was: Assigned)
Dupe of issue 715751
►

Sign in to add a comment