Skip to content
This repository has been archived by the owner on Aug 4, 2022. It is now read-only.

Commit

Permalink
Bug 1578073 - Remove native calls in Histogram.java; r=ng
Browse files Browse the repository at this point in the history
Getting the JNI calls here to work requires a good amount of webrtc.org
machinery. It might be worth setting that up the next time we do an upstream
merge, but for now, it is a lot simpler to just remove the affected code,
given that we are not interested in collecting this data anyway.

Differential Revision: https://phabricator.services.mozilla.com/D61860
  • Loading branch information
dminor committed Feb 11, 2020
1 parent 75ef671 commit 973a2d5
Showing 1 changed file with 2 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -27,18 +27,13 @@ private Histogram(long handle) {
}

static public Histogram createCounts(String name, int min, int max, int bucketCount) {
return new Histogram(nativeCreateCounts(name, min, max, bucketCount));
return new Histogram(0);
}

static public Histogram createEnumeration(String name, int max) {
return new Histogram(nativeCreateEnumeration(name, max));
return new Histogram(0);
}

public void addSample(int sample) {
nativeAddSample(handle, sample);
}

private static native long nativeCreateCounts(String name, int min, int max, int bucketCount);
private static native long nativeCreateEnumeration(String name, int max);
private static native void nativeAddSample(long handle, int sample);
}

0 comments on commit 973a2d5

Please sign in to comment.