- 06 Nov, 2018 40 commits
-
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/2f08425d51e7..228942c6f98d Created with: gclient setdep -r src-internal@228942c6f98d The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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. BUG=chromium:856278 TBR=mmoss@chromium.org Change-Id: Id8db7f14667919fb18b5b74a0e595e4ef48c22bd Reviewed-on: https://chromium-review.googlesource.com/c/1319370Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#605611}
-
Zhuoyu Qian authored
https://google.github.io/styleguide/cppguide.html#Namespace_Names > Namespace names are all lower-case. This CL has no behavior changes. Bug: 889726 Change-Id: Id1a9e25e4c241587268fd9c891a65d3ead832929 Reviewed-on: https://chromium-review.googlesource.com/c/1319389Reviewed-by:
Kent Tamura <tkent@chromium.org> Commit-Queue: Zhuoyu Qian <zhuoyu.qian@samsung.com> Cr-Commit-Position: refs/heads/master@{#605610}
-
Jungshik Shin authored
It has two changes: https://chromium.googlesource.com/chromium/deps/icu.git/+log/42d5027..834113 One is to cherry-pick an integer overflow fix. The other is to fix ISO-2022-JP encoding. Now that 373 code points are encoded as expected. Rebaseline the test results (delete them because all the code points in those files pass). Bug: 901255, 900059 Test: See bug 900059 Test: LayoutTests: external/wpt/encoding/legacy-mb-japanese/iso-2022-jp Test: https://www.w3.org/International/tests/repo/run?base=encoding&batch=encoding-dbl-byte&test=legacy-mb-japanese/iso-2022-jp/iso2022jp-encode-form.html Change-Id: I474ac74c270b77c46f786b28d04494fadb753694 Reviewed-on: https://chromium-review.googlesource.com/c/1317129Reviewed-by:
Joshua Bell <jsbell@chromium.org> Commit-Queue: Jungshik Shin <jshin@chromium.org> Cr-Commit-Position: refs/heads/master@{#605609}
-
Keishi Hattori authored
Enables HeapCompaction for kIncrementalIdleGC and kIncrementalV8FollowupGC. Bug: 864425 Cq-Include-Trybots: 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 Commit-Queue: Keishi Hattori <keishi@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#603477} Change-Id: I506868b5ea224714491fb4dfd9403e5311550255 Reviewed-on: https://chromium-review.googlesource.com/c/1316954Reviewed-by:Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#605608}
-
Charlie Harrison authored
Bug: 898795 Change-Id: I143b0705647563e732e662701f3878126139cb1a Reviewed-on: https://chromium-review.googlesource.com/c/1318419 Commit-Queue: Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Cr-Commit-Position: refs/heads/master@{#605607}
-
Azeem Arshad authored
Bug: 896861 Change-Id: Ida470c2da886409de0864da4f3a4dfdca117e03e Reviewed-on: https://chromium-review.googlesource.com/c/1289575Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Alan Cutter <alancutter@chromium.org> Reviewed-by:
Ryan Hansberry <hansberry@chromium.org> Commit-Queue: Azeem Arshad <azeemarshad@chromium.org> Cr-Commit-Position: refs/heads/master@{#605606}
-
Lan Wei authored
In order to match with test_driver Actions API, we need to change the button type from string to int in the pointerActionSequence API and the related web platform tests and layout tests. Bug: 893480 Change-Id: I151849d31038cf40a2a603196e0f616e15d89072 Reviewed-on: https://chromium-review.googlesource.com/c/1297637 Commit-Queue: Lan Wei <lanwei@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Cr-Commit-Position: refs/heads/master@{#605605}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/a9bbd8684936..d7db17b992e4 git log a9bbd8684936..d7db17b992e4 --date=short --no-merges --format='%ad %ae %s' 2018-11-05 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision bf7ad46d..11d7305a (605401:605505) Created with: gclient setdep -r src/third_party/webrtc@d7db17b992e4 The AutoRoll server is located here: https://autoroll.skia.org/r/webrtc-chromium-autoroll 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=luci.chromium.try:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng BUG=chromium:None TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I582f1055289c2c0315749b93949f0ebb28444850 Reviewed-on: https://chromium-review.googlesource.com/c/1318161Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#605604}
-
Leon Han authored
This CL removes the typemapping of (blink.mojom.Referrer, content::Referrer) and leaves a TODO: replace content::Referrer with blink::mojom::Referrer everywhere in the future. Next step we can move blink/public/platform/referrer.mojom into blink/public/mojom/. BUG=789854 Change-Id: I70b52fe762a858d6425177604749ce4e85a7fc9c Reviewed-on: https://chromium-review.googlesource.com/c/1317348Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Leon Han <leon.han@intel.com> Cr-Commit-Position: refs/heads/master@{#605603}
-
Emircan Uysaler authored
Bug: 326740 Change-Id: Iea381d99984249796fb0e11279174a553ea9ce8a Reviewed-on: https://chromium-review.googlesource.com/c/1313328Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Henrik Boström <hbos@chromium.org> Commit-Queue: Emircan Uysaler <emircan@chromium.org> Cr-Commit-Position: refs/heads/master@{#605602}
-
Scott Graham authored
Initializing a struct member with the value of an enum would previously fail to compile; implement this. This is slightly awkward because the front end does not scope the enum value to the enum like the C++ (and JS) bindings but rather only to the library. So the compiler needs to know the type to which it's being assigned and use that to determine how the enum's value is intended to be qualified. Bug: 883496 Change-Id: Id8a9f75d07bb0acb11963ecbd37d4915b36c28e0 Reviewed-on: https://chromium-review.googlesource.com/c/1313286 Commit-Queue: Scott Graham <scottmg@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#605601}
-
Yuta Kitamura authored
This CL simplifies lengthy function calls via Platform::Current() that obtain the current thread and the main thread by using Thread::Current() and Thread::MainThread(). Also now-redundant null checks for those functions are removed. Bug: 826203 Change-Id: Iedfd7f09b5334dc84fa921cdca84ca39ad9b3af9 Reviewed-on: https://chromium-review.googlesource.com/c/1318773Reviewed-by:
-
Eric Foo authored
Change-Id: I759a8630ea1ae636bfc3954006a2fa5fd6b5299d Reviewed-on: https://chromium-review.googlesource.com/c/1318732 Commit-Queue: Eric Foo <efoo@chromium.org> Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Cr-Commit-Position: refs/heads/master@{#605599}
-
Hirokazu Honda authored
This adds video frame flags, --frame_validator=, check : compares md5 values computed from video frame with the expected md5 values. Test fails if there is an unexpected video frame. dump : outputs video frames to files BUG=chromium:856562 TEST=VDA unittest with --test_import TEST=VDA unittest with --test_import --frame_validator=check TEST=VDA unittest with --frame_validator=check TEST=VDA unittest with --frame_validator=check,dump TEST=VDA unittest with --frame_validator=dump TEST=VDA unittest with --gtest_filter=VideoDecodeAcceleratorTest.DISABLED_GenMD5 --gtest_also_run_disabled_tests Change-Id: I1511ef07a2bd01231e9daf1038776324c22e3da5 Reviewed-on: https://chromium-review.googlesource.com/c/1309390Reviewed-by:Alexandre Courbot <acourbot@chromium.org> Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#605598}
-
Krishna Govind authored
R=cmasso@chromium.org Change-Id: Id3b9912d9b06e0baae9e98e4e51d9da53ec64e6b Reviewed-on: https://chromium-review.googlesource.com/c/1319331Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#605597}
-
Eric Foo authored
Change-Id: I80a812501aeb3c4d894d687326dac3777b2f54c5 Reviewed-on: https://chromium-review.googlesource.com/c/1318587Reviewed-by:
Ryan Tseng <hinoka@chromium.org> Commit-Queue: Eric Foo <efoo@chromium.org> Cr-Commit-Position: refs/heads/master@{#605596}
-
Yuta Kitamura authored
This CL simplifies lengthy function calls via Platform::Current() that obtain the current thread and the main thread by using Thread::Current() and Thread::MainThread(). Bug: 826203 Change-Id: I18a106751518e361aecfb6e82152359c1b00f3d1 Reviewed-on: https://chromium-review.googlesource.com/c/1319129Reviewed-by:
-
https://crbug.com/889567Makoto Shimazu authored
Bug: 889567 Change-Id: I790222c0e74692214bc10daff7a6aa00478a4b88 Reviewed-on: https://chromium-review.googlesource.com/c/1319109 Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#605594}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/3d2886849216..c238958c1ca3 git log 3d2886849216..c238958c1ca3 --date=short --no-merges --format='%ad %ae %s' 2018-11-06 chrome-bot@chromium.org Update config settings by config-updater. Created with: gclient setdep -r src/third_party/chromite@c238958c1ca3 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll 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=chrome-os-gardeners@chromium.org Change-Id: I07cd28e68dc926a69573e8627859e0e287e92467 Reviewed-on: https://chromium-review.googlesource.com/c/1318164Reviewed-by:
-
Yoshifumi Inoue authored
This patch changes |NGInlineNode::AssociateItemsWithInlines()| to associate |NGInlineItem| to |LayoutText| correctly by skipping |NGInlineItem| without layout object. Before this patch, |LayoutText| is assocaited |NGIninlineItem| objects after the |NGIninlineItem.layout_object == nullptr|. This patch is a preparation of the patch[1]. [1] http://crrev.com/c/1309562 Make Element#innterText to utilize NGInlineItem instead of NGPaintFragment Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: I41f16ea13ca1e0641cb2a42331d64fa2cd6387ac Reviewed-on: https://chromium-review.googlesource.com/c/1317217 Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#605592}
-
Wenzhao Zang authored
This reverts commit 2bdc43ad. Reason for revert: crbug.com/900373 Original change's description: > viz: Turn off or crop primary plane when possible > > For many fullscreen cases, the rest of the renderpass quads are solid > black quads. In that case, we can turn off the primary plane, and not > having to scan out an full screen of black pixels saves a good deal of > power. In other cases, we can crop down the primary plane down to the > content rectangle (youtube channel logo, for example) and save power. > > Averaging battery draw measurement over about a minute of play back > for a fullscreen youtube video, power savings for soraka is around > 500mW (6.5W to 6.0W) and 300mW for kevin (4.4W to 4.1W), when we > turn off the primary plane. > > Change-Id: I7946d7acab334bffdf2bb49a1d9c424cf46fc610 > Reviewed-on: https://chromium-review.googlesource.com/c/1045256 > Reviewed-by: Daniele Castagna <dcastagna@chromium.org> > Reviewed-by: Dongseong Hwang <dongseong.hwang@intel.com> > Commit-Queue: Kristian H. Kristensen <hoegsberg@chromium.org> > Cr-Commit-Position: refs/heads/master@{#600415} TBR=dongseong.hwang@intel.com,dcastagna@chromium.org,hoegsberg@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 900373 Change-Id: I37857bca0cda656c2e2a83dc04312b9c599976e7 Reviewed-on: https://chromium-review.googlesource.com/c/1318547 Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org> Reviewed-by:
Wenzhao (Colin) Zang <wzang@chromium.org> Cr-Commit-Position: refs/heads/master@{#605591}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=abdulsyed@chromiue.org Change-Id: I06691e83b83420cc54891b0ad95c123470fca502 Reviewed-on: https://chromium-review.googlesource.com/c/1319172Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#605590}
-
Timothy Loh authored
This CL makes a few improvements in searching for the Crostini Terminal: - Allow partial matches (bug 840699). Instead of requiring the entire word 'terminal', allowa match from just 'ter' (but still require a few characters to avoid over-triggering). - Add additional keywords 'linux', 'crostini' (bug 858822). - Add unlocalized (English) keywords. Users may still search for these as localizations of Linux/Crostini tend to be tranliterations. Bug: 840699, 858822 Change-Id: Iabff74d9d8a61c2c2abcc50f9683b381c5dce126 Reviewed-on: https://chromium-review.googlesource.com/c/1280092 Commit-Queue: Timothy Loh <timloh@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#605589}
-
Heng-Ruey Hsu authored
API doesn't ask user to call SetPhotoOptions before TakePhoto. Therefore TakePhoto should enable blob streams as well if SetPhotoOptions is not called. BUG=b:114676133, chromium:900900 TEST=./capture_unittest --gtest_filter="*VideoCaptureDeviceTests/*" and take photo in CCA Change-Id: I72d4f1394fe95aaca768cb366a341796f55fe336 Reviewed-on: https://chromium-review.googlesource.com/c/1314074Reviewed-by:Miguel Casas <mcasas@chromium.org> Commit-Queue: Heng-ruey Hsu <henryhsu@chromium.org> Cr-Commit-Position: refs/heads/master@{#605588}
-
David Bokan authored
When an Element is made the global rootScroller, it can hide the URL bar like Document scrolling does normally. However, the Element's LayoutBox, like the LayoutView, is not resized in this case. This means there's a newly exposed region at the bottom of the screen that's visible beyond the Element's overflow rect. LayoutView already has an exception in the PaintLayerClipper for this case. This CL extends the exception to the root scroller. This fixes hit testing that descends into child PaintLayers of the root scroller's layer. In addition, it also caches an "IsGlobalRootScroller" bit in LayoutObject for performance, just like the effective bit added in https://crrev.com/2685a6b3e82be7711c6d3b02f225b1b90bb9bf46 Also added tests to ensure this works in an iframe. Bug: 798719 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Iaf87db5569ebbe1aa82f20d3cf19868a12ed9539 Reviewed-on: https://chromium-review.googlesource.com/c/1287176 Commit-Queue: David Bokan <bokan@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#605587}
-
John Budorick authored
This reverts commit ba0225d1. Reason for revert: breaks internal compilation; see e.g. http://shortn/_9928H1Hk0R Original change's description: > Reland "Android: turning on bytecode checker" > > This is a reland of 0301a791 > > I don't think this would have caused a runtime chromeos error, since it's > a build time android change. If it pops up again that I broke the test, > I will investigate further. > > TBR=retry > > Original change's description: > > Android: turning on bytecode checker > > > > This arg was defaulting to false when it should have defaulted to true. > > > > TBR=turning off bytecode checks for certain dirs > > > > Bug: 874854 > > Change-Id: I248ccab0ed52079106b07e43a302927bfa4414c8 > > Reviewed-on: https://chromium-review.googlesource.com/c/1309973 > > Commit-Queue: Sam Maier <smaier@chromium.org> > > Reviewed-by: agrieve <agrieve@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#605068} > > Bug: 874854 > Change-Id: Idc95750d4465ca53b72db26a98ce74724fe21968 > Reviewed-on: https://chromium-review.googlesource.com/c/1318071 > Reviewed-by: Sam Maier <smaier@chromium.org> > Commit-Queue: Sam Maier <smaier@chromium.org> > Cr-Commit-Position: refs/heads/master@{#605347} TBR=agrieve@chromium.org,smaier@chromium.org Change-Id: I4bb41ab999431fd2df511838d10db598cd5c0a87 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 874854 Reviewed-on: https://chromium-review.googlesource.com/c/1319333Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#605586}
-
Nektarios Paisios authored
This patch also fixes the bug whereby Jaws wasn't announcing the visited state on a link. We need to update the accessibility tree if there are any changes to the way a layout object is painted. To be safe, we update the whole subtree. The update needs to happen after the style change, not before, so as to catch any changes to the visited link state, color and font. R=dmazzoni@chromium.org, aleventhal@chromium.org Change-Id: I03f512a95f8101c603a712fb3d62e7b31a59da5e Bug: 898706 Reviewed-on: https://chromium-review.googlesource.com/c/1298173 Commit-Queue: Nektarios Paisios <nektar@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#605585}
-
sangwoo.ko authored
Stop notifying events via old api and replace old API with new API. This CL is a refactor and has no intended behavior change. Bug: 842194 Change-Id: I5e11e7323940acbbdc31bd73d599480c7312e8b9 Reviewed-on: https://chromium-review.googlesource.com/c/1314508 Commit-Queue: Sang Woo Ko <sangwoo108@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#605584}
-
siyua authored
The previous title has translation issue. Bug: 897998 Change-Id: I59471c13de7c099bd8b7755d09c67d83e517f6b4 Reviewed-on: https://chromium-review.googlesource.com/c/1316835Reviewed-by:
Evan Stade <estade@chromium.org> Commit-Queue: Siyu An <siyua@chromium.org> Cr-Commit-Position: refs/heads/master@{#605583}
-
rajendrant authored
The fields ps, sid in chrome-proxy header are deprecated. So removing them. https://g3doc.corp.google.com/wireless/speed/flywheel/g3doc/protocol/syntax_proxy.md#requests Bug: 893728 Change-Id: I752938b6defd6e55405bd40151341ab50486c014 Reviewed-on: https://chromium-review.googlesource.com/c/1316677 Commit-Queue: rajendrant <rajendrant@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#605582}
-
Fan Yang authored
With this change, client initiated stream ID is 0, 2, 4..., and server initiated stream ID is 1, 3, 5... If client starts with v99, and server does not support v99, the version negotiation is not supported. Merge internal change: 218572979 R=rch@chromium.org Change-Id: I922820118ce229e1ae188a3c2d1a16cf3b34c2a6 Reviewed-on: https://chromium-review.googlesource.com/c/1318355 Commit-Queue: Fan Yang <fayang@chromium.org> Reviewed-by:
Ryan Hamilton <rch@chromium.org> Cr-Commit-Position: refs/heads/master@{#605581}
-
Dmitry Gozman authored
- Merge TargetRegistry into DevToolsSession. - Allow flatten children on any targets. Bug: 775132 Change-Id: Idaafb019f9881a937f3e4b2d1abab7a3e89af06e Reviewed-on: https://chromium-review.googlesource.com/c/1316748 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#605580}
-
Dmitry Titov authored
This reverts commit dc7a451a. Reason for revert: Broke tests on Mac (at least): https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.12%20Tests/16499 Broken test output: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8930653299367123984/+/steps/network_service_browser_tests_on_Intel_GPU_on_Mac_on_Mac-10.12.6/0/logs/SaveCardBubbleViewsFullFormBrowserTest.StrikeDatabase_Upload_AddStrikeIfSaveFails/0 Original change's description: > Add StrikeDatabase browsertests > > The tests in this CL have been landed and reverted a couple of times; > they work, but end up being a little flaky on Win7. We've had a hard > time getting a reproducible setup, and the Chrome testing team have > verified our feature works as intended, so for the time being we're > marking the tests as flaky on Windows and will follow up. > > (Sheriffs: If somehow the landing of this CL causes additional > flakes, feel free to revert. Flake analysis hasn't shown anything > outside of Win7.) > > Bug: 884817 > Change-Id: Iec545cb861bfc1aa3d50746c3a413257c9a9ec82 > Reviewed-on: https://chromium-review.googlesource.com/c/1316666 > Commit-Queue: Jared Saul <jsaul@google.com> > Reviewed-by: Evan Stade <estade@chromium.org> > Cr-Commit-Position: refs/heads/master@{#605507} TBR=estade@chromium.org,jsaul@google.com Change-Id: I1e408b6395c80c425d982544bfdd88e01231db7b No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 884817 Reviewed-on: https://chromium-review.googlesource.com/c/1319190Reviewed-by:
Dmitry Titov <dimich@chromium.org> Commit-Queue: Dmitry Titov <dimich@chromium.org> Cr-Commit-Position: refs/heads/master@{#605579}
-
Mitsuru Oshima authored
Bug: 901402 Test: manual. The window title fades in after overview animation ends (in clamshell, tablet, from home launcher). and hides immediately when exiting. Change-Id: I06b25ebea428439ec6e7ab2751db74409971855f Reviewed-on: https://chromium-review.googlesource.com/c/1316363 Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#605578}
-
siyua authored
Make a new CL since the master branch has changed much. The unresolved comments in the previous CL are addressed here. 1) Add vector icon for https://icons.googleplex.com/#icon=ic_lightbulb_outline&search=light%20bulb 2) Add components factory for finished and action required views. 3) Update the text string in action-required dialog. 4) Move image, title and LocalCardMigrationOfferView to LocalCardMigrationDialogView. 5) Update MigratableCardView to add different content depending on the card migration status. Have uploaded implementation screenshots in the bug below (comment#8). Bug: 897998 Change-Id: I34d6f112a872944ab3ba1cb96f3aaa1665e4173d Reviewed-on: https://chromium-review.googlesource.com/c/1300473 Commit-Queue: Siyu An <siyua@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#605577}
-
Ken Rockot authored
This is a reland of 14c0b604 Original change's description: > [mojo] Add IPC support to core shared library > > Updates the mojo_core shared library so that it is actually suitable > for production IPC use. Namely: > > - MojoInitialize now spins up a background thread for I/O and gives > it to the core impl. > - A flag is added to Initialize(), allowing shared library consumers > to initialize themselves as a broker process. > - The thunks helper correctly loads the mojo_core library with > RTLD_DEEPBIND set, avoiding heap crossover within the core impl > due to e.g. malloc and free lazy-binding to different heaps > - Fixes the thunks header to compile as C once again, as it was > missing some struct keywords. > - Adds MojoShutdown to the core ABI as a necessary call for shared > library consumers who want clean shutdown. > > This also adds a new multiprocess test consuming the shared library > to tie together and validate all of the above changes. > > Bug: 809320 > Change-Id: Ic1c56d99c86fd4a2dc7f812ee152994ced35ece6 > Reviewed-on: https://chromium-review.googlesource.com/c/1306347 > Commit-Queue: Ken Rockot <rockot@google.com> > Reviewed-by: Reilly Grant <reillyg@chromium.org> > Cr-Commit-Position: refs/heads/master@{#605109} TBR=reillyg@chromium.org Bug: 809320 Change-Id: I8a8b2c1acfb16bf42823f266dffbae8e897075f3 Reviewed-on: https://chromium-review.googlesource.com/c/1318061Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#605576}
-
Shinya Kawanaka authored
Due to disk shortage, `all` cannot be built for Mac Goma Canary (clobber). Let's build chrome instead. The disk of "Chromium Mac Goma GCE Staging" is also getting short, so I changed this builder, too. Bug: 899425 Change-Id: I58a960ad8455c78bc7ae871298edf78aed650f7b Reviewed-on: https://chromium-review.googlesource.com/c/1316952 Commit-Queue: Shinya Kawanaka <shinyak@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Fumitoshi Ukai <ukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#605575}
-
Xiaocheng Hu authored
This patch converts the layout test with W3C testharness to reduce usage of deprecated js-test.js. This patch is also a preparation for using Element.innerText for text dump. As the layout tests adds float style to all <span>, without this patch, innerText dump can be confusing. Bug: 887148 Change-Id: Ia8cafbc996fe1f4ae66350b565b7611bb7ec35e5 Reviewed-on: https://chromium-review.googlesource.com/c/1318357 Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:
-
chromium-autoroll authored
The AutoRoll server is located here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll 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=luci.chromium.try:fuchsia_arm64_cast_audio;luci.chromium.try:fuchsia_x64_cast_audio TBR=cr-fuchsia+bot@chromium.org Change-Id: I9af8ed01ad2fae3e51e99a5a484e874c31ae8938 Reviewed-on: https://chromium-review.googlesource.com/c/1318162Reviewed-by:
-
Eliot Courtney authored
This clips the requested screen capture area to the desktop window for ArcScreenCaptureSession. AuraDesktopCapturer already does this. Bug: b/118675936 Test: cts passes Change-Id: I3795d21bf34f05e7f71e46f3fcc8ff2e50cec506 Reviewed-on: https://chromium-review.googlesource.com/c/1316985Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Commit-Queue: Eliot Courtney <edcourtney@chromium.org> Cr-Commit-Position: refs/heads/master@{#605572}
-
