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

Side by Side Diff: ui/views/cocoa/bridged_content_view.mm

Issue 2228833002: MacViews: Fix behavior of move and select commands when selection direction changes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@use_text_commands
Patch Set: Created 4 years, 4 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 #import "ui/views/cocoa/bridged_content_view.h" 5 #import "ui/views/cocoa/bridged_content_view.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #import "base/mac/mac_util.h" 8 #import "base/mac/mac_util.h"
9 #import "base/mac/scoped_nsobject.h" 9 #import "base/mac/scoped_nsobject.h"
10 #include "base/strings/sys_string_conversions.h" 10 #include "base/strings/sys_string_conversions.h"
(...skipping 765 matching lines...) Expand 10 before | Expand all | Expand 10 after
776 } 776 }
777 777
778 - (void)moveLeft:(id)sender { 778 - (void)moveLeft:(id)sender {
779 [self handleAction:ui::TextEditCommand::MOVE_LEFT 779 [self handleAction:ui::TextEditCommand::MOVE_LEFT
780 keyCode:ui::VKEY_LEFT 780 keyCode:ui::VKEY_LEFT
781 domCode:ui::DomCode::ARROW_LEFT 781 domCode:ui::DomCode::ARROW_LEFT
782 eventFlags:0]; 782 eventFlags:0];
783 } 783 }
784 784
785 - (void)moveUp:(id)sender { 785 - (void)moveUp:(id)sender {
786 [self handleAction:ui::TextEditCommand::MOVE_TO_BEGINNING_OF_LINE 786 [self handleAction:ui::TextEditCommand::MOVE_UP
787 keyCode:ui::VKEY_UP 787 keyCode:ui::VKEY_UP
788 domCode:ui::DomCode::ARROW_UP 788 domCode:ui::DomCode::ARROW_UP
789 eventFlags:0]; 789 eventFlags:0];
790 } 790 }
791 791
792 - (void)moveDown:(id)sender { 792 - (void)moveDown:(id)sender {
793 [self handleAction:ui::TextEditCommand::MOVE_TO_END_OF_LINE 793 [self handleAction:ui::TextEditCommand::MOVE_DOWN
794 keyCode:ui::VKEY_DOWN 794 keyCode:ui::VKEY_DOWN
795 domCode:ui::DomCode::ARROW_DOWN 795 domCode:ui::DomCode::ARROW_DOWN
796 eventFlags:0]; 796 eventFlags:0];
797 } 797 }
798 798
799 - (void)moveWordForward:(id)sender { 799 - (void)moveWordForward:(id)sender {
800 [self handleAction:ui::TextEditCommand::MOVE_WORD_FORWARD 800 [self handleAction:ui::TextEditCommand::MOVE_WORD_FORWARD
801 keyCode:ui::VKEY_UNKNOWN 801 keyCode:ui::VKEY_UNKNOWN
802 domCode:ui::DomCode::NONE 802 domCode:ui::DomCode::NONE
803 eventFlags:0]; 803 eventFlags:0];
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
846 } 846 }
847 847
848 - (void)moveToBeginningOfDocument:(id)sender { 848 - (void)moveToBeginningOfDocument:(id)sender {
849 [self handleAction:ui::TextEditCommand::MOVE_TO_BEGINNING_OF_DOCUMENT 849 [self handleAction:ui::TextEditCommand::MOVE_TO_BEGINNING_OF_DOCUMENT
850 keyCode:ui::VKEY_HOME 850 keyCode:ui::VKEY_HOME
851 domCode:ui::DomCode::HOME 851 domCode:ui::DomCode::HOME
852 eventFlags:ui::EF_CONTROL_DOWN]; 852 eventFlags:ui::EF_CONTROL_DOWN];
853 } 853 }
854 854
855 - (void)pageDown:(id)sender { 855 - (void)pageDown:(id)sender {
856 [self handleAction:ui::TextEditCommand::MOVE_TO_END_OF_LINE 856 [self handleAction:ui::TextEditCommand::MOVE_PAGE_DOWN
857 keyCode:ui::VKEY_NEXT 857 keyCode:ui::VKEY_NEXT
858 domCode:ui::DomCode::PAGE_DOWN 858 domCode:ui::DomCode::PAGE_DOWN
859 eventFlags:0]; 859 eventFlags:0];
860 } 860 }
861 861
862 - (void)pageUp:(id)sender { 862 - (void)pageUp:(id)sender {
863 [self handleAction:ui::TextEditCommand::MOVE_TO_BEGINNING_OF_LINE 863 [self handleAction:ui::TextEditCommand::MOVE_PAGE_UP
864 keyCode:ui::VKEY_PRIOR 864 keyCode:ui::VKEY_PRIOR
865 domCode:ui::DomCode::PAGE_UP 865 domCode:ui::DomCode::PAGE_UP
866 eventFlags:0]; 866 eventFlags:0];
867 } 867 }
868 868
869 - (void)moveBackwardAndModifySelection:(id)sender { 869 - (void)moveBackwardAndModifySelection:(id)sender {
870 [self handleAction:ui::TextEditCommand::MOVE_BACKWARD_AND_MODIFY_SELECTION 870 [self handleAction:ui::TextEditCommand::MOVE_BACKWARD_AND_MODIFY_SELECTION
871 keyCode:ui::VKEY_UNKNOWN 871 keyCode:ui::VKEY_UNKNOWN
872 domCode:ui::DomCode::NONE 872 domCode:ui::DomCode::NONE
873 eventFlags:0]; 873 eventFlags:0];
(...skipping 15 matching lines...) Expand all
889 889
890 - (void)moveWordBackwardAndModifySelection:(id)sender { 890 - (void)moveWordBackwardAndModifySelection:(id)sender {
891 [self 891 [self
892 handleAction:ui::TextEditCommand::MOVE_WORD_BACKWARD_AND_MODIFY_SELECTION 892 handleAction:ui::TextEditCommand::MOVE_WORD_BACKWARD_AND_MODIFY_SELECTION
893 keyCode:ui::VKEY_UNKNOWN 893 keyCode:ui::VKEY_UNKNOWN
894 domCode:ui::DomCode::NONE 894 domCode:ui::DomCode::NONE
895 eventFlags:0]; 895 eventFlags:0];
896 } 896 }
897 897
898 - (void)moveUpAndModifySelection:(id)sender { 898 - (void)moveUpAndModifySelection:(id)sender {
899 [self handleAction:ui::TextEditCommand:: 899 [self handleAction:ui::TextEditCommand::MOVE_UP_AND_MODIFY_SELECTION
900 MOVE_TO_BEGINNING_OF_LINE_AND_MODIFY_SELECTION
901 keyCode:ui::VKEY_UP 900 keyCode:ui::VKEY_UP
902 domCode:ui::DomCode::ARROW_UP 901 domCode:ui::DomCode::ARROW_UP
903 eventFlags:ui::EF_SHIFT_DOWN]; 902 eventFlags:ui::EF_SHIFT_DOWN];
904 } 903 }
905 904
906 - (void)moveDownAndModifySelection:(id)sender { 905 - (void)moveDownAndModifySelection:(id)sender {
907 [self 906 [self handleAction:ui::TextEditCommand::MOVE_DOWN_AND_MODIFY_SELECTION
908 handleAction:ui::TextEditCommand::MOVE_TO_END_OF_LINE_AND_MODIFY_SELECTION 907 keyCode:ui::VKEY_DOWN
909 keyCode:ui::VKEY_DOWN 908 domCode:ui::DomCode::ARROW_DOWN
910 domCode:ui::DomCode::ARROW_DOWN 909 eventFlags:ui::EF_SHIFT_DOWN];
911 eventFlags:ui::EF_SHIFT_DOWN];
912 } 910 }
913 911
914 - (void)moveToBeginningOfLineAndModifySelection:(id)sender { 912 - (void)moveToBeginningOfLineAndModifySelection:(id)sender {
915 [self handleAction:ui::TextEditCommand:: 913 [self handleAction:ui::TextEditCommand::
916 MOVE_TO_BEGINNING_OF_LINE_AND_MODIFY_SELECTION 914 MOVE_TO_BEGINNING_OF_LINE_AND_MODIFY_SELECTION
917 keyCode:ui::VKEY_HOME 915 keyCode:ui::VKEY_HOME
918 domCode:ui::DomCode::HOME 916 domCode:ui::DomCode::HOME
919 eventFlags:ui::EF_SHIFT_DOWN]; 917 eventFlags:ui::EF_SHIFT_DOWN];
920 } 918 }
921 919
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
953 951
954 - (void)moveToBeginningOfDocumentAndModifySelection:(id)sender { 952 - (void)moveToBeginningOfDocumentAndModifySelection:(id)sender {
955 [self handleAction:ui::TextEditCommand:: 953 [self handleAction:ui::TextEditCommand::
956 MOVE_TO_BEGINNING_OF_DOCUMENT_AND_MODIFY_SELECTION 954 MOVE_TO_BEGINNING_OF_DOCUMENT_AND_MODIFY_SELECTION
957 keyCode:ui::VKEY_HOME 955 keyCode:ui::VKEY_HOME
958 domCode:ui::DomCode::HOME 956 domCode:ui::DomCode::HOME
959 eventFlags:ui::EF_CONTROL_DOWN | ui::EF_SHIFT_DOWN]; 957 eventFlags:ui::EF_CONTROL_DOWN | ui::EF_SHIFT_DOWN];
960 } 958 }
961 959
962 - (void)pageDownAndModifySelection:(id)sender { 960 - (void)pageDownAndModifySelection:(id)sender {
963 [self 961 [self handleAction:ui::TextEditCommand::MOVE_PAGE_DOWN_AND_MODIFY_SELECTION
964 handleAction:ui::TextEditCommand::MOVE_TO_END_OF_LINE_AND_MODIFY_SELECTION 962 keyCode:ui::VKEY_NEXT
965 keyCode:ui::VKEY_NEXT 963 domCode:ui::DomCode::PAGE_DOWN
966 domCode:ui::DomCode::PAGE_DOWN 964 eventFlags:ui::EF_SHIFT_DOWN];
967 eventFlags:ui::EF_SHIFT_DOWN];
968 } 965 }
969 966
970 - (void)pageUpAndModifySelection:(id)sender { 967 - (void)pageUpAndModifySelection:(id)sender {
971 [self handleAction:ui::TextEditCommand:: 968 [self handleAction:ui::TextEditCommand::MOVE_PAGE_UP_AND_MODIFY_SELECTION
972 MOVE_TO_BEGINNING_OF_LINE_AND_MODIFY_SELECTION
973 keyCode:ui::VKEY_PRIOR 969 keyCode:ui::VKEY_PRIOR
974 domCode:ui::DomCode::PAGE_UP 970 domCode:ui::DomCode::PAGE_UP
975 eventFlags:ui::EF_SHIFT_DOWN]; 971 eventFlags:ui::EF_SHIFT_DOWN];
976 } 972 }
977 973
978 - (void)moveParagraphForwardAndModifySelection:(id)sender { 974 - (void)moveParagraphForwardAndModifySelection:(id)sender {
979 [self 975 [self handleAction:ui::TextEditCommand::
980 handleAction:ui::TextEditCommand::MOVE_TO_END_OF_LINE_AND_MODIFY_SELECTION 976 MOVE_PARAGRAPH_FORWARD_AND_MODIFY_SELECTION
981 keyCode:ui::VKEY_DOWN 977 keyCode:ui::VKEY_DOWN
982 domCode:ui::DomCode::ARROW_DOWN 978 domCode:ui::DomCode::ARROW_DOWN
983 eventFlags:ui::EF_CONTROL_DOWN | ui::EF_SHIFT_DOWN]; 979 eventFlags:ui::EF_CONTROL_DOWN | ui::EF_SHIFT_DOWN];
984 } 980 }
985 981
986 - (void)moveParagraphBackwardAndModifySelection:(id)sender { 982 - (void)moveParagraphBackwardAndModifySelection:(id)sender {
987 [self handleAction:ui::TextEditCommand:: 983 [self handleAction:ui::TextEditCommand::
988 MOVE_TO_BEGINNING_OF_LINE_AND_MODIFY_SELECTION 984 MOVE_PARAGRAPH_BACKWARD_AND_MODIFY_SELECTION
989 keyCode:ui::VKEY_UP 985 keyCode:ui::VKEY_UP
990 domCode:ui::DomCode::ARROW_UP 986 domCode:ui::DomCode::ARROW_UP
991 eventFlags:ui::EF_CONTROL_DOWN | ui::EF_SHIFT_DOWN]; 987 eventFlags:ui::EF_CONTROL_DOWN | ui::EF_SHIFT_DOWN];
992 } 988 }
993 989
994 - (void)moveWordRight:(id)sender { 990 - (void)moveWordRight:(id)sender {
995 [self handleAction:ui::TextEditCommand::MOVE_WORD_RIGHT 991 [self handleAction:ui::TextEditCommand::MOVE_WORD_RIGHT
996 keyCode:ui::VKEY_RIGHT 992 keyCode:ui::VKEY_RIGHT
997 domCode:ui::DomCode::ARROW_RIGHT 993 domCode:ui::DomCode::ARROW_RIGHT
998 eventFlags:ui::EF_CONTROL_DOWN]; 994 eventFlags:ui::EF_CONTROL_DOWN];
(...skipping 340 matching lines...) Expand 10 before | Expand all | Expand 10 after
1339 } 1335 }
1340 1336
1341 return [super accessibilityAttributeValue:attribute]; 1337 return [super accessibilityAttributeValue:attribute];
1342 } 1338 }
1343 1339
1344 - (id)accessibilityHitTest:(NSPoint)point { 1340 - (id)accessibilityHitTest:(NSPoint)point {
1345 return [hostedView_->GetNativeViewAccessible() accessibilityHitTest:point]; 1341 return [hostedView_->GetNativeViewAccessible() accessibilityHitTest:point];
1346 } 1342 }
1347 1343
1348 @end 1344 @end
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698