Skip to content

Commit 6e75e7d

Browse files
cwrenAndroid (Google) Code Review
authored andcommitted
Merge "Instrument printing service for metrics collection." into mnc-dev
2 parents f92a1fd + dcc34fd commit 6e75e7d

2 files changed

Lines changed: 6 additions & 0 deletions

File tree

packages/PrintSpooler/src/com/android/printspooler/model/PrintSpoolerService.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@
4646
import android.util.Slog;
4747
import android.util.Xml;
4848

49+
import com.android.internal.logging.MetricsLogger;
4950
import com.android.internal.os.HandlerCaller;
5051
import com.android.internal.util.FastXmlSerializer;
5152
import com.android.printspooler.R;
@@ -493,6 +494,7 @@ public boolean setPrintJobState(PrintJobId printJobId, int state, String error)
493494
Slog.i(LOG_TAG, "[STATE CHANGED] " + printJob);
494495
}
495496

497+
MetricsLogger.histogram(this, "print_job_state", state);
496498
switch (state) {
497499
case PrintJobInfo.STATE_COMPLETED:
498500
case PrintJobInfo.STATE_CANCELED:

packages/PrintSpooler/src/com/android/printspooler/ui/PrintActivity.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@
7272
import android.widget.Spinner;
7373
import android.widget.TextView;
7474

75+
import com.android.internal.logging.MetricsLogger;
7576
import com.android.printspooler.R;
7677
import com.android.printspooler.model.MutexFileProvider;
7778
import com.android.printspooler.model.PrintSpoolerProvider;
@@ -325,6 +326,7 @@ public void onResume() {
325326
if (mState != STATE_INITIALIZING && mCurrentPrinter != null) {
326327
mPrinterRegistry.setTrackedPrinter(mCurrentPrinter.getId());
327328
}
329+
MetricsLogger.count(this, "print_preview", 1);
328330
}
329331

330332
@Override
@@ -1074,6 +1076,8 @@ private void cancelPrint() {
10741076
private void confirmPrint() {
10751077
setState(STATE_PRINT_CONFIRMED);
10761078

1079+
MetricsLogger.count(this, "print_confirmed", 1);
1080+
10771081
updateOptionsUi();
10781082
addCurrentPrinterToHistory();
10791083

0 commit comments

Comments
 (0)