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

Side by Side Diff: extensions/browser/app_window/app_window.cc

Issue 789533002: Fullscreen: make fullscreen requests come from RenderFrame (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "extensions/browser/app_window/app_window.h" 5 #include "extensions/browser/app_window/app_window.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 912 matching lines...) Expand 10 before | Expand all | Expand 10 after
923 923
924 void AppWindow::NavigationStateChanged(content::WebContents* source, 924 void AppWindow::NavigationStateChanged(content::WebContents* source,
925 content::InvalidateTypes changed_flags) { 925 content::InvalidateTypes changed_flags) {
926 if (changed_flags & content::INVALIDATE_TYPE_TITLE) 926 if (changed_flags & content::INVALIDATE_TYPE_TITLE)
927 native_app_window_->UpdateWindowTitle(); 927 native_app_window_->UpdateWindowTitle();
928 else if (changed_flags & content::INVALIDATE_TYPE_TAB) 928 else if (changed_flags & content::INVALIDATE_TYPE_TAB)
929 native_app_window_->UpdateWindowIcon(); 929 native_app_window_->UpdateWindowIcon();
930 } 930 }
931 931
932 void AppWindow::ToggleFullscreenModeForTab(content::WebContents* source, 932 void AppWindow::ToggleFullscreenModeForTab(content::WebContents* source,
933 const GURL& origin,
933 bool enter_fullscreen) { 934 bool enter_fullscreen) {
934 const Extension* extension = GetExtension(); 935 const Extension* extension = GetExtension();
935 if (!extension) 936 if (!extension)
936 return; 937 return;
937 938
938 if (!IsExtensionWithPermissionOrSuggestInConsole( 939 if (!IsExtensionWithPermissionOrSuggestInConsole(
939 APIPermission::kFullscreen, extension, source->GetRenderViewHost())) { 940 APIPermission::kFullscreen, extension, source->GetRenderViewHost())) {
940 return; 941 return;
941 } 942 }
942 943
(...skipping 152 matching lines...) Expand 10 before | Expand all | Expand 10 after
1095 region.bounds.x(), 1096 region.bounds.x(),
1096 region.bounds.y(), 1097 region.bounds.y(),
1097 region.bounds.right(), 1098 region.bounds.right(),
1098 region.bounds.bottom(), 1099 region.bounds.bottom(),
1099 region.draggable ? SkRegion::kUnion_Op : SkRegion::kDifference_Op); 1100 region.draggable ? SkRegion::kUnion_Op : SkRegion::kDifference_Op);
1100 } 1101 }
1101 return sk_region; 1102 return sk_region;
1102 } 1103 }
1103 1104
1104 } // namespace extensions 1105 } // namespace extensions
OLDNEW
« content/renderer/render_frame_impl.cc ('K') | « extensions/browser/app_window/app_window.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698