Android: Get the cache directory using path service.

Android: Get the cache directory using path service.

"/data/local" is not writable on non-rooted devices and
while running tests in an APK.

BUG=125059
TEST=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=138981

[John Grabowski, 2012-05-24 21:48:17]

http://codereview.chromium.org/10443021/diff/1/ipc/ipc_channel_posix_unittest.cc
File ipc/ipc_channel_posix_unittest.cc (right):

http://codereview.chromium.org/10443021/diff/1/ipc/ipc_channel_posix_unittest...
ipc/ipc_channel_posix_unittest.cc:101: static std::string
GetConnectionSocketName();
these should return 'const std::string&'

[nilesh, 2012-05-24 22:27:51]

http://codereview.chromium.org/10443021/diff/1/ipc/ipc_channel_posix_unittest.cc
File ipc/ipc_channel_posix_unittest.cc (right):

http://codereview.chromium.org/10443021/diff/1/ipc/ipc_channel_posix_unittest...
ipc/ipc_channel_posix_unittest.cc:101: static std::string
GetConnectionSocketName();
On 2012/05/24 21:48:18, John Grabowski wrote:
> these should return 'const std::string&'

They are not returning a reference to a member variable. These static methods
create the string when they are called.

[dmac, 2012-05-24 22:46:30]

lgtm

[John Grabowski, 2012-05-25 01:12:59]

On 2012/05/24 22:46:30, dmac wrote:
> lgtm

ok then; LGTM

[commit-bot: I haz the power, 2012-05-25 01:21:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nileshagrawal@chromium.org/10443021/8002

[commit-bot: I haz the power, 2012-05-25 02:56:30]

Change committed as 138981