Follow-ups/Tweaks for Embedded Flash Fullscreen.

chrome/browser/ui/views/frame/browser_view.cc

Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index 75e64af9578e207b40ef91e6dc6e06dabb6949a6..698a89c974c9f4a8109e1528d47a0a0ce89f5caa 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -2318,12 +2318,8 @@ void BrowserView::ProcessFullscreen(bool fullscreen,
   browser_->WindowFullscreenStateChanged();
 
   if (fullscreen) {
-    if (!chrome::IsRunningInAppMode() &&
-        type != FOR_METRO &&
-        !ShouldUseImmersiveFullscreenForUrl(url)) {

[sky, 2013/10/11 01:56:28]

Why are you removing the !ShouldUseImm...?

[miu, 2013/10/11 03:30:30]

It's checked in the call to UpdateFullscreenExitBubbleContent() and so it's now
redundant to have here.

-      fullscreen_bubble_.reset(new FullscreenExitBubbleViews(
-          this, url, bubble_type));
-    }
+    if (!chrome::IsRunningInAppMode() && type != FOR_METRO)
+      UpdateFullscreenExitBubbleContent(url, bubble_type);
   } else {
 #if defined(OS_WIN) && !defined(USE_AURA)
     if (omnibox_win) {