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

Side by Side Diff: chromecast/browser/cast_content_browser_client.cc

Issue 2821473002: Service CreateNewWindow on the UI thread with a new mojo interface (Closed)
Patch Set: MakeShared goodness Created 3 years, 7 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
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 "chromecast/browser/cast_content_browser_client.h" 5 #include "chromecast/browser/cast_content_browser_client.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
(...skipping 432 matching lines...) Expand 10 before | Expand all | Expand 10 after
443 } else { 443 } else {
444 LOG(ERROR) << "Invalid host for client certificate request: " 444 LOG(ERROR) << "Invalid host for client certificate request: "
445 << requesting_url.host() 445 << requesting_url.host()
446 << " with render_process_id: " 446 << " with render_process_id: "
447 << render_process_id; 447 << render_process_id;
448 return NULL; 448 return NULL;
449 } 449 }
450 } 450 }
451 451
452 bool CastContentBrowserClient::CanCreateWindow( 452 bool CastContentBrowserClient::CanCreateWindow(
453 int opener_render_process_id, 453 content::RenderFrameHost* opener,
454 int opener_render_frame_id,
455 const GURL& opener_url, 454 const GURL& opener_url,
456 const GURL& opener_top_level_frame_url, 455 const GURL& opener_top_level_frame_url,
457 const GURL& source_origin, 456 const GURL& source_origin,
458 content::mojom::WindowContainerType container_type, 457 content::mojom::WindowContainerType container_type,
459 const GURL& target_url, 458 const GURL& target_url,
460 const content::Referrer& referrer, 459 const content::Referrer& referrer,
461 const std::string& frame_name, 460 const std::string& frame_name,
462 WindowOpenDisposition disposition, 461 WindowOpenDisposition disposition,
463 const blink::mojom::WindowFeatures& features, 462 const blink::mojom::WindowFeatures& features,
464 bool user_gesture, 463 bool user_gesture,
465 bool opener_suppressed, 464 bool opener_suppressed,
466 content::ResourceContext* context,
467 bool* no_javascript_access) { 465 bool* no_javascript_access) {
468 *no_javascript_access = true; 466 *no_javascript_access = true;
469 return false; 467 return false;
470 } 468 }
471 469
472 void CastContentBrowserClient::ExposeInterfacesToRenderer( 470 void CastContentBrowserClient::ExposeInterfacesToRenderer(
473 service_manager::BinderRegistry* registry, 471 service_manager::BinderRegistry* registry,
474 content::RenderProcessHost* render_process_host) { 472 content::RenderProcessHost* render_process_host) {
475 registry->AddInterface( 473 registry->AddInterface(
476 base::Bind(&media::MediaCapsImpl::AddBinding, 474 base::Bind(&media::MediaCapsImpl::AddBinding,
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
568 process_type, dumps_path, false /* upload */); 566 process_type, dumps_path, false /* upload */);
569 // StartUploaderThread() even though upload is diferred. 567 // StartUploaderThread() even though upload is diferred.
570 // Breakpad-related memory is freed in the uploader thread. 568 // Breakpad-related memory is freed in the uploader thread.
571 crash_handler->StartUploaderThread(); 569 crash_handler->StartUploaderThread();
572 return crash_handler; 570 return crash_handler;
573 } 571 }
574 #endif // !defined(OS_ANDROID) 572 #endif // !defined(OS_ANDROID)
575 573
576 } // namespace shell 574 } // namespace shell
577 } // namespace chromecast 575 } // namespace chromecast
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698