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

Side by Side Diff: content/browser/renderer_host/render_widget_host_impl.h

Issue 598483002: Fix WeakPtrFactory member ordering in content (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changed order of friend class Created 6 years, 2 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_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_ 5 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_ 6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
7 7
8 #include <deque> 8 #include <deque>
9 #include <list> 9 #include <list>
10 #include <map> 10 #include <map>
(...skipping 809 matching lines...) Expand 10 before | Expand all | Expand 10 after
820 bool suppress_next_char_events_; 820 bool suppress_next_char_events_;
821 821
822 bool pending_mouse_lock_request_; 822 bool pending_mouse_lock_request_;
823 bool allow_privileged_mouse_lock_; 823 bool allow_privileged_mouse_lock_;
824 824
825 // Keeps track of whether the webpage has any touch event handler. If it does, 825 // Keeps track of whether the webpage has any touch event handler. If it does,
826 // then touch events are sent to the renderer. Otherwise, the touch events are 826 // then touch events are sent to the renderer. Otherwise, the touch events are
827 // not sent to the renderer. 827 // not sent to the renderer.
828 bool has_touch_handler_; 828 bool has_touch_handler_;
829 829
830 base::WeakPtrFactory<RenderWidgetHostImpl> weak_factory_;
831
832 scoped_ptr<SyntheticGestureController> synthetic_gesture_controller_; 830 scoped_ptr<SyntheticGestureController> synthetic_gesture_controller_;
833 831
834 scoped_ptr<TouchEmulator> touch_emulator_; 832 scoped_ptr<TouchEmulator> touch_emulator_;
835 833
836 // Receives and handles all input events. 834 // Receives and handles all input events.
837 scoped_ptr<InputRouter> input_router_; 835 scoped_ptr<InputRouter> input_router_;
838 836
839 scoped_ptr<TimeoutMonitor> hang_monitor_timeout_; 837 scoped_ptr<TimeoutMonitor> hang_monitor_timeout_;
840 838
841 #if defined(OS_WIN) 839 #if defined(OS_WIN)
842 std::list<HWND> dummy_windows_for_activation_; 840 std::list<HWND> dummy_windows_for_activation_;
843 #endif 841 #endif
844 842
845 int64 last_input_number_; 843 int64 last_input_number_;
846 844
847 int next_browser_snapshot_id_; 845 int next_browser_snapshot_id_;
848 typedef std::map<int, 846 typedef std::map<int,
849 base::Callback<void(const unsigned char*, size_t)> > PendingSnapshotMap; 847 base::Callback<void(const unsigned char*, size_t)> > PendingSnapshotMap;
850 PendingSnapshotMap pending_browser_snapshots_; 848 PendingSnapshotMap pending_browser_snapshots_;
851 849
850 base::WeakPtrFactory<RenderWidgetHostImpl> weak_factory_;
851
852 DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostImpl); 852 DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostImpl);
853 }; 853 };
854 854
855 } // namespace content 855 } // namespace content
856 856
857 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_ 857 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
OLDNEW
« no previous file with comments | « content/browser/net/view_http_cache_job_factory.cc ('k') | content/browser/renderer_host/render_widget_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698