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

Unified Diff: chrome/browser/ui/webui/net_internals_ui_browsertest.cc

Issue 8474001: Add a timeline view to about:net-internals. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Update comments Created 9 years, 1 month 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/ui/webui/net_internals_ui_browsertest.cc
===================================================================
--- chrome/browser/ui/webui/net_internals_ui_browsertest.cc (revision 109671)
+++ chrome/browser/ui/webui/net_internals_ui_browsertest.cc (working copy)
@@ -228,6 +228,7 @@
AddLibrary(FilePath(FILE_PATH_LITERAL("net_internals/main.js")));
AddLibrary(FilePath(FILE_PATH_LITERAL("net_internals/prerender_view.js")));
AddLibrary(FilePath(FILE_PATH_LITERAL("net_internals/test_view.js")));
+ AddLibrary(FilePath(FILE_PATH_LITERAL("net_internals/timeline_view.js")));
}
void NetInternalsTest::SetUpOnMainThread() {
@@ -302,6 +303,48 @@
}
////////////////////////////////////////////////////////////////////////////////
+// timeline_view.js
+////////////////////////////////////////////////////////////////////////////////
+
+// TODO(mmenke): Add tests for labels and DataSeries.
+
+// Tests setting and updating range.
+IN_PROC_BROWSER_TEST_F(NetInternalsTest, NetInternalsTimelineViewRange) {
+ EXPECT_TRUE(RunJavascriptAsyncTest("netInternalsTimelineViewRange"));
+}
+
+// Tests using the scroll bar.
+IN_PROC_BROWSER_TEST_F(NetInternalsTest, NetInternalsTimelineViewScrollbar) {
+ EXPECT_TRUE(RunJavascriptAsyncTest("netInternalsTimelineViewScrollbar"));
+}
+
+// Tests case of having no events.
+IN_PROC_BROWSER_TEST_F(NetInternalsTest, NetInternalsTimelineViewNoEvents) {
+ EXPECT_TRUE(RunJavascriptAsyncTest("netInternalsTimelineViewNoEvents"));
+}
+
+// Dumps a log file to memory, modifies its events, loads it again, and
+// makes sure the range is correctly set and not automatically updated.
+IN_PROC_BROWSER_TEST_F(NetInternalsTest, NetInternalsTimelineViewLoadLog) {
+ EXPECT_TRUE(RunJavascriptAsyncTest("netInternalsTimelineViewLoadLog"));
+}
+
+// Zooms out twice, and then zooms in once.
+IN_PROC_BROWSER_TEST_F(NetInternalsTest, NetInternalsTimelineViewZoomOut) {
+ EXPECT_TRUE(RunJavascriptAsyncTest("netInternalsTimelineViewZoomOut"));
+}
+
+// Zooms in as much as allowed, and zooms out once.
+IN_PROC_BROWSER_TEST_F(NetInternalsTest, NetInternalsTimelineViewZoomIn) {
+ EXPECT_TRUE(RunJavascriptAsyncTest("netInternalsTimelineViewZoomIn"));
+}
+
+// Tests case of all events having the same time.
+IN_PROC_BROWSER_TEST_F(NetInternalsTest, NetInternalsTimelineViewDegenerate) {
+ EXPECT_TRUE(RunJavascriptAsyncTest("netInternalsTimelineViewDegenerate"));
+}
+
+////////////////////////////////////////////////////////////////////////////////
// dns_view.js
////////////////////////////////////////////////////////////////////////////////

Powered by Google App Engine
This is Rietveld 408576698