diff options
author | Nico Weber <thakis@chromium.org> | 2014-09-08 14:50:37 -0700 |
---|---|---|
committer | Nico Weber <thakis@chromium.org> | 2014-09-08 21:53:46 +0000 |
commit | 7543ca10a7ad464cfb8a7ee927f089298ff0eb78 (patch) | |
tree | 07383ac3fdf43afee191b66600e5113956dd45a6 /apps | |
parent | a0d051ed00e459f970a784e1419ba26414824fc6 (diff) | |
download | chromium_src-7543ca10a7ad464cfb8a7ee927f089298ff0eb78.zip chromium_src-7543ca10a7ad464cfb8a7ee927f089298ff0eb78.tar.gz chromium_src-7543ca10a7ad464cfb8a7ee927f089298ff0eb78.tar.bz2 |
Revert "Add AppWindow.setVisibleOnAllWorkspaces." (https://codereview.chromium.org/469993003)
Speculative, to see if it helps with
==12563== WARNING: MemorySanitizer: use-of-uninitialized-value
#0 0x7f834414fd2c in views::DesktopWindowTreeHostX11::InitX11Window(views::Widget::InitParams const&) ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc:1164
#1 0x7f834414dc6e in views::DesktopWindowTreeHostX11::Init(aura::Window*, views::Widget::InitParams const&) ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc:259
#2 0x7f83441408e9 in views::DesktopNativeWidgetAura::InitNativeWidget(views::Widget::InitParams const&) ui/views/widget/desktop_aura/desktop_native_widget_aura.cc:420
#3 0x7f834411722f in views::Widget::Init(views::Widget::InitParams const&) ui/views/widget/widget.cc:358
#4 0x7f834473bc8b in ChromeNativeAppWindowViews::InitializeDefaultWindow(extensions::AppWindow::CreateParams const&) chrome/browser/ui/views/apps/chrome_native_app_window_views.cc:231
#5 0x7f834473f52d in ChromeNativeAppWindowViews::InitializeWindow(extensions::AppWindow*, extensions::AppWindow::CreateParams const&) chrome/browser/ui/views/apps/chrome_native_app_window_views.cc:672
#6 0x7f834c3d8bbe in apps::NativeAppWindowViews::Init(extensions::AppWindow*, extensions::AppWindow::CreateParams const&) apps/ui/views/native_app_window_views.cc:46
#7 0x7f834460443c in ChromeAppsClient::CreateNativeAppWindowImpl(extensions::AppWindow*, extensions::AppWindow::CreateParams const&) chrome/browser/ui/views/apps/chrome_apps_client_views.cc:14
#8 0x7f834be98433 in extensions::AppWindow::Init(GURL const&, extensions::AppWindowContents*, extensions::AppWindow::CreateParams const&) extensions/browser/app_window/app_window.cc:281
#9 0x7f834bd6bbfb in extensions::AppWindowCreateFunction::RunAsync() extensions/browser/api/app_window/app_window_api.cc:296 #10 0x7f834bee4510 in AsyncExtensionFunction::Run() extensions/browser/extension_function.cc:452
#11 0x7f834bee9771 in extensions::ExtensionFunctionDispatcher::DispatchWithCallbackInternal(ExtensionHostMsg_Request_Params const&, content::RenderViewHost*, content::RenderFrameHost*, base::Callback<void (ExtensionFunction::ResponseType, base::ListValue const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&)> const&) extensions/browser/extension_function_dispatcher.cc:392
#12 0x7f834bee86b6 in extensions::ExtensionFunctionDispatcher::Dispatch(ExtensionHostMsg_Request_Params const&, content::RenderViewHost*) extensions/browser/extension_function_dispatcher.cc:309
#13 0x7f834bedac91 in OnRequest extensions/browser/extension_host.cc:342
#14 0x7f8345add9d2 in content::WebContentsImpl::OnMessageReceived(content::RenderViewHost*, content::RenderFrameHost*, IPC::Message const&) content/browser/web_contents/web_contents_impl.cc:526
#15 0x7f83459847fe in content::RenderViewHostImpl::OnMessageReceived(IPC::Message const&) content/browser/renderer_host/render_view_host_impl.cc:894
Started happening here
http://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20MSan%20Tests/builds/98
(see stdio for symbols), and this looks like the only change in that
build.
This reverts commit c76ef7326840ea3eeb3fa96fb88f5269b5ec55a7.
BUG=384644
TBR=jackhou@chromium.org
Review URL: https://codereview.chromium.org/550413002
Cr-Commit-Position: refs/heads/master@{#293792}
Diffstat (limited to 'apps')
-rw-r--r-- | apps/ui/views/native_app_window_views.cc | 4 | ||||
-rw-r--r-- | apps/ui/views/native_app_window_views.h | 1 |
2 files changed, 0 insertions, 5 deletions
diff --git a/apps/ui/views/native_app_window_views.cc b/apps/ui/views/native_app_window_views.cc index e185b8f..4579c73 100644 --- a/apps/ui/views/native_app_window_views.cc +++ b/apps/ui/views/native_app_window_views.cc @@ -401,8 +401,4 @@ bool NativeAppWindowViews::CanHaveAlphaEnabled() const { return widget_->IsTranslucentWindowOpacitySupported(); } -void NativeAppWindowViews::SetVisibleOnAllWorkspaces(bool always_visible) { - widget_->SetVisibleOnAllWorkspaces(always_visible); -} - } // namespace apps diff --git a/apps/ui/views/native_app_window_views.h b/apps/ui/views/native_app_window_views.h index 931faba..9988b17 100644 --- a/apps/ui/views/native_app_window_views.h +++ b/apps/ui/views/native_app_window_views.h @@ -161,7 +161,6 @@ class NativeAppWindowViews : public extensions::NativeAppWindow, virtual void SetContentSizeConstraints(const gfx::Size& min_size, const gfx::Size& max_size) OVERRIDE; virtual bool CanHaveAlphaEnabled() const OVERRIDE; - virtual void SetVisibleOnAllWorkspaces(bool always_visible) OVERRIDE; // web_modal::WebContentsModalDialogHost implementation. virtual gfx::NativeView GetHostView() const OVERRIDE; |