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

Unified Diff: components/metrics/call_stack_profile_metrics_provider_unittest.cc

Issue 2530043002: Set process phases in the StackSamplingProfiler. (Closed)
Patch Set: improved comment 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
Index: components/metrics/call_stack_profile_metrics_provider_unittest.cc
diff --git a/components/metrics/call_stack_profile_metrics_provider_unittest.cc b/components/metrics/call_stack_profile_metrics_provider_unittest.cc
index 727b506d3891aaffa43c7fc227d548062ad670ab..c81e23922ac460f6d69b52b7326a8c848e629249 100644
--- a/components/metrics/call_stack_profile_metrics_provider_unittest.cc
+++ b/components/metrics/call_stack_profile_metrics_provider_unittest.cc
@@ -41,7 +41,7 @@ struct ExpectedProtoEntry {
};
struct ExpectedProtoSample {
- uint32_t process_phases; // Bit-field of expected phases.
+ uint32_t process_milestones; // Bit-field of expected milestones.
const ExpectedProtoEntry* entries;
int entry_count;
int64_t entry_repeats;
@@ -61,7 +61,7 @@ class ProfilesFactory {
ProfilesFactory(){};
~ProfilesFactory(){};
- ProfilesFactory& AddPhase(int phase);
+ ProfilesFactory& AddMilestone(int milestone);
ProfilesFactory& NewProfile(int duration_ms, int interval_ms);
ProfilesFactory& NewSample();
ProfilesFactory& AddFrame(size_t module, uintptr_t offset);
@@ -73,13 +73,13 @@ class ProfilesFactory {
private:
Profiles profiles_;
- uint32_t process_phases_ = 0;
+ uint32_t process_milestones_ = 0;
DISALLOW_COPY_AND_ASSIGN(ProfilesFactory);
};
-ProfilesFactory& ProfilesFactory::AddPhase(int phase) {
- process_phases_ |= 1 << phase;
+ProfilesFactory& ProfilesFactory::AddMilestone(int milestone) {
+ process_milestones_ |= 1 << milestone;
return *this;
}
@@ -93,7 +93,7 @@ ProfilesFactory& ProfilesFactory::NewProfile(int duration_ms, int interval_ms) {
ProfilesFactory& ProfilesFactory::NewSample() {
profiles_.back().samples.push_back(Sample());
- profiles_.back().samples.back().process_phases = process_phases_;
+ profiles_.back().samples.back().process_milestones = process_milestones_;
return *this;
}
@@ -181,10 +181,10 @@ void CallStackProfileMetricsProviderTest::VerifyProfileProto(
SCOPED_TRACE("sample " + base::IntToString(s));
const CallStackProfile::Sample& proto_sample = stack.sample().Get(s);
- uint32_t process_phases = 0;
+ uint32_t process_milestones = 0;
for (int i = 0; i < proto_sample.process_phase().size(); ++i)
- process_phases |= 1U << proto_sample.process_phase().Get(i);
- EXPECT_EQ(expected.samples[s].process_phases, process_phases);
+ process_milestones |= 1U << proto_sample.process_phase().Get(i);
+ EXPECT_EQ(expected.samples[s].process_milestones, process_milestones);
ASSERT_EQ(expected.samples[s].entry_count, proto_sample.entry().size());
ASSERT_TRUE(proto_sample.has_count());
@@ -379,7 +379,7 @@ TEST_F(CallStackProfileMetricsProviderTest, RepeatedStacksUnordered) {
.NewProfile(100, 10)
.DefineModule(module_name, module_path, module_base_address)
- .AddPhase(0)
+ .AddMilestone(0)
.NewSample()
.AddFrame(0, module_base_address + 0x10)
.NewSample()
@@ -389,7 +389,7 @@ TEST_F(CallStackProfileMetricsProviderTest, RepeatedStacksUnordered) {
.NewSample()
.AddFrame(0, module_base_address + 0x10)
- .AddPhase(1)
+ .AddMilestone(1)
.NewSample()
.AddFrame(0, module_base_address + 0x10)
.NewSample()
@@ -464,7 +464,7 @@ TEST_F(CallStackProfileMetricsProviderTest, RepeatedStacksOrdered) {
.NewProfile(100, 10)
.DefineModule(module_name, module_path, module_base_address)
- .AddPhase(0)
+ .AddMilestone(0)
.NewSample()
.AddFrame(0, module_base_address + 0x10)
.NewSample()
@@ -474,7 +474,7 @@ TEST_F(CallStackProfileMetricsProviderTest, RepeatedStacksOrdered) {
.NewSample()
.AddFrame(0, module_base_address + 0x10)
- .AddPhase(1)
+ .AddMilestone(1)
.NewSample()
.AddFrame(0, module_base_address + 0x10)
.NewSample()
« no previous file with comments | « components/metrics/call_stack_profile_metrics_provider.cc ('k') | components/metrics/proto/execution_context.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698