Rename fullscreen_exit_bubble_* to exclusive_access_bubble_*

chrome/browser/ui/fullscreen/exclusive_access_bubble.h

Index: chrome/browser/ui/fullscreen/exclusive_access_bubble.h
diff --git a/chrome/browser/ui/fullscreen/fullscreen_exit_bubble.h b/chrome/browser/ui/fullscreen/exclusive_access_bubble.h
similarity index 82%
rename from chrome/browser/ui/fullscreen/fullscreen_exit_bubble.h
rename to chrome/browser/ui/fullscreen/exclusive_access_bubble.h
index 65af1d1bfcc051bf158584f22a04fb041a06663a..e5da267bd797c716b69448c9ac6003dca77343a4 100644
--- a/chrome/browser/ui/fullscreen/fullscreen_exit_bubble.h
+++ b/chrome/browser/ui/fullscreen/exclusive_access_bubble.h
@@ -2,11 +2,11 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_UI_FULLSCREEN_FULLSCREEN_EXIT_BUBBLE_H_
-#define CHROME_BROWSER_UI_FULLSCREEN_FULLSCREEN_EXIT_BUBBLE_H_
+#ifndef CHROME_BROWSER_UI_FULLSCREEN_EXCLUSIVE_ACCESS_BUBBLE_H_
+#define CHROME_BROWSER_UI_FULLSCREEN_EXCLUSIVE_ACCESS_BUBBLE_H_
 
 #include "base/timer/timer.h"
-#include "chrome/browser/ui/fullscreen/fullscreen_exit_bubble_type.h"
+#include "chrome/browser/ui/fullscreen/exclusive_access_bubble_type.h"
 #include "ui/gfx/animation/animation_delegate.h"
 #include "ui/gfx/point.h"
 #include "url/gurl.h"
@@ -17,12 +17,12 @@ namespace gfx {
 class Rect;
 }
 
-class FullscreenExitBubble : public gfx::AnimationDelegate {
+class ExclusiveAccessBubble : public gfx::AnimationDelegate {

[sky, 2014/12/12 00:00:37]

FullscreenXXX is rather self descriptive. ExclusiveAccess isn't. All this code
needs comments to clarify what exclusive access means.

[Sriram, 2014/12/12 21:56:39]

Added comment to explain what and when this is used. The part of the reason for
the rename is that the class is used in multiple cases - i.e. fullscreen as well
as mouse lock (and keyboard lock in near future).

On 2014/12/12 00:00:37, sky wrote:

  public:
-  explicit FullscreenExitBubble(Browser* browser,
+  explicit ExclusiveAccessBubble(Browser* browser,
                                 const GURL& url,
-                                FullscreenExitBubbleType bubble_type);
-  ~FullscreenExitBubble() override;
+                                ExclusiveAccessBubbleType bubble_type);
+  ~ExclusiveAccessBubble() override;
 
  protected:
   static const int kPaddingPx;           // Amount of padding around the link
@@ -89,27 +89,27 @@ class FullscreenExitBubble : public gfx::AnimationDelegate {
   GURL url_;
 
   // The type of the bubble; controls e.g. which buttons to show.
-  FullscreenExitBubbleType bubble_type_;
+  ExclusiveAccessBubbleType bubble_type_;
 
  private:
   // Timer to delay before allowing the bubble to hide after it's initially
   // shown.
-  base::OneShotTimer<FullscreenExitBubble> initial_delay_;
+  base::OneShotTimer<ExclusiveAccessBubble> initial_delay_;
 
   // Timer to see how long the mouse has been idle.
-  base::OneShotTimer<FullscreenExitBubble> idle_timeout_;
+  base::OneShotTimer<ExclusiveAccessBubble> idle_timeout_;
 
   // Timer to poll the current mouse position.  We can't just listen for mouse
   // events without putting a non-empty HWND onscreen (or hooking Windows, which
   // has other problems), so instead we run a low-frequency poller to see if the
   // user has moved in or out of our show/hide regions.
-  base::RepeatingTimer<FullscreenExitBubble> mouse_position_checker_;
+  base::RepeatingTimer<ExclusiveAccessBubble> mouse_position_checker_;
 
   // The most recently seen mouse position, in screen coordinates.  Used to see
   // if the mouse has moved since our last check.
   gfx::Point last_mouse_pos_;
 
-  DISALLOW_COPY_AND_ASSIGN(FullscreenExitBubble);
+  DISALLOW_COPY_AND_ASSIGN(ExclusiveAccessBubble);
 };
 
-#endif  // CHROME_BROWSER_UI_FULLSCREEN_FULLSCREEN_EXIT_BUBBLE_H_
+#endif  // CHROME_BROWSER_UI_FULLSCREEN_EXCLUSIVE_ACCESS_BUBBLE_H_