From fbf685861664ad5fb16d07c048d364998bb12fcd Mon Sep 17 00:00:00 2001 From: Steve McKay Date: Fri, 9 Jun 2017 15:28:28 -0700 Subject: Pass doc uri in Intent data. As opposed to an extra. Use of ACTION_VIEW was atypical. "Data" is the natural home for a Uri. Bug: 62293172 Test: Treehugger Change-Id: I8e2b9cd36292591dbc88d2a6e1137b2243c96197 --- src/com/android/documentsui/files/ActionHandler.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/com/android/documentsui/files/ActionHandler.java b/src/com/android/documentsui/files/ActionHandler.java index fccff2022..2757d5d3e 100644 --- a/src/com/android/documentsui/files/ActionHandler.java +++ b/src/com/android/documentsui/files/ActionHandler.java @@ -676,9 +676,9 @@ public class ActionHandler extends AbstractActionHa @Override public void showInspector(DocumentInfo doc) { + Metrics.logUserAction(mActivity, Metrics.USER_ACTION_INSPECTOR); Intent intent = new Intent(mActivity, DocumentInspectorActivity.class); - intent.putExtra(Intent.ACTION_VIEW, doc.derivedUri); - Metrics.logUserAction(mActivity.getBaseContext(), Metrics.USER_ACTION_INSPECTOR); + intent.setData(doc.derivedUri); mActivity.startActivity(intent); } -- cgit v1.2.3-59-g8ed1b