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

Unified Diff: chrome/browser/resources/net_internals/log_util.js

Issue 9581021: [refactor] Split up SourceTracker into SourceTracker + EventsTracker. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update based on mmenke comments Created 8 years, 10 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/net_internals/log_util.js
diff --git a/chrome/browser/resources/net_internals/log_util.js b/chrome/browser/resources/net_internals/log_util.js
index 7d2cdfdef8bbb4a0b0276099554571cd04a2e615..78c7e342e50fdbf576926ccbe453cb2c75b78acf 100644
--- a/chrome/browser/resources/net_internals/log_util.js
+++ b/chrome/browser/resources/net_internals/log_util.js
@@ -63,13 +63,14 @@ log_util = (function() {
oldLogDump.constants.clientInfo.numericDate) {
numericDate = oldLogDump.constants.clientInfo.numericDate;
}
- var logDump = createLogDump(userComments,
- Constants,
- g_browser.sourceTracker.getAllCapturedEvents(),
- oldLogDump.polledData,
- getTabData_(),
- numericDate,
- securityStripping);
+ var logDump = createLogDump(
+ userComments,
+ Constants,
+ EventsTracker.getInstance().getAllCapturedEvents(),
+ oldLogDump.polledData,
+ getTabData_(),
+ numericDate,
+ securityStripping);
return JSON.stringify(logDump, null, ' ');
}
@@ -79,13 +80,14 @@ log_util = (function() {
*/
function onUpdateAllCompleted(userComments, callback, securityStripping,
polledData) {
- var logDump = createLogDump(userComments,
- Constants,
- g_browser.sourceTracker.getAllCapturedEvents(),
- polledData,
- getTabData_(),
- timeutil.getCurrentTime(),
- securityStripping);
+ var logDump = createLogDump(
+ userComments,
+ Constants,
+ EventsTracker.getInstance().getAllCapturedEvents(),
+ polledData,
+ getTabData_(),
+ timeutil.getCurrentTime(),
+ securityStripping);
callback(JSON.stringify(logDump, null, ' '));
}
@@ -214,7 +216,7 @@ log_util = (function() {
MainView.getInstance().onLoadLog(opt_fileName);
// Delete all events. This will also update all logObservers.
- g_browser.sourceTracker.deleteAllSourceEntries();
+ EventsTracker.getInstance().deleteAllLogEntries();
// Inform all the views that a log file is being loaded, and pass in
// view-specific saved state, if any.
@@ -224,7 +226,7 @@ log_util = (function() {
var view = categoryTabSwitcher.findTabById(tabIds[i]).contentView;
view.onLoadLogStart(logDump.polledData, logDump.tabData[tabIds[i]]);
}
- g_browser.sourceTracker.onReceivedLogEntries(validEvents);
+ EventsTracker.getInstance().addLogEntries(validEvents);
var numInvalidEvents = logDump.events.length -
(validEvents.length + numDeprecatedPassiveEvents);
« no previous file with comments | « chrome/browser/resources/net_internals/index.js ('k') | chrome/browser/resources/net_internals/log_view_painter.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698