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

Unified Diff: fuzzer/go/frontend/data/report.go

Issue 1676593002: Remove old count and overview fuzzer elements (Closed) Base URL: https://skia.googlesource.com/buildbot@filter-ui
Patch Set: merged in above Created 4 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
« no previous file with comments | « fuzzer/elements.html ('k') | fuzzer/go/frontend/data/report_test.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: fuzzer/go/frontend/data/report.go
diff --git a/fuzzer/go/frontend/data/report.go b/fuzzer/go/frontend/data/report.go
index fac09893b9f814aa0a8f6a1962e87761fd47f20d..1d280cfaebe902545b1fee71a844f461513f6432 100644
--- a/fuzzer/go/frontend/data/report.go
+++ b/fuzzer/go/frontend/data/report.go
@@ -76,11 +76,10 @@ func newBuilder() *treeReportBuilder {
type fuzzReportCache struct {
// All the data goes in here, in no particular order
rawData FuzzReportTree
- // Generated, sorted caches
- FullReport FuzzReportTree
- SummaryReport FuzzReportTree
+ // Generated, sorted cache
+ FullReport FuzzReportTree
- // If data is in rawData, but not in SummaryReport or FullReport, the trees should be
+ // If data is in rawData, but not in FullReport, the trees should be
// rebuilt
isDirty bool
}
@@ -92,14 +91,6 @@ var currentData = newBuilder()
// without disturbing the data shown to users.
var stagingData = newBuilder()
-func FindFuzzSummary(category string) FuzzReportTree {
- cache, found := currentData.caches[category]
- if !found {
- return FuzzReportTree{}
- }
- return cache.SummaryReport
-}
-
// FindFuzzDetails returns the detailed fuzz reports for a file name, function name, and line number.
// If functionName is "" or lineNumber is -1, all reports are shown.
func FindFuzzDetails(category, fileName, functionName string, lineNumber int) (FuzzReportTree, error) {
@@ -145,14 +136,6 @@ func (function *FunctionFuzzReport) filterByLineNumber(lineNumber int) {
}
}
-func CategoryOverview(category string) FuzzReportTree {
- overview, found := currentData.caches[category]
- if found {
- return overview.SummaryReport
- }
- return FuzzReportTree{}
-}
-
// FindFuzzDetailForFuzz returns a tree containing the single
// report with the given name, or an error, it it doesn't exist.
func FindFuzzDetailForFuzz(category, name string) (FuzzReportTree, error) {
@@ -365,26 +348,9 @@ func (r *treeReportBuilder) getTreeSortedByTotal(category string) FuzzReportTree
return cache.FullReport
}
-// getSummarySortedByTotal gets the summary FuzzReport for a fuzz category
-// sorted by total number of fuzzes.
-func (r *treeReportBuilder) getSummarySortedByTotal(category string) FuzzReportTree {
- cache, found := r.caches[category]
- if !found {
- glog.Warningf("Could not find report tree for category %s", category)
- return FuzzReportTree{}
- }
- if cache.isDirty {
- r.mutex.Lock()
- defer r.mutex.Unlock()
- cache.rebuildSortedReports()
- }
- return cache.SummaryReport
-}
-
// rebuildSortedReports creates the sorted reports for a given cache.
func (c *fuzzReportCache) rebuildSortedReports() {
c.FullReport = c.getClonedSortedReport(true)
- c.SummaryReport = c.getClonedSortedReport(false)
c.isDirty = false
}
« no previous file with comments | « fuzzer/elements.html ('k') | fuzzer/go/frontend/data/report_test.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698