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

Side by Side Diff: content/renderer/render_view_impl.cc

Issue 2177203002: Exit flash fullscreen before opening new windows/popups Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "content/renderer/render_view_impl.h" 5 #include "content/renderer/render_view_impl.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <cmath> 8 #include <cmath>
9 #include <memory> 9 #include <memory>
10 10
(...skipping 1472 matching lines...) Expand 10 before | Expand all | Expand 10 after
1483 } 1483 }
1484 1484
1485 // blink::WebViewClient ------------------------------------------------------ 1485 // blink::WebViewClient ------------------------------------------------------
1486 1486
1487 WebView* RenderViewImpl::createView(WebLocalFrame* creator, 1487 WebView* RenderViewImpl::createView(WebLocalFrame* creator,
1488 const WebURLRequest& request, 1488 const WebURLRequest& request,
1489 const WebWindowFeatures& features, 1489 const WebWindowFeatures& features,
1490 const WebString& frame_name, 1490 const WebString& frame_name,
1491 WebNavigationPolicy policy, 1491 WebNavigationPolicy policy,
1492 bool suppress_opener) { 1492 bool suppress_opener) {
1493 #if defined(ENABLE_PLUGINS)
1494 if (PepperPluginInstanceImpl* plugin = GetFocusedPepperPlugin()) {
1495 // Opening a window should exit fullscreen, as it does for HTML5 fullscreen.
1496 plugin->FlashSetFullscreen(false, false);
1497 }
1498 #endif
1493 ViewHostMsg_CreateWindow_Params params; 1499 ViewHostMsg_CreateWindow_Params params;
1494 params.opener_id = GetRoutingID(); 1500 params.opener_id = GetRoutingID();
1495 params.user_gesture = WebUserGestureIndicator::isProcessingUserGesture(); 1501 params.user_gesture = WebUserGestureIndicator::isProcessingUserGesture();
1496 if (GetContentClient()->renderer()->AllowPopup()) 1502 if (GetContentClient()->renderer()->AllowPopup())
1497 params.user_gesture = true; 1503 params.user_gesture = true;
1498 params.window_container_type = WindowFeaturesToContainerType(features); 1504 params.window_container_type = WindowFeaturesToContainerType(features);
1499 params.session_storage_namespace_id = session_storage_namespace_id_; 1505 params.session_storage_namespace_id = session_storage_namespace_id_;
1500 if (frame_name != "_blank") 1506 if (frame_name != "_blank")
1501 params.frame_name = base::UTF16ToUTF8(base::StringPiece16(frame_name)); 1507 params.frame_name = base::UTF16ToUTF8(base::StringPiece16(frame_name));
1502 params.opener_render_frame_id = 1508 params.opener_render_frame_id =
(...skipping 1599 matching lines...) Expand 10 before | Expand all | Expand 10 after
3102 return render_frame->focused_pepper_plugin(); 3108 return render_frame->focused_pepper_plugin();
3103 } 3109 }
3104 frame = frame->traverseNext(false); 3110 frame = frame->traverseNext(false);
3105 } 3111 }
3106 3112
3107 return nullptr; 3113 return nullptr;
3108 } 3114 }
3109 #endif 3115 #endif
3110 3116
3111 } // namespace content 3117 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698