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

Issue 609798 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Not on Chrome
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Crash is seen on clicking camera icon while creating supervised user

Project Member Reported by sc00335...@techmahindra.com, May 6 2016

Issue description

Version: 52.0.2724.0/8286.0.0 (Official Build) dev-channel quawks,daisy,spring,pit
OS: Chrome os

What steps will reproduce the problem?
(1) Sign in to user and sign out >> From 3 dot menu in uber tray click "add supervised user" >> Give correct password and sign in
(2) Now in Create supervised user overlay without giving name click on camera icon to change profile icon

Expected: No crash should be seen while clicking camera icon in supervised overlay.
Actual: Instead crash is seen on clicking camera icon.

NOTE: Crash is also seen while clicking camera icon after signing in to valid account for first time.

Crash ids: Crash ID 5735104a00000000 (Chrome), Crash ID be17104a00000000 (Chrome)

This is a regression issue as it is working fine in 50.0.2661.91/7978.66.0 stable channel daisy.

 
Cc: songsuk@chromium.org pucchakayala@chromium.org
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on Spring using chrome version 52.0.2724.0/8286.0.0
Stack trace
==================
Thread 0 CRASHED [SIGABRT @ 0x000058ec ] MAGIC SIGNATURE THREAD
0xb0f37636	(libc-2.19.so + 0x00016636 )	__libc_do_syscall
0xb0f47313	(libc-2.19.so -raise.c:56 )	raise
0xb0f482ff	(libc-2.19.so -abort.c:89 )	abort
0xb4e975d1	(libbase.so -debugger_posix.cc:249 )	base::debug::BreakDebugger
0xb5540f6f	(chrome -media_stream_devices_controller.cc:427 )	MediaStreamDevicesController::RunCallback
0xb5541039	(chrome -media_stream_devices_controller.cc:295 )	MediaStreamDevicesController::PermissionDenied
0xb5c3e2a3	(chrome -webui_login_view.cc:449 )	chromeos::WebUILoginView::RequestMediaAccessPermission
0xb3c25e37	(libcontent.so -web_contents_impl.cc:2254 )	content::WebContentsImpl::RequestMediaAccessPermission
0xb3b835bf	(libcontent.so -media_stream_ui_proxy.cc:114 )	content::MediaStreamUIProxy::Core::RequestAccess
0xb3b825ff	(libcontent.so -bind_internal.h:181 )	base::internal::Invoker<base::IndexSequence<0u, 1u>, base::internal::BindState<base::internal::RunnableAdapter<void (content::MediaStreamUIProxy::Core::*)(std::unique_ptr<content::MediaStreamRequest, std::default_delete<content::MediaStreamRequest> >)>, void(content::MediaStreamUIProxy::Core*, std::unique_ptr<content::MediaStreamRequest, std::default_delete<content::MediaStreamRequest> >), base::internal::UnretainedWrapper<content::MediaStreamUIProxy::Core>, base::internal::PassedWrapper<std::unique_ptr<content::MediaStreamRequest, std::default_delete<content::MediaStreamRequest> > > >, base::internal::InvokeHelper<false, void, base::internal::RunnableAdapter<void (content::MediaStreamUIProxy::Core::*)(std::unique_ptr<content::MediaStreamRequest, std::default_delete<content::MediaStreamRequest> >)> >, void()>::Run
0xb4e7ebc3	(libbase.so -callback.h:397 )	base::debug::TaskAnnotator::RunTask
0xb4e81f43	(libbase.so -message_loop.cc:484 )	base::MessageLoop::RunTask
0xb4e82479	(libbase.so -message_loop.cc:493 )	base::MessageLoop::DeferOrRunPendingTask
0xb4e82527	(libbase.so -message_loop.cc:610 )	base::MessageLoop::DoWork
0xb4e7d3cf	(libbase.so -message_pump_libevent.cc:230 )	base::MessagePumpLibevent::Run
0xb4ec2329	(libbase.so -run_loop.cc:35 )	base::RunLoop::Run
0xb62f3c4d	(chrome -chrome_browser_main.cc:1866 )	ChromeBrowserMainParts::MainMessageLoopRun
0xb3a2b58f	(libcontent.so -browser_main_loop.cc:972 )	content::BrowserMainLoop::RunMainMessageLoopParts
0xb3a2c8ed	(libcontent.so -browser_main_runner.cc:154 )	content::BrowserMainRunnerImpl::Run
0xb3a28605	(libcontent.so -browser_main.cc:46 )	content::BrowserMain
0xb39cf631	(libcontent.so -content_main_runner.cc:742 )	content::ContentMainRunnerImpl::Run
0xb39ce7f1	(libcontent.so -content_main.cc:20 )	content::ContentMain
0xb547b92b	(chrome -chrome_main.cc:84 )	ChromeMain
0xb0f37307	(libc-2.19.so -libc-start.c:285 )	__libc_start_main
0xb547b803	(chrome + 0x002c1803 )	_start
0xb68e3b2f	(chrome -elf-init.c:87 )	__libc_csu_init
0xb51a39bf	(ld-2.19.so + 0x0000b9bf )	_dl_sort_fini
Cc: joh...@chromium.org mlamouri@chromium.org tedc...@chromium.org lshang@chromium.org brettw@chromium.org
Owner: benwells@chromium.org
Status: Assigned (was: Untriaged)
Added some people who modified media_stream_devices_controller.cc in the past.

Ben. Please help assign to the right person.

Cc: wuchengli@chromium.org posciak@chromium.org
Components: -OS>Kernel>Camera
Cc: x...@chromium.org abodenha@chromium.org
Reproduced on samus with 52.0.2724.0/8283.0.0
Cc: benwells@chromium.org raymes@chromium.org
Owner: tsergeant@chromium.org
punting to Tim :) (and cc'ing Raymes)
Status: Started (was: Assigned)
I've looked into this, there was some recently (R391390) some logic added to the login view to allow SAML login pages to use the webcam. This logic causes the MediaStreamDevicesController to try to run its callback twice when processing a request from a WebUI page.

The fix is pretty simple, I'll have a CL up shortly.
Status: Fixed (was: Started)
The bug was fixed independently by R394273.
Status: Verified (was: Fixed)
Verified on 52.0.2743.0/8350.3.0 daisy
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-52; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-52 label, otherwise remove Merge-TBD label. Thanks.
Labels: -Merge-TBD
[Bulk edit]

Our blockerbot script was offline; it was recently brought back online, and thus labeled many old issues (including this one) erroneously.  Removing Merge-TBD label since all milestones for this issue are already completed; no further work should be done.

Sign in to add a comment