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

Side by Side Diff: content/renderer/render_thread_impl.h

Issue 1005893002: Detect and clean up discarded mojo connections to deleted RenderFrames. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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 (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 #ifndef CONTENT_RENDERER_RENDER_THREAD_IMPL_H_ 5 #ifndef CONTENT_RENDERER_RENDER_THREAD_IMPL_H_
6 #define CONTENT_RENDERER_RENDER_THREAD_IMPL_H_ 6 #define CONTENT_RENDERER_RENDER_THREAD_IMPL_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 588 matching lines...) Expand 10 before | Expand all | Expand 10 after
599 int gpu_rasterization_msaa_sample_count_; 599 int gpu_rasterization_msaa_sample_count_;
600 bool is_threaded_gpu_rasterization_enabled_; 600 bool is_threaded_gpu_rasterization_enabled_;
601 bool is_impl_side_painting_enabled_; 601 bool is_impl_side_painting_enabled_;
602 bool is_lcd_text_enabled_; 602 bool is_lcd_text_enabled_;
603 bool is_distance_field_text_enabled_; 603 bool is_distance_field_text_enabled_;
604 bool is_zero_copy_enabled_; 604 bool is_zero_copy_enabled_;
605 bool is_one_copy_enabled_; 605 bool is_one_copy_enabled_;
606 bool is_elastic_overscroll_enabled_; 606 bool is_elastic_overscroll_enabled_;
607 unsigned use_image_texture_target_; 607 unsigned use_image_texture_target_;
608 608
609 struct PendingRenderFrameConnect 609 class PendingRenderFrameConnect
610 : public base::RefCounted<PendingRenderFrameConnect> { 610 : public base::RefCounted<PendingRenderFrameConnect>,
611 public mojo::ErrorHandler {
612 public:
611 PendingRenderFrameConnect( 613 PendingRenderFrameConnect(
614 int routing_id,
612 mojo::InterfaceRequest<mojo::ServiceProvider> services, 615 mojo::InterfaceRequest<mojo::ServiceProvider> services,
613 mojo::ServiceProviderPtr exposed_services); 616 mojo::ServiceProviderPtr exposed_services);
614 617
615 mojo::InterfaceRequest<mojo::ServiceProvider> services; 618 mojo::InterfaceRequest<mojo::ServiceProvider>& services() {
616 mojo::ServiceProviderPtr exposed_services; 619 return services_;
620 }
621
622 mojo::ServiceProviderPtr& exposed_services() { return exposed_services_; }
617 623
618 private: 624 private:
619 friend class base::RefCounted<PendingRenderFrameConnect>; 625 friend class base::RefCounted<PendingRenderFrameConnect>;
620 626
621 ~PendingRenderFrameConnect(); 627 ~PendingRenderFrameConnect() override;
628
629 void OnConnectionError() override;
630
631 int routing_id_;
632 mojo::InterfaceRequest<mojo::ServiceProvider> services_;
633 mojo::ServiceProviderPtr exposed_services_;
622 }; 634 };
623 635
624 typedef std::map<int, scoped_refptr<PendingRenderFrameConnect>> 636 typedef std::map<int, scoped_refptr<PendingRenderFrameConnect>>
625 PendingRenderFrameConnectMap; 637 PendingRenderFrameConnectMap;
626 PendingRenderFrameConnectMap pending_render_frame_connects_; 638 PendingRenderFrameConnectMap pending_render_frame_connects_;
627 639
628 DISALLOW_COPY_AND_ASSIGN(RenderThreadImpl); 640 DISALLOW_COPY_AND_ASSIGN(RenderThreadImpl);
629 }; 641 };
630 642
631 #if defined(COMPILER_MSVC) 643 #if defined(COMPILER_MSVC)
632 #pragma warning(pop) 644 #pragma warning(pop)
633 #endif 645 #endif
634 646
635 } // namespace content 647 } // namespace content
636 648
637 #endif // CONTENT_RENDERER_RENDER_THREAD_IMPL_H_ 649 #endif // CONTENT_RENDERER_RENDER_THREAD_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | content/renderer/render_thread_impl.cc » ('j') | content/renderer/render_thread_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698