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

Issue 3002603002: Remove race condition in Observatory memory page (Closed)

Created:
3 years, 4 months ago by cbernaschina
Modified:
3 years, 4 months ago
Reviewers:
bkonyi, rmacnak, siva
CC:
reviews_dartlang.org, turnidge, rmacnak, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Remove race condition in Observatory memory page During the computation of the hover labels in the memory dashboard graph the format functions were reading shared state that can be out of sync if an update is ongoing. To fix the error is enough to use the local copies instead. To prevent this error in the future the format functions now are marked as static and the analyzer will signal if they are accessing shared state. R=asiva@google.com, rmacnak@google.com Committed: https://github.com/dart-lang/sdk/commit/470749c38c1ba50689ef80f0a1a013b834684f97

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -9 lines) Patch
M runtime/observatory/lib/src/elements/memory/graph.dart View 3 chunks +9 lines, -9 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
cbernaschina
3 years, 4 months ago (2017-08-14 16:49:12 UTC) #2
siva
lgtm
3 years, 4 months ago (2017-08-14 20:17:22 UTC) #3
rmacnak
lgtm
3 years, 4 months ago (2017-08-14 20:38:29 UTC) #4
cbernaschina
3 years, 4 months ago (2017-08-14 21:39:18 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
470749c38c1ba50689ef80f0a1a013b834684f97 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698