New issue
Advanced search Search tips

Issue 117068 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2012
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug-Regression

Restricted
  • Only users with Commit permission may comment.



Sign in to add a comment

Crash generated when uploading image from picker

Project Member Reported by mbollu@chromium.org, Mar 7 2012

Issue description

Chrome Version: <Copy from: 'about:version'> 19.0.1061.0 dev
OS: Win7
Crash ID: f618bccab37827a

URL (if applicable) where crash occurred:

Can you reproduce this crash?

What steps will reproduce this crash (or if it's not reproducible,
what were you doing just before the crash)?
1. Install chrome.
2. Install Mememator app.
3. Launch Mememator app.
4. Try to upload the image.

Result:
Chrome crashes with crash id: f618bccab37827a

Stack Trace from crash/:
Thread 0 *CRASHED* ( EXCEPTION_ACCESS_VIOLATION_READ @ 0x00000000 )

0x5ff59c34	 [chrome.dll	 - web_intent_picker_views.cc:258]	`anonymous namespace'::ServiceButtonsView::GetPreferredSize()
0x60544b43	 [chrome.dll	 - grid_layout.cc:609]	views::ColumnSet::CalculateSize()
0x6054522d	 [chrome.dll	 - grid_layout.cc:860]	views::GridLayout::SizeRowsAndColumns(bool,int,int,gfx::Size *)
0x6054555c	 [chrome.dll	 - grid_layout.cc:805]	views::GridLayout::Layout(views::View *)
0x5f3fb1f6	 [chrome.dll	 - view.cc:498]	views::View::Layout()
0x5ff5a8fb	 [chrome.dll	 - web_intent_picker_views.cc:606]	WebIntentPickerViews::OnModelChanged(WebIntentPickerModel *)
0x5ff57fab	 [chrome.dll	 - web_intent_picker_model.cc:28]	WebIntentPickerModel::AddInstalledService(std::basic_string<wchar_t,std::char_traits<wchar_t>,std::allocator<wchar_t> > const &,GURL const &,WebIntentPickerModel::Disposition)
0x5fe1a625	 [chrome.dll	 - web_intent_picker_controller.cc:277]	WebIntentPickerController::OnIntentsQueryDone(int,std::vector<webkit_glue::WebIntentServiceData,std::allocator<webkit_glue::WebIntentServiceData> > const &)
0x5ff59b1c	 [chrome.dll	 - web_intents_registry.cc:133]	WebIntentsRegistry::OnWebDataServiceRequestDone(int,WDTypedResult const *)
0x5f526041	 [chrome.dll	 - web_data_service.cc:674]	WebDataService::RequestCompleted(int)
0x5f15716c	 [chrome.dll	 - bind_internal.h:1254]	base::internal::Invoker<2,base::internal::BindState<base::internal::RunnableAdapter<void ( CaptureVideoDecoder::*)(media::VideoCapture *)>,void (CaptureVideoDecoder *,media::VideoCapture *),void (CaptureVideoDecoder *,media::VideoCapture *)>,void (CaptureVideoDecoder *,media::VideoCapture *)>::Run(base::internal::BindStateBase *)
0x5f1464b9	 [chrome.dll	 - message_loop.cc:458]	MessageLoop::RunTask(base::PendingTask const &)
0x5f1452c7	 [chrome.dll	 - message_loop.cc:660]	MessageLoop::DoWork()
0x5f2b6a13	 [chrome.dll	 - message_pump_win.cc:203]	base::MessagePumpForUI::DoRunLoop()
0x5f508ea2	 [chrome.dll	 - message_pump_win.cc:51]	base::MessagePumpWin::RunWithDispatcher(base::MessagePump::Delegate *,base::MessagePumpWin::Dispatcher *)
0x5f144e27	 [chrome.dll	 - message_loop.cc:390]	MessageLoop::RunHandler()
0x5f508e23	 [chrome.dll	 - message_loop.cc:776]	MessageLoopForUI::RunWithDispatcher(base::MessagePumpWin::Dispatcher *)
0x5f508b57	 [chrome.dll	 - chrome_browser_main.cc:1901]	ChromeBrowserMainParts::MainMessageLoopRun(int *)
0x5f508660	 [chrome.dll	 - browser_main_loop.cc:449]	content::BrowserMainLoop::RunMainMessageLoopParts()
0x5f5085ba	 [chrome.dll	 - browser_main_runner.cc:94]	`anonymous namespace'::BrowserMainRunnerImpl::Run()
0x5f1b66cb	 [chrome.dll	 - browser_main.cc:21]	BrowserMain(content::MainFunctionParams const &)
0x5f14186d	 [chrome.dll	 - content_main_runner.cc:271]	`anonymous namespace'::RunNamedProcessTypeMain(std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &,content::MainFunctionParams const &,content::ContentMainDelegate *)
0x5f1417f2	 [chrome.dll	 - content_main_runner.cc:500]	`anonymous namespace'::ContentMainRunnerImpl::Run()
0x5f134d61	 [chrome.dll	 - content_main.cc:35]	content::ContentMain(HINSTANCE__ *,sandbox::SandboxInterfaceInfo *,content::ContentMainDelegate *)
0x5f134cec	 [chrome.dll	 - chrome_main.cc:28]	ChromeMain
0x010e54a6	 [chrome.exe	 - client_util.cc:424]	MainDllLoader::Launch(HINSTANCE__ *,sandbox::SandboxInterfaceInfo *)
0x010e48ec	 [chrome.exe	 - chrome_exe_main_win.cc:36]	wWinMain
0x01104e2e	 [chrome.exe	 - crt0.c:263]	__tmainCRTStartup
0x75d43399	 [kernel32.dll	 + 0x00013399]	BaseThreadInitThunk
0x77d29ef1	 [ntdll.dll	 + 0x00039ef1]	__RtlUserThreadStart
0x77d29ec4	 [ntdll.dll	 + 0x00039ec4]	_RtlUserThreadStart

*Please note that issues filed with no information filled in above
will be marked as WontFix*

 
Labels: -Type-Bug Type-Regression
Labels: OS-Windows
Owner: binji@chromium.org
Status: Assigned

Comment 4 by mbollu@chromium.org, Mar 12 2012

Labels: ReleaseBlock-Dev
This issue is reproducible on 19.0.1067.0 (Official Build 126101). 

Comment 5 by mbollu@chromium.org, Mar 12 2012

Intent 'Pick' is failing for apps Mememator, Imagemator, Inspirationmator, Cloud File Picker.
mbollu: Can you describe the failure?  Also this is likely a separate issue so can you file a new bug?

Comment 7 by mbollu@chromium.org, Mar 12 2012

Labels: -ReleaseBlock-Dev ReleaseBlock-Beta
Apologizes and clarifying on Comment 5.

Intent 'Pick' is crashing but not failing for apps Mememator, Imagemator, Inspirationmator, Cloud File Picker. We can reproduce the crash by following steps in - Steps To Reproduce. It's not a different issue and it's happening on Windows but not on Linux, Mac.

Intent 'Share' at http://examples.webintents.org/intents/share/share.html is also crashing.

Since it's a Demo feature I am making it ReleaseBlock-Beta.
Ben, what's the status on this one?

Comment 9 by binji@google.com, Mar 12 2012

In the CQ, should be in today.
Project Member

Comment 10 by bugdroid1@chromium.org, Mar 14 2012

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=126723

------------------------------------------------------------------------
r126723 | binji@chromium.org | Wed Mar 14 13:31:53 PDT 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/views/web_intent_picker_views.cc?r1=126723&r2=126722&pathrev=126723

[Web Intents] Fix crash in views picker.
Crash occurs because the previous code would early-out when there were no services installed, which would prevent creating the LayoutManager. As a result, the layout manager was NULL.

BUG= 117068 
TEST=none
TBR=sky@chromium.org

Review URL: https://chromiumcodereview.appspot.com/9651024
------------------------------------------------------------------------

Comment 11 by binji@chromium.org, Mar 14 2012

Status: Fixed
Project Member

Comment 12 by bugdroid1@chromium.org, Oct 13 2012

Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Project Member

Comment 13 by bugdroid1@chromium.org, Mar 9 2013

Labels: -Type-Regression -Feature-WebIntents -Area-UI Cr-UI-Browser-WebIntents Type-Bug-Regression Cr-UI

Sign in to add a comment