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

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

Issue 9298032: Field trial to investigate higher weightings for hits when calculating Omnibox prerender confidence. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: stringprintf Created 8 years, 11 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/network_action_predictor/network_action_predictor.js
diff --git a/chrome/browser/resources/network_action_predictor/network_action_predictor.js b/chrome/browser/resources/network_action_predictor/network_action_predictor.js
index bc8d0aa5fcc5954fd38f8fe5e13c06e2070752e8..11343cb9dd65c7ac4e10aeafe53e7e3091320710 100644
--- a/chrome/browser/resources/network_action_predictor/network_action_predictor.js
+++ b/chrome/browser/resources/network_action_predictor/network_action_predictor.js
@@ -14,29 +14,31 @@ function requestNetworkActionPredictorDb() {
/**
* Callback from backend with the database contents. Sets up some globals and
* calls to create the UI.
- * @param {boolean} enabled Whether or not NetworkActionPredictor is enabled.
- * @param {Array} database The database as a flattened list.
+ * @param {Dictionary} database Information about NetworkActionPredictor
+ * including the database as a flattened list, a boolean indicating if the
+ * system is enabled and the current hit weight.
*/
-function updateDatabaseTable(enabled, database) {
+function updateDatabaseTable(database) {
console.debug('Updating Table NAP DB');
var filter = $('filter');
filter.disabled = false;
filter.onchange = function() {
- updateDatabaseView(enabled, database);
+ updateDatabaseView(database);
};
- updateDatabaseView(enabled, database);
+ updateDatabaseView(database);
}
/**
* Updates the table from the database.
- * @param {boolean} enabled Whether or not NetworkActionPredictor is enabled.
- * @param {Array} database The database as a flattened list.
+ * @param {Dictionary} database Information about NetworkActionPredictor
+ * including the database as a flattened list, a boolean indicating if the
+ * system is enabled and the current hit weight.
*/
-function updateDatabaseView(enabled, database) {
+function updateDatabaseView(database) {
var databaseSection = $('databaseTableBody');
- var showEnabled = database && enabled;
+ var showEnabled = database.enabled && database.db;
$('enabledMode').hidden = !showEnabled;
$('disabledMode').hidden = showEnabled;
@@ -49,8 +51,8 @@ function updateDatabaseView(enabled, database) {
// Clear any previous list.
databaseSection.textContent = '';
- for (var i = 0; i < database.length; ++i) {
- var entry = database[i];
+ for (var i = 0; i < database.db.length; ++i) {
+ var entry = database.db[i];
if (!filter.checked || entry.confidence > 0) {
var row = document.createElement('tr');
@@ -69,6 +71,7 @@ function updateDatabaseView(enabled, database) {
}
}
$('countBanner').textContent = 'Entries: ' + databaseSection.children.length;
+ $('countBanner').textContent += ' Hit Weight: ' + database.hit_weight;
}
document.addEventListener('DOMContentLoaded', requestNetworkActionPredictorDb);

Powered by Google App Engine
This is Rietveld 408576698