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

Side by Side Diff: chrome/browser/chromeos/frame/browser_frame_chromeos.cc

Issue 5747001: Add the virtual keyboard to BrowserFrameChromeos. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 10 years 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/frame/browser_frame_chromeos.h" 5 #include "chrome/browser/chromeos/frame/browser_frame_chromeos.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "chrome/browser/views/frame/browser_view.h" 8 #include "chrome/browser/ui/views/frame/browser_view.h"
9 #include "chrome/browser/views/frame/opaque_browser_frame_view.h" 9 #include "chrome/browser/ui/views/frame/opaque_browser_frame_view.h"
10 #include "chrome/browser/views/frame/popup_non_client_frame_view.h" 10 #include "chrome/browser/ui/views/frame/popup_non_client_frame_view.h"
11 #if defined(TOUCH_UI)
sky 2010/12/09 22:19:03 conditional includes should be grouped after non-c
bryeung 2010/12/10 17:23:19 Done.
12 #include "chrome/browser/ui/views/frame/touch_browser_frame_view.h"
13 #endif
11 #include "chrome/common/chrome_switches.h" 14 #include "chrome/common/chrome_switches.h"
12 15
13 // static (Factory method.) 16 // static (Factory method.)
14 BrowserFrame* BrowserFrame::Create(BrowserView* browser_view, 17 BrowserFrame* BrowserFrame::Create(BrowserView* browser_view,
15 Profile* profile) { 18 Profile* profile) {
16 chromeos::BrowserFrameChromeos* frame = 19 chromeos::BrowserFrameChromeos* frame =
17 new chromeos::BrowserFrameChromeos(browser_view, profile); 20 new chromeos::BrowserFrameChromeos(browser_view, profile);
18 frame->Init(); 21 frame->Init();
19 return frame; 22 return frame;
20 } 23 }
(...skipping 10 matching lines...) Expand all
31 34
32 void BrowserFrameChromeos::Init() { 35 void BrowserFrameChromeos::Init() {
33 // NOTE: This logic supersedes the logic in BrowserFrameGtk::Init() 36 // NOTE: This logic supersedes the logic in BrowserFrameGtk::Init()
34 // by always setting browser_frame_view_. 37 // by always setting browser_frame_view_.
35 bool is_popup = browser_view()->IsBrowserTypePopup(); 38 bool is_popup = browser_view()->IsBrowserTypePopup();
36 if (is_popup) { 39 if (is_popup) {
37 // ChromeOS Panels should always use PopupNonClientFrameView. 40 // ChromeOS Panels should always use PopupNonClientFrameView.
38 set_browser_frame_view(new PopupNonClientFrameView()); 41 set_browser_frame_view(new PopupNonClientFrameView());
39 } else { 42 } else {
40 // Default FrameView. 43 // Default FrameView.
44 #if defined(TOUCH_UI)
sky 2010/12/09 22:19:03 Add a factory method/function some where so that y
bryeung 2010/12/10 17:23:19 Done.
45 set_browser_frame_view(new TouchBrowserFrameView(this, browser_view()));
46 #else
41 set_browser_frame_view(new OpaqueBrowserFrameView(this, browser_view())); 47 set_browser_frame_view(new OpaqueBrowserFrameView(this, browser_view()));
48 #endif
42 } 49 }
43 50
44 BrowserFrameGtk::Init(); 51 BrowserFrameGtk::Init();
45 52
46 if (!is_popup) { 53 if (!is_popup) {
47 // On chromeos we want windows to always render as active. 54 // On chromeos we want windows to always render as active.
48 GetNonClientView()->DisableInactiveRendering(true); 55 GetNonClientView()->DisableInactiveRendering(true);
49 } 56 }
50 } 57 }
51 58
52 bool BrowserFrameChromeos::IsMaximized() const { 59 bool BrowserFrameChromeos::IsMaximized() const {
53 if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kChromeosFrame)) 60 if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kChromeosFrame))
54 return WindowGtk::IsMaximized(); 61 return WindowGtk::IsMaximized();
55 bool is_popup = browser_view()->IsBrowserTypePopup(); 62 bool is_popup = browser_view()->IsBrowserTypePopup();
56 return !IsFullscreen() && (!is_popup || WindowGtk::IsMaximized()); 63 return !IsFullscreen() && (!is_popup || WindowGtk::IsMaximized());
57 } 64 }
58 65
59 } // namespace chromeos 66 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698