Headless shell crashes loading youtube.com |
|||
Issue descriptionTo reproduce: chrome --headless --remote-debugging-port=9222 https://www.youtube.com #0 0x7f85d289c4e7 base::debug::StackTrace::StackTrace() #1 0x7f85d289c05f base::debug::(anonymous namespace)::StackDumpSignalHandler() #2 0x7f85d25c5330 <unknown> #3 0x7f85d03f2827 content::PushMessagingManager::Core::RegisterOnUI() #4 0x7f85d289cfb6 base::debug::TaskAnnotator::RunTask() #5 0x7f85d28c083d base::MessageLoop::RunTask() #6 0x7f85d28c10b8 base::MessageLoop::DoWork() #7 0x7f85d28c261a base::(anonymous namespace)::WorkSourceDispatch() #8 0x7f85cc444e04 g_main_context_dispatch #9 0x7f85cc445048 <unknown> #10 0x7f85cc4450ec g_main_context_iteration #11 0x7f85d28c2486 base::MessagePumpGlib::Run() #12 0x7f85d28c0557 base::MessageLoop::RunHandler() #13 0x7f85d28ea15e base::RunLoop::Run() #14 0x7f85d02126cc content::BrowserMainLoop::MainMessageLoopRun() #15 0x7f85d0212515 content::BrowserMainLoop::RunMainMessageLoopParts() #16 0x7f85d0215ecd content::BrowserMainRunnerImpl::Run() #17 0x7f85d44560ff headless::HeadlessContentMainDelegate::RunProcess() #18 0x7f85d080d16c content::RunNamedProcessTypeMain() #19 0x7f85d080d93c content::ContentMainRunnerImpl::Run() #20 0x7f85d080c4f0 content::ContentMain() #21 0x7f85d4451253 headless::(anonymous namespace)::RunContentMain() #22 0x7f85d44512ca headless::HeadlessBrowserMain() #23 0x7f85d32a9a45 headless::HeadlessShellMain() #24 0x7f85d2f65665 ChromeMain #25 0x7f85c9a3df45 __libc_start_main
,
Feb 23 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ea60d1f2365d4715d25c4bdfb13c6749a2302a8f commit ea60d1f2365d4715d25c4bdfb13c6749a2302a8f Author: peter <peter@chromium.org> Date: Thu Feb 23 15:14:21 2017 Don't assume that the permission manager exists. It's valid for a browser context's permission manager to return NULL. Handle this case in Push Messaging by immediately rejecting the request instead. BUG= 695418 Review-Url: https://codereview.chromium.org/2706243007 Cr-Commit-Position: refs/heads/master@{#452489} [modify] https://crrev.com/ea60d1f2365d4715d25c4bdfb13c6749a2302a8f/content/browser/push_messaging/push_messaging_manager.cc
,
Feb 23 2017
,
Feb 23 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by skyos...@chromium.org
, Feb 23 2017