Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(833)

Unified Diff: chrome/browser/ui/views/apps/chrome_native_app_window_views.h

Issue 947343003: Split out Ash/Aura parts of ChromeNativeAppWindowViews. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ready for review. Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/apps/chrome_native_app_window_views.h
diff --git a/chrome/browser/ui/views/apps/chrome_native_app_window_views.h b/chrome/browser/ui/views/apps/chrome_native_app_window_views.h
index 06dd8286551aa8da737147eac9e4206c0fb193cd..b4a657c61277b8eddea01b91a46465f0403f9887 100644
--- a/chrome/browser/ui/views/apps/chrome_native_app_window_views.h
+++ b/chrome/browser/ui/views/apps/chrome_native_app_window_views.h
@@ -7,28 +7,16 @@
#include "base/memory/scoped_ptr.h"
#include "extensions/components/native_app_window/native_app_window_views.h"
-#include "ui/views/context_menu_controller.h"
namespace apps {
class AppWindowFrameView;
}
-#if defined(USE_ASH)
-namespace ash {
-class ImmersiveFullscreenController;
-}
-#endif
-
class DesktopKeyboardCapture;
class ExtensionKeybindingRegistryViews;
-namespace views {
-class MenuRunner;
-}
-
class ChromeNativeAppWindowViews
- : public native_app_window::NativeAppWindowViews,
- public views::ContextMenuController {
+ : public native_app_window::NativeAppWindowViews {
public:
ChromeNativeAppWindowViews();
~ChromeNativeAppWindowViews() override;
@@ -43,20 +31,18 @@ class ChromeNativeAppWindowViews
virtual void InitializeDefaultWindow(
const extensions::AppWindow::CreateParams& create_params);
+ virtual gfx::NativeView GetImeWindowContainer();
virtual void InitializePanelWindow(
tapted 2015/02/25 05:54:35 nit: keep this with InitializeDefaultWindow
jackhou1 2015/02/25 22:57:51 Done.
const extensions::AppWindow::CreateParams& create_params);
+ virtual gfx::Rect GetPanelWindowInitialBounds(gfx::Size preferred_size_);
tapted 2015/02/25 05:54:35 const gfx::Size& preferred_size
jackhou1 2015/02/25 22:57:51 Done.
virtual views::NonClientFrameView* CreateStandardDesktopAppFrame();
+ virtual apps::AppWindowFrameView* CreateNonStandardAppFrame();
// ui::BaseWindow implementation.
gfx::Rect GetRestoredBounds() const override;
ui::WindowShowState GetRestoredState() const override;
bool IsAlwaysOnTop() const override;
- // Overridden from views::ContextMenuController:
- void ShowContextMenuForView(views::View* source,
- const gfx::Point& p,
- ui::MenuSourceType source_type) override;
-
// WidgetDelegate implementation.
gfx::ImageSkia GetWindowAppIcon() override;
gfx::ImageSkia GetWindowIcon() override;
@@ -85,11 +71,6 @@ class ChromeNativeAppWindowViews
const extensions::AppWindow::CreateParams& create_params) override;
private:
- FRIEND_TEST_ALL_PREFIXES(ShapedAppWindowTargeterTest,
- ResizeInsetsWithinBounds);
-
- apps::AppWindowFrameView* CreateNonStandardAppFrame();
-
// True if the window is fullscreen or fullscreen is pending.
bool is_fullscreen_;
@@ -105,18 +86,6 @@ class ChromeNativeAppWindowViews
// The class that registers for keyboard shortcuts for extension commands.
scoped_ptr<ExtensionKeybindingRegistryViews> extension_keybinding_registry_;
-#if defined(USE_ASH)
- // Used to put non-frameless windows into immersive fullscreen on ChromeOS. In
- // immersive fullscreen, the window header (title bar and window controls)
- // slides onscreen as an overlay when the mouse is hovered at the top of the
- // screen.
- scoped_ptr<ash::ImmersiveFullscreenController>
- immersive_fullscreen_controller_;
-#endif // defined(USE_ASH)
-
- // Used to show the system menu.
- scoped_ptr<views::MenuRunner> menu_runner_;
-
// Used to capture all keyboard events including task switching sequence.
scoped_ptr<DesktopKeyboardCapture> desktop_keyboard_capture_;

Powered by Google App Engine
This is Rietveld 408576698