)]}' { "log": [ { "commit": "32fd18324a06cbe20b8f5cafcb995f84e37e1be9", "tree": "b8ac3513f5d5c049aa7f050fd4a4f59bf6d5a36a", "parents": [ "40c113e567c8fc516e83ccf19c4ce471d4cb5459" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Nov 30 16:33:28 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Nov 30 16:34:01 2018 +0000" }, "message": "hiddenapi: Fix typo\n\nTest: n/a\nChange-Id: I40c2732310c9d8464f01c146debf359fb253ff95\n" }, { "commit": "69af30e3626fd65c4bd1ce3b89fe8cd2ff271caa", "tree": "4c6daa7c6ef506214dd858e6b559f8fd70de7941", "parents": [ "6f2d7b997ed8f2f5f952f5115fbf06c31d8ea7da" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Nov 30 15:42:02 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Nov 30 15:55:07 2018 +0000" }, "message": "hiddenapi: Turn CHECK into a warning\n\nWe create a dummy hiddenapi-flags.csv file for ART buildbots but\nRecent change to `hiddenapi` started enforcing that every dex member\nhas a corresponding CSV entry. Turn the CHECK into a LOG(WARNING) for\nthe time being to unblock the bots.\n\nTest: rely on the ART Buildbot\nChange-Id: I02565a0f11039dc28d395872d99adc7067d7e426\n" }, { "commit": "91690d37ad9d947e4e72b5622b9fbf6a9391274d", "tree": "729fac9e3c27017b8199bcd509c5f9bfd2de3473", "parents": [ "f2970cd870948a6ee1c8ecd30c9c3147d05aa0be" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Sun Nov 04 18:07:23 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Nov 29 16:42:10 2018 +0000" }, "message": "hiddenapi: Convert API lists to a flags CSV file\n\nAccess flags for hiddenapi have been managed as a series of test files,\none file per flag. This requries too many changes every time a flag is\nchanged or a new flag is introduced. Change `hiddenapi` and `veridex`\nto expect flags as a CSV file in the format:\n\n \u003capi signature\u003e,\u003cflag1\u003e,...,\u003cflagN\u003e\n\nTest: m, phone boots\nTest: m test-art\nTest: m appcompat\nChange-Id: Iffa64b36ffc5524779d5f57c3c6f0c0e84bfc606\n" }, { "commit": "9c8957dd84679fa2ea32cbfe80d7dbe2f4a490fc", "tree": "d1b592788516838a264925026ef672b0c7617095", "parents": [ "23b729e17ed437d19583f70e63d1fc8a85b5dae4" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Nov 27 22:24:57 2018 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Nov 28 09:29:51 2018 -0800" }, "message": "Fix \"performance-for-range-copy\" error.\n\nNot sure why this only shows up with -std\u003dc++17, but it\u0027s clearly\ncorrect:\n\n art/tools/hiddenapi/hiddenapi.cc:1063:45: error: the loop variable\u0027s type is not a reference type; this creates a copy in each iteration; consider making this a reference [performance-for-range-copy,-warnings-as-errors]\n\nBug: http://b/111067277\nTest: builds\nChange-Id: I3796d275b142db3de40e7a855a8130ecea6212ae\n" }, { "commit": "dcfa89bfc06a6c211bbb64fa81313eaf6454ab67", "tree": "4aaf6da88861029e387525957bf68796cf95648c", "parents": [ "fd667d4165aefe51010e4fe3bd81b820ed9884ef" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Oct 31 11:04:10 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Nov 22 12:49:26 2018 +0000" }, "message": "Rename and reorganize hiddenapi::ApiList\n\nChange values of ApiList flags to make them easier to extend in the\nfuture and unify naming across all components. Light greylist is now\njust \"Greylist\", dark greylist becomes \"GreylistMaxO\".\n\nNote that the version code in \"GreylistMaxO\" must also include any\nmaintenance releases, i.e. entries on \"GreylistMaxO\" are accessible\nto apps with targetSdkVersion\u003c\u003d27 (O MR1).\n\nTest: m, phone boots\nTest: m test-art\nChange-Id: I9622e0646eb265008a8bb2652270876ae95dac84\n" }, { "commit": "2a487eb89b67d7ff030368e4e4a6ed0a3ac39003", "tree": "bedffc5f4ee16d07c957d06cdab5148c69bd6a56", "parents": [ "875b4f26517ce215342746f07e41cda4c1991237" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 19 11:41:22 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 20 13:04:36 2018 -0800" }, "message": "ART: Fix performance-inefficient-vector-operation\n\nBug: 32619234\nTest: m test-art-host\nChange-Id: I286e27424ac39c0fed308811dc3004e734c0560c\n" }, { "commit": "976b01f62ad8fa2e4a25f64c152839eb37552f9e", "tree": "5b6206e8a069f5b824e7c96582188f1601a4afa0", "parents": [ "fe59955fc41e277bf1c60378202ba785abb1e4a8" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Nov 12 10:46:14 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Nov 12 10:49:29 2018 +0000" }, "message": "hiddenapi: Initialize unused value in DexFile::MapItem\n\nThe build tool adds a new MapItem into the MapList of a dex file\nwhich points to the new hiddenapi flags section. However, hiddenapi\nwould not initialize the `unused_` field in MapItem and therefore\nresult in non-deterministic builds.\n\nBug: 119308882\nTest: invoke hiddenapi twice of the same dex, diff the two files\nChange-Id: I00d0dc909a3424f3eb2776cf87719c074f264e74\n" }, { "commit": "47cd272d15f41109b3dacb21cfa509d18a03e011", "tree": "f2693adde283631362b3acc2c573d7280632621f", "parents": [ "6e32b0059b4c3073c601018da3ca315ad568525a" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Oct 23 12:50:02 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Oct 30 11:04:29 2018 +0000" }, "message": "Remove HiddenApiAccessFlags, move content to hiddenapi::\n\nHiddenapi code in runtime/ has all code in the hiddenapi:: namespace\ninstead of using a class as a wrapper. Refactor HiddenApiAccessFlags\nfor consistency. Also turn ApiList into `enum class` for stricter\ntype checks.\n\nTest: m test-art\nChange-Id: Ifb3c443ea43860476abd4fd3d4934cd14e2cdcc1\n" }, { "commit": "20c765f645fa9be77e045463c5064d41211a2815", "tree": "dcb9dfb6bf5363d1e933589f22339dc999b96703", "parents": [ "ce2a00daa92670a4fc01ef59fdbc3769a846f69c" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Sat Oct 27 21:45:15 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Sun Oct 28 20:00:03 2018 +0000" }, "message": "Revert^4: Add dex item for hiddenapi flags\n\nMove hiddenapi access flags to own data section so as to:\n(a) increase amount of information stored per method/field\n(b) use encoding which can be supported long-term.\n\nThe dex item is implemented as:\n - array of offsets indexed by class def index\n - streams of uleb-128 encoded flags.\nOffsets in array point to the beginning of the flags stream\nfor the given class def. Flags are encoded in the same order\nas fields and methods are encoded in class data. Zero offset\nmeans that the class either does not have class data, or all\nof its flags are zero.\n\nThe patch updates:\n(a) libdexfile with data structure declarations and accessor\n methods,\n(b) hiddenapi tool to create the new item from hiddenapi lists\n and insert it into the given dex file,\n(c) dexlayout to copy the flags into compact dex,\n(d) dex file verifier to verify the item.\nIt also removes skipping of verification for boot class path\ndex files as those now pass DexFileVerifier, and removes the\nneed for removing the flags for JVMTI.\n\nThe size increase is 450 KB.\n\nThis reverts commit d33d318685ec4a1c9e7995c914c104ab6487513b.\n\nChange-Id: Id00e0efb38ee1eab8d7ed5c645a7778b6b94b849\nTest: phone boots\nTest: m test-art\n" }, { "commit": "d33d318685ec4a1c9e7995c914c104ab6487513b", "tree": "3bb8a93f499ac537d445e90533598571f3d2d120", "parents": [ "fe3a979f6214e2ea94a9aed4c573c6a6f64c7096" ], "author": { "name": "Luman Qu", "email": "quluman@google.com", "time": "Fri Oct 26 16:53:25 2018 +0000" }, "committer": { "name": "Luman Qu", "email": "quluman@google.com", "time": "Fri Oct 26 16:53:25 2018 +0000" }, "message": "Revert \"Revert^2: Add dex item for hiddenapi flags\"\n\nThis reverts commit fe3a979f6214e2ea94a9aed4c573c6a6f64c7096.\n\nReason for revert: Caught by droidcop https://b.corp.google.com/issues/118481351\n\nChange-Id: I17cc3586c9ae3b49839633d9543edce2937c388d\n" }, { "commit": "fe3a979f6214e2ea94a9aed4c573c6a6f64c7096", "tree": "b31c1748dcd30a5bf76c2186b4c0124fb18b9dd6", "parents": [ "d9a0437b0d6cb1438bf21284a2502cbfe293db06" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Oct 25 17:24:38 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Oct 25 17:25:17 2018 +0000" }, "message": "Revert^2: Add dex item for hiddenapi flags\n\nMove hiddenapi access flags to own data section so as to:\n(a) increase amount of information stored per method/field\n(b) use encoding which can be supported long-term.\n\nThe dex item is implemented as:\n - array of offsets indexed by class def index\n - streams of uleb-128 encoded flags.\nOffsets in array point to the beginning of the flags stream\nfor the given class def. Flags are encoded in the same order\nas fields and methods are encoded in class data. Zero offset\nmeans that the class either does not have class data, or all\nof its flags are zero.\n\nThe patch updates:\n(a) libdexfile with data structure declarations and accessor\n methods,\n(b) hiddenapi tool to create the new item from hiddenapi lists\n and insert it into the given dex file,\n(c) dexlayout to copy the flags into compact dex,\n(d) dex file verifier to verify the item.\nIt also removes skipping of verification for boot class path\ndex files as those now pass DexFileVerifier, and removes the\nneed for removing the flags for JVMTI.\n\nThe size increase is 450 KB.\n\nThis reverts commit d9a0437b0d6cb1438bf21284a2502cbfe293db06.\n\nTest: phone boots\nTest: m test-art\nChange-Id: I224bf1b587529f36ea00a666680c4e4c8ca16a42\n" }, { "commit": "d9a0437b0d6cb1438bf21284a2502cbfe293db06", "tree": "3bb8a93f499ac537d445e90533598571f3d2d120", "parents": [ "fd2aa2bf72563870c9261f248f992a20ac4837a8" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Oct 24 12:34:06 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Oct 24 12:37:49 2018 +0000" }, "message": "Revert \"Add dex item for hiddenapi flags\"\n\nThis reverts commit 0dbc363f32d075017e1c4fb5e17715e3f12d0157.\n\nReason for revert: marlin running out of space\n\nChange-Id: I80218af6408812b639b7c807bf8cb2a1d5239d94\n" }, { "commit": "0dbc363f32d075017e1c4fb5e17715e3f12d0157", "tree": "950dfed748f63863db6b95d7b854940171b544c6", "parents": [ "62ada4cd2c9a7ef5d1ab4c538d660f8a6a7f7571" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Sep 10 09:14:30 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Oct 23 14:51:01 2018 +0100" }, "message": "Add dex item for hiddenapi flags\n\nMove hiddenapi access flags to own data section so as to:\n(a) increase amount of information stored per method/field\n(b) use encoding which can be supported long-term.\n\nThe dex item is implemented as:\n - array of offsets indexed by class def index\n - streams of uleb-128 encoded flags.\nOffsets in array point to the beginning of the flags stream\nfor the given class def. Flags are encoded in the same order\nas fields and methods are encoded in class data. Zero offset\nmeans that the class either does not have class data, or all\nof its flags are zero.\n\nThe patch updates:\n(a) libdexfile with data structure declarations and accessor\n methods,\n(b) hiddenapi tool to create the new item from hiddenapi lists\n and insert it into the given dex file,\n(c) dexlayout to copy the flags into compact dex,\n(d) dex file verifier to verify the item.\nIt also removes skipping of verification for boot class path\ndex files as those now pass DexFileVerifier, and removes the\nneed for removing the flags for JVMTI.\n\nThe size increase is 450 KB.\n\nTest: phone boots\nTest: m test-art\nChange-Id: Idec301db540cf164fccc97136d1df4abb8f758bd\n" }, { "commit": "9b031f7e4834b263ae531409a9f6c82cfdc89477", "tree": "4b5fdfc3ecc12b5b7e32959bec0c26595b87d2ab", "parents": [ "c71f38bdf7dca39a070ba25d436edcd758551f42" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 04 11:03:34 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 09 14:38:45 2018 -0700" }, "message": "ART: Refactor for bugprone-argument-comment\n\nHandles smaller subdirectories.\n\nBug: 116054210\nTest: WITH_TIDY\u003d1 mmma art\nChange-Id: I81c8f3396b9922684b68e9f3d8fccefe364f1279\n" }, { "commit": "bbc6e7edb5fca4a61ac53dd9bce79cb4f0bb3403", "tree": "0fbce767bc383358cf4cd65aafc74140e1850982", "parents": [ "19379b58bd433da91230e4fe6cd96e7416d16adc" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Aug 24 16:58:47 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 28 11:06:07 2018 +0100" }, "message": "Use \u0027final\u0027 and \u0027override\u0027 specifiers directly in ART.\n\nRemove all uses of macros \u0027FINAL\u0027 and \u0027OVERRIDE\u0027 and replace them with\n\u0027final\u0027 and \u0027override\u0027 specifiers. Remove all definitions of these\nmacros as well, which were located in these files:\n- libartbase/base/macros.h\n- test/913-heaps/heaps.cc\n- test/ti-agent/ti_macros.h\n\nART is now using C++14; the \u0027final\u0027 and \u0027override\u0027 specifiers have\nbeen introduced in C++11.\n\nTest: mmma art\nChange-Id: I256c7758155a71a2940ef2574925a44076feeebf\n" }, { "commit": "396dc08833659c78409567b28f7bb25e9254fbb0", "tree": "22e8989313c1040e47e010077d6d144ab08c34f1", "parents": [ "0d3f480343f00d971ec55b197b0825a540777e32" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Aug 06 12:29:57 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Aug 07 13:44:46 2018 -0700" }, "message": "Move hiddenapi to ClassAccessor\n\nTest: test-art-host\nBug: 79758018\n\nChange-Id: Ib0848cc69ffaa7735f43d88e9719cc43953e4a93\n" }, { "commit": "345c0ed11aff80cc7ab13005f8b48914cfcf4578", "tree": "49cc9a89e34afe207769e9ce40c6020c6be3cebc", "parents": [ "98afa11c3cd8517bd28d1cad7aacaf0179c905f0" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Aug 03 10:26:44 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Aug 06 18:14:35 2018 +0100" }, "message": "hiddenapi: Handle visibility through inheritance\n\nFramework uses a pattern in which a public method is defined in\na package-private parent class, but exposed outside of package\nvia a public child class. Improve the definition of stub member\nvisibility in `hiddenapi` to understand this pattern.\n\nBug: 76424618\nTest: m appcompat, SpannableStringInternal.length() public\nChange-Id: Ie9c1653142a4991d6de5460be5abd074aa03b0a0\n" }, { "commit": "da821c9046ded3050cd322b515bc9687af0cfd7f", "tree": "a524e9032b385179352477ea138553dfb382529a", "parents": [ "c06bfc9b18e6421ac60e7b27f8f1e90b45772f52" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jul 12 11:13:15 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jul 12 11:13:15 2018 +0100" }, "message": "hiddenapi: Fix map insertion bug\n\n`hiddenapi` would use std::map::insert to overwrite an entry, which\nis wrong and will not overwrite the entry. This caused all entries\nto be classified as private. Fix the bug.\n\nBug: 79409988\nTest: m appcompat, check hiddenapi-public-list.txt not empty\nChange-Id: Ie1c1fad2dfff9ff0332b5196d605d512d6dedc15\n" }, { "commit": "0b6de0c0de6c760a6bdd45a37e92055f662ad4ff", "tree": "e6b1398d87f31392a0dc1ff99c1ba10efbbb782e", "parents": [ "2258c2ef5f6cb25ff12a1dc6dfac1f868892c226" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jun 28 11:56:41 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Jul 03 17:21:53 2018 +0100" }, "message": "hiddenapi: Add \u0027list\u0027 command to generate public/private API lists\n\nIn order to make decisions about non-SDK API restrictions, we need\na complete list of public and private class members in the boot\nclass path. This was previously done by Doclava but having to lower\nits Java view of the framework to dex level has proven cumbersome\nand error prone.\n\nThis patch adds a new command to the `hiddenapi` build tool which\nbuilds the class hierarchy of boot class path and attempts to resolve\neach member of android.jar against it. Resolved members are dumped\ninto a public API text file, the rest into a private API text file.\n\nNote that the resolution is not strictly the same as in ART and we\nmay err on the side of caution. This should be revisited to comply\nwith the spec.\n\nBug: 79409988\nTest: m out/target/common/obj/PACKAGING/hiddenapi-private-list.txt\nChange-Id: Ia82bcaad9347344aacf8dc6f7b093f769cd853ec\n" }, { "commit": "003e64b4108125d6f59bc1b663fd71864abadab9", "tree": "488f963dc86f905245df46a6ba99e534b1666edd", "parents": [ "d20a4d76c33cd6e609ad6b1b3cde09fdcbdde05c" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Jun 27 13:20:52 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jun 28 19:43:13 2018 +0100" }, "message": "hiddenapi: Refactor before new features are added\n\nRefactor the hiddenapi build tool to take a command name as its\nfirst argument and create a notion of a \"class path\" group of dex\nfiles. These changes will be needed for creating a new command\nthat resolves SDK stubs against the boot class path.\n\nBug: 79409988\nTest: m hiddenapi\nTest: m test-art-host-gtest-hiddenapi_test\nTest: art/test.py -r -t 674-hiddenapi -t 999-redefine-hiddenapi\nChange-Id: I93c36154cc8c5e8c0e9414cf02e2c6ea298ae0e8\n" }, { "commit": "1f010164bea4714e9f653c52e8948c5d5305040b", "tree": "e69b2c3f4d2645d4097655f8914325945fe5dbbd", "parents": [ "a7d37cf735f05cfc3a3585a5c44eb65d8a279874" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue May 15 08:59:32 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed May 16 13:01:34 2018 -0700" }, "message": "Move ArtDexFileLoader to libdexfile\n\nAdds a dependency from libdexfile on libartbase, but allows the use of\nMemMap loaders, etc.\n\nBug: 78652467\nTest: make -j 40 checkbuild\nChange-Id: I15cf33893ca9192050762f8350a3cc1e39f88dc5\n" }, { "commit": "79e2607ab50163bfdc283f4a49decec26a216df5", "tree": "0cdb2c65412b64a55ddf1a486ab3d961a0301a3e", "parents": [ "b4b91460b2d01d36ddcb7d55111ccd8faf390c2a" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Apr 06 17:58:50 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Apr 13 08:29:30 2018 -0700" }, "message": "Move profile dependent modules to libartbase\n\nMove mem_map and zip_archive to libartbase. This should be the last two\nremaining modules that profile_compilation_info is dependent upon.\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make and boot a device\n\nChange-Id: I136ee23e426aa8ec7441e3d3f1978f1bebf4b562\n" }, { "commit": "cebbb65c19a511e76c5024b9e6713bf74be3f7c3", "tree": "e984d49e422bf90b3453457a6077b98f60b60022", "parents": [ "722093c305de5646759318fdeedc022c08a6723c" ], "author": { "name": "Mathew Inwood", "email": "mathewi@google.com", "time": "Mon Mar 19 15:15:06 2018 +0000" }, "committer": { "name": "Mathew Inwood", "email": "mathewi@google.com", "time": "Mon Mar 19 15:15:06 2018 +0000" }, "message": "Fix find_api_violations script.\n\n- The log tag seems to have changed, no longer zygote[64]\n- Add option to output just method signatures, for adding to the light\n greylist.\n\nTest: $ find_api_violations.pl -s -b --nolightgrey \u003c bugreport.txt\nChange-Id: I31fd21c0d77e23188c010a412db41b4bcdc53ee8\n" }, { "commit": "5ebc4cc1489a1992fbee969e05478a474c2ec111", "tree": "b43b248304741e6e6c8590f512f57d2678cf93e5", "parents": [ "d395e73e9ed7c3fb5e8a48c3f3141a8997d4a82b" ], "author": { "name": "Mathew Inwood", "email": "mathewi@google.com", "time": "Thu Mar 08 09:47:46 2018 +0000" }, "committer": { "name": "Mathew Inwood", "email": "mathewi@google.com", "time": "Fri Mar 09 11:32:31 2018 +0000" }, "message": "Quick script to get API access violations.\n\nTest: $ adb logcat | perl find_api_violations.pl\n\n$ perl find_api_violations.pl --help\n\nDESCRIPTION\n This script parses API violations from \"adb logcat\". Output is in CSV\n format with columns \"package\", \"symbol\", \"list\".\n\n The package name is mapped from a PID, parsed from the same log. To ensure\n you get all packages names, you should process the logcat from device boot\n time.\n\nSYNOPSIS\n adb logcat | perl find_api_violations.pl \u003e violations.csv\n cat bugreport.txt | perl find_api_violations.pl --bugreport \u003e violations.csv\n\nOPTIONS\n --[no]lightgrey\n (Don\u0027t) show light grey list accesses (default true)\n\n --[no]darkgrey\n (Don\u0027t) show dark grey list accesses (default true)\n\n --[no]black\n (Don\u0027t) show black list accesses (default true)\n\n --bugreport|-b\n Process a bugreport, rather than raw logcat\n\n --help\n\nBug: 64382372\nChange-Id: Id4baf10412b016a4bfd80bd3bd25cf9ca437601c\n" }, { "commit": "c431b9dc4b23cc950eb313695258df5d89f53b22", "tree": "422273559c3ae52caff0c6b1cf1a62a8312f0e26", "parents": [ "f46f46cf5bd32788d5252b7107628a66594a5e98" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Mar 02 12:01:51 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Mar 05 13:58:20 2018 -0800" }, "message": "Move most of runtime/base to libartbase/base\n\nEnforce the layering that code in runtime/base should not depend on\nruntime by separating it into libartbase. Some of the code in\nruntime/base depends on the Runtime class, so it cannot be moved yet.\nAlso, some of the tests depend on CommonRuntimeTest, which itself needs\nto be factored (in a subsequent CL).\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make -j 50 test-art-host\n\nChange-Id: I8b096c1e2542f829eb456b4b057c71421b77d7e2\n" }, { "commit": "67bf42e89592c3a1c648f927f2ce3ccb189a1161", "tree": "054d5b7adf7cc62d4d2a2118a70c0fbdd1751610", "parents": [ "d961043ff1dd6fddb68aa90c1f939cfafec24219" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Feb 26 16:43:04 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Feb 28 15:08:21 2018 -0800" }, "message": "Header library to remove dependence on runtime/\n\nAdd a new header library to remove libdexfile and others\u0027 dependence on\nruntime (typically runtime/base) includes in libdexfile. Also a small step\nto tease dexlayout and profman away from relying on these as well.\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make -j 50 test-art-host-gtest\n\nChange-Id: I38e2fe399a75f4bc6318c77a71954c00ea73ec2b\n" }, { "commit": "fcbe15ce26930c0c4bb33998fc2abbe4f8770dbe", "tree": "6513a2eaf1251e93578916b5171dc42984de7da3", "parents": [ "b40fa7c33075292beeb6840ac679ffd08fd1f719" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Feb 15 09:41:13 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Feb 15 15:15:13 2018 -0800" }, "message": "Build debug library libdexfiled\n\nAdd a debug version of libdexfile for use when CHECKs, etc., are\nenabled.\n\nBug: 22322814\nTest: make -j 50 checkbuild\nChange-Id: I263138611bcdde9dbc8f619bcb2a0a6d3a09fbf6\n" }, { "commit": "0225f8e2939a9340cb7dcebfcfe7996a2bd9bce9", "tree": "fd32362dff4ecdc8ff8b0d800a1b46fb0d9d0104", "parents": [ "c7e546ff3963a1d51b1f100d308db735bd19f736" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jan 31 08:52:24 2018 +0000" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jan 31 01:24:53 2018 -0800" }, "message": "Revert \"Revert \"Make libdexfile build independent of runtime dir\"\"\n\nThis reverts commit 787784f9effb126b5d0d3dc97d544c4a477b5daf.\n\nReason for revert: Bot configuration issue.\n\nChange-Id: I6a10bb4a9571f89c7e4dd095f9157e830a44e2de\nBug: 22322814\nTest: make -j 50 checkbuild\n" }, { "commit": "787784f9effb126b5d0d3dc97d544c4a477b5daf", "tree": "1ff6d9fd284a4dd2a337165dff20e71137997c9d", "parents": [ "b40b7e73469339a6b667b4a2e2b8690112a74dc9" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 18:48:51 2018 +0000" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 18:48:51 2018 +0000" }, "message": "Revert \"Make libdexfile build independent of runtime dir\"\n\nThis reverts commit b40b7e73469339a6b667b4a2e2b8690112a74dc9.\n\nReason for revert: on device libdexfile.so missing\n\nChange-Id: I9bd61a98bef870400580e8c991cb061d3f57fa72\n" }, { "commit": "b40b7e73469339a6b667b4a2e2b8690112a74dc9", "tree": "7e8a34d9c7894c4c946f674f19f0a0a512b95184", "parents": [ "9690ad794b324ba54e936608881ac0f62538b97a" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jan 25 17:11:07 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 01:17:58 2018 -0800" }, "message": "Make libdexfile build independent of runtime dir\n\nRemove libdexfile\u0027s dependency on utils.cc and move utf.cc into\n/dex. Remove libdexfile\u0027s constituent sources from libart and\nuse libdexfile wherever libart is. Also remove some ART-specific\ninterfaces. Libdexfile\u0027s tests remain to be converted, plus\nmoving the files to a new directory peer to runtime/.\n\nBug: 22322814\nTest: make -j 50 test-art-host\nChange-Id: Ifaf695216e4a0e43d3aa377984d933f7a2a243c2\n" }, { "commit": "1a824a5c1070648971ce9593a1dd71cdd8cf2f98", "tree": "1ff6d9fd284a4dd2a337165dff20e71137997c9d", "parents": [ "0406e1e59970399393f53923704e1b9a828b2173" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jan 26 15:45:30 2018 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Jan 29 19:42:40 2018 +0000" }, "message": "Remove hiddenapi access flags in FixedUpDexFile\n\nThe hiddenapi tool will mess with the access flags of fields and\nmethods in order to record which are \u0027@hide\u0027. We need to undo this\nbefore passing any dex files down to jvmti agents.\n\nTest: ./test.py --host -j50\nBug: 72550707\nBug: 64382372\n\nChange-Id: Ibc9a96a6b541c06844f276db009ac29514f7a3bb\n" }, { "commit": "f6a8a557e0e3099a2c458a81a4b48623989330a5", "tree": "e82439e60546643d34b888fd79c04ae7b4d899c0", "parents": [ "1372c9772d37f43147d3791c998173811f04594c" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Jan 15 18:10:50 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jan 18 14:47:33 2018 +0000" }, "message": "Set runtime hidden API access flags\n\nThe hiddenapi tool encodes new access flags into boot class path\ndex files, but we do not want to use those at runtime. They are\ncumbersome to decode and interfere with other flags.\n\nThis patch introduces new runtime access flags kAccHiddenApiBits,\ntwo consecutive bits in the space occupied by intrinsic ordinals\nwhich are used to encode the four categories of APIs in\nHiddenApiAccessFlags::ApiList. ClassLinker converts one encoding\nto the other when a new method/field is loaded from DexFile.\n\nDexHiddenAccessFlags have been made an internal class of new\nHiddenApiAccessFlags class which contains all the code related\nto encoding/decoding these flags for both DexFile and runtime.\n\nTest: m test-art-host\nBug: 64382372\nChange-Id: Ie055dc3440c44e8815030e652f08d9ee3dba69e5\n" }, { "commit": "0baa7cc3f49091b9bf8dc344be3090810606cd18", "tree": "ed8d6a786dab1e9367ad05949d0d81d83c66ce4a", "parents": [ "4572ade694e35f19c3b2d2c68a573eb8e40a7ccb" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Jan 17 09:59:08 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Jan 17 09:59:08 2018 +0000" }, "message": "Make hiddenapi gtest host-only\n\nThe tool itself is meant to be used while building boot class path\ndex files. No point in running it on target.\n\nBug: 64382372\nTest: m test-art-target-gtest\nChange-Id: Ieb0a2f1dd22d035ed194abd5360925598c333616\n" }, { "commit": "4572ade694e35f19c3b2d2c68a573eb8e40a7ccb", "tree": "d4e8de7a0f170604288cd93e66a3c6dbdb0befe0", "parents": [ "34554c9a6aadd66845a25d1b23a64759f6692e32" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Jan 17 09:26:47 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Jan 17 09:41:28 2018 +0000" }, "message": "Fix memory leak in hiddenapi_test\n\nForgot to put an instance of ZipArchive into unique_ptr.\n\nBug: 64382372\nTest: m test-art-host-gtest-hiddenapi_test\nChange-Id: I3bed0a20e55018f4e904750e0f4aeded8f93c4cc\n" }, { "commit": "2b9c35be35a759ba2032692648f5bbcb1e7e78c8", "tree": "4825fdb4a4ecd27922dc9c945b30423534e67010", "parents": [ "6d4c343ee5db18f039aeb3e07ff8d3c1fd37c3a0" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Jan 12 15:44:43 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Jan 16 15:32:12 2018 +0000" }, "message": "Introduce new build tool: hiddenapi\n\nNew tool `hiddenapi` iterates over all class members inside given\nDEX files and modifies their access flags if their signatures\nappear on one of two lists - greylist and blacklist - provided as\ntext file inputs. These access flags denote to the runtime that\nthe marked methods/fields should be treated as internal APIs with\nrestricted access.\n\nTwo bits of information are encoded in the DEX access flags. These\nare encoded as unsigned LEB128 values in DEX and so as to not\nincrease the size of the DEX, different modifiers were chosen to\ncarry the information under different circumstances.\n\nFirst bit is encoded as the inversion of visibility access flags\n(bits 2:0). At most one of these flags can be set at any given time.\nInverting these bits therefore produces a value where at least two\nbits are set and there is never any loss of information.\n\nSecond bit is encoded differently for each given type of class\nmember as there is no single unused bit such that setting it would\nnot increase the size of the LEB128 encoding.\n - Bit 5 for fields as it carries no other meaning\n - Bit 5 for non-native methods, as `synchronized` can only be set\n on native methods\n - Bit 9 for native methods, as it carries no meaning and bit 8\n (native) will make the LEB128 encoding at least two bytes long\n\nThis tool is meant to be applied on boot class path DEX files and\nas such, this encoding is not part of the DEX specification and may\nchange in the future. Access flags returned by ClassDataItemIterator\nare stripped of these hidden flags and thus fully transparent to the\nruntime.\n\nTest: m test-art-host\nBug: 64382372\nChange-Id: Ifc237ff8a35a8b470b7fc682a9cb879370d1e6e9\n" } ] }