Merge "Fix document management permission enforcement." into klp-dev (fc98ea37) · Commits · e / os / android_packages_apps_DocumentsUI

src/com/android/documentsui/TestActivity.java

+50 −21

Original line number	Diff line number	Diff line
		@@ -33,10 +33,14 @@ import libcore.io.IoUtils;
		import libcore.io.Streams;

		import java.io.InputStream;
		import java.io.OutputStream;

		public class TestActivity extends Activity {
		private static final String TAG = "TestActivity";

		private static final int CODE_READ = 42;
		private static final int CODE_WRITE = 43;

		private TextView mResult;

		@Override
		@@ -49,10 +53,10 @@ public class TestActivity extends Activity {
		view.setOrientation(LinearLayout.VERTICAL);

		final CheckBox multiple = new CheckBox(context);
		multiple.setText("\nALLOW_MULTIPLE\n");
		multiple.setText("ALLOW_MULTIPLE");
		view.addView(multiple);
		final CheckBox localOnly = new CheckBox(context);
		localOnly.setText("\nLOCAL_ONLY\n");
		localOnly.setText("LOCAL_ONLY");
		view.addView(localOnly);

		Button button;
		@@ -70,7 +74,7 @@ public class TestActivity extends Activity {
		if (localOnly.isChecked()) {
		intent.putExtra(Intent.EXTRA_LOCAL_ONLY, true);
		}
		startActivityForResult(intent, 42);
		startActivityForResult(intent, CODE_READ);
		}
		});
		view.addView(button);
		@@ -89,7 +93,7 @@ public class TestActivity extends Activity {
		if (localOnly.isChecked()) {
		intent.putExtra(Intent.EXTRA_LOCAL_ONLY, true);
		}
		startActivityForResult(intent, 42);
		startActivityForResult(intent, CODE_READ);
		}
		});
		view.addView(button);
		@@ -108,7 +112,7 @@ public class TestActivity extends Activity {
		if (localOnly.isChecked()) {
		intent.putExtra(Intent.EXTRA_LOCAL_ONLY, true);
		}
		startActivityForResult(intent, 42);
		startActivityForResult(intent, CODE_READ);
		}
		});
		view.addView(button);
		@@ -129,7 +133,7 @@ public class TestActivity extends Activity {
		if (localOnly.isChecked()) {
		intent.putExtra(Intent.EXTRA_LOCAL_ONLY, true);
		}
		startActivityForResult(intent, 42);
		startActivityForResult(intent, CODE_READ);
		}
		});
		view.addView(button);
		@@ -146,7 +150,7 @@ public class TestActivity extends Activity {
		if (localOnly.isChecked()) {
		intent.putExtra(Intent.EXTRA_LOCAL_ONLY, true);
		}
		startActivityForResult(intent, 42);
		startActivityForResult(intent, CODE_WRITE);
		}
		});
		view.addView(button);
		@@ -165,7 +169,7 @@ public class TestActivity extends Activity {
		if (localOnly.isChecked()) {
		intent.putExtra(Intent.EXTRA_LOCAL_ONLY, true);
		}
		startActivityForResult(Intent.createChooser(intent, "Kittens!"), 42);
		startActivityForResult(Intent.createChooser(intent, "Kittens!"), CODE_READ);
		}
		});
		view.addView(button);
		@@ -178,20 +182,45 @@ public class TestActivity extends Activity {

		@Override
		protected void onActivityResult(int requestCode, int resultCode, Intent data) {
		mResult.setText("resultCode=" + resultCode + ", data=" + String.valueOf(data));
		mResult.setText(null);
		String result = "resultCode=" + resultCode + ", data=" + String.valueOf(data);

		if (requestCode == CODE_READ) {
		final Uri uri = data != null ? data.getData() : null;
		if (uri != null) {
		InputStream is = null;
		try {
		is = getContentResolver().openInputStream(uri);
		final int length = Streams.readFullyNoClose(is).length;
		Log.d(TAG, "read length=" + length);
		result += "; read length=" + length;
		} catch (Exception e) {
		result += "; ERROR";
		Log.w(TAG, "Failed to read " + uri, e);
		} finally {
		IoUtils.closeQuietly(is);
		}
		} else {
		result += "no uri?";
		}
		} else if (requestCode == CODE_WRITE) {
		final Uri uri = data != null ? data.getData() : null;
		if (uri != null) {
		OutputStream os = null;
		try {
		os = getContentResolver().openOutputStream(uri);
		os.write("THE COMPLETE WORKS OF SHAKESPEARE".getBytes());
		} catch (Exception e) {
		result += "; ERROR";
		Log.w(TAG, "Failed to write " + uri, e);
		} finally {
		IoUtils.closeQuietly(os);
		}
		} else {
		result += "no uri?";
		}
		}

		Log.d(TAG, result);
		mResult.setText(result);
		}
		}

src/com/android/documentsui/model/RootInfo.java

+7 −0

Original line number	Diff line number	Diff line
		@@ -52,6 +52,13 @@ public class RootInfo {
		root.summary = getCursorString(cursor, Root.COLUMN_SUMMARY);
		root.documentId = getCursorString(cursor, Root.COLUMN_DOCUMENT_ID);
		root.availableBytes = getCursorLong(cursor, Root.COLUMN_AVAILABLE_BYTES);

		// TODO: remove this hack
		if ("com.google.android.apps.docs.storage".equals(root.authority)) {
		root.flags &= ~(Root.FLAG_PROVIDES_AUDIO \| Root.FLAG_PROVIDES_IMAGES
		\| Root.FLAG_PROVIDES_VIDEO);
		}

		return root;
		}