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

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: associated with IPC channel Created 3 years, 8 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 445 matching lines...) Expand 10 before | Expand all | Expand 10 after
456 const GURL& opener_top_level_frame_url, 456 const GURL& opener_top_level_frame_url,
457 const GURL& source_origin, 457 const GURL& source_origin,
458 content::mojom::WindowContainerType container_type, 458 content::mojom::WindowContainerType container_type,
459 const GURL& target_url, 459 const GURL& target_url,
460 const content::Referrer& referrer, 460 const content::Referrer& referrer,
461 const std::string& frame_name, 461 const std::string& frame_name,
462 WindowOpenDisposition disposition, 462 WindowOpenDisposition disposition,
463 const blink::mojom::WindowFeatures& features, 463 const blink::mojom::WindowFeatures& features,
464 bool user_gesture, 464 bool user_gesture,
465 bool opener_suppressed, 465 bool opener_suppressed,
466 content::ResourceContext* context,
467 bool* no_javascript_access) { 466 bool* no_javascript_access) {
468 *no_javascript_access = true; 467 *no_javascript_access = true;
469 return false; 468 return false;
470 } 469 }
471 470
472 void CastContentBrowserClient::ExposeInterfacesToRenderer( 471 void CastContentBrowserClient::ExposeInterfacesToRenderer(
473 service_manager::BinderRegistry* registry, 472 service_manager::BinderRegistry* registry,
474 content::RenderProcessHost* render_process_host) { 473 content::RenderProcessHost* render_process_host) {
475 registry->AddInterface( 474 registry->AddInterface(
476 base::Bind(&media::MediaCapsImpl::AddBinding, 475 base::Bind(&media::MediaCapsImpl::AddBinding,
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
568 process_type, dumps_path, false /* upload */); 567 process_type, dumps_path, false /* upload */);
569 // StartUploaderThread() even though upload is diferred. 568 // StartUploaderThread() even though upload is diferred.
570 // Breakpad-related memory is freed in the uploader thread. 569 // Breakpad-related memory is freed in the uploader thread.
571 crash_handler->StartUploaderThread(); 570 crash_handler->StartUploaderThread();
572 return crash_handler; 571 return crash_handler;
573 } 572 }
574 #endif // !defined(OS_ANDROID) 573 #endif // !defined(OS_ANDROID)
575 574
576 } // namespace shell 575 } // namespace shell
577 } // namespace chromecast 576 } // namespace chromecast
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698