Refactor MemMap::MapAnonymous().

Remove the address argument from the shortcut overload and
introduce one more shortcut overload. This makes it easier
to find all uses where we pass non-null address hint.

Remove `requested_begin` parameter from some constructors
where we were always passing null. Rewrite some tests to
use the reservation API.

Test: m test-art-host-gtest
Test: testrunner.py --host --optimizing
Bug: 118408378
Change-Id: Ibbbb96667e7cc11cf7fea119892463d8dbc9a8b5

1 files changed, 2 insertions, 0 deletions

diff --git a/runtime/gc/heap_test.cc b/runtime/gc/heap_test.cc
index a133a1058c..606228cf36 100644
--- a/runtime/gc/heap_test.cc
+++ b/runtime/gc/heap_test.cc
@@ -38,6 +38,8 @@ class HeapTest : public CommonRuntimeTest {
                                      16 * KB,
                                      PROT_READ,
                                      /*low_4gb=*/ true,
+                                     /*reuse=*/ false,
+                                     /*reservation=*/ nullptr,
                                      &error_msg);
     ASSERT_TRUE(reserved_.IsValid()) << error_msg;
     CommonRuntimeTest::SetUp();