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

Unified Diff: ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h

Issue 215553002: Report some PNaCl UMA earlier, instead of in one chunk at the end. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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: ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h
diff --git a/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h b/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h
index 5a20181cd9d5b757e9dafd811a6d0b68677af7f7..4de09633836102d43a1603f5ad713d1a38d63b2c 100644
--- a/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h
+++ b/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h
@@ -34,13 +34,6 @@ class PnaclOptions;
class PnaclResources;
class TempFile;
-struct PnaclTimeStats {
- int64_t pnacl_llc_load_time;
- int64_t pnacl_compile_time;
- int64_t pnacl_ld_load_time;
- int64_t pnacl_link_time;
-};
-
class PnaclTranslateThread {
public:
PnaclTranslateThread();
@@ -70,7 +63,7 @@ class PnaclTranslateThread {
// Send bitcode bytes to the translator. Called from the main thread.
void PutBytes(std::vector<char>* data, int count);
- const PnaclTimeStats& GetTimeStats() const { return time_stats_; }
+ int64_t GetCompileTime() const { return compile_time_; }
private:
// Starts an individual llc or ld subprocess used for translation.
@@ -117,7 +110,7 @@ class PnaclTranslateThread {
// Associated with buffer_cond_
bool done_;
- PnaclTimeStats time_stats_;
+ int64_t compile_time_;
// Data about the translation files, owned by the coordinator
const Manifest* manifest_;

Powered by Google App Engine
This is Rietveld 408576698