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

Unified Diff: chrome/test/data/extensions/api_test/input_ime/background.js

Issue 2386823002: [Extensions] Remove ExtensionFunction::SetError() (Closed)
Patch Set: lazyboy's Created 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/extensions/chrome_extension_function.cc ('k') | extensions/browser/extension_function.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/extensions/api_test/input_ime/background.js
diff --git a/chrome/test/data/extensions/api_test/input_ime/background.js b/chrome/test/data/extensions/api_test/input_ime/background.js
index b82f6d08d91b9571172e58c50eec5f6924425fa7..383d3347aaa4755785aea2e3e9796db1786c1c0d 100644
--- a/chrome/test/data/extensions/api_test/input_ime/background.js
+++ b/chrome/test/data/extensions/api_test/input_ime/background.js
@@ -33,19 +33,20 @@ function commitTextTest() {
}
-function setCandidateWindowPropertiesTest() {
- chrome.input.ime.setCandidateWindowProperties({
- "engineID": "test",
- "properties": {
- "visible": true,
- "cursorVisible": false,
- "vertical": true,
- "pageSize": 6,
- "auxiliaryText": "notes",
- "auxiliaryTextVisible": true
- }
- }, chrome.test.callbackPass());
-}
+// Disabled: crbug.com/641425.
+// function setCandidateWindowPropertiesTest() {
+// chrome.input.ime.setCandidateWindowProperties({
+// "engineID": "test",
+// "properties": {
+// "visible": true,
+// "cursorVisible": false,
+// "vertical": true,
+// "pageSize": 6,
+// "auxiliaryText": "notes",
+// "auxiliaryTextVisible": true
+// }
+// }, chrome.test.callbackPass());
+// }
function setCandidatesTest() {
@@ -113,17 +114,24 @@ function setCursorPositionTest() {
// }, chrome.test.callbackPass());
// }
-function deleteSurroundingText() {
- chrome.input.ime.deleteSurroundingText({
- "engineID": "test",
- "contextID": 1,
- "offset": -1,
- "length": 1
- }, chrome.test.callbackPass());
-}
+// Disabled: crbug.com/641425.
+// function deleteSurroundingText() {
+// chrome.input.ime.deleteSurroundingText({
+// "engineID": "test",
+// "contextID": 1,
+// "offset": -1,
+// "length": 1
+// }, chrome.test.callbackPass());
+// }
-chrome.test.runTests([setCompositionTest, clearCompositionTest,
- commitTextTest, setCandidateWindowPropertiesTest,
- setCandidatesTest, setCursorPositionTest,
- /*setMenuItemsTest, updateMenuItemsTest, */
- deleteSurroundingText]);
+chrome.test.runTests([
+ setCompositionTest,
+ clearCompositionTest,
+ commitTextTest,
+ // setCandidateWindowPropertiesTest,
+ setCandidatesTest,
+ setCursorPositionTest,
+ // setMenuItemsTest,
+ // updateMenuItemsTest,
+ // deleteSurroundingText,
+]);
« no previous file with comments | « chrome/browser/extensions/chrome_extension_function.cc ('k') | extensions/browser/extension_function.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698