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

Issue 634987 link

Starred by 6 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Dec 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

Toggling "automatically hide and show menu bar" hangs every renderer

Project Member Reported by ellyjo...@chromium.org, Aug 5 2016

Issue description

If you toggle this system setting (in Settings > General) while Chrome is running, every renderer process will hang, like so:

    2473 Thread_6836792   DispatchQueue_1: com.apple.main-thread  (serial)
    + 2456 start  (in Google Chrome Helper)  load address 0x1023e6000 + 0xb44  []
    + ! 2456 main  (in Google Chrome Helper) + 522  [chrome_exe_main_mac.c:89]
    + !   2456 ChromeMain  (in Google Chrome Framework) + 58  [chrome_main.cc:85]
    + !     2456 content::ContentMain(content::ContentMainParams const&)  (in Google Chrome Framework)  load address 0x10261d000 + 0x145f9a6  [content_main.cc:20]
    + !       2456 content::ContentMainRunnerImpl::Run()  (in Google Chrome Framework)  load address 0x10261d000 + 0x1460760  [content_main_runner.cc:785]
    + !         2456 content::RendererMain(content::MainFunctionParams const&)  (in Google Chrome Framework)  load address 0x10261d000 + 0x4ef1914  [renderer_main.cc:198]
    + !           2456 base::RunLoop::Run()  (in Google Chrome Framework)  load address 0x10261d000 + 0x18e3001  [run_loop.cc:36]
    + !             2456 base::MessagePumpCFRunLoopBase::Run(base::MessagePump::Delegate*)  (in Google Chrome Framework)  load address 0x10261d000 + 0x18c87f4  [message_pump_mac.mm:246]
    + !               2456 base::MessagePumpNSRunLoop::DoRun(base::MessagePump::Delegate*)  (in Google Chrome Framework)  load address 0x10261d000 + 0x18c901e  [message_pump_mac.mm:606]
    + !                 2456 -[NSRunLoop(NSRunLoop) runMode:beforeDate:]  (in Foundation) + 270  [0x7fff888b5ed9]
    + !                   2456 CFRunLoopRunSpecific  (in CoreFoundation) + 296  [0x7fff89692ed8]
    + !                     2456 __CFRunLoopRun  (in CoreFoundation) + 2347  [0x7fff89693a6b]
    + !                       2456 __CFRunLoopDoBlocks  (in CoreFoundation) + 341  [0x7fff89693d05]
    + !                         2456 __CFRUNLOOP_IS_CALLING_OUT_TO_A_BLOCK__  (in CoreFoundation) + 12  [0x7fff896b44ac]
    + !                           2456 ___CFXRegistrationPost_block_invoke  (in CoreFoundation) + 63  [0x7fff896aab4f]
    + !                             2456 __CFNOTIFICATIONCENTER_IS_CALLING_OUT_TO_AN_OBSERVER__  (in CoreFoundation) + 12  [0x7fff896aabbc]
    + !                               2456 CFNotificationProc(__CFNotificationCenter*, void*, __CFString const*, void const*, __CFDictionary const*)  (in HIToolbox) + 316  [0x7fff939e774f]
    + !                                 2456 BroadcastInternal(unsigned int, void*, unsigned char)  (in HIToolbox) + 135  [0x7fff9398e0ff]
    + !                                   2456 MenuBarAutoHidingListener(unsigned int, void*, void*)  (in HIToolbox) + 66  [0x7fff93b06deb]
    + !                                     2456 UpdateAllAggregateUIModes(MenuBarAnimationStyle, unsigned char)  (in HIToolbox) + 136  [0x7fff9396efc4]
    + !                                       2456 MenuBarInstance::ForEachMenuBarDo(void (MenuBarInstance*) block_pointer)  (in HIToolbox) + 47  [0x7fff9396f01f]
    + !                                         2456 MenuBarInstance::UpdateAggregateUIMode(MenuBarAnimationStyle, unsigned char, unsigned char, unsigned char)  (in HIToolbox) + 715  [0x7fff9396f3a7]
    + !                                           2456 MenuBarInstance::Hide(MenuBarAnimationStyle, unsigned char, unsigned char, unsigned char*, unsigned char)  (in HIToolbox) + 245  [0x7fff93b059b7]
    + !                                             2456 MenuBarInstance::ForEachWindowDo(unsigned char, bool (OpaqueWindowPtr*, unsigned int) block_pointer)  (in HIToolbox) + 177  [0x7fff9396f9fd]
    + !                                               2456 MBWindows::GetWindowOnDisplay(unsigned int, unsigned char)  (in HIToolbox) + 282  [0x7fff9396fc62]
    + !                                                 2456 ShowHide  (in HIToolbox) + 34  [0x7fff939808df]
    + !                                                   2456 _ShowHideWindows  (in HIToolbox) + 282  [0x7fff939809ff]
    + !                                                     2456 WindowData::PrepareForVisibility()  (in HIToolbox) + 153  [0x7fff9398132d]
    + !                                                       2456 HIView::Render(unsigned int, CGContext*)  (in HIToolbox) + 54  [0x7fff93981c10]
    + !                                                         2456 HIView::DrawComposited(short, OpaqueGrafPtr*, __HIShape const*, unsigned int, HIView*, CGContext*)  (in HIToolbox) + 1031  [0x7fff9398201d]
    + !                                                           2456 HIView::RecursiveDrawComposited(__HIShape const*, __HIShape const*, unsigned int, HIView*, CGContext*, unsigned char, double)  (in HIToolbox) + 1293  [0x7fff93982869]
    + !                                                             2456 HIView::RecursiveDrawComposited(__HIShape const*, __HIShape const*, unsigned int, HIView*, CGContext*, unsigned char, double)  (in HIToolbox) + 557  [0x7fff93982589]
    + !                                                               2456 HIView::SendDraw(short, OpaqueGrafPtr*, __HIShape const*, CGContext*)  (in HIToolbox) + 337  [0x7fff93982c07]
    + !                                                                 2456 SendEventToEventTargetWithOptions  (in HIToolbox) + 43  [0x7fff93965aab]
    + !                                                                   2456 SendEventToEventTargetInternal(OpaqueEventRef*, OpaqueEventTargetRef*, HandlerCallRec*)  (in HIToolbox) + 404  [0x7fff93965c48]
    + !                                                                     2456 DispatchEventToHandlers(EventTargetRec*, OpaqueEventRef*, HandlerCallRec*)  (in HIToolbox) + 1231  [0x7fff939667be]
    + !                                                                       2456 HIView::EventHandler(OpaqueEventHandlerCallRef*, OpaqueEventRef*, void*)  (in HIToolbox) + 2962  [0x7fff9397239e]
    + !                                                                         2456 HIMenuBarView::DrawSelf(short, __HIShape const*, CGContext*)  (in HIToolbox) + 559  [0x7fff9398365b]
    + !                                                                           2456 HIMenuBarView::DrawOnce(CGRect, CGRect, bool, bool, CGContext*)  (in HIToolbox) + 1216  [0x7fff93983c18]
    + !                                                                             2456 _HIThemeDrawAppleMenuTitle  (in HIToolbox) + 531  [0x7fff93987282]
    + !                                                                               2456 _HIThemeCUIDrawWithRenderer  (in HIToolbox) + 174  [0x7fff93987ee6]
    + !                                                                                 2456 _HIThemeCUIDrawWithOptions  (in HIToolbox) + 81  [0x7fff93987f52]
    + !                                                                                   2456 CUIDraw  (in CoreUI) + 175  [0x7fff8fdee992]
    + !                                                                                     2456 CUIRenderer::Draw(CGRect, CGContext*, __CFDictionary const*, __CFDictionary const**)  (in CoreUI) + 2341  [0x7fff8fdec065]
    + !                                                                                       2456 CUICoreThemeRenderer::DrawMenuTitle(CUIDescriptor const*, CUIReturnInfo&)  (in CoreUI) + 850  [0x7fff8fe61238]
    + !                                                                                         2456 CUIRenderer::DrawImage(CGRect, long, CUIDescriptor const*) const  (in CoreUI) + 6651  [0x7fff8fdfd6dd]
    + !                                                                                           2456 CUIRenderer::CreateImageByApplyingEffectsToImage(CUIDescriptor const*, long, __CFArray const*, CGImage*, double, unsigned char, CGBlendMode&) const  (in CoreUI) + 15627  [0x7fff8fdfa71b]
    + !                                                                                             2456 +[CUIShapeEffectStack sharedCIContext]  (in CoreUI) + 23  [0x7fff8fe0a7f5]
    + !                                                                                               2456 pthread_once  (in libsystem_pthread.dylib) + 57  [0x7fff90920b95]
    + !                                                                                                 2456 _os_once  (in libsystem_platform.dylib) + 41  [0x7fff9387dfc4]
    + !                                                                                                   2456 __pthread_once_handler  (in libsystem_pthread.dylib) + 65  [0x7fff90920bf6]
    + !                                                                                                     2456 CreateSharedCIContext  (in CoreUI) + 80  [0x7fff8fe0a84e]
    + !                                                                                                       2456 -[CIContext initWithOptions:]  (in CoreImage) + 357  [0x7fff88c0a446]
    + !                                                                                                         2456 +[CIContext(Internal) internalCLContextWithOptions:glContext:]  (in CoreImage) + 495  [0x7fff88c0f8a0]
    + !                                                                                                           2456 CI::CLModernContext::init()  (in CoreImage) + 185  [0x7fff88d1e30f]
    + !                                                                                                             2456 CI::CLModernContext::determineDeviceLimits()  (in CoreImage) + 142  [0x7fff88d1e8a0]
    + !                                                                                                               2454 void CI::minValueForDevices(_cl_device_id**, unsigned int, unsigned int, unsigned long&, unsigned long)  (in CoreImage) + 53  [0x7fff88d27f29]
    + !                                                                                                               : 1362 bool CI::valueForDevice(_cl_device_id*, unsigned int, unsigned long&)  (in CoreImage) + 26  [0x7fff88d27f78]
    + !                                                                                                               : | 1353 clGetDeviceInfo  (in OpenCL) + 98  [0x7fff97872413]

The bug still manifests with --no-sandbox.
 
Reproduced with 54.0.2820.0 (current canary) and 53.0.2785.46 (current beta).
From Console:

8/5/16 1:23:44.560 PM Google Chrome Helper[59435]: [CL_INVALID_DEVICE] : OpenCL Error : clGetDeviceInfo failed: Failed to retrieve device information! Invalid device
8/5/16 1:23:44.560 PM Google Chrome Helper[59437]: [CL_INVALID_DEVICE] : OpenCL Error : clGetDeviceInfo failed: Failed to retrieve device information! Invalid device
8/5/16 1:23:44.560 PM Google Chrome Helper[59436]: [CL_INVALID_DEVICE] : OpenCL Error : clGetDeviceInfo failed: Failed to retrieve device information! Invalid device
8/5/16 1:23:44.560 PM Google Chrome Helper[59435]: Unable to determine parameter value. This will likely cause issues with CL context rendering.
8/5/16 1:23:44.560 PM Google Chrome Helper[59437]: Unable to determine parameter value. This will likely cause issues with CL context rendering.
8/5/16 1:23:44.560 PM Google Chrome Helper[59436]: Unable to determine parameter value. This will likely cause issues with CL context rendering.

Elly noted that with --no-sandbox, though, we don't call SandboxWarmup, so #3 is a red herring. Not sure why disabling the sandbox doesn't fix this.
Labels: -Pri-1 Pri-2
Labels: Stability-Hang

Comment 7 by rsesek@chromium.org, Dec 13 2016

 Issue 673880  has been merged into this issue.

Comment 8 by meh...@chromium.org, Dec 13 2016

ellyjones@ I can reproduce it in Chrome Stable Version 55.0.2883.95 (64-bit), but not in latest Canary Version 57.0.2950.0 (64-bit). Can you still reproduce it in latest Canary?

Comment 9 by shrike@chromium.org, Dec 14 2016

mehmet@ - just FYI it may be awhile before ellyjones@ gets a chance to respond to your question.
Cc: pinkerton@chromium.org
Status: Untriaged (was: Available)
Marking for re-triage so it gets attention. 
I'll try to find the CL range, that might have fixed this issue.
Okay, I found the range with the CL's that fixed that issue:

https://chromium.googlesource.com/chromium/src/+log/7eae0b7129b93217f4f024dfb4087de989e3029e..7d47e639710463d061bb385e60d8395161958284

rsesek: Could it be that your change has fixed that issue in M57? https://codereview.chromium.org/2509273005


Yes I suppose it's possible, but that's a bit odd given what we found in #4.
Maybe #4 is a retina / non-retina thing? I am using a non-retina Macbook Air 11.
Status: WontFix (was: Untriaged)
[mac triage] Since we can no longer repro, and this is a bit niche, I don't think there's anything left to do here.
Cc: jamaladass@google.com jhunch@google.com
Issue 711427 has been merged into this issue.

Sign in to add a comment