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

Unified Diff: chrome/browser/ui/browser.cc

Issue 158253002: Tabs being screen-captured will fullscreen within their tab contents area. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed scheib's review comments. Rebased. Created 6 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/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 87acf52c3f42ef8b404ee151571318bd5cbfd81b..38858828e5aafd426d309236d30dd0eb946d22eb 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -981,6 +981,8 @@ void Browser::ActiveTabChanged(WebContents* old_contents,
// is updated.
window_->OnActiveTabChanged(old_contents, new_contents, index, reason);
+ fullscreen_controller_->OnTabDetachedFromView(old_contents);
sky 2014/02/12 21:30:13 Is it possible to make fullscreencontroller observ
miu 2014/02/12 21:49:14 Yes, there are definitely ordering dependencies, b
+
// Discarded tabs always get reloaded.
if (tab_strip_model_->IsTabDiscarded(index)) {
LOG(WARNING) << "Reloading discarded tab at " << index;

Powered by Google App Engine
This is Rietveld 408576698