[Extensions] Convert some SyncExtensionFunctions

chrome/test/data/extensions/api_test/input_ime/background.js

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 973c4532702169d72b6f3d6981608ba78460b68e..b82f6d08d91b9571172e58c50eec5f6924425fa7 100644
--- a/chrome/test/data/extensions/api_test/input_ime/background.js
+++ b/chrome/test/data/extensions/api_test/input_ime/background.js
@@ -79,38 +79,39 @@ function setCursorPositionTest() {
 }
 
 
-
-function setMenuItemsTest() {
-  chrome.input.ime.setMenuItems({
-    "engineID": "test",
-    "items": [{
-      "id": "Menu 1",
-      "label": "Menu 1",
-      "style": "check",
-      "visible": true,
-      "enabled": true
-    }, {
-      "id": "Menu 2",
-      "label": "Menu 2",
-      "style": "radio",
-      "visible": true,
-      "enabled": true
-    }]
-  }, chrome.test.callbackPass());
-}
-
-function updateMenuItemsTest() {
-  chrome.input.ime.updateMenuItems({
-    "engineID": "test",
-    "items": [{
-      "id": "Menu 1",
-      "enabled": false
-    }, {
-      "id": "Menu 2",
-      "visible": false,
-    }]
-  }, chrome.test.callbackPass());
-}
+// Disabled: crbug.com/641425.

[lazyboy, 2016/08/29 18:01:49]

Add this change to CL description?

These don't work because InputMethodEngine isn't (and wasn't) available?

[Devlin, 2016/08/29 20:29:54]

Adjusted cl description.  These actually don't work because the input method
engine returns false for Set/UpdateMenuItems(), and appears to always have.

+// function setMenuItemsTest() {
+//   chrome.input.ime.setMenuItems({
+//     "engineID": "test",
+//     "items": [{
+//       "id": "Menu 1",
+//       "label": "Menu 1",
+//       "style": "check",
+//       "visible": true,
+//       "enabled": true
+//     }, {
+//       "id": "Menu 2",
+//       "label": "Menu 2",
+//       "style": "radio",
+//       "visible": true,
+//       "enabled": true
+//     }]
+//   }, chrome.test.callbackPass());
+// }
+
+// Disabled: crbug.com/641425.
+// function updateMenuItemsTest() {
+//   chrome.input.ime.updateMenuItems({
+//     "engineID": "test",
+//     "items": [{
+//       "id": "Menu 1",
+//       "enabled": false
+//     }, {
+//       "id": "Menu 2",
+//       "visible": false,
+//     }]
+//   }, chrome.test.callbackPass());
+// }
 
 function deleteSurroundingText() {
   chrome.input.ime.deleteSurroundingText({
@@ -124,5 +125,5 @@ function deleteSurroundingText() {
 chrome.test.runTests([setCompositionTest, clearCompositionTest,
                       commitTextTest, setCandidateWindowPropertiesTest,
                       setCandidatesTest, setCursorPositionTest,
-                      setMenuItemsTest, updateMenuItemsTest,
+                      /*setMenuItemsTest, updateMenuItemsTest, */
                       deleteSurroundingText]);