Avi (use Gerrit)
2014/10/29 17:40:57
Wait, what?
This is clearly ownership semantics,
Wait, what?
This is clearly ownership semantics, so why would you use a scoped_ptr in the
interface (correct!) and then rip out its guts and manage the object directly?
Can't you do a LazyInstance instead, at least?
oshima
2014/10/31 15:54:45
This needs to be set by embedder so we can't use S
On 2014/10/29 17:40:57, Avi wrote:
> Wait, what?
>
> This is clearly ownership semantics, so why would you use a scoped_ptr in the
> interface (correct!) and then rip out its guts and manage the object directly?
>
> Can't you do a LazyInstance instead, at least?
This needs to be set by embedder so we can't use Snigleton nor LazyInstance, but
I changed to set the instance to JavaScriptDialogManager impl. I hope you like
it.
Issue 666533007: Move JavaScriptDialogManager, JavascriptAppModalDialogViews to components/app_modal_dialogs
(Closed)
Created 6 years, 2 months ago by oshima
Modified 6 years, 1 month ago
Reviewers: msw, Avi (use Gerrit), pkotwicz, Yoyo Zhou, Lei Zhang, blundell, Jiang Jiang
Base URL: https://chromium.googlesource.com/chromium/src@master
Comments: 154