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

Unified Diff: chrome/test/data/extensions/api_test/bluetooth_low_energy/remove_service/runtest.js

Issue 1976453002: Add permission parsing and the removeService API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@idl_changes
Patch Set: Created 4 years, 7 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/test/data/extensions/api_test/bluetooth_low_energy/remove_service/manifest.json ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/extensions/api_test/bluetooth_low_energy/remove_service/runtest.js
diff --git a/chrome/test/data/extensions/api_test/bluetooth_low_energy/register_service/runtest.js b/chrome/test/data/extensions/api_test/bluetooth_low_energy/remove_service/runtest.js
similarity index 53%
copy from chrome/test/data/extensions/api_test/bluetooth_low_energy/register_service/runtest.js
copy to chrome/test/data/extensions/api_test/bluetooth_low_energy/remove_service/runtest.js
index e6970790e0b6d6c09a0ef495337564d4de86a727..ef06b5627d18e2a5cd4a08a2b41dbdb3a11eabd7 100644
--- a/chrome/test/data/extensions/api_test/bluetooth_low_energy/register_service/runtest.js
+++ b/chrome/test/data/extensions/api_test/bluetooth_low_energy/remove_service/runtest.js
@@ -18,26 +18,36 @@ function failOnSuccess() {
return false;
}
-var service1 = { uuid: '00001234-0000-1000-8000-00805f9b34fb', isPrimary: true }
-var service2 = { uuid: '00001234-0000-1000-8000-00805f9b34fb', isPrimary: true }
-chrome.bluetoothLowEnergy.createService(service1, function(serviceId) {
+var service = { uuid: '00001234-0000-1000-8000-00805f9b34fb', isPrimary: true }
+// Create then remove a service.
+chrome.bluetoothLowEnergy.createService(service, function(serviceId) {
if (failOnError(serviceId))
return;
- chrome.bluetoothLowEnergy.registerService(serviceId, function() {
+ chrome.bluetoothLowEnergy.removeService(serviceId, function() {
if (failOnError('result'))
return;
- // Registering again, this should fail.
+ // Registering a removed service. Should fail.
chrome.bluetoothLowEnergy.registerService(serviceId, function() {
if (failOnSuccess())
return;
- chrome.bluetoothLowEnergy.createService(service2, function(serviceId) {
+
+ // Create, register then remove a service.
+ chrome.bluetoothLowEnergy.createService(service, function(serviceId) {
if (failOnError(serviceId))
return;
- // Registering another service, this should work.
chrome.bluetoothLowEnergy.registerService(serviceId, function() {
if (failOnError('result'))
return;
- chrome.test.succeed();
+ chrome.bluetoothLowEnergy.removeService(serviceId, function() {
+ if (failOnError('result'))
+ return;
+ // Removing an fake service. Should fail.
+ chrome.bluetoothLowEnergy.removeService('fake_id', function() {
+ if (failOnSuccess())
+ return;
+ chrome.test.succeed();
+ });
+ });
});
});
});
« no previous file with comments | « chrome/test/data/extensions/api_test/bluetooth_low_energy/remove_service/manifest.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698