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

Unified Diff: chrome/browser/resources/local_discovery/local_discovery.js

Issue 23851008: Added cache flush on network change to ServiceDiscoveryHostClient (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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
Index: chrome/browser/resources/local_discovery/local_discovery.js
diff --git a/chrome/browser/resources/local_discovery/local_discovery.js b/chrome/browser/resources/local_discovery/local_discovery.js
index 4ec0c33a9eb77098f4d9273dc06c55e48a5f7dec..6ff4865f9fffb2eb7274cbc37cbc6a892f14b774 100644
--- a/chrome/browser/resources/local_discovery/local_discovery.js
+++ b/chrome/browser/resources/local_discovery/local_discovery.js
@@ -207,17 +207,7 @@ cr.define('local_discovery', function() {
}
}
- var numberPrinters = $('register-device-list').children.length;
- $('printer-num').textContent = generateNumberPrintersAvailableText(
- numberPrinters);
-
- if (numberPrinters == 0) {
- $('register-message').textContent = loadTimeData.getString(
- 'noPrintersOnNetworkExplanation');
- } else {
- $('register-message').textContent = loadTimeData.getString(
- 'registerConfirmMessage');
- }
+ updateUIToReflectNumberOfLocalDevices();
}
/**
@@ -248,6 +238,29 @@ cr.define('local_discovery', function() {
}
}
+ function onDeviceCacheFlushed() {
+ for (var deviceName in devices) {
+ devices[deviceName].removeDevice();
+ delete devices[deviceName];
+ }
+
+ updateUIToReflectNumberOfLocalDevices();
+ }
+
+ function updateUIToReflectNumberOfLocalDevices() {
+ var numberPrinters = $('register-device-list').children.length;
+ $('printer-num').textContent = generateNumberPrintersAvailableText(
+ numberPrinters);
+
+ if (numberPrinters == 0) {
+ $('register-message').textContent = loadTimeData.getString(
+ 'noPrintersOnNetworkExplanation');
+ } else {
+ $('register-message').textContent = loadTimeData.getString(
+ 'registerConfirmMessage');
+ }
+ }
+
/**
* Announce that a registration succeeeded.
@@ -331,6 +344,7 @@ cr.define('local_discovery', function() {
onRegistrationFailed: onRegistrationFailed,
onUnregisteredDeviceUpdate: onUnregisteredDeviceUpdate,
onRegistrationConfirmedOnPrinter: onRegistrationConfirmedOnPrinter,
- onCloudDeviceListAvailable: onCloudDeviceListAvailable
+ onCloudDeviceListAvailable: onCloudDeviceListAvailable,
+ onDeviceCacheFlushed: onDeviceCacheFlushed
};
});

Powered by Google App Engine
This is Rietveld 408576698