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

Side by Side Diff: third_party/WebKit/Source/core/editing/InputMethodControllerTest.cpp

Issue 2681023003: Remove replaceComposition() calls in finishComposingText. (Closed)
Patch Set: Created 3 years, 10 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 #include "core/editing/InputMethodController.h" 5 #include "core/editing/InputMethodController.h"
6 6
7 #include "core/dom/Document.h" 7 #include "core/dom/Document.h"
8 #include "core/dom/Element.h" 8 #include "core/dom/Element.h"
9 #include "core/dom/Range.h" 9 #include "core/dom/Range.h"
10 #include "core/editing/Editor.h" 10 #include "core/editing/Editor.h"
(...skipping 935 matching lines...) Expand 10 before | Expand all | Expand 10 after
946 Vector<CompositionUnderline> underlines; 946 Vector<CompositionUnderline> underlines;
947 underlines.push_back(CompositionUnderline(0, 5, Color(255, 0, 0), false, 0)); 947 underlines.push_back(CompositionUnderline(0, 5, Color(255, 0, 0), false, 0));
948 editable->focus(); 948 editable->focus();
949 949
950 document().setTitle(emptyString()); 950 document().setTitle(emptyString());
951 controller().setComposition("foo", underlines, 0, 3); 951 controller().setComposition("foo", underlines, 0, 3);
952 EXPECT_STREQ("beforeinput.isComposing:true;input.isComposing:true;", 952 EXPECT_STREQ("beforeinput.isComposing:true;input.isComposing:true;",
953 document().title().utf8().data()); 953 document().title().utf8().data());
954 954
955 document().setTitle(emptyString()); 955 document().setTitle(emptyString());
956 controller().finishComposingText(InputMethodController::KeepSelection); 956 controller().commitText("bar", underlines, 0);
957 // Last pair of InputEvent should also be inside composition scope. 957 // Last pair of InputEvent should also be inside composition scope.
958 EXPECT_STREQ("beforeinput.isComposing:true;input.isComposing:true;", 958 EXPECT_STREQ("beforeinput.isComposing:true;input.isComposing:true;",
959 document().title().utf8().data()); 959 document().title().utf8().data());
960 } 960 }
961 961
962 TEST_F(InputMethodControllerTest, CompositionInputEventForReplace) { 962 TEST_F(InputMethodControllerTest, CompositionInputEventForReplace) {
963 createHTMLWithCompositionInputEventListeners(); 963 createHTMLWithCompositionInputEventListeners();
964 964
965 // Simulate composition in the |contentEditable|. 965 // Simulate composition in the |contentEditable|.
966 Vector<CompositionUnderline> underlines; 966 Vector<CompositionUnderline> underlines;
(...skipping 19 matching lines...) Expand all
986 underlines.push_back(CompositionUnderline(0, 5, Color(255, 0, 0), false, 0)); 986 underlines.push_back(CompositionUnderline(0, 5, Color(255, 0, 0), false, 0));
987 987
988 document().setTitle(emptyString()); 988 document().setTitle(emptyString());
989 controller().setComposition("hello", underlines, 5, 5); 989 controller().setComposition("hello", underlines, 5, 5);
990 EXPECT_STREQ("beforeinput.data:hello;input.data:hello;", 990 EXPECT_STREQ("beforeinput.data:hello;input.data:hello;",
991 document().title().utf8().data()); 991 document().title().utf8().data());
992 992
993 // Confirm the ongoing composition. 993 // Confirm the ongoing composition.
994 document().setTitle(emptyString()); 994 document().setTitle(emptyString());
995 controller().finishComposingText(InputMethodController::KeepSelection); 995 controller().finishComposingText(InputMethodController::KeepSelection);
996 EXPECT_STREQ( 996 EXPECT_STREQ("compositionend.data:hello;", document().title().utf8().data());
997 "beforeinput.data:hello;input.data:hello;compositionend.data:hello;",
998 document().title().utf8().data());
999 } 997 }
1000 998
1001 TEST_F(InputMethodControllerTest, CompositionInputEventForDelete) { 999 TEST_F(InputMethodControllerTest, CompositionInputEventForDelete) {
1002 createHTMLWithCompositionInputEventListeners(); 1000 createHTMLWithCompositionInputEventListeners();
1003 1001
1004 // Simulate composition in the |contentEditable|. 1002 // Simulate composition in the |contentEditable|.
1005 Vector<CompositionUnderline> underlines; 1003 Vector<CompositionUnderline> underlines;
1006 underlines.push_back(CompositionUnderline(0, 5, Color(255, 0, 0), false, 0)); 1004 underlines.push_back(CompositionUnderline(0, 5, Color(255, 0, 0), false, 0));
1007 1005
1008 document().setTitle(emptyString()); 1006 document().setTitle(emptyString());
(...skipping 146 matching lines...) Expand 10 before | Expand all | Expand 10 after
1155 1153
1156 controller().commitText(String("string"), underlines, 0); 1154 controller().commitText(String("string"), underlines, 0);
1157 1155
1158 ASSERT_EQ(1u, document().markers().markers().size()); 1156 ASSERT_EQ(1u, document().markers().markers().size());
1159 1157
1160 EXPECT_EQ(9u, document().markers().markers()[0]->startOffset()); 1158 EXPECT_EQ(9u, document().markers().markers()[0]->startOffset());
1161 EXPECT_EQ(15u, document().markers().markers()[0]->endOffset()); 1159 EXPECT_EQ(15u, document().markers().markers()[0]->endOffset());
1162 } 1160 }
1163 1161
1164 } // namespace blink 1162 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/editing/InputMethodController.cpp ('k') | third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698