Android: adds test intents for memory pressure.

Android: adds test intents for memory pressure.

Allows using:
adb shell am start -a org.chromium.content_shell.action.ACTION_LOW_MEMORY \
-n org.chromium.content_shell_apk/.ContentShellActivity

to simulate memory pressure signals.

BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207960

[bulach, 2013-06-17 17:28:26]

ptal

[rmcilroy, 2013-06-17 18:22:43]

https://codereview.chromium.org/17108012/diff/1/base/android/java/src/org/chr...
File base/android/java/src/org/chromium/base/MemoryPressureListener.java
(right):

https://codereview.chromium.org/17108012/diff/1/base/android/java/src/org/chr...
base/android/java/src/org/chromium/base/MemoryPressureListener.java:19: public
class MemoryPressureListener {
Does this need to be public?

https://codereview.chromium.org/17108012/diff/1/chrome/android/testshell/java...
File
chrome/android/testshell/java/src/org/chromium/chrome/testshell/ChromiumTestShellActivity.java
(right):

https://codereview.chromium.org/17108012/diff/1/chrome/android/testshell/java...
chrome/android/testshell/java/src/org/chromium/chrome/testshell/ChromiumTestShellActivity.java:36:
* Sending an intent with this action will cause it to send a memory pressure
signal
replace "cause it to send a" with "simulate a" (similarly below and in
ContentShell)

https://codereview.chromium.org/17108012/diff/1/chrome/android/testshell/java...
chrome/android/testshell/java/src/org/chromium/chrome/testshell/ChromiumTestShellActivity.java:114:
MemoryPressureListener.simulateMemoryPressureSignal(TRIM_MEMORY_MODERATE);
return here, or put getUrlFromIntent in an "else {" branch?

[bulach, 2013-06-18 09:08:55]

thanks ross! all addressed, ptal

https://codereview.chromium.org/17108012/diff/1/base/android/java/src/org/chr...
File base/android/java/src/org/chromium/base/MemoryPressureListener.java
(right):

https://codereview.chromium.org/17108012/diff/1/base/android/java/src/org/chr...
base/android/java/src/org/chromium/base/MemoryPressureListener.java:19: public
class MemoryPressureListener {
On 2013/06/17 18:22:43, Ross Mcilroy wrote:
> Does this need to be public?

yep, unfortunately it'll be used by packages outside base.

https://codereview.chromium.org/17108012/diff/1/chrome/android/testshell/java...
File
chrome/android/testshell/java/src/org/chromium/chrome/testshell/ChromiumTestShellActivity.java
(right):

https://codereview.chromium.org/17108012/diff/1/chrome/android/testshell/java...
chrome/android/testshell/java/src/org/chromium/chrome/testshell/ChromiumTestShellActivity.java:36:
* Sending an intent with this action will cause it to send a memory pressure
signal
On 2013/06/17 18:22:43, Ross Mcilroy wrote:
> replace "cause it to send a" with "simulate a" (similarly below and in
> ContentShell)

Done.

https://codereview.chromium.org/17108012/diff/1/chrome/android/testshell/java...
chrome/android/testshell/java/src/org/chromium/chrome/testshell/ChromiumTestShellActivity.java:114:
MemoryPressureListener.simulateMemoryPressureSignal(TRIM_MEMORY_MODERATE);
On 2013/06/17 18:22:43, Ross Mcilroy wrote:
> return here, or put getUrlFromIntent in an "else {" branch?

Done.

[rmcilroy, 2013-06-18 11:14:58]

lgtm

Thanks Marcus.

[bulach, 2013-06-19 09:37:25]

+yfriedman for content/shell/android OWNERS

[bulach, 2013-06-21 13:36:24]

ping? :)

[Yaron, 2013-06-21 17:01:25]

lgtm

Sorry didn't see this before

[commit-bot: I haz the power, 2013-06-21 17:46:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/17108012/7001

[commit-bot: I haz the power, 2013-06-21 23:50:16]

Change committed as 207960