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

Unified Diff: chrome/browser/resources/offline_pages/offline_internals_browser_proxy.js

Issue 2089423002: [Offline Pages] Connect the offline page logs to the internals page. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@i4
Patch Set: remove unneeded consts Created 4 years, 6 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/offline_pages/offline_internals_browser_proxy.js
diff --git a/chrome/browser/resources/offline_pages/offline_internals_browser_proxy.js b/chrome/browser/resources/offline_pages/offline_internals_browser_proxy.js
index 7ab1de4f007ecd268e26c61c37d6124fc3302c5d..60ac94dd1f4162f42b27780b9fab1fba1cee3c47 100644
--- a/chrome/browser/resources/offline_pages/offline_internals_browser_proxy.js
+++ b/chrome/browser/resources/offline_pages/offline_internals_browser_proxy.js
@@ -28,6 +28,14 @@ var OfflinePage;
*/
var SavePageRequest;
+/**
+ * @typedef {{
+ * modelIsLogging: boolean,
+ * queueIsLogging: boolean
+ * }}
+ */
+var IsLogging;
+
cr.define('offlineInternals', function() {
/** @interface */
function OfflineInternalsBrowserProxy() {}
@@ -60,6 +68,32 @@ cr.define('offlineInternals', function() {
* pages are deleted.
*/
deleteSelectedPages: function(ids) {},
+
+ /**
+ * Sets whether to record logs for stored pages.
+ * @param {boolean} shouldLog True if logging should be enabled.
+ */
+ setRecordPageModel: function(shouldLog) {},
+
+ /**
+ * Sets whether to record logs for scheduled requests.
+ * @param {boolean} shouldLog True if logging should be enabled.
+ */
+ setRecordRequestQueue: function(shouldLog) {},
+
+ /**
+ * Gets the currently recorded logs.
+ * @return {!Promise<!Array<string>>} A promise firing when the
+ * logs are retrieved.
+ */
+ getEventLogs: function() {},
+
+ /**
+ * Gets the state of logging (on/off).
+ * @return {!Promise<!IsLogging>} A promise firing when the state
+ * is retrieved.
+ */
+ getLoggingState: function() {},
};
/**
@@ -88,6 +122,26 @@ cr.define('offlineInternals', function() {
/** @override */
deleteSelectedPages: function(ids) {
return cr.sendWithPromise('deleteSelectedPages', ids);
+ },
+
+ /** @override */
+ setRecordPageModel: function(shouldLog) {
+ chrome.send('setRecordPageModel');
+ },
+
+ /** @override */
+ setRecordRequestQueue: function(shouldLog) {
+ chrome.send('setRecordRequestQueue');
+ },
+
+ /** @override */
+ getEventLogs: function() {
+ return cr.sendWithPromise('getEventLogs');
+ },
+
+ /** @override */
+ getLoggingState: function() {
+ return cr.sendWithPromise('getLoggingState');
}
};
« no previous file with comments | « chrome/browser/resources/offline_pages/offline_internals.js ('k') | chrome/browser/ui/webui/offline_internals_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698