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

Unified Diff: chrome/browser/extensions/api/tab_capture/tab_capture_performancetest.cc

Issue 127353002: Clean-up: Move tabCapture API test data files out of experimental dir. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 6 years, 11 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/extensions/api/tab_capture/tab_capture_performancetest.cc
diff --git a/chrome/browser/extensions/api/tab_capture/tab_capture_performancetest.cc b/chrome/browser/extensions/api/tab_capture/tab_capture_performancetest.cc
index 5cbd22d53fc16aa376dd5d632b0576993cc2aa52..be080ab3503e4a71bcbac1fb5067ca8051f5ed14 100644
--- a/chrome/browser/extensions/api/tab_capture/tab_capture_performancetest.cc
+++ b/chrome/browser/extensions/api/tab_capture/tab_capture_performancetest.cc
@@ -154,7 +154,7 @@ class TabCapturePerformanceTest
trace_analyzer::Query::EventPhaseIs(TRACE_EVENT_PHASE_INSTANT));
analyzer->FindEvents(query, &events);
if (events.size() < 20) {
- LOG(INFO) << "Not enough events of type " << event_name << " found.";
+ LOG(ERROR) << "Not enough events of type " << event_name << " found.";
return false;
}
@@ -163,7 +163,7 @@ class TabCapturePerformanceTest
events.end() - 3);
trace_analyzer::RateStats stats;
if (!GetRateStats(rate_events, &stats, NULL)) {
- LOG(INFO) << "GetRateStats failed";
+ LOG(ERROR) << "GetRateStats failed";
return false;
}
double mean_ms = stats.mean_us / 1000.0;
@@ -195,8 +195,7 @@ class TabCapturePerformanceTest
// but libjingle currently doesn't allow that.
// page += HasFlag(kDisableVsync) ? "&fps=300" : "&fps=30";
page += "&fps=30";
- ASSERT_TRUE(RunExtensionSubtest("tab_capture/experimental", page))
- << message_;
+ ASSERT_TRUE(RunExtensionSubtest("tab_capture", page)) << message_;
ASSERT_TRUE(tracing::EndTracing(&json_events));
scoped_ptr<trace_analyzer::TraceAnalyzer> analyzer;
analyzer.reset(trace_analyzer::TraceAnalyzer::Create(json_events));

Powered by Google App Engine
This is Rietveld 408576698