- 30 Jul, 2018 22 commits
-
-
Deepti Gandluri authored
Bug:v8:6532 Change-Id: Ie983fa561654f86597b8f45c5ce11f993846bfe6 Reviewed-on: https://chromium-review.googlesource.com/1145893 Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Reviewed-by:
Michael Starzinger <mstarzinger@chromium.org> Reviewed-by:
Ben Titzer <titzer@chromium.org> Cr-Commit-Position: refs/heads/master@{#54796}
-
Frank Tang authored
Bug: v8:7993,v8:7994 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: I058a2b8181b933dce9f71dab2b03304a424ae0ea Reviewed-on: https://chromium-review.googlesource.com/1154237 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
Daniel Ehrenberg <littledan@chromium.org> Reviewed-by:
Sathya Gunasekaran <gsathya@chromium.org> Cr-Commit-Position: refs/heads/master@{#54795}
-
Frank Tang authored
Bug: v8:7979 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: Ic30da6f85b49fd34ee58faf253d9e976a460153c Reviewed-on: https://chromium-review.googlesource.com/1150873Reviewed-by:
-
Frank Tang authored
Reflect spec change in https://github.com/tc39/proposal-intl-relative-time/pull/81 Bug: v8:7992 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: Ic2153f8355b8380d2986b34d46eb6bb6771d4b59 Reviewed-on: https://chromium-review.googlesource.com/1154236Reviewed-by:
-
Vasili Skurydzin authored
Change-Id: I991686d6890909510d2a3e7101230995ff521063 Reviewed-on: https://chromium-review.googlesource.com/1152081Reviewed-by:
Junliang Yan <jyan@ca.ibm.com> Commit-Queue: Junliang Yan <jyan@ca.ibm.com> Cr-Commit-Position: refs/heads/master@{#54792}
-
Michael Lippautz authored
Tbr: cbruni@chromium.org No-try: true Change-Id: Ica8c64e0956d4ca362646814e675104f94e77fac Reviewed-on: https://chromium-review.googlesource.com/1155585Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#54791}
-
Rodrigo Bruno authored
From the promotion code point of view, a ThinString being forwarded to an ExternalString is exactly the same as an actual ExternalString being forwarded to its promoted copy. These changes provide a way to disambiguate both scenarios since they are different for external memory accounting. Bug: chromium:867902 Change-Id: I6fd56ee5e0f8900318466108273ab26e936eb439 Reviewed-on: https://chromium-review.googlesource.com/1152975 Commit-Queue: Rodrigo Bruno <rfbpb@google.com> Reviewed-by:
-
Michael Starzinger authored
R=ahaas@chromium.org TEST=cctest/test-wasm-shared-engine Change-Id: I1f1b643089206805bc88e8898db6bf94093291b5 Reviewed-on: https://chromium-review.googlesource.com/1154916Reviewed-by:
Andreas Haas <ahaas@chromium.org> Commit-Queue: Michael Starzinger <mstarzinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#54789}
-
Michael Lippautz authored
The following are ready for iwyu: - src/heap/mark-compact.h - src/heap/objects-visiting.h Bug: v8:7490 Change-Id: I4cb9b1146586adcef8c0e0cf187ca363fe6e9a8d Reviewed-on: https://chromium-review.googlesource.com/1154970Reviewed-by:
-
Peter Marshall authored
Fixes: src/parsing/parse-info.h src/parsing/preparsed-scope-data.h src/profiler/heap-profiler.h src/profiler/heap-snapshot-generator.h src/profiler/heap-snapshot-generator-inl.h' src/property.h src/prototype.h src/prototype-inl.h Bug: v8:7754,v8:7490,v8:7965 Change-Id: Iba14292cdfbbaf29d5b989c16d579a1ee0edcc25 Reviewed-on: https://chromium-review.googlesource.com/1154967 Commit-Queue: Peter Marshall <petermarshall@chromium.org> Reviewed-by:
Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#54787}
-
jgruber authored
For technical reasons, we used to serialize off-heap trampolines with a RelocInfo containing a single OFF_HEAP_TARGET entry. This RelocInfo was never needed at runtime. It also ended up being somewhat misleading because printing an embedded code object would show the OFF_HEAP_TARGET entry (belonging to the trampoline) at the first 'real' instruction. With this CL, we explicitly serialize an empty byte array as the reloc info for each off-heap trampoline, i.e. the snapshot will never contain such off-heap target reloc infos. Bug: v8:6666, v8:7969 Change-Id: If6fa85a438d093ed5dcea07ce0de1db49a224d28 Reviewed-on: https://chromium-review.googlesource.com/1146643 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#54786}
-
Michael Lippautz authored
Bug: v8:7490 Change-Id: Id6e50a201732b4ca1fd1f69f8c43298e216ce95b Reviewed-on: https://chromium-review.googlesource.com/1154788Reviewed-by:
-
Marja Hölttä authored
BUG=v8:7308 Change-Id: Ibf87a807202d8f1e0e7c095bf6b3790b0f0954bf Reviewed-on: https://chromium-review.googlesource.com/1154907Reviewed-by:
-
Florian Sattler authored
AdvanceUntil allows the Utf16CharacterStream to advance until a charater is found that passes the check. Bug: v8:7926 Change-Id: Iae39fb24194aa0ee2f544a55a7847956aa324b64 Reviewed-on: https://chromium-review.googlesource.com/1151303 Commit-Queue: Florian Sattler <sattlerf@google.com> Reviewed-by:
Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/master@{#54783}
-
Jaroslav Sevcik authored
This CL introduces InstanceTypeChecker namespace with type checkers for types used by the optimizing compiler. The type checkers are global functions. Examples: bool InstanceTypeChecker::IsJSFunction(InstanceType); bool InstanceTypeChecker::IsAllocationSite(InstanceType); The CL also uses these type checkers in the HeapObject::IsXXXX() functions to ensure consistency. Bug: v8:7790 Change-Id: I7eb9e79ca28da02fd489201f532e566d0d13243a Reviewed-on: https://chromium-review.googlesource.com/1154529Reviewed-by:
Michael Stanton <mvstanton@chromium.org> Commit-Queue: Jaroslav Sevcik <jarin@chromium.org> Cr-Commit-Position: refs/heads/master@{#54782}
-
Michael Lippautz authored
Bug: v8:7490 Change-Id: I6bed10a6389d83b1941ba894f06b2d9540e84139 Reviewed-on: https://chromium-review.googlesource.com/1152733Reviewed-by:
-
Peter Marshall authored
Fixes: src/map-updater.h src/objects/property-descriptor-object.h src/objects/prototype-info-inl.h src/objects/regexp-match-info.h src/objects/shared-function-info-inl.h Bug: v8:7754,v8:7490,v8:7965 Change-Id: I7ae9dc86491c8e147d628f5fd8362534e861b15e Reviewed-on: https://chromium-review.googlesource.com/1154221Reviewed-by:
-
Peter Marshall authored
Bug: v8:7754,v8:7490,v8:7965 Change-Id: I574dbade6dc21204461be4d83b138f7341016305 Reviewed-on: https://chromium-review.googlesource.com/1152819Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Peter Marshall <petermarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#54779}
-
Marja Hölttä authored
This reverts commit 13401d2b. Reason for revert: gc stress breakage Original change's description: > [in-place weak refs] Use WeakArrayList in wasm > > Now we can remove FixedArrayOfWeakCells (this was the last user). > > BUG=v8:7308 > > Change-Id: I7b74f7833288b20d8f4d098a4afce640ea6db823 > Reviewed-on: https://chromium-review.googlesource.com/1150170 > Commit-Queue: Marja Hölttä <marja@chromium.org> > Reviewed-by: Michael Lippautz <mlippautz@chromium.org> > Reviewed-by: Andreas Haas <ahaas@chromium.org> > Cr-Commit-Position: refs/heads/master@{#54776} TBR=marja@chromium.org,mlippautz@chromium.org,ahaas@chromium.org Change-Id: I1bc1ad7e27813aeaaf191efdf74b865d676475f5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:7308 Reviewed-on: https://chromium-review.googlesource.com/1154787Reviewed-by:
-
jgruber authored
When disassembling code (in particular embedded builtins), try to print better information about root-relative accesses. For example: REX.W movq rdx,[r13+0x548] REX.W movq rax,[r13+0x10a8] turns into REX.W movq rdx,[r13+0x548] (root (0x1ff420d0ccd9 <FixedArray[1672]>)) REX.W movq rax,[r13+0x10a8] (external reference (check_object_type)) This is a band-aid solution until we come up with something better. It does not understand multi-instruction sequences (such as loads from the builtins constants table), assumes every kRootRegister-relative access is actually root-relative (i.e. the register is not initialized to some other value), and is limited to a particular instruction pattern. Bug: v8:6666,v8:7969 Change-Id: I35af92e8233c9bb0f2ad6ba0e86bd0ab69177205 Reviewed-on: https://chromium-review.googlesource.com/1146806 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
-
Marja Hölttä authored
Now we can remove FixedArrayOfWeakCells (this was the last user). BUG=v8:7308 Change-Id: I7b74f7833288b20d8f4d098a4afce640ea6db823 Reviewed-on: https://chromium-review.googlesource.com/1150170 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/b222950..b668f37 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/f30572c..e26243c TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: I782a848d130ee7bc6f8ecd4553dd21dcbedeafd6 Reviewed-on: https://chromium-review.googlesource.com/1154628Reviewed-by:
V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#54775}
-
- 29 Jul, 2018 2 commits
-
-
Jungshik Shin authored
This reverts commit c83fae06. Reason for revert: the CL that required the revert of this CL was relanded. https://chromium-review.googlesource.com/c/v8/v8/+/1154247 Original change's description: > Revert "[Intl] Move Number.toLocaleString from js to c++" > > This reverts commit a895f01a. > > Reason for revert: Needed for other revert: > https://chromium-review.googlesource.com/c/v8/v8/+/1152767 > > Original change's description: > > [Intl] Move Number.toLocaleString from js to c++ > > > > > > Bug: v8:7960 > > Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng > > Change-Id: I21a524b620e210cce625a1a00e68c0b687187087 > > Reviewed-on: https://chromium-review.googlesource.com/1144659 > > Commit-Queue: Frank Tang <ftang@chromium.org> > > Reviewed-by: Jungshik Shin <jshin@chromium.org> > > Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#54741} > > TBR=jshin@chromium.org,gsathya@chromium.org,bstell.chromium.org@gmail.com,ftang@chromium.org > > Change-Id: I060fa2834dde5e1b4cc71923cc066d97bce2a33b > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: v8:7960 > Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng > Reviewed-on: https://chromium-review.googlesource.com/1152787 > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Commit-Queue: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#54746} TBR=machenbach@chromium.org,jshin@chromium.org,gsathya@chromium.org,bstell.chromium.org@gmail.com,ftang@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:7960 Change-Id: I7a7a67db77b9b5a181f1751a03186eb7e8b271a0 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng Reviewed-on: https://chromium-review.googlesource.com/1154248 Commit-Queue: Jungshik Shin <jshin@chromium.org> Reviewed-by:
Jungshik Shin <jshin@chromium.org> Cr-Commit-Position: refs/heads/master@{#54774}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/79a29b1..b222950 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/1e61f60..e05f902 TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: I62a3628dd0b3ce06b48813111a194c00a55705dd Reviewed-on: https://chromium-review.googlesource.com/1154433 Commit-Queue: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Reviewed-by:
-
- 28 Jul, 2018 9 commits
-
-
Frank Tang authored
This is a reland of 51ad234f With a manual layout rebaseline of js/fast/string-prototype-properties [1], this CL can be relanded without breaking the layout test. [1] https://chromium-review.googlesource.com/c/chromium/src/+/1154289 also marks the test for skipping until this fix is rolled to Chromium. Original change's description: > [Intl] move localeCompare to C++ > > > Bug: v8:7958 > Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng > Change-Id: I84a27dda5205c8581a7ffe37213d685cc49974fa > Reviewed-on: https://chromium-review.googlesource.com/1144644 > Commit-Queue: Frank Tang <ftang@chromium.org> > Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> > Cr-Commit-Position: refs/heads/master@{#54740} TBR=ftang@chromium.org, gsathya@chromium.org Bug: v8:7958 Test: layout test: js/fast/string-prototype-properties Change-Id: Ic546349fcbc935917ded018801f7d942e50565d5 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng Reviewed-on: https://chromium-review.googlesource.com/1154247 Commit-Queue: Jungshik Shin <jshin@chromium.org> Reviewed-by:
-
Sergiy Byelozyorov authored
TBR=sergiyb@chromium.org No-Try: true Bug: chromium:838266 Change-Id: I898579f780e40ee0fb5fd1948a92a6c500242dfd Reviewed-on: https://chromium-review.googlesource.com/1154219Reviewed-by:
Sergiy Byelozyorov <sergiyb@chromium.org> Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Cr-Commit-Position: refs/heads/master@{#54771}
-
Sergiy Byelozyorov authored
TBR=sergiyb@chromium.org No-Try: true Bug: chromium:838266 Change-Id: I0c288eeaf28569d671220ab320aaec677825bb37 Reviewed-on: https://chromium-review.googlesource.com/1154218Reviewed-by:
-
Sergiy Byelozyorov authored
TBR=sergiyb@chromium.org No-Try: true Bug: chromium:842224 Change-Id: Icd00b7b4b86e1cc2de0234c69632eb5e6a63b11a Reviewed-on: https://chromium-review.googlesource.com/1154216Reviewed-by:
-
Matheus Marchini authored
The metadata introduced in this patch will be useful for postmortem tools to inspect Contexts and ScopeInfos (see https://github.com/nodejs/llnode/issues/211). R=bmeurer@google.com, yangguo@google.com Change-Id: I927fcab4014d128bd782046c1ecb9ee045723e95 Reviewed-on: https://chromium-review.googlesource.com/1153858Reviewed-by:
-
Sergiy Byelozyorov authored
TBR=sergiyb@chromium.org No-Try: true Bug: chromium:842224 Change-Id: Ic7c6093331f1bd35fffbfc98cd3f1d0ed3571eb4 Reviewed-on: https://chromium-review.googlesource.com/1154212 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by:
-
Sergiy Byelozyorov authored
TBR=sergiyb@chromium.org No-Try: true Bug: chromium:842224 Change-Id: I63c96969382fff550a64f80d35a7576d84092a0a Reviewed-on: https://chromium-review.googlesource.com/1154211 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/adaab11..79a29b1 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/933672b..1e61f60 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/079812c..d795ab8 Rolling v8/third_party/fuchsia-sdk: https://chromium.googlesource.com/chromium/src/third_party/fuchsia-sdk/+log/12961e5..e4f38ff TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: I483a9915a057d44369212720b0a2d69ebe1b0ca3 Reviewed-on: https://chromium-review.googlesource.com/1154199 Commit-Queue: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Reviewed-by:
-
Sergiy Byelozyorov authored
TBR=sergiyb@chromium.org No-Try: true Bug: chromium:838864 Change-Id: Ie0e6e1742344bfd69ec109f55fcccbaca0d66d66 Reviewed-on: https://chromium-review.googlesource.com/1154209 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by:
-
- 27 Jul, 2018 7 commits
-
-
Deepti Gandluri authored
Functions/variables for some atomic memory operators use type, representation interchangeably. Fix to make it consistent. Bug: v8:7754, v8:6532 Change-Id: I16ae35b72728739aee4bc67287317c2fd4a9088a Reviewed-on: https://chromium-review.googlesource.com/1152462Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/master@{#54763}
-
Frank Tang authored
Bug: v8:7979 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: Icf646f5b9888239acb100a4238bbccc288e93153 Reviewed-on: https://chromium-review.googlesource.com/1150918Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Frank Tang <ftang@chromium.org> Cr-Commit-Position: refs/heads/master@{#54762}
-
Jakob Kummerow authored
The ToBigInt conversion can have side effects, so the check for neutered-ness must happen afterwards. Bug: chromium:867776 Change-Id: I6e550c77a284da4cf132c21a6c3b1ed8f34eedc9 Reviewed-on: https://chromium-review.googlesource.com/1153553 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#54761}
-
Deepti Gandluri authored
This reverts commit 670fa86f. Reason for revert: Causes gc-stress bots to fail - https://logs.chromium.org/logs/v8/buildbucket/cr-buildbucket.appspot.com/8939819990688690880/+/steps/Check_-_d8/0/logs/object-spread-ic/0 https://logs.chromium.org/logs/v8/buildbucket/cr-buildbucket.appspot.com/8939819696982258192/+/steps/Check_-_d8/0/logs/object-spread-ic/0 Original change's description: > [CloneObjectIC] overwrite monomorphic/polymorphic feedback if deprecated > > An object with a deprecated Map which has already been cached in > CloneObjectIC feedback is still a valid Map for fast cloning --- but > to be consistent with other ICs, deprecated maps are ignored, and are > expected to be transitioned away from. > > If the source object has a deprecated map, the instance is migrated. > > BUG=v8:7611, chromium:867958 > R=jkummerow@chromium.org, mvstanton@chromium.org > > Change-Id: I9771b00400fb4dda45a62e874a31d9b50630d847 > Reviewed-on: https://chromium-review.googlesource.com/1152414 > Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> > Commit-Queue: Caitlin Potter <caitp@igalia.com> > Cr-Commit-Position: refs/heads/master@{#54758} TBR=jkummerow@chromium.org,mvstanton@chromium.org,caitp@igalia.com Change-Id: Id17652ad6f3f09adb43848069549ad146d48b2d7 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:7611, chromium:867958 Reviewed-on: https://chromium-review.googlesource.com/1153747Reviewed-by:
Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/master@{#54760}
-
Sergiy Byelozyorov authored
This reverts commit ba023371. Reason for revert: decided to use CIPD package instead, see reasoning here: http://shortn/_MgZHcjC60Q. Original change's description: > [tools] Add an option to checkout v8-perf repo before running perf tests > > R=machenbach@chromium.org > > No-Try: true > Bug: chromium:838864 > Change-Id: I02e38778ec4c33a20f4b2e93fb22615baa316883 > Reviewed-on: https://chromium-review.googlesource.com/1151295 > Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#54723} TBR=machenbach@chromium.org,sergiyb@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. No-Try: true No-Tree-Checks: true Bug: chromium:838864 Change-Id: I65a5b075c6fede61b59528ba92f6803011f187c7 Reviewed-on: https://chromium-review.googlesource.com/1153687 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by:
-
Caitlin Potter authored
An object with a deprecated Map which has already been cached in CloneObjectIC feedback is still a valid Map for fast cloning --- but to be consistent with other ICs, deprecated maps are ignored, and are expected to be transitioned away from. If the source object has a deprecated map, the instance is migrated. BUG=v8:7611, chromium:867958 R=jkummerow@chromium.org, mvstanton@chromium.org Change-Id: I9771b00400fb4dda45a62e874a31d9b50630d847 Reviewed-on: https://chromium-review.googlesource.com/1152414Reviewed-by:
-
Brian Stell authored
Also removed an obsolete test that is covered by test262/intl402 Bug: v8:7954, v8:5751 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: I41113653cd27c165e6f0a52e4b63bb9ddc553cba Reviewed-on: https://chromium-review.googlesource.com/1150453 Commit-Queue: Jungshik Shin <jshin@chromium.org> Reviewed-by:
-
