- 10 Nov, 2020 40 commits
-
-
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: I982dd85364561c588f4457dc80047fba48082a33 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528801Reviewed-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@{#825711}
-
sandromaggi authored
This refactors WaitForDocumentReadyState (and GetDocument ReadyState) to take an (optional) ElementFinder::Result instead of an optional Selector. An optional ElementFinder::Result is here defined as having an empty node_frame_id (which will map to the main frame). Bug: b/2523176 Change-Id: I7db89cc0ec2ede071616b30db42c1f8de9ab2d0b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526441 Commit-Queue: Sandro Maggi <sandromaggi@google.com> Reviewed-by:
Mathias Carlen <mcarlen@chromium.org> Cr-Commit-Position: refs/heads/master@{#825710}
-
Yuta Hijikata authored
The change is mostly mechanical replacing defined(OS_CHROMEOS) with BUILDFLAG(IS_CHROMEOS_ASH) and GN variable is_chromeos with is_ash with some special cases (For those cases please refer to http://go/lacros-macros). The patch is made in preparation to switching lacros build from target_os=linux to target_os=chromeos. This will prevent lacros from changing behaviour after the switch. Bug: 1052397 Change-Id: Ib9e67a3cccd20549836019db36c507d71db70aa2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2494202 Commit-Queue: Yuta Hijikata <ythjkt@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Cr-Commit-Position: refs/heads/master@{#825709}
-
Ben Reich authored
Move the JavaScript script output from being saved in the devtools_code_coverage directory instead moving into a subdirectory called scripts. This avoids pollution of the devtools_code_coverage directory as merged coverage files will be saved there. R=noel@chromium.org Bug: 1113941 Test: browser_tests --gtest_filter=*FilesApp* --devtools-code-coverage Change-Id: I109846b21d2d31e9a677f68f360409e2ecb3897e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527903Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Ben Reich <benreich@chromium.org> Auto-Submit: Ben Reich <benreich@chromium.org> Cr-Commit-Position: refs/heads/master@{#825708}
-
sandromaggi authored
The previous implementation used a differing strategy based on the optional selector. This CL changes to use the |EvaluateParams| based approach for both cases. This is step 1 in a plan to refactor * WaitForDocumentReadyState * WaitForDocumentToBecomeInteractive to become one. Bug: b/172542134 Change-Id: I5d8cd765e4cb4e922bd3f5e65a03678caa0d52c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523176 Commit-Queue: Sandro Maggi <sandromaggi@google.com> Reviewed-by:
-
Yusuf Sengul authored
Bug: 1147244 Change-Id: Ib0931d9d8090c343483a1d2c0bcf98ddc2c7d18b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527687Reviewed-by:
Rakesh Soma <rakeshsoma@google.com> Commit-Queue: Yusuf Sengul <yusufsn@google.com> Cr-Commit-Position: refs/heads/master@{#825706}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/0833ef41..4f7c1ce4 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: I7cc1004b02a6dc76d3f8c713c0668f162b0a88ac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527538Reviewed-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@{#825705}
-
Kenichi Ishibashi authored
It owns remotes to ServiceWorkerResource{Reader,Writer} which can be disconnected at any time. Handle these disconnections to provide graceful error propagations. This CL doesn't add tests for some comparison cases (copying and resuming). Follow-up CLs will add these tests. Bug: 1133143 Change-Id: Ie8d76b93ff0ffbf3bfde4ddfe2dfe98b2ab920d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2525702Reviewed-by:Makoto Shimazu <shimazu@chromium.org> Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Cr-Commit-Position: refs/heads/master@{#825704}
-
Jérémie Boulic authored
In //ui/file_manager/file_manager/common/js: - file_type.js - file_type_unittest.js - lru_cache.js - lru_cache_unittest.js Bug: 1133186 Change-Id: Id2b0dc75b4bc18969e6e7b862d0c9282baa0ba06 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2521886 Auto-Submit: Jeremie Boulic <jboulic@chromium.org> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Commit-Queue: Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#825703}
-
Bartek Nowierski authored
We're currently evaluating performance of PartitionAlloc-Everywhere (PA-E). Any regressions from zeroing would be inedvartantly attributed to PA-E, as it hugely increases number of memory allocations going through this path. The best course of action would be to postone zeroing until PA-E is fully evaluated and enabled in trunk. Then we should re-enable zeroing and properly re-evaluate it in the new world. Change-Id: I06ee7c63d10b2a14c76a7bd51faab7b616621c88 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2521881Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Chris Palmer <palmer@chromium.org> Commit-Queue: Bartek Nowierski <bartekn@chromium.org> Cr-Commit-Position: refs/heads/master@{#825702}
-
Trent Begin authored
This change adds an implementation of the net error diagnostics dialog for Chrome OS. Now if the connectivity diagnostics webui feature is enabled, the new application is launched, otherwise the legacy chrome app is launched. Bug: 1144261 Change-Id: I924cc5c2fbedfb1a7a5cba90fcd6bbf0b3f48855 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510799 Commit-Queue: Trent Begin <tbegin@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Edward Jung (EMEA) <edwardjung@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#825701}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/b6999d94c051..ca2031075565 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 ricea@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: chromium:1064151,chromium:1078229,chromium:1131897,chromium:1136305,chromium:1141619,chromium:1146194,chromium:1147190 Tbr: ricea@google.com Change-Id: I275b0cdd2cb00115c4574053e978cac57d40d75b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528799Reviewed-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@{#825700}
-
Yuchen Liu authored
b6f2d0699633 [Android] Fix un-paired namespace declare for higher __ANDROID_API__ Change-Id: I6894e4f9a9455f180fdca0b81e8540ff4613f727 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527799 Commit-Queue: Yuchen Liu <yucliu@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Auto-Submit: Yuchen Liu <yucliu@chromium.org> Reviewed-by:Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#825699}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/1d532cbd0baa..eba8b028ddbc 2020-11-10 adsrini@google.com Merge "Add a proto for frametimeline events" 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: I7aaee782b151bcd021fc3c44ee1ec566eb38e58f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528610Reviewed-by:
-
Alan Cutter authored
This CL migrates the Chrome OS JSON config for the Google Docs preinstalled web app (located here [0]) up into the Chrome binary. This default web app is going to be used by Chrome Windows/Mac/Linux so this CL is deduping the offline data into one place. A separate CL will clean up the redundant JSON config. [0]: https://chrome-internal.googlesource.com/chromeos/overlays/chromeos-overlay/+/master/chromeos-base/chromeos-default-apps/files/web_apps/docs.json Bug: 1104692 Change-Id: I4f713bf1dec230fe4c6f3c691d4d954ac0c5f340 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2524809 Commit-Queue: Alan Cutter <alancutter@chromium.org> Reviewed-by:
calamity <calamity@chromium.org> Reviewed-by:
Glen Robertson <glenrob@chromium.org> Cr-Commit-Position: refs/heads/master@{#825697}
-
Steven Valdez authored
Design: https://docs.google.com/document/d/1-4n1RLaJs8ANw34TsS6BVt7ARa7lK70F6aHFG2CL3Rc/edit#heading=h.6a92f2gfl9le Bug: 1133310 Change-Id: I70210d3f6ac82450607acd89a78d08a53dd52ef4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2436763 Commit-Queue: Steven Valdez <svaldez@chromium.org> Reviewed-by:
David Van Cleve <davidvc@chromium.org> Cr-Commit-Position: refs/heads/master@{#825696}
-
Robert Ogden authored
This CL implements a small class that will keep track of all Retry-After responses from prefetched origins. Not done in this CL, saved for follow-ups: * Integrating with BrowsingDataRemover * Persisting to prefs Bug: 1146210 Change-Id: I19aa8f8272da40a88ca457286a1f8aac46640424 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2522193Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Commit-Queue: Robert Ogden <robertogden@chromium.org> Cr-Commit-Position: refs/heads/master@{#825695}
-
Yuta Hijikata authored
The change is mostly mechanical replacing defined(OS_CHROMEOS) with BUILDFLAG(IS_CHROMEOS_ASH) and GN variable is_chromeos with is_ash with some special cases (For those cases please refer to http://go/lacros-macros). The patch is made in preparation to switching lacros build from target_os=linux to target_os=chromeos. This will prevent lacros from changing behaviour after the switch. Bug: 1052397 Change-Id: I34cfe1a8d474a2b146138ce2502c4cb1875c37cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2494194Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: Yuta Hijikata <ythjkt@chromium.org> Cr-Commit-Position: refs/heads/master@{#825694}
-
James Vecore authored
Change-Id: I686ea0b14646a318aa4e6e1e2da47cde5022f6d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526629 Auto-Submit: James Vecore <vecore@google.com> Reviewed-by:
Josh Nohle <nohle@chromium.org> Reviewed-by:
Ryan Hansberry <hansberry@chromium.org> Commit-Queue: James Vecore <vecore@google.com> Cr-Commit-Position: refs/heads/master@{#825693}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1604931688-79a6ca0d5aca76d3d27163162f00dd49eb38f0e8.profdata to chrome-win32-master-1604955437-61247d42a6140f126d61086b5421948904b6ffb0.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-win32-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:win-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: Ibef9174ab493d69e5426c561d503463deba1bff3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528609Reviewed-by:
-
Mohamed Amir Yosef authored
... upon opt-in to account store passwords. Bug: 1145971 Change-Id: Ib8f9e07db5ddf4267afe42324bcdd76ec9841a4d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526103Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Cr-Commit-Position: refs/heads/master@{#825691}
-
Luciano Pacheco authored
Elements: <track-info-panel>, <track-list>, <control-panel> and <audio-player>. Move the <link> tag inside the <template> because this is enforced by polymer_modulizer() this requires changing from rel="import" to normal rel="stylesheet" which in turn requires to changed the URL to request the CSS. Fix all closure errors. Bug: 1133186 Change-Id: I633ef710307afa8b1237a188a7b4d8932cbc0cdb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526004Reviewed-by:
Jeremie Boulic <jboulic@chromium.org> Commit-Queue: Jeremie Boulic <jboulic@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#825690}
-
Morten Stenshorne authored
This is a partial revert of patch set 4 in CL:2515399 (to the initial patch set). The change in patch set 4 was done in order to improve readability, but with one oversight: We'd set a non-zero distance when actually within range, since we'd always subtract one edge from another. This helps fix 6 tests, but the tests themselves also had to be updated, because legacy PositionForPoint() for multicol is broken when hitting above or below columns, in that it assumed UNIX / Mac editing behavior, which makes a difference when hitting just above the character at e.g. offset 2 in the first column. UNIX / Mac would return 0 as startOffset ("above the first line... okay, we're before the text, then"), while Windows should return 2 ("we're above the first line, but the nearest character is the one at offset 2 - use that"). Force Windows editing behavior in the tests, to fix this. The PositionForPoint() implementation for LayoutNG multicol is now behaving way better than the legacy engine, which allows us to add the missing parts to those tests (what we tested for vertical-rl was really limited due to legacy bugs). Legacy was also off-by-one for RTL columns, when hitting near the middle of a column gap (assigning one pixel too many to one column, and one too little to the other column). Rebaseline the tests. Everything now passes in LayoutNG, but there are obviously failures when running with the legacy engine now. Bug: 829028 Change-Id: Iecdecf1454318e645fa91bac29119df0fc7ca377 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526402Reviewed-by:Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#825689}
-
Fletcher Woodruff authored
Add a function to cancel the currently running scan job to LorgnetteManagerClient. Has the limitation that it only works when one scan is running. Bug: b:171821976 Change-Id: Ib60cb27477ee6c26110655dbdd1a408795774585 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523437 Commit-Queue: Fletcher Woodruff <fletcherw@chromium.org> Reviewed-by:
Jesse Schettler <jschettler@chromium.org> Cr-Commit-Position: refs/heads/master@{#825688}
-
Kent Tamura authored
This CL remove DEFINE_LAYOUT_OBJECT_TYPE_CASTS() for LayoutInline, and converts DEFINE_LAYOUT_OBJECT_TYPE_CASTS() to DowncastTraits<> for LayoutQuote. This CL has no behavior changes. Bug: 891908 Change-Id: I2ade507d24cb59b9d99c419cc566cd6b00d315ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528512 Auto-Submit: Kent Tamura <tkent@chromium.org> Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:
-
Brian Geffon authored
Node channel messages need to be packed. Bug: b:172869067 Change-Id: I30ac9d44347fef9e0b74267d6d09ec8589e4d3e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527532Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Brian Geffon <bgeffon@chromium.org> Cr-Commit-Position: refs/heads/master@{#825686}
-
Yu Su authored
- Add a ChipRenderParams class to wrap the params used when displaying the chip - add new LensController methods to retrieve chip params, generate Lens intent and launch Lens. Will implement these methods in a follow up clank cl. - update LensQueryParams to be a temperate wrapper object for Lens init params. Will split out Lens init params into a separate wrapper object once we have the Direct intent SDK. Change-Id: If90ac5f407889069fe2f58ef210055d2c144df46 Bug: 1145748, b/171252338 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527724 Commit-Queue: Yu Su <yusuyoutube@google.com> Reviewed-by:
Ben Goldberger <benwgold@google.com> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Sinan Sahin <sinansahin@google.com> Cr-Commit-Position: refs/heads/master@{#825685}
-
Mark Mentovai authored
In a change from x86_64, macOS on arm64 requires MAP_JIT for code not using the hardened runtime. However, if MAP_JIT is specified for signed code lacking the JIT entitlement, mmap and mprotect fail with EINVAL, so it’s still not possible to use MAP_JIT everywhere it could conceivably be required. A wasm attempt to mprotect a region as PROT_READ | PROT_WRITE | PROT_EXEC was succeeding in signed builds but failing in non-hardened-runtime processes with EPERM as a result of the mapping lacking MAP_JIT. In a previous (post-facto drive-by) review of this code (https://chromium-review.googlesource.com/c/2078792), I noted “I would have preferred the caller coming in with affirmative knowledge of whether it wanted JIT or not” but the callers aren’t necessarily aware of the entitlement regime they’re expected to run under. Bug: 1144200 Change-Id: I2edde64922dfbd8ba7158cc741b0e82d680235b8 CQ-Include-Trybots: luci.chromium.try:mac-arm64-rel Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523446Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Mark Mentovai <mark@chromium.org> Cr-Commit-Position: refs/heads/master@{#825684}
-
Alex Moshchuk authored
This modifies an existing test, DynamicIsolatedOriginTest.NewBrowsingInstanceInOldProcess, to explicitly exercise a scenario where a process's minimum BrowsingInstance ID changes, and where this may affect a CanAccessDataForOrigin decision. In particular, this would happen if the new minimum BrowsingInstance ID makes a particular origin eligible for isolation. Currently, the minimum BrowsingInstance ID stored for a process in ChildProcessSecurityPolicy does not change even if the corresponding BrowsingInstance goes away, so the outcome of CanAccessDataForOrigin can't change. This will change once multiple BrowsingInstances are tracked for each process, rather than just a minimum ID, in issue 1135539. In particular, after this change, CanAccessDataForOrigin may start denying access to an origin which is considered isolated in all of the process's remaining BrowsingInstances, due to citadel enforcement. Bug: 1135539 Change-Id: I8d388f6099f224172b8a77129362de8300e37ecc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528128Reviewed-by:
-
Curt Clemens authored
E.g. var(--google-gray-refresh-700). The values on all of the pages outside of settings had hard-coded colors. Change-Id: Idefc2c9746f13f02521712a2e7740eaaca97b62f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523662 Commit-Queue: Curt Clemens <cclem@google.com> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#825682}
-
Maggie Chen authored
It's split into AddSurfaceDamageToDamageList(), AddRenderPassFilterDamageToDamageList() and FindQuadWithOverlayDamage() for better readability. These new functions are called based on the following. AddSurfaceDamageToDamageList() is called per surface, form CopyPasses(), HandleSurfaceQuad() and EmitSurfaceContent(). AddRenderPassFilterDamageToDamageList() is called per render pass, from CopyQuadsToPass(). FindQuadWithOverlayDamage() is called from CopyQuadsToPass() where we set the |overlay_damage_index| in shared_quad_state. Bug: 1117235 Change-Id: Ibbd2ada71cc0fc0deaaed1b7da1c475d35f910f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2493836 Commit-Queue: Maggie Chen <magchen@chromium.org> Reviewed-by:
weiliangc <weiliangc@chromium.org> Cr-Commit-Position: refs/heads/master@{#825681}
-
Kent Tamura authored
* Use RenderingTest::GetLayoutBoxByElementId() to reduce the usage of ToLayoutBox(). * Use Element::GetLayoutBox() instead of Element::GetLayoutObject() to reduce the usage of ToLayoutBox(). This CL has no behavior changes. Bug: 891908 Change-Id: I5d18e3ff061d0d4b7953fbfb21c0d314bf78a776 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526665 Commit-Queue: Koji Ishii <kojii@chromium.org> Auto-Submit: Kent Tamura <tkent@chromium.org> Reviewed-by:
-
Noel Gordon authored
The PageHandler and UI components can depend on each other, which is a subject of an upcoming CL. This CL to avoid the `gn check` errors that would result with separate build rules for these items in that upcoming CL. Preparatory patch. No change in behavior. Bug: 1113981 Tbr: alex Change-Id: Ifeb988d1ce11e548bcb2b2d56021f2d89d8be526 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528514Reviewed-by:
-
danielng authored
Adding metrics that will be used to record information about the borealis installation flow. These will be used to help measure the performance, usage and reliability of the installer. Cq-Depend: 2509072 Bug: b/171758529 Change-Id: I4c078f388ba67cf858cacea78b102211d003edba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2515863 Commit-Queue: Daniel Ng <danielng@google.com> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Caitlin Fischer <caitlinfischer@google.com> Reviewed-by:
Nic Hollingum <hollingum@google.com> Cr-Commit-Position: refs/heads/master@{#825678}
-
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-sdk-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 Cq-Include-Trybots: luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-deterministic-dbg;luci.chromium.try:fuchsia-x64-cast Tbr: chrome-fuchsia-gardener@grotations.appspotmail.com Change-Id: Iac1af4c9052003030bb23e54b25ea3afa443728b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528714Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/59992a0fea40..c32d29e103de 2020-11-09 jalyn@microsoft.com Update KeyboardShortcutAction enum to match browser 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:1140386,chromium:174309 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Ieea34c21c7d2dbdd00ea56c1b1ee96f0a5bde0d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527798Reviewed-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: I169467009bc38ede203b8b654528f321dc815105 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528715Reviewed-by:
-
Carlos Frias authored
Enable Sensors control policies for launch in M88: DefaultSensorsSetting SensorsAllowedForUrls SensorsBlockedForUrls Bug: 1108589 Change-Id: I9d643a4a6edb0264341d34e3358c8e1be35440eb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362999 Commit-Queue: Carlos Frias <carlos.frias@microsoft.com> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#825674}
-
mlcui authored
- Update JSDoc type of uiLanguage to include the missing undefined - Fix exceptions if uiLanguage is undefined (doesn't happen in practice) - Add missing dependencies to the computed binding - Prevent extra comma from appearing if input method is not found Bug: None Change-Id: I5bc3dc3849bbb29e92c4a636c511dba69b60aa3e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2525562Reviewed-by:
My Nguyen <myy@chromium.org> Commit-Queue: Michael Cui <mlcui@google.com> Cr-Commit-Position: refs/heads/master@{#825673}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/9b7f81867ddd..1d532cbd0baa 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: I1cc15bba26848b99a0faaf457b55ae1b3f602e58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528713Reviewed-by:
-
