[Sampling profiler] Remove heap profiler OnSampleCompleted interface

This interface is unused within CallStackProfileBuilder. The equivalent
functionality will be supported eventually within the class but will
occur via the metadata mechanism.

Bug: 851163
Change-Id: I3bd8430de70c973c3e4a293118403312b977f8d1
Reviewed-on: https://chromium-review.googlesource.com/c/1351533Reviewed-by: Xi Cheng <chengx@chromium.org>
Commit-Queue: Mike Wittman <wittman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611935}

components/metrics/call_stack_profile_builder.cc

@@ -62,14 +62,6 @@ CallStackProfileBuilder::~CallStackProfileBuilder() = default;
 // static
 void CallStackProfileBuilder::OnSampleCompleted(
     std::vector<base::StackSamplingProfiler::Frame> frames) {
-  OnSampleCompleted(std::move(frames), 1);
-}
-
-// TODO(wittman): record |count| as per-Stacksample metadata in the new proto
-// format.
-void CallStackProfileBuilder::OnSampleCompleted(
-    std::vector<base::StackSamplingProfiler::Frame> frames,
-    size_t /*count*/) {
   // Write CallStackProfile::Stack protobuf message.
   CallStackProfile::Stack stack;
 

components/metrics/call_stack_profile_builder.h

@@ -45,11 +45,6 @@ class CallStackProfileBuilder
   void OnProfileCompleted(base::TimeDelta profile_duration,
                           base::TimeDelta sampling_period) override;
 
-  // The function is used by sampling heap profiler. Its samples already come
-  // with different counts.
-  void OnSampleCompleted(std::vector<base::StackSamplingProfiler::Frame> frames,
-                         size_t count);
-
   // Sets the callback to use for reporting browser process profiles. This
   // indirection is required to avoid a dependency on unnecessary metrics code
   // in child processes.