Separate projection mode from rest of touch HUD

ash/shell.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ASH_SHELL_H_
 #define ASH_SHELL_H_
 
 #include <utility>
 #include <vector>
 
@@ skipping 444 matching lines @@
     return root_window_host_factory_.get();
   }
 
   LauncherModel* launcher_model() {
     return launcher_model_.get();
   }
 
   // Returns the launcher delegate, creating if necesary.
   LauncherDelegate* GetLauncherDelegate();
 
+  void SetTouchHudProjectionEnabled(bool enabled);
+
+  void ToggleTouchHudProjection();

[sky, 2013/06/26 21:07:44]

Can't the caller trivially implement this rather than another method?

[mohsen, 2013/06/27 00:44:31]

Yes, it can. Removed this function.

+
+  bool is_touch_hud_projection_enabled() const {

[sky, 2013/06/26 21:07:44]

Style guide says function should match member. So either rename member to
'is_touch_...' or nuke the is_ from this. I tend to think is_tou... is cleaner.

[mohsen, 2013/06/27 00:44:31]

Done.

+    return touch_hud_projection_enabled_;
+  }
+
  private:
   FRIEND_TEST_ALL_PREFIXES(ExtendedDesktopTest, TestCursor);
   FRIEND_TEST_ALL_PREFIXES(WindowManagerTest, MouseEventCursors);
   FRIEND_TEST_ALL_PREFIXES(WindowManagerTest, TransformActivate);
   friend class internal::RootWindowController;
   friend class test::ShellTestApi;
   friend class shell::WindowWatcher;
 
   typedef std::pair<aura::Window*, gfx::Rect> WindowAndBoundsPair;
 
@@ skipping 122 matching lines @@
   views::corewm::CursorManager cursor_manager_;
 
   ObserverList<ShellObserver> observers_;
 
   // Used by ash/shell.
   content::BrowserContext* browser_context_;
 
   // For testing only: simulate that a modal window is open
   bool simulate_modal_window_open_for_testing_;
 
+  bool touch_hud_projection_enabled_;
+
   DISALLOW_COPY_AND_ASSIGN(Shell);
 };
 
 }  // namespace ash
 
 #endif  // ASH_SHELL_H_