Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2011 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_H_ | 5 #ifndef CONTENT_RENDERER_RENDER_VIEW_H_ |
| 6 #define CONTENT_RENDERER_RENDER_VIEW_H_ | 6 #define CONTENT_RENDERER_RENDER_VIEW_H_ |
| 7 #pragma once | 7 #pragma once |
| 8 | 8 |
| 9 #include <deque> | 9 #include <deque> |
| 10 #include <map> | 10 #include <map> |
| (...skipping 640 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 651 virtual void OnSetFocus(bool enable); | 651 virtual void OnSetFocus(bool enable); |
| 652 virtual void OnWasHidden(); | 652 virtual void OnWasHidden(); |
| 653 virtual void OnWasRestored(bool needs_repainting); | 653 virtual void OnWasRestored(bool needs_repainting); |
| 654 virtual bool SupportsAsynchronousSwapBuffers() OVERRIDE; | 654 virtual bool SupportsAsynchronousSwapBuffers() OVERRIDE; |
| 655 virtual void OnImeSetComposition( | 655 virtual void OnImeSetComposition( |
| 656 const string16& text, | 656 const string16& text, |
| 657 const std::vector<WebKit::WebCompositionUnderline>& underlines, | 657 const std::vector<WebKit::WebCompositionUnderline>& underlines, |
| 658 int selection_start, | 658 int selection_start, |
| 659 int selection_end) OVERRIDE; | 659 int selection_end) OVERRIDE; |
| 660 virtual void OnImeConfirmComposition(const string16& text) OVERRIDE; | 660 virtual void OnImeConfirmComposition(const string16& text) OVERRIDE; |
| 661 virtual ui::TextInputType GetTextInputType() OVERRIDE; | 661 virtual WebKit::WebTextInputType GetTextInputType() OVERRIDE; |
|
James Su
2011/09/23 01:46:48
I think we should use ui::TextInputType here.
| |
| 662 virtual WebKit::WebRect GetCaretBounds() OVERRIDE; | |
|
James Su
2011/09/23 01:46:48
We should use ui::Rect.
| |
| 662 virtual bool CanComposeInline() OVERRIDE; | 663 virtual bool CanComposeInline() OVERRIDE; |
| 663 | 664 |
| 664 private: | 665 private: |
| 665 // For unit tests. | 666 // For unit tests. |
| 666 friend class ExternalPopupMenuTest; | 667 friend class ExternalPopupMenuTest; |
| 667 friend class PepperDeviceTest; | 668 friend class PepperDeviceTest; |
| 668 friend class RenderViewTest; | 669 friend class RenderViewTest; |
| 669 | 670 |
| 670 FRIEND_TEST_ALL_PREFIXES(ExternalPopupMenuRemoveTest, RemoveOnChange); | 671 FRIEND_TEST_ALL_PREFIXES(ExternalPopupMenuRemoveTest, RemoveOnChange); |
| 671 FRIEND_TEST_ALL_PREFIXES(ExternalPopupMenuTest, NormalCase); | 672 FRIEND_TEST_ALL_PREFIXES(ExternalPopupMenuTest, NormalCase); |
| (...skipping 583 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1255 // bunch of stuff, you should probably create a helper class and put your | 1256 // bunch of stuff, you should probably create a helper class and put your |
| 1256 // data and methods on that to avoid bloating RenderView more. You can use | 1257 // data and methods on that to avoid bloating RenderView more. You can use |
| 1257 // the Observer interface to filter IPC messages and receive frame change | 1258 // the Observer interface to filter IPC messages and receive frame change |
| 1258 // notifications. | 1259 // notifications. |
| 1259 // --------------------------------------------------------------------------- | 1260 // --------------------------------------------------------------------------- |
| 1260 | 1261 |
| 1261 DISALLOW_COPY_AND_ASSIGN(RenderView); | 1262 DISALLOW_COPY_AND_ASSIGN(RenderView); |
| 1262 }; | 1263 }; |
| 1263 | 1264 |
| 1264 #endif // CONTENT_RENDERER_RENDER_VIEW_H_ | 1265 #endif // CONTENT_RENDERER_RENDER_VIEW_H_ |
| OLD | NEW |