- 05 Mar, 2019 40 commits
-
-
Bryan McQuade authored
This does not change the behavior of how existing metrics are associated with sources, but creates source entries for all same-doc navs. Follow up changes can start associating metrics with same doc navs as it makes sense to do so. Change-Id: Ib244c70ab83fde810bcd30c7f1c78169bbf8337a Bug: 917463 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1489896 Commit-Queue: Bryan McQuade <bmcquade@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Cr-Commit-Position: refs/heads/master@{#637796}
-
Guohui Deng authored
Rename it to overlay_plane_id, which is a better name, and consistent with the name of its counterpart in VideoFrameMetadata. Bug: 925450 Test: None. Change-Id: I965604451b442558dabc65f7e1fcb91315b38881 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1497612Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Commit-Queue: Guohui Deng <guohuideng@chromium.org> Cr-Commit-Position: refs/heads/master@{#637795}
-
Alexandre Frechette authored
Match the names and config to the finch studies we're launching server side in ManualTranslateInProductHelp.gcl. Bug: 922216 Change-Id: Iaae80d1172fade2e451c29ddfbb85301244278af Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1492735Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Alexandre Frechette <frechette@chromium.org> Cr-Commit-Position: refs/heads/master@{#637794}
-
Ben Wagner authored
This also updates the list of excluded files, removing the files which are added and removing files which have been removed from upstream. This also marks the '.h' files as public, which has limited utility since gn check does not inspect angle bracket includes. Bug: 931719 Change-Id: I97bab5651aa8e52b8bfd7d569f0308250dc74ca3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1500850 Commit-Queue: Ben Wagner <bungeman@chromium.org> Reviewed-by:
Dominik Röttsches <drott@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#637793}
-
Xiaohan Wang authored
This reverts commit a104770b. Reason for revert: The original crash has been fixed in Chrome 72 but it is unclear how that's fixed. Even if we still need this checks for investigation, they have made into M73 beta/stable which should provide sufficient data. Original change's description: > media: Use CHECK in CdmAdapter::SetTimer() > > This is for bug investigation and will be reverted after the bug > investigation is over. See Bug. > > Bug: 887761 > Test: No functionality change. > Change-Id: I98c7dff8a46ac4d7f4847711d24ef5e2ba45cc42 > Reviewed-on: https://chromium-review.googlesource.com/c/1409350 > Reviewed-by: John Rummell <jrummell@chromium.org> > Commit-Queue: Xiaohan Wang <xhwang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#622571} TBR=xhwang@chromium.org,jrummell@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 887761 Change-Id: I4dbfb86ea2eb656a358818e7721a489311c44065 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503900Reviewed-by:
John Rummell <jrummell@chromium.org> Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Commit-Queue: John Rummell <jrummell@chromium.org> Auto-Submit: Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#637792}
-
siashah authored
Bug: 934915 Change-Id: Ia697a61e903d4ddc64ba3c648ec7a47942f16348 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1492484 Commit-Queue: Siddharth Shah <siashah@chromium.org> Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Reviewed-by:
Jared Saul <jsaul@google.com> Reviewed-by:
Fabio Tirelo <ftirelo@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#637791}
-
Eric Lawrence authored
A regression was introduced when the Windows file selection dialog was updated to use the IFileDialog interface, causing file extensions whose type descriptions that are missing in the registry (e.g. *.json) to be omitted from the Type dropdown on the file dialog. Bug: 936987 Change-Id: I58074e8aed6e4ed35ac0501c434faf20a29272f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1495802 Commit-Queue: Eric Lawrence [MSFT] <ericlaw@microsoft.com> Reviewed-by:
Robert Liao <robliao@chromium.org> Reviewed-by:
Patrick Monette <pmonette@chromium.org> Cr-Commit-Position: refs/heads/master@{#637790}
-
Owen Min authored
Disable the test by sheriff because it's flaky. Bug: 938289 Change-Id: I9e9bd5820444562e5485c0a1b445960278cc97dd Tbr: derat@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504115Reviewed-by:
Owen Min <zmin@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#637789}
-
Bruce Dawson authored
The new ARM64 Win32 builder was taking 4-6 hours to build 'all', with goma support. This builder does have to do more build steps due to its cross-compiling nature, but that's ridiculous. It's only using -j 80 (some machines use -j 150), but analysis proved that linking was the biggest problem. That is almost certainly because this build is using symbol_level=2. On a machine without an SSD and with only 30 GB of memory this is a recipe for thrashing. Here are some build result timings from the first ever fully successful build, which summarize the issue nicely: Longest build steps: 406.1 weighted s to build unit_tests.exe, unit_tests.exe.pdb (2028.0 s CPU time) 414.3 weighted s to build browser_tests.exe, browser_tests.exe.pdb (2068.2 s CPU time) 418.7 weighted s to build sync_performance_tests.exe, sync_performance_tests.exe.pdb (2092.9 s CPU time) 433.6 weighted s to build components_unittests.exe, components_unittests.exe.pdb (2185.1 s CPU time) 608.7 weighted s to build captured_sites_interactive_tests.exe, captured_sites_interactive_tests.exe.pdb (3042.6 s CPU time) 1562.7 weighted s to build win_clang_x64/v8_context_snapshot_generator.exe, win_clang_x64/v8_context_snapshot_generator.exe.pdb (1568.4 s CPU time) Time by build-step type: 4985.1 s weighted time to generate 22921 .obj files (395193.4 s CPU time) 9895.1 s weighted time to generate 382 PEFile (linking) files (50980.6 s CPU time) 15529.8 s weighted time (492396.4 s CPU time, 31.7x parallelism) 33749 build steps completed, average of 2.17/s Probably we'd get better results with less parallel linking, but let's just set a better symbol_level, and not spend > 63% of time on linking. Setting a higher -j value is a separate issue. Bug: 938085 Change-Id: I82b8307ad33365af2eac5b568554a42a322ea7d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1502512 Commit-Queue: Dirk Pranke <dpranke@chromium.org> Auto-Submit: Bruce Dawson <brucedawson@chromium.org> Reviewed-by: -
Scott Little authored
Change-Id: Id5abf61919005b1dedf9907be9b03f070d3cc893 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1500596Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Scott Little <sclittle@chromium.org> Cr-Commit-Position: refs/heads/master@{#637787}
-
Elly Fong-Jones authored
Bug: None Change-Id: Ie8cb697360308f9fb3de17a81f1ab26d42944188 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503997 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Auto-Submit: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#637786}
-
Ian Kilpatrick authored
In LayoutNG LayoutObject::IsRenderedLegend() is on a very performance sensitive codepath. This changes the first part of the function just to perform a bit-check, inlines it, and then delegates to IsRenderedLegendInternal(). This improves performance on the microbenchmarks across the board. Based on stack-sampled-metrics this should improve layout performance by 0.5-1%. Bug: 635619 Change-Id: I0332fd15200604c783cbd178cd2f0a0b85f36862 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1490018 Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#637785}
-
Qingsi Wang authored
This reverts commit affe2337. The commit 8ce00191 is also reverted to rebase. Reason for revert: crbug.com/931916 TBR=msramek@chromium.org Original change's description: > Use a separate socket for sending in mDNS code. > > TBR=thestig@chromium.org > > Bug: 899310, 905422 > Change-Id: I5ea07adb3d3241af2f0056299bdd4b5b7b0d479a > Reviewed-on: https://chromium-review.googlesource.com/c/1367366 > Commit-Queue: Qingsi Wang <qingsi@google.com> > Reviewed-by: Eric Orth <ericorth@chromium.org> > Reviewed-by: Martin Šrámek <msramek@chromium.org> > Cr-Commit-Position: refs/heads/master@{#625092} Bug: 899310, 905422 Change-Id: I5bc329f417131dfdbabaa0d041fe9c47395da5e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1498716 Commit-Queue: Qingsi Wang <qingsi@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Eric Orth <ericorth@chromium.org> Cr-Commit-Position: refs/heads/master@{#637784}
-
Stefan Zager authored
IntersectionGeometry is designed to be consumed by blink-internal code using IntersectionObserver; IntersectionObserverEntry is designed to implement the IDL for javascript. This CL divides the classes along those lines, and also makes IntersectionGeometry a true POD type by eliminating LayoutObject* members. R=chrishtr@chromium.org Change-Id: I5bb37aef8e5962cca813efc37f0f0d2c2c6798c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1501312Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: Stefan Zager <szager@chromium.org> Cr-Commit-Position: refs/heads/master@{#637783}
-
dpapad authored
This method was used by the old bookmark manager component extension, which no longer exists. Also removing bookmark related strings that are not used anywhere anymore. Bug: None Change-Id: I3d3f5ba8eeb20019372ca4658c359f48cbfdd470 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1497421 Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Auto-Submit: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#637782}
-
Katie D authored
Bug: 905419 Change-Id: I652369e5e276c2ead9d1e84a15758ff4548d2e80 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1497338Reviewed-by:
David Roger <droger@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#637781}
-
Vadim Gorbachev (bmsdave) authored
related CL: https://chromium-review.googlesource.com/c/v8/v8/+/1484302 Bug: chromium:935197 Change-Id: Iaabd72f3d11f4966b97c548e8f9b1bf222e9956e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1492335 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Sergiy Belozorov <sergiyb@chromium.org> Reviewed-by:
Johannes Henkel <johannes@chromium.org> Cr-Commit-Position: refs/heads/master@{#637780}
-
Min Qin authored
BUG=934941 Change-Id: I66a4aa9d8a547719eb206545e480527e6a690759 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1501219Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#637779}
-
Oksana Zhuravlova authored
This change replaces the usage of InterfaceProvider with DocumentInterfaceBroker in AudioContext. Bug: 936482 Change-Id: Ic3d3ad8f92e5a7554742bdaea07ae13e8fd9887c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1497378Reviewed-by:
Hongchan Choi <hongchan@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#637778}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/1b849dd8336e..59db7d4f1098 Created with: gclient setdep -r src-internal@59db7d4f1098 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. CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=shend@chromium.org,zmin@chromium.org,dvadym@chromium.org,petewil@chromium.org Change-Id: Iafe859f0c1f29b399ab59a5e963bf9ef51d28938 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503912Reviewed-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@{#637777}
-
Luna Lu authored
This allows us to specify value as following: oversized-images * equivalent to oversized-images *(inf) oversized-images 'self' equivalent to oversized-images 'self'(inf); <other-origin>(2.0) Note that by default the allowed threshold is (2.0). This aligns with current layout tests. By default, oversized-images is enabled on * with value inf. Note that this CL does not update FP implementation of oversized-images policy. It is separated into another CL BUG: 924568 Change-Id: I789f5e6f35561ab7c1472e73ad974787ed4aef3e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1480231Reviewed-by:
Ian Clelland <iclelland@chromium.org> Commit-Queue: Luna Lu <loonybear@chromium.org> Cr-Commit-Position: refs/heads/master@{#637776}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/7949f215c118..d36c08623d90 git log 7949f215c118..d36c08623d90 --date=short --no-merges --format='%ad %ae %s' 2019-03-05 titovartem@google.com Add support for simulcast streams in QualityAnalyzingVideoDecoder. 2019-03-05 nisse@webrtc.org Fix mis-spelled TODO items Created with: gclient setdep -r src/third_party/webrtc@d36c08623d90 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 TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I8cdbbd429848fc2a77bd7024dc66ae09b6b0e136 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503959Reviewed-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@{#637775}
-
chromium-autoroll authored
This CL may cause a small binary size increase, roughly proportional to how long it's been since our last AFDO profile roll. For larger increases (around or exceeding 100KB), please file a bug against gbiv@chromium.org. Additional context: https://crbug.com/805539 Please note that, despite rolling to chrome/android, this profile is used for both Linux and Android. The AutoRoll server is located here: https://autoroll.skia.org/r/afdo-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=gbiv@chromium.org Change-Id: I4b54a2ccb26c43eadcf9edc7ceb95aa435cb09c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503937Reviewed-by:
-
Dave Tapuska authored
The feature hasn't shipped yet so we need a bit more time to cover the launch of it. BUG=930906 Change-Id: I053cf1df7933c79ce724c81452ee9cbfa18f7128 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504036Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/adbf3bef6872..5b3bf586f36d git log adbf3bef6872..5b3bf586f36d --date=short --no-merges --format='%ad %ae %s' 2019-03-05 recipe-roller@chromium.org Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@5b3bf586f36d The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-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=agable@chromium.org Change-Id: Ie81fa287ac3d828ad6fe6b3b706ef3aa3e1a9327 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503560Reviewed-by:
-
Peter Boström authored
Bug: chromium:935878 Change-Id: Ie12606060794e6519b0a793037daa80bdf71c47c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503907 Commit-Queue: Peter Boström <pbos@chromium.org> Commit-Queue: Collin Baker <collinbaker@chromium.org> Auto-Submit: Peter Boström <pbos@chromium.org> Reviewed-by:
Collin Baker <collinbaker@chromium.org> Cr-Commit-Position: refs/heads/master@{#637771}
-
Owen Min authored
Disable flaky test Bug: 938200 TBR: petewil@chromium.org Change-Id: Iecbe1a710e39771cf3001f42e8f5e053c9fe6700 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503999Reviewed-by:
-
Qingsi Wang authored
Bug: 899310, 931916 Change-Id: Iae166ec6ac0814ab761cf55365cc17768ecaa9fb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1500782 Commit-Queue: Qingsi Wang <qingsi@chromium.org> Reviewed-by:
-
Suman Kancherla authored
This refactoring separates the responsibilities of VRBrowserRendererThreadWin and VrUiHost for drawing UI. Existing code requires two calls to show/hide UI. That has been reduced to one. That way, VrUiHost can just call appropriate setters and expect the UI to drawn appropriately. In the future, this can be extended to displaying toasts etc. Bug: 913502 Change-Id: I129e885603b8898461ec0de012007b852ea89268 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1500415 Commit-Queue: Suman Kancherla <sumankancherla@chromium.org> Reviewed-by:
Bill Orr <billorr@chromium.org> Reviewed-by:
Klaus Weidner <klausw@chromium.org> Cr-Commit-Position: refs/heads/master@{#637768}
-
Chris Hamilton authored
This retires the kIsLoading property from the page node as part of the refactor that aims to get rid of the "bag of properties" from node impls. BUG=910288 Change-Id: I6ca90dcfb4096a2b399f243a0da6e40c78fe21af Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1497340Reviewed-by:
Will Harris <wfh@chromium.org> Reviewed-by:
Sigurður Ásgeirsson <siggi@chromium.org> Commit-Queue: Chris Hamilton <chrisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#637767}
-
Pedro Amaral authored
This CL hides/shows the bottom bar when the phone orientation changes from portrait to landscape. The bottom bar has two parts: an Android View and a composited toolbar. This CL hides the Android View by setting it to GONE and hides the composited view by setting the bottom control height to 0. The bottom bar code was modified to make the composited view's visibility be tied to the browser controls height. Native pages previously did not support changing the bottom controls height so this CL also adds that functionality to native pages. ToolbarSwipeLayout previously did not track the visibility of the bottom bar so this CL adds that ability. The other part of this CL is showing the top toolbar buttons when the bottom bar is hidden. This was done by setting them to GONE and tracking their visibility state with |ToolbarPhone#mHidingToolbarButtons|. Bug: 929164 Change-Id: Ide09baf7521a8986abeec6b03b952baf8d91925a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1482002Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Reviewed-by:
Pedro Amaral <amaralp@chromium.org> Cr-Commit-Position: refs/heads/master@{#637766}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/2f1c1d7eeebb..f094bac94992 git log 2f1c1d7eeebb..f094bac94992 --date=short --no-merges --format='%ad %ae %s' 2019-03-05 courtneygo@google.com Allow enabling trace annotations Created with: gclient setdep -r src/third_party/angle@f094bac94992 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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: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=cwallez@chromium.org Change-Id: I00bb260f30845cb0a77da25dc39300b68cd0803f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503958Reviewed-by:
-
Sergey Ulanov authored
IP_MULTICAST_IF is currently broken on Fuchsia. Disable it temporarily until the issue is fixed. Bug: 938101 Change-Id: I5d8af571f069eff573fa5ab8ac06010c869984ca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503911 Auto-Submit: Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Fabrice de Gans-Riberi <fdegans@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#637764}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/5fb83a97087d..86f6ac086c8b git log 5fb83a97087d..86f6ac086c8b --date=short --no-merges --format='%ad %ae %s' 2019-03-05 33432579+alan-baker@users.noreply.github.com Add a test for two back-edges in a continue (#2418) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@86f6ac086c8b The AutoRoll server is located here: https://autoroll.skia.org/r/spirv-tools-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=dsinclair@chromium.org Change-Id: I1080693cb980c87ea13a6c37146ef553bb0bd9c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503853Reviewed-by:
-
Tien Mai authored
Added a registry key that can be used to override the default GLS. Bug: 938488 Change-Id: I7cd91ce32765e721921c8c35ac151c7c52d04dab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503623Reviewed-by:
Roger Tawa <rogerta@chromium.org> Commit-Queue: Tien Mai <tienmai@chromium.org> Cr-Commit-Position: refs/heads/master@{#637762}
-
Eric Seckler authored
Includes changes to TrackEventThreadLocalEventSink due to changed proto field names. TBR: skyostil@chromium.org Change-Id: I00549d8fadea9fc6245bd2bb9d64a3fc01b5c96c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503379 Commit-Queue: Eric Seckler <eseckler@chromium.org> Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Cr-Commit-Position: refs/heads/master@{#637761}
-
Owen Min authored
This reverts commit 1c3948d8. Reason for revert: The patch caused multiple content_unittest flaky. https://bugs.chromium.org/p/chromium/issues/detail?id=938126 This CL could be relanded once these flaky tests are fixed or disabled. Original change's description: > Run pending tasks in ~TestBrowserThreadBundle. > > This makes sure we run all remaining tasks in tests. This should be the > case even for ExecutionMode::QUEUED as documented here: > https://cs.chromium.org/chromium/src/base/test/scoped_task_environment.h?l=42 > > Also lift the restriction to only flush tasks in !MOCK_TIME && !QUEUED modes > which is unnecessary now that TestBrowserThreadBundle is-a > ScopedTaskEnvironment (http://crrev.com/616622). > > Bug: 708584 > Change-Id: I52773e27d3875bd246bb1cd7eea92c9e11fc70bf > Reviewed-on: https://chromium-review.googlesource.com/c/1480460 > Commit-Queue: Gabriel Charette <gab@chromium.org> > Reviewed-by: Gabriel Charette <gab@chromium.org> > Cr-Commit-Position: refs/heads/master@{#637288} TBR=gab@chromium.org,knollr@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 708584 Bug: 938126 Change-Id: I8b43a5c90b8f28eee2817a94cd556b2742e2f58c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503993Reviewed-by:
-
Jan Krcal authored
This CL adds tests for remote changes for autofill wallet metadata. It uses parametrized tests because the same behaviour and conflict resolution should trigger for initial sync additions, non-initial additions as well as for non-initial updated. Bug: 936013 Change-Id: Ia06ce99dbf9f3da0c2e2130d0bcb0dc4caa80b0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1491312 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
-
Nicolas authored
New tests check that BrowserSwitcherService writes "cache.dat" and "sitelistcache.dat" files in AppData\Local (on Windows). They also verify the file contents. Change-Id: I3ee197c2013c9c7c616c46655b552f629d32a6c3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1500973 Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Cr-Commit-Position: refs/heads/master@{#637758}
-
Leonard Grey authored
Motivation: https://chromium-review.googlesource.com/c/chromium/src/+/1492047 Change-Id: I89d2fc98569ddcd948e1eeb7e27faac3c5afc587 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503629 Commit-Queue: Avi Drissman <avi@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#637757}
-
