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

Unified Diff: third_party/WebKit/Source/devtools/front_end/resources/ApplicationPanelSidebar.js

Issue 2914293002: [IndexedDB] [DevTools] Right-click 'Refresh' on database now updates object store view, also fixed … (Closed)
Patch Set: Better handling of null database in view, removed extra lines in test file, moved utility test func… Created 3 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
Index: third_party/WebKit/Source/devtools/front_end/resources/ApplicationPanelSidebar.js
diff --git a/third_party/WebKit/Source/devtools/front_end/resources/ApplicationPanelSidebar.js b/third_party/WebKit/Source/devtools/front_end/resources/ApplicationPanelSidebar.js
index 6b2d25e6b90ae567d4a9d0fc06bc222cf231818b..bb45f1bd1531d00a0c10566f52281e045eeba906 100644
--- a/third_party/WebKit/Source/devtools/front_end/resources/ApplicationPanelSidebar.js
+++ b/third_party/WebKit/Source/devtools/front_end/resources/ApplicationPanelSidebar.js
@@ -1086,7 +1086,9 @@ Resources.IndexedDBTreeElement = class extends Resources.StorageCategoryTreeElem
refreshIndexedDB() {
for (var indexedDBModel of SDK.targetManager.models(Resources.IndexedDBModel))
- indexedDBModel.refreshDatabaseNames();
+ indexedDBModel.refreshDatabaseNames(); // Adds IDBDatabaseTreeElements.
dgozman 2017/06/02 21:48:41 I think we should not rely on this face. After all
kristipark 2017/06/03 01:42:44 Unfortunately unless refreshDatabaseNames is calle
+ for (var i = 0; i < this._idbDatabaseTreeElements.length; ++i)
+ this._idbDatabaseTreeElements[i]._refreshIndexedDB();
}
/**
@@ -1197,7 +1199,7 @@ Resources.IDBDatabaseTreeElement = class extends Resources.BaseStorageTreeElemen
}
_refreshIndexedDB() {
- this._model.refreshDatabaseNames();
+ this._model.refreshDatabase(this._databaseId);
}
/**

Powered by Google App Engine
This is Rietveld 408576698