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

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

Issue 1877073003: Respect the order of input messages from browser to renderer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add DCHECKs for thread Created 4 years, 6 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 | « content/renderer/render_frame_impl.cc ('k') | content/renderer/render_view_impl.cc » ('j') | 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 #ifndef CONTENT_RENDERER_RENDER_VIEW_IMPL_H_ 5 #ifndef CONTENT_RENDERER_RENDER_VIEW_IMPL_H_
6 #define CONTENT_RENDERER_RENDER_VIEW_IMPL_H_ 6 #define CONTENT_RENDERER_RENDER_VIEW_IMPL_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 415 matching lines...) Expand 10 before | Expand all | Expand 10 after
426 SSLStatus GetSSLStatusOfFrame(blink::WebFrame* frame) const override; 426 SSLStatus GetSSLStatusOfFrame(blink::WebFrame* frame) const override;
427 const std::string& GetAcceptLanguages() const override; 427 const std::string& GetAcceptLanguages() const override;
428 #if defined(OS_ANDROID) 428 #if defined(OS_ANDROID)
429 void UpdateTopControlsState(TopControlsState constraints, 429 void UpdateTopControlsState(TopControlsState constraints,
430 TopControlsState current, 430 TopControlsState current,
431 bool animate) override; 431 bool animate) override;
432 #endif 432 #endif
433 void ConvertViewportToWindowViaWidget(blink::WebRect* rect) override; 433 void ConvertViewportToWindowViaWidget(blink::WebRect* rect) override;
434 gfx::RectF ElementBoundsInWindow(const blink::WebElement& element) override; 434 gfx::RectF ElementBoundsInWindow(const blink::WebElement& element) override;
435 float GetDeviceScaleFactorForTest() const override; 435 float GetDeviceScaleFactorForTest() const override;
436 bool HasAddedInputHandler() const override;
436 437
437 gfx::Point ConvertWindowPointToViewport(const gfx::Point& point); 438 gfx::Point ConvertWindowPointToViewport(const gfx::Point& point);
438 439
439 bool uses_temporary_zoom_level() const { return uses_temporary_zoom_level_; } 440 bool uses_temporary_zoom_level() const { return uses_temporary_zoom_level_; }
440 441
441 // Please do not add your stuff randomly to the end here. If there is an 442 // Please do not add your stuff randomly to the end here. If there is an
442 // appropriate section, add it there. If not, there are some random functions 443 // appropriate section, add it there. If not, there are some random functions
443 // nearer to the top you can add it to. 444 // nearer to the top you can add it to.
444 445
445 protected: 446 protected:
(...skipping 481 matching lines...) Expand 10 before | Expand all | Expand 10 after
927 // session, this info is sent to the browser along with other drag/drop info. 928 // session, this info is sent to the browser along with other drag/drop info.
928 DragEventSourceInfo possible_drag_event_info_; 929 DragEventSourceInfo possible_drag_event_info_;
929 930
930 // NOTE: stats_collection_observer_ should be the last members because their 931 // NOTE: stats_collection_observer_ should be the last members because their
931 // constructors call the AddObservers method of RenderViewImpl. 932 // constructors call the AddObservers method of RenderViewImpl.
932 std::unique_ptr<StatsCollectionObserver> stats_collection_observer_; 933 std::unique_ptr<StatsCollectionObserver> stats_collection_observer_;
933 934
934 typedef std::map<cc::SharedBitmapId, cc::SharedBitmap*> BitmapMap; 935 typedef std::map<cc::SharedBitmapId, cc::SharedBitmap*> BitmapMap;
935 BitmapMap disambiguation_bitmaps_; 936 BitmapMap disambiguation_bitmaps_;
936 937
938 bool has_added_input_handler_;
939
937 // --------------------------------------------------------------------------- 940 // ---------------------------------------------------------------------------
938 // ADDING NEW DATA? Please see if it fits appropriately in one of the above 941 // ADDING NEW DATA? Please see if it fits appropriately in one of the above
939 // sections rather than throwing it randomly at the end. If you're adding a 942 // sections rather than throwing it randomly at the end. If you're adding a
940 // bunch of stuff, you should probably create a helper class and put your 943 // bunch of stuff, you should probably create a helper class and put your
941 // data and methods on that to avoid bloating RenderView more. You can 944 // data and methods on that to avoid bloating RenderView more. You can
942 // use the Observer interface to filter IPC messages and receive frame change 945 // use the Observer interface to filter IPC messages and receive frame change
943 // notifications. 946 // notifications.
944 // --------------------------------------------------------------------------- 947 // ---------------------------------------------------------------------------
945 948
946 DISALLOW_COPY_AND_ASSIGN(RenderViewImpl); 949 DISALLOW_COPY_AND_ASSIGN(RenderViewImpl);
947 }; 950 };
948 951
949 } // namespace content 952 } // namespace content
950 953
951 #endif // CONTENT_RENDERER_RENDER_VIEW_IMPL_H_ 954 #endif // CONTENT_RENDERER_RENDER_VIEW_IMPL_H_
OLDNEW
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | content/renderer/render_view_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698