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

Unified Diff: runtime/vm/thread.h

Issue 2617513004: Revert "Added isolate + thread high watermark tracking to Observatory" (Closed)
Patch Set: Created 3 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
« no previous file with comments | « runtime/vm/isolate.cc ('k') | runtime/vm/thread.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/thread.h
diff --git a/runtime/vm/thread.h b/runtime/vm/thread.h
index d9cf1204c1ce19f3bbd3b192c6fcf7bc50569410..e73c44bc024888165911a3fe04bcfad082854564 100644
--- a/runtime/vm/thread.h
+++ b/runtime/vm/thread.h
@@ -264,22 +264,6 @@ class Thread : public BaseThread {
bool ZoneIsOwnedByThread(Zone* zone) const;
- void IncrementMemoryUsage(uint value) {
- current_thread_memory_ += value;
- if (current_thread_memory_ > memory_high_watermark_) {
- memory_high_watermark_ = current_thread_memory_;
- }
- }
-
- void DecrementMemoryUsage(uint value) {
- ASSERT(current_thread_memory_ >= value);
- current_thread_memory_ -= value;
- }
-
- uint memory_high_watermark() const { return memory_high_watermark_; }
-
- void ResetHighWatermark() { memory_high_watermark_ = current_thread_memory_; }
-
// The reusable api local scope for this thread.
ApiLocalScope* api_reusable_scope() const { return api_reusable_scope_; }
void set_api_reusable_scope(ApiLocalScope* value) {
@@ -697,8 +681,6 @@ class Thread : public BaseThread {
OSThread* os_thread_;
Monitor* thread_lock_;
Zone* zone_;
- uint current_thread_memory_;
- uint memory_high_watermark_;
ApiLocalScope* api_reusable_scope_;
ApiLocalScope* api_top_scope_;
StackResource* top_resource_;
« no previous file with comments | « runtime/vm/isolate.cc ('k') | runtime/vm/thread.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698