- 21 May, 2018 40 commits
-
-
Tom Anderson authored
BUG=747675 R=thakis TBR=dpranke Change-Id: I8d3cdab732a10d81480f6476fbdccff2c8ef34e5 Reviewed-on: https://chromium-review.googlesource.com/754033Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#560338}
-
Sergey Berezin authored
This removes the support for system-wide Xcode. R=huangml@chromium.org Bug: 835036 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I925587983d618593771522a28ccd4bdf0cd16d78 Reviewed-on: https://chromium-review.googlesource.com/1064829Reviewed-by:
Menglu Huang <huangml@chromium.org> Commit-Queue: Sergey Berezin <sergeyberezin@chromium.org> Cr-Commit-Position: refs/heads/master@{#560337}
-
Tom Anderson authored
On Mac, they discovered long ago that quitting after holding Cmd+Q was a bad idea since apps underneath Chrome would get the repeated Cmd+Q key event and would also close [1]. Their solution was to delay quitting until the shortcut is released. To give users confirmation that Chrome is quitting, all browser windows are faded out. The same issue occurs on Linux and Windows (I've unintentionally closed many background apps writing CL [2]), so this CL implements the Mac solution. [1] http://dev.chromium.org/developers/design-documents/confirm-to-quit-experiment [2] https://chromium.googlesource.com/chromium/src.git/+/6cbd505d3cb04e3e108e61a74f82421b66f0f655 BUG=243164 R=sky Change-Id: Ia3de4acaf463cd5b202743d44f763c6d52fbe963 Reviewed-on: https://chromium-review.googlesource.com/1066924Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#560336}
-
Yafei Duan authored
Renaming SystemDownloadManagerStub to StubSystemDownloadManager for consistency with StubOfflinePageModel. Bug: NONE Change-Id: I7cb2d9d13b7c31a44c24b030d33125d38e1efd2f Reviewed-on: https://chromium-review.googlesource.com/1063036 Commit-Queue: Yafei Duan <romax@chromium.org> Reviewed-by:
Peter Williamson <petewil@chromium.org> Cr-Commit-Position: refs/heads/master@{#560335}
-
Eric Roman authored
Change-Id: I2451fcbfd31bb70628d138838ec95327cb8af91b Reviewed-on: https://chromium-review.googlesource.com/1066599Reviewed-by:
Bence Béky <bnc@chromium.org> Commit-Queue: Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#560334}
-
Christian Biesinger authored
As pointed out by Koji in https://chromium-review.googlesource.com/c/chromium/src/+/1064855, we should not clear the paint fragment if IsIntermediateLayout(). That was an oversight in the original CL. R=eae@chromium.org,kojii@chromium.org Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I93d899dd554edc227c190d2ebad3eca725531c9a Reviewed-on: https://chromium-review.googlesource.com/1067423 Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#560333}
-
Lei Zhang authored
Instead of writing PDF data out to disk and reading it back in again in a different process, write out the data to shared memory and share that with the receiving process. Send the rendering results back using shared memory as well. Change-Id: If80fccdda1237313c6b4d4caa87cfea4b73c2439 Reviewed-on: https://chromium-review.googlesource.com/1055202Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Jay Civelli <jcivelli@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#560332}
-
Fredrik Söderquist authored
Previously we didn't know in what context said method was called, so we didn't know if zoom ought to be applied or not. Add LayoutSVGRoot::UnscaledIntrinsicSizingInfo and change current callers of ComputeIntrinsicSizingInfo to use it. Then make ComputeIntrinsicSizingInfo scale the intrinsic size. Bug: 722055 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: If668e85bcea1028e48a3bc98c74d07435d425338 Reviewed-on: https://chromium-review.googlesource.com/1067397 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:
Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#560331}
-
Nico Weber authored
Ran `tools/clang/scripts/upload_revision.py 332838`. Bug: 844392 Change-Id: I1a6288f8ba56f52bc2ec7c9c50ac2af4bf590b4f Reviewed-on: https://chromium-review.googlesource.com/1067108Reviewed-by:
Peter Collingbourne <pcc@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#560330}
-
Nico Weber authored
I added them to the clang/tot memory bots in https://chromium-review.googlesource.com/1060293 and they mostly pass. Disable the two tests that don't pass. TBR=nyquist,dmazzoni Bug: 843511 Change-Id: Iefc4f1450e3da57ad7854513587e1c9e2744a75e Reviewed-on: https://chromium-review.googlesource.com/1067168Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
-
edchin authored
Following general coding guideline: "Don't gift wrap". Might also be described as returning early. Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I75a1f467b56853207a6d7ec5516eb853b1cca9a4 Reviewed-on: https://chromium-review.googlesource.com/1067249 Commit-Queue: edchin <edchin@chromium.org> Reviewed-by:
edchin <edchin@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#560328}
-
Marco Paniconi authored
https://chromium.googlesource.com/webm/libvpx.git/+log/d99abe9a9ad7..e27a331778c4 $ git log d99abe9a9..e27a33177 --date=short --no-merges --format='%ad %ae %s' 2018-05-18 marpan vp9-svc: Fix on disabling inter_layer prediction. 2018-05-18 marpan vp9-svc: Fix issue with reseting lst_fb_idx. 2018-05-15 johannkoenig Revert "Revert "update libwebm"" 2018-05-18 johannkoenig iosbuild.sh: portable sed usage 2018-05-16 marpan vp9-svc: Skip find_predictors based on ref_frame_flags. 2018-05-16 marpan vp0-svc: Small code cleanup in nonrd-pickmode. 2018-05-14 marpan vp9-svc: Enable scene detection and re-encode for SVC. 2018-05-15 jingning Use the updated best rd cost for transform block search Created with: roll-dep src/third_party/libvpx/source/libvpx R=johannkoenig@google.com Change-Id: I5ba816c96cde73a5cfaecb4e520dc0f54078a1a8 Reviewed-on: https://chromium-review.googlesource.com/1066082Reviewed-by:
Johann Koenig <johannkoenig@google.com> Commit-Queue: Marco Paniconi <marpan@google.com> Cr-Commit-Position: refs/heads/master@{#560327}
-
Tom Anderson authored
R=dpranke Change-Id: I880c2cc2a2702db6ebbbd90120753587971bb2be Reviewed-on: https://chromium-review.googlesource.com/1067581Reviewed-by:
-
sczs authored
Bug: 845192 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I66a8d2c45021c27674c937572c1b706895f2eb8c Reviewed-on: https://chromium-review.googlesource.com/1067482Reviewed-by:
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/435282162ed6..4dc5a454052e $ git log 435282162..4dc5a4540 --date=short --no-merges --format='%ad %ae %s' 2018-05-21 caryclark remove defines and add commas Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=allanmac@chromium.org Change-Id: I8861976abcafea5a46efbec13001a7a944f942b2 Reviewed-on: https://chromium-review.googlesource.com/1067506Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#560324}
-
Justin Cohen authored
Bug: 833562 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ic0c915d3458b20dd0c660cdfd18f317be061bb85 Reviewed-on: https://chromium-review.googlesource.com/1066575Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Commit-Queue: Justin Cohen <justincohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#560323}
-
Fredrik Söderquist authored
Rather than exposing the document root's LayoutObject, add an bunch of methods to achieve the same thing as the returned object is used for. Reuse the same names as used on FrameView. Also rename the method on LayoutImage to something more appropriate. Bug: 722055 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I6d006206365988c2ad8068db7251414529ed3bca Reviewed-on: https://chromium-review.googlesource.com/1065997 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#560322}
-
Philip Rogers authored
CompositedLayerMapping's background_layer was only used for fixed root backgrounds (see: PLC::NeedsFixedRootBackgroundLayer) because the main graphics layer scrolled. With root layer scrolling, all fixed backgrounds are painted on the main graphics layer, and we no longer need CLM's background_layer. Bug: 823365 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I1546edb5fa1588b7499328b1ec56327e379f847b Reviewed-on: https://chromium-review.googlesource.com/1067076Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Steve Kobes <skobes@chromium.org> Commit-Queue: Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#560321}
-
Yuri Wiitala authored
1. Removed people long-gone from the project (by several years). 2. Added xjz@ for cast streaming files. 3. Fixed/Added "# COMPONENT" comments to help redirect new crbugs. Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: I1a549a4f85e76f563a162263e90268de57daddab Reviewed-on: https://chromium-review.googlesource.com/1065089Reviewed-by:
Xiangjun Zhang <xjz@chromium.org> Reviewed-by:
John Rummell <jrummell@chromium.org> Commit-Queue: Yuri Wiitala <miu@chromium.org> Cr-Commit-Position: refs/heads/master@{#560320}
-
James Hawkins authored
This fixes a regression in the sense that Views login recently became the default for some set of users, and SmartLock probably never worked for login on Views (but it did work on WebUI login). R=jdufault@chromium.org Bug: 844532 Test: none yet, need to write E2E Change-Id: I736f2ea3e8b7fc4d961852c5fb649f03a904d333 Reviewed-on: https://chromium-review.googlesource.com/1066935Reviewed-by:
Jacob Dufault <jdufault@chromium.org> Commit-Queue: James Hawkins <jhawkins@chromium.org> Cr-Commit-Position: refs/heads/master@{#560319}
-
Aaron Leventhal authored
According to the IA2 spec here: https://wiki.linuxfoundation.org/accessibility/iaccessible2/textattributes "If an attribute is not specified and if the table shows that there is a default value, the default value should be assumed." This makes the a11y tree less noisy, smaller and easier to debug, and is similar to how Firefox works. Bug: 844147 Change-Id: I3f7827d10d9659e5e758827a806da0b7f6598e41 Reviewed-on: https://chromium-review.googlesource.com/1066112Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Cr-Commit-Position: refs/heads/master@{#560318}
-
Xiaocheng Hu authored
Mojo doesn't allow removing an uncalled mojo callback while the message pipe is still open. On Android, SpellCheckerSessionBridge::DisconnectSession() can discard pending requests while the message pipe is still open, violating the requirement. This patch fixes it by running the callbacks with empty result before removing them. Bug: 843845 Change-Id: I2ab4d51eeb43e63d9715eb42187d51656886fea5 Reviewed-on: https://chromium-review.googlesource.com/1065148Reviewed-by:
Rachel Blum <groby@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#560317}
-
Min Chen authored
Bug: 818331 Change-Id: Ifcaa7adcb25fe099047d5f9d73d6b0bd78d7fff8 Reviewed-on: https://chromium-review.googlesource.com/1067508Reviewed-by:
Dan Erat <derat@chromium.org> Reviewed-by:
Qiang Xu <warx@google.com> Commit-Queue: Min Chen <minch@chromium.org> Cr-Commit-Position: refs/heads/master@{#560316}
-
Menglu Huang authored
Bug: 837241 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I8f4db54b6b1e0df50c54cbc13a47e574497b2c6a Reviewed-on: https://chromium-review.googlesource.com/1036482Reviewed-by:
Sergey Berezin <sergeyberezin@chromium.org> Commit-Queue: Menglu Huang <huangml@chromium.org> Cr-Commit-Position: refs/heads/master@{#560315}
-
Hidy Han authored
Change-Id: I0643ca9e2d4e5d75cad3a76d8d005a6916c2fd13 Reviewed-on: https://chromium-review.googlesource.com/1063250 Commit-Queue: Hidy Han <hidyhan@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#560314}
-
Rune Lillesveen authored
We would skip style recalc for <content> fallback descendants because <content> elements are not part of the flat tree, hence they will not have a ParentComputedStyle() and we would think it's because it's display:none. Instead traverse into <content> children when <content> needs reattachment. Bug: 843520 Change-Id: I2b7f6b44c600371422e86750308231b60e2f5abd Reviewed-on: https://chromium-review.googlesource.com/1065682Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#560313}
-
Emircan Uysaler authored
When WebRTC sets timestamp to zero for the remote video, it indicates that we want to skip render smoothness algorithm and render asap. This CL covers this case by skipping setting REFERENCE_TIME when timestamp is zero. Bug: webrtc:9135 Change-Id: Ie6cf7a3a7c07cae400541f222611a6efb2ecd943 Reviewed-on: https://chromium-review.googlesource.com/1066915Reviewed-by:
Qiang Chen <qiangchen@chromium.org> Commit-Queue: Emircan Uysaler <emircan@chromium.org> Cr-Commit-Position: refs/heads/master@{#560312}
-
Emily Hanley authored
There is a case where no benchmarks could be uploading perf results from a shard (ie they are all disabled). Bug: 842145 Change-Id: Iea52461dee8e8a7c4475396d9af9f0339cdfaf3e Reviewed-on: https://chromium-review.googlesource.com/1066080Reviewed-by:
Ned Nguyen <nednguyen@google.com> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#560311}
-
Lily Houghton authored
This refactors the chromedriver unit tests, particularly those in net_util_unittest, sync_websocket_unittest, and websocket_unittest, as well as the TestHttpServer class, to use the network service's HttpServer class (instead of the one in net/). Bug: 821000 Change-Id: Icc4479b656936f38e86d104004d1dd703d845dda Reviewed-on: https://chromium-review.googlesource.com/1038544 Commit-Queue: Lily Houghton <lilyhoughton@chromium.org> Reviewed-by:
John Chen <johnchen@chromium.org> Reviewed-by:
Helen Li <xunjieli@chromium.org> Cr-Commit-Position: refs/heads/master@{#560310}
-
dpapad authored
Bug: None Change-Id: Ibb2e05e68dce90a0cc41a8e6e88b497fd4f683c0 Reviewed-on: https://chromium-review.googlesource.com/1066918Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#560309}
-
Fan Yang authored
Merge internal change: 197190958 R=rch@chromium.org Change-Id: Ic0a737873c63d26b9831e5f00e72737eba209bd3 Reviewed-on: https://chromium-review.googlesource.com/1067359Reviewed-by:
Ryan Hamilton <rch@chromium.org> Commit-Queue: Fan Yang <fayang@chromium.org> Cr-Commit-Position: refs/heads/master@{#560308}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/7ca7a59f0202..421df92ae3d4 $ git log 7ca7a59f0..421df92ae --date=short --no-merges --format='%ad %ae %s' 2018-05-21 nednguyen Improve error handling when browser cannot be found in serially_executed_browser_test_case Created with: roll-dep src/third_party/catapult BUG=chromium:845147 The AutoRoll server is located here: https://catapult-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=sullivan@chromium.org Change-Id: I8be4845e37314ec28da9d60ed848ddaeacb54626 Reviewed-on: https://chromium-review.googlesource.com/1067438Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#560307}
-
sawtelle authored
Currently, hovering the mouse over the Picture-in-Picture window sets the opacity of the layer used to darken the video to 0.2, which is not particularly noticeable. With this change, the opacity of the used layer has been adjusted to .4 to create a more distinct effect. Bug: 843219 Change-Id: I8b39e62dec60c142c3b72309e3df526ee370d901 Reviewed-on: https://chromium-review.googlesource.com/1060476Reviewed-by:
apacible <apacible@chromium.org> Commit-Queue: Steven Sawtelle <sawtelle@google.com> Cr-Commit-Position: refs/heads/master@{#560306}
-
Koji Ishii authored
CL:1056647 invalidated inline items in LayoutNGText when LayoutObjects are moved without full notifications. However, it did not handle when LayoutInline is moved and it has LayoutNGText descendants. In such cases, InsertChildNode is called only for the LayoutInline. This patch supports the case as well. Bug: 636993 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I9aaefec24f3dd67f9d3b257af35e079d07418b71 Reviewed-on: https://chromium-review.googlesource.com/1065540Reviewed-by:
-
Rayan Kanso authored
Boilerplate for getting settled background fetches that have been persisted to disk. TBR=avi@chromium.org Bug: 826257 Change-Id: I8a49663df9841fc736af3edad3c946c84da8443c Reviewed-on: https://chromium-review.googlesource.com/1057623 Commit-Queue: Rayan Kanso <rayankans@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#560304}
-
Derek Cheng authored
A pending request could get added to LazyBackgroundTaskQueue while the extension is not enabled (e.g. if it crashed and is waiting to be reloaded). This appears to be by design. In such a case we do not attempt to create the background host. The request (and subsequent requests) could become stuck if the background host does not get created. This patch changes LazyBackgroundTaskQueue to listen for extension loaded, and creates a background host if there are pending requests. This patch also makes sure we only create a PendingTasksList entry if we are enqueueing a request to it. Bug: 835017 Change-Id: Ie2aff8cfb620a1867b033c3474a3277a283fb258 Reviewed-on: https://chromium-review.googlesource.com/1058083Reviewed-by:
-
Allen Vicencio authored
Removes adding of "smb" as a standard scheme and removes SmbUrl from SmbService Bug: chromium:757625 Change-Id: Icbb0ec7d7563f8196e134167b02b199a48cecf9e Reviewed-on: https://chromium-review.googlesource.com/1067507Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Commit-Queue: Allen Vicencio <allenvic@chromium.org> Cr-Commit-Position: refs/heads/master@{#560302}
-
Ken Rockot authored
Ensures no EDK dependencies get introduced into mojo/public (non-test), and cleans up some other incorrect DEPS allowances. TBR=jcivelli@chromium.org Bug: None Change-Id: I60030b0734ff73869442d5880b50d1b2289721af Reviewed-on: https://chromium-review.googlesource.com/1067489Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#560301}
-
Fredrik Söderquist authored
Since we reassociate the SVGResource synchronously, we need to also make sure that the associated SVGResources object is updated, else it could end up pointing to the old <pattern> LayoutObject. Bug: 845040 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I026b46abcd510485ddfe33622cb5f5aa1bab9610 Reviewed-on: https://chromium-review.googlesource.com/1065779Reviewed-by:
-
Ted Choc authored
BUG=844787 Change-Id: I966b385a928075c25eebdb7abfbdbbfb0ab2db95 Reviewed-on: https://chromium-review.googlesource.com/1067477Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#560299}
-
