- 19 Nov, 2020 40 commits
-
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/3a578ae35233..16973d613fed If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-linux-chromium Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: I02219e631374e971a799836b882f7210ea92988b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2549401Reviewed-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@{#829152}
-
Chrome Cunningham authored
This makes reset() match the spec. All pending/active work should be aborted and any late arriving outputs should not make it to JS. Bug: 1121340 Change-Id: I4963a39023320e081e2d2c13bba2d43fed990f91 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2541646 Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Reviewed-by:
Dan Sanders <sandersd@chromium.org> Cr-Commit-Position: refs/heads/master@{#829151}
-
anina koehler authored
Bug: None Change-Id: I921ea0c990f1b20f6e993e136e472f14df6d2175 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2539195 Commit-Queue: Anina Koehler <aninak@google.com> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Alexander Hendrich <hendrich@chromium.org> Cr-Commit-Position: refs/heads/master@{#829150}
-
Hajime Hoshi authored
WebFilesystem doesn't have notions of handlers or connections, then it should be safe to allow pages with WebFilesystem to be put into the BFcache. Let's allow WebFilesystem for BFcache in order to improve the BFcache hit rate. This is implemented behind a flag and the feature is disabled by default. This means that this CL doesn't change the behavior by default. We might enable the feature later. Bug: 1144133 Change-Id: Idf5d9cfb8a4365ae65743c2becc7a4ef8ec66999 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531155 Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Cr-Commit-Position: refs/heads/master@{#829149}
-
Internal Frameworks Autoroller authored
TBR=bling-team@google.com Change-Id: I0c44f09a9606445e7de459a882c47b34c4c0a5f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2549721Reviewed-by:
Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#829148}
-
Maksim Sadym authored
Reland "Fix node screenshot. Part 1. Add `captureBeyondViewport` param to CDP command `captureScreenshot`." This reverts commit 85d844e3. Reason for revert: reland initial commit with disabled flaky test. Original change's description: > Revert "Fix node screenshot. Part 1. Add `captureBeyondViewport` param to CDP command `captureScreenshot`." > > This reverts commit 114bc6ab. > > Reason for revert: CaptureScreenshotTest.CaptureScreenshotBeyondViewport_InnerScrollbarsAreShown is failing consistently on linux-chromeos-rel: > > https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-rel/42849 > https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-rel/42850 > > Original change's description: > > Fix node screenshot. Part 1. Add `captureBeyondViewport` param to CDP command `captureScreenshot`. > > > > 1. Add `captureBeyondViewport` param to CDP command `captureScreenshot`. > > 2. Use `WebSettingsImpl::SetMainFrameClipsContent(false)` to force the whole page to be rendered. > > 3. Set `hide_scrollbars` to avoid artificial scrollbars. > > 4. Set `record_whole_document` to force the whole page to be rendered. > > 5. Removed DCHECK from `third_party/blink/renderer/core/frame/visual_viewport.cc`, as soon as it didn't expect to have preferences to be changed "on the flight". > > > > Details: https://bugs.chromium.org/p/chromium/issues/detail?id=1003629#c37 > > > > Screenshots: > > > > - Before: https://i.imgur.com/yt6WZRx.png > > - Patchset #1: https://i.imgur.com/VsocJ3L.png - artificial scrollbars. > > - Patchset #3: > > * https://imgur.com/UVqpzUQ - line breaks are not exactly the same as in original view. > > * https://imgur.com/MXpgHOl > > * https://imgur.com/haxemcr - not aligned with the node. > > * https://imgur.com/RbdlYVT > > - Patchset #18: > > * https://imgur.com/EWUmn0O > > * https://imgur.com/DUrQ1yF > > * https://imgur.com/cT5oBSi - perfectly aligned. > > * https://imgur.com/Jy4UWtf - no artificial scrollbars (because of the view extended). > > > > Bug: 1003629 > > Change-Id: I6bbc85cd0995626a8b1fb748ec9048c9d586200e > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2470876 > > Commit-Queue: Maksim Sadym <sadym@chromium.org> > > Auto-Submit: Maksim Sadym <sadym@chromium.org> > > Reviewed-by: Andrey Kosyakov <caseq@chromium.org> > > Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#827440} > > TBR=dgozman@chromium.org,wangxianzhu@chromium.org,pdr@chromium.org,caseq@chromium.org,sigurds@chromium.org,sadym@chromium.org > > Change-Id: Iaa9b6ee9f5584adbb4a521808e2d201834a6a6dc > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 1003629 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2538543 > Reviewed-by: Scott Little <sclittle@chromium.org> > Commit-Queue: Scott Little <sclittle@chromium.org> > Cr-Commit-Position: refs/heads/master@{#827505} TBR=dgozman@chromium.org,wangxianzhu@chromium.org,pdr@chromium.org,caseq@chromium.org,sclittle@chromium.org,sigurds@chromium.org,sadym@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1003629 Change-Id: I2d394d77d0235762c61449ff8e90573de4c0c5d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545524Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Maksim Sadym <sadym@chromium.org> Cr-Commit-Position: refs/heads/master@{#829147}
-
Timothy Loh authored
This CL makes the CrosUsbDetector unmount filesystems from USB drives before sharing them with a VM. Currently USB drives are not allowed by the permission broker unless the VID:PID is explicitly allowed in enterprise policy, although we intend to change this behaviour after M88 branches. Mounted archive files from the USB will be unmounted, but zip files require the files-zip-mount flag turned on first (see bug 912236). The attach flow is restructured from a recursive flow (retry from start upon error) to a linear flow, making it easier to reason about. A subsequent CL will update the settings UI pages to show an error if unmounting (or any other step in the attach process) fails. Bug: 1146325 Change-Id: Id0ea4805fbdf901b56391929c2c4ef204380d9e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2517231 Commit-Queue: Timothy Loh <timloh@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Nicholas Verne <nverne@chromium.org> Reviewed-by:
Jeremie Boulic <jboulic@chromium.org> Cr-Commit-Position: refs/heads/master@{#829146}
-
Sergei Glazunov authored
This is a prerequisite for a subsequent CL that will allow us to enable cookies and tags/refcounts independently for a given partition. In addition, fix a regression in PartitionAllocTest.Bookkeeping, which occurs when BRP is enabled. Bug: 1073933 Change-Id: I7557e89e4dc51082ba043a0697f974a7d30c8c5a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2540984 Commit-Queue: Sergei Glazunov <glazunov@google.com> Reviewed-by:
Bartek Nowierski <bartekn@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/8645ae9d..47d4ef3f Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I3688635721a8938e984d22ef6c663c4d73b12cb0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548965Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#829144}
-
Daniel Cheng authored
Bug: none Change-Id: I5f9ad4c4b10090b8fb9d76acf86f24cf71d5d1c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548964 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Auto-Submit: Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#829143}
-
Zhaoliang Ma authored
There is no need to create |input_buffers_| when import mode is used, this CL limits the input buffer creation to non-import mode. Bug: b:172209195 Test: apprtc on atlas Change-Id: Idc42cc6b87add2564fc2b31d771477d9c71aefeb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548886 Commit-Queue: Zhaoliang Ma <zhaoliang.ma@intel.com> Reviewed-by:
Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#829142}
-
chromium-autoroll authored
Roll Chrome Linux PGO profile from chrome-linux-master-1605743960-c6eb23d92bd5ebb2b738686f930937312c075c5e.profdata to chrome-linux-master-1605765502-afac6c1491dbda42eb7f9fa994b3ce280c680afd.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-linux-chromium Please CC pgo-profile-sheriffs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: I840cc76540ec3bce66c0eca9629659eb3268c044 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2549420Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/004bf3ee3fb4..d40c6d7f3bef If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC keishi@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: None Tbr: keishi@google.com Change-Id: I66d9828a630dad5fde1829e679fcf9f80f8abddd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548807Reviewed-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@{#829140}
-
Paul Wankadia authored
https://chromium.googlesource.com/external/github.com/google/re2.git/+log/a84f9718f35f..9e5430536b59 $ git log a84f9718f..9e5430536 --date=short --no-merges --format='%ad %ae %s' 2020-11-17 junyer Make benchmarks use substrings of a random text buffer. 2020-11-15 twpayne Add note about Go Unicode character classes 2020-10-26 junyer Fix symbol visibility and add test coverage. 2020-10-13 junyer Get the conditional right this time. Sigh. 2020-10-13 junyer Don't support ParseFrom() on MSVC. It can cause ICEs. Created with: roll-dep src/third_party/re2/src R=thakis@chromium.org Change-Id: I46f07ce3400f21210bed61e07825f340a6232d30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548962 Commit-Queue: Paul Wankadia <junyer@chromium.org> Commit-Queue: Max Moroz <mmoroz@chromium.org> Reviewed-by:
Max Moroz <mmoroz@chromium.org> Cr-Commit-Position: refs/heads/master@{#829139}
-
Antonio Sartori authored
The Content Security Policies code under services/network and the relative mojo types still misses support for a few CSP directives. We need to support all CSP directives in order to use it as a replacement to the Blink types, and also if we want to add Content Security Policies to the Policy Container (see attached bugs). This CL implements support for the directive 'trusted-types'. Bug: 1021462,1149272 Change-Id: I78797427f9239e1b57aca3e71d4de83e9897e7a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2539907Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Commit-Queue: Antonio Sartori <antoniosartori@chromium.org> Cr-Commit-Position: refs/heads/master@{#829138}
-
Avi Drissman authored
@available works now for macOS 11.0; use it. Fixed: 1105187, 1115294 Change-Id: Icecd3b71b913e474629aa2e9c8be4936954e9689 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545080 Commit-Queue: Dominik Röttsches <drott@chromium.org> Auto-Submit: Avi Drissman <avi@chromium.org> Reviewed-by:
Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/master@{#829137}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/8b76d37f9f9a..8d63a7fc4924 2020-11-19 syoussefi@chromium.org Basic support for running deqp khr gles32 tests If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC cnorthrop@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md 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-asan;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: cnorthrop@google.com Change-Id: I643a27b8cd6fcfe59c71120178f12f9fbf298a2e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548607Reviewed-by:
-
Alexandre Courbot authored
When a dynamic resolution change even happens, we dismiss all the pictures that were sent to the client. Although it should never happen, if a reuse picture event takes place, it will re-insert the picture ID into the "at client" map, messing with reference count and resulting in broken playback. Make sure this does not happen by clearing the entry for a picture ID when it gets imported again. BUG=b:161759267 BUG=b:165035340 BUG=b:172881995 TEST=DRC with Youtube Android app on Trogdor now takes place smoothly. TEST=CtsMediaTestCases android.media.cts.AdaptivePlaybackTest#testH264Adaptive passes on Trogdor Change-Id: I503a6aacbe8647398fe502eded3ed6c799aa2faa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548588 Commit-Queue: Alexandre Courbot <acourbot@chromium.org> Auto-Submit: Alexandre Courbot <acourbot@chromium.org> Reviewed-by:
Chih-Yu Huang <akahuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#829135}
-
Brian Geffon authored
Fix MSAN complaint about uninitialized memory. BUG=chromium:1149805 TBR=rockot@google.com Change-Id: Ic8956c31730e6859ce68d2da047970d4e1429693 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548900Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Mike West <mkwst@chromium.org> Auto-Submit: Brian Geffon <bgeffon@chromium.org> Cr-Commit-Position: refs/heads/master@{#829134}
-
Hiroki Nakagawa authored
This CL adds a chrome flag "enable-prerender2" so that we can easily test the feature without building a custom Chrome. This chrome flag stands on the existing blink::features::kPrerender2 flag. This chrome flag has 3 variations: - Disabled (Default) - Enabled - Enabled without activation "Enabled without activation" runs prerendering but doesn't activate the prerendered page. With this param, content::NavigationRequest looks up the prerendered page, but destroys it and then issues a network request like regular navigation. Bug: 1148647 Change-Id: I9734eb2f2a63153dbc69bdf9a8b3ce0ca535979f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543724 Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org> Reviewed-by:
Takashi Toyoshima <toyoshim@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#829133}
-
Eugene But authored
This may happen if memory footprint was not recorded in the previous session. Bug: 1103752 Change-Id: Ib840b2a12cd1c6c6e432b2e00d2dfbbcf113cbe1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548184 Commit-Queue: Olivier Robin <olivierrobin@chromium.org> Auto-Submit: Eugene But <eugenebut@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#829132}
-
Luciano Pacheco authored
Change Audio Player to configure the ContentMetadataProvider to load the SharedWorker as modules based on the feature flag AudioPlayerJsModules. Bug: 1133186, 1150718 Change-Id: Idd07cac5ffeb6503a2aeab352421214dfc42bce5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548106 Commit-Queue: Luciano Pacheco <lucmult@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
-
wutao authored
When ImageView has an |image_| with different size than the |image_size_|, it will resize and draw the |image_|. The quality is not as good as if we resize the |image_| to be the same as the |image_size_| with |RESIZE_BEST| method. Bug: b/168615460 Test: manual Change-Id: I1a843ea7d86d003cd71f14061147b1afc81b5769 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2546478 Commit-Queue: Tao Wu <wutao@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#829130}
-
Swapnil authored
The browsertests of WebRtc policies are moved from policy_browsertests to a separate file. Bug: 1148739 Change-Id: If1a762a038521ebd0fd973e3042698ed5900d2cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2542502 Commit-Queue: Swapnil Gupta <swapnilgupta@google.com> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Cr-Commit-Position: refs/heads/master@{#829129}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/021b77e7f0c7..3a578ae35233 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-mac-chromium Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: If11933eba3348e3b14daf71c2f46830e3eb1d31c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548467Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/ad797e966480..a06769a8ea10 2020-11-19 szuend@chromium.org Replace property with getter in TextPrompt.js If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: chromium:1011811 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I923e5cbee73ec68e6ce4d1497df053dd7a20694e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548806Reviewed-by:
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/3a578ae35233..16973d613fed 2020-11-19 treehugger-gerrit@google.com Merge "Add test for profiler CPU guardrail." If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: I17fb8ca1efc3a96782dc6c711665d56e633bf96a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548609Reviewed-by:
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-aemu-chromium-autoroll Please CC chrome-fuchsia-gardener@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: chrome-fuchsia-gardener@grotations.appspotmail.com Change-Id: I191edd19882123c1ec5c4fde98c543f17d48123c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548195Reviewed-by:
-
Bo Majewski authored
The purpose of script loader is to enable loading scripts at runtime. The secondary purpose is to centralize code for creating script tags. Bug: 1113981 Change-Id: I8780d8315964ba14e2d24854d9da1e81a8d2e862 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2547744 Commit-Queue: Bo Majewski <majewski@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#829124}
-
Alan Cutter authored
This is a reland of d4e5f661 Re-fixed the Windows deps requirements. Also verified gn check locally on Linux cross compile target for Windows. https://chromium-review.googlesource.com/c/chromium/src/+/2546142/3/chrome/browser/win/conflicts/BUILD.gn Original change's description: > Reland "Extract browser_process.h/cc into own gn target" > > This is a reland of 86556051 > > The original CL missed a dep requirement in the Chrome branded > Windows build. This has been fixed: > https://chromium-review.googlesource.com/c/chromium/src/+/2543873/2/chrome/browser/win/conflicts/BUILD.gn > > Original change's description: > > Extract browser_process.h/cc into own gn target > > > > This CL pulls out browser_process.h and browser_process.cc into their > > own gn target so that other gn targets can depend on it without creating > > cyclic dependencies with //chrome/browser:browser. > > > > This is required for CL: > > https://chromium-review.googlesource.com/c/chromium/src/+/2528519 > > > > Bug: 1147410, 883570, 925153 > > Change-Id: I17ef1f0b1683c4abb7588a6ef24ec89f15647545 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529309 > > Commit-Queue: Alan Cutter <alancutter@chromium.org> > > Reviewed-by: Filip Gorski <fgorski@chromium.org> > > Reviewed-by: Glen Robertson <glenrob@chromium.org> > > Auto-Submit: Alan Cutter <alancutter@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#828077} > > Bug: 1147410 > Bug: 883570 > Bug: 925153 > Change-Id: I18faaddc3850ceafb924bda6d8e197f886e79433 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543873 > Reviewed-by: Glen Robertson <glenrob@chromium.org> > Reviewed-by: Filip Gorski <fgorski@chromium.org> > Commit-Queue: Alan Cutter <alancutter@chromium.org> > Cr-Commit-Position: refs/heads/master@{#828442} Bug: 1147410 Bug: 883570 Bug: 925153 Change-Id: Ia8249933997b6f23f8c0ca63ab927bb70ce6849b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2546142Reviewed-by:
Glen Robertson <glenrob@chromium.org> Reviewed-by:
Filip Gorski <fgorski@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#829123}
-
Andrey Kosyakov authored
- disable sending of missed frames to frame observers added after we cancel frame in ExternalBeginFrameSourceMojo to avoid sending multiple frames, or sending frames at the time when the client does not intend. - force full frames (i.e. animation_only = false) till we have a first frame drawn. We used to do this on the client side, but this really belongs to the back-end (and maybe further down to the renderer). Bug: 1149042 Change-Id: Id1d79fbd4d6f355ebf6b9a441a5df772c7cf3b4a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2538488Reviewed-by:
Sunny Sachanandani <sunnyps@chromium.org> Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#829122}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/6e44bd85f3a3..0c019226a958 2020-11-19 dproy@chromium.org [tbmv2] Add a way to add alert groupings to histograms If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC nuskos@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:chromeos-kevin-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 Bug: chromium:1150233 Tbr: nuskos@google.com Change-Id: I5b3f180d56a3cc409deaffb08e2ce4e611d940b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548318Reviewed-by:
-
Alexandre Courbot authored
GetFrame() returns VideoFrames with a destruction observer than puts the frame back into the pool once the client is done with it. However, when a reconfiguration takes place, we are now working with a new set of frames, and we should not put the old ones back into the pool. Make sure this does not happen by invalidating the weak pointers the destruction observers has been posted with when switching to a new set of frames. BUG=b:161759267 BUG=b:165035340 BUG=b:172881995 TEST=CtsMediaTestCases android.media.cts.AdaptivePlaybackTest#testH264Adaptive passes on Trogdor TEST=GtsExoPlayerTestCases com.google.android.exoplayer.gts.DashTest passes on Trogdor Change-Id: Ia1a28fb64d5659eb1eedf0b96ad4bc344e67fdad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548587 Auto-Submit: Alexandre Courbot <acourbot@chromium.org> Commit-Queue: Chih-Yu Huang <akahuang@chromium.org> Reviewed-by:
-
Austin Tankiang authored
These broke due to https://crrev.com/c/1630010. Mark these links as rel="opener" to workaround this for now. Bug: 1149257 Change-Id: I4d21a03811fdef4bc7165ffdd172e004e6107d05 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2547858Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium f1bad64e. With Chromium commits locally applied on WPT: 0977dd12 "Move native paint worklet off thread" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=smcgruer@google.com No-Export: true Cq-Include-Trybots: luci.chromium.try:linux-wpt-identity-fyi-rel,linux-wpt-payments-fyi-rel Change-Id: I44898f3e29b2a182514a3404888e6e0675f9c58a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548920Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#829118}
-
Polina Bondarenko authored
Before starting ARC container or VM try to load a snapshot if available. ARC data snapshots are loaded only for MGS. If there is no snapshot or the process fails, start ARC session as usual. BUG=b:161221001 TEST=components_unittests TEST=unit_tests Change-Id: I7e222f9abd11b0471a40a475c73931886582bf55 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2498488 Auto-Submit: Polina Bondarenko <pbond@chromium.org> Commit-Queue: Polina Bondarenko <pbond@chromium.org> Reviewed-by:
Oleksandr Peletskyi <peletskyi@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Cr-Commit-Position: refs/heads/master@{#829117}
-
Rebekah Potter authored
This reverts commit 18dac5f7. Reason for revert: Speculative revert. The IDs for these resources don't match what they should in the uprev failure described at https://bugs.chromium.org/p/chromium/issues/detail?id=1150466 It's likely this CL triggered the bot issue, but probably isn't the root cause (needs further investigation). Original change's description: > WebUI: Auto-generated grd file for chrome://usb-internals. > > Bug: 1132403 > Change-Id: Ie44b69c074794cbd10a08289e389e502e8e94ccc > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2539892 > Reviewed-by: Rebekah Potter <rbpotter@chromium.org> > Commit-Queue: dpapad <dpapad@chromium.org> > Cr-Commit-Position: refs/heads/master@{#828130} TBR=dpapad@chromium.org,rbpotter@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1132403 Change-Id: I77d53bf592318c98e560b2f193cda0edd8dff0d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548650 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#829116}
-
Polina Bondarenko authored
A new static library for components/arc/enterprise is needed to be able to call ArcDataSnapshotdManager from ArcSessionImpl. BUG=b:161221001 TEST=compile Change-Id: I8da4385cdc2d1bad74721a2c623631aa9894ff98 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543063Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/458ab69a..8645ae9d Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ia676a2140e6016adfaf425d435ddcaf6a9e2463c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548885Reviewed-by:
-
Luciano Pacheco authored
Add the optimize_webui() for the SharedWorker within Audio Player. Change Audio Player initialization to switch the SharedWorker script url based on the feature flag `AudioPlayerJsModules`. To be able to check the state of the feature flag refactored the initialization to: 1. Use the load_time_data.js because that's how we send the state of the flag from the C++ to the JS. 2. Delayed the initialization of MetadataModel to be after the load_time_data initialization because MetadataModel is what starts the SharedWorker. 3. Change the call site to MetadataModel which is in fetchMetadata_() to wait for MetadataModel initialization. No change in behavior, so the current tests cover this all. I've tested manually and run the integration tests with the flag on and off. Test: browser_tests --gtest_filter="AudioPlayer*:OpenAudioFiles*" Bug: 1133186 Change-Id: I62bd50a4ef17c0b799112727c0540fc5dc8f58d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2546306 Commit-Queue: Jeremie Boulic <jboulic@chromium.org> Reviewed-by:
-
