- 06 Jan, 2021 40 commits
-
-
Tibor Goldschwendt authored
We are still using them. Specifically, this CL extends the following histograms: * NewTabPage.BackgroundService.Collections.RequestLatency* * NewTabPage.BackgroundService.Images.RequestLatency* * NewTabPage.CustomizedShortcuts * NewTabPage.CustomizeLocalImageBackgroundAction * NewTabPage.CustomizeShortcutAction * NewTabPage.LogoDownloadOutcome * NewTabPage.LogoDownloadTime * NewTabPage.LogoImageDownloaded * NewTabPage.LogoShownTime2 * NewTabPage.Modules.ShownTime * NewTabPage.Modules.Usage* * NewTabPage.Promos.RequestLatency2* * NewTabPage.RecipeTasks.RecipeClick * NewTabPage.RecipeTasks.RelatedSearchClick * NewTabPage.ShoppingTasks.ProductClick * NewTabPage.ShoppingTasks.RelatedSearchClick * NewTabPage.TileTitle* * NewTabPage.URLState Fixed: 1151982, 1156215, 1151981 Change-Id: Id31e792fe3d4b205ea8d1fd1ea5f17d9c5f4157b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611634 Auto-Submit: Tibor Goldschwendt <tiborg@chromium.org> Commit-Queue: Mark Pearson <mpearson@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Cr-Commit-Position: refs/heads/master@{#840673}
-
cfredric authored
Bug: 1143756 Change-Id: I7c8f9b94c817a577367f377ed2d7a0dd3c81ca24 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523414Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Maksim Orlovich <morlovich@chromium.org> Reviewed-by:
Lily Chen <chlily@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Auto-Submit: Chris Fredrickson <cfredric@chromium.org> Cr-Commit-Position: refs/heads/master@{#840672}
-
Andrew Grieve authored
Noticed a significant delay between "Finished" log message and process terminating. Turns out this call to _exit(0) saves 2 seconds off of a 4.6 second run for content_common_mojo_bindings. Bug: 1143399 Change-Id: I0beb911033ddb7f96027796afac0c65ac5306df2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611950Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#840671}
-
Justin Cohen authored
UIImageWriteToSavedPhotosAlbum appears to force a png instead of a gif. Use PHAssetCreationRequest instead to preserve the gif. Bug: 1159431 Change-Id: I3640085f97d6f20fe62d96c8ac02d2bbed353585 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2595254 Commit-Queue: Justin Cohen <justincohen@chromium.org> Auto-Submit: Justin Cohen <justincohen@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Reviewed-by:
Sebastien Lalancette <seblalancette@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#840670}
-
Andrew Grieve authored
The exception was not being checked for, and so never shown. Bug: None Change-Id: Ia61ed3cb2379defe087262d61374fbe6c6675217 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611450 Auto-Submit: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Commit-Queue: Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#840669}
-
Steven Bennetts authored
Bug: none Change-Id: Ibfa838ae47f5004566deb0d0ef9973531ce0f340 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2597781 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Azeem Arshad <azeemarshad@chromium.org> Cr-Commit-Position: refs/heads/master@{#840668}
-
Dominic Mazzoni authored
Bug: 1153721, 1153720 Change-Id: Ib808bb4e00028ba584e2c9de40830ffeb9d78122 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2605568Reviewed-by:
Brian White <bcwhite@chromium.org> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#840667}
-
dpapad authored
This is in preparation of auto-generating grd entries for the PDF Viewer. Bug: 1132403 Change-Id: I695823e98d672a611ddc136eebb1afe40693f2af Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2612098Reviewed-by:
Daniel Hosseinian <dhoss@chromium.org> Commit-Queue: dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#840666}
-
Alice Wang authored
This CL adds tests to check SigninStartAccessPoint logging. Bug: 1163476 Change-Id: I35ff0b27780d537e97109dc3f0fa71c2785a6ad1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611716Reviewed-by:
Tanmoy Mollik <triploblastic@chromium.org> Commit-Queue: Alice Wang <aliceywang@chromium.org> Cr-Commit-Position: refs/heads/master@{#840665}
-
Jesse Schettler authored
The histogram records the number of detected scanners. Bug: 1059779 Change-Id: I2db04c38c6ddf7dc4a3d7e3ff7df69e67076c743 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2596428Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Reviewed-by:
Caitlin Fischer <caitlinfischer@google.com> Commit-Queue: Jesse Schettler <jschettler@chromium.org> Cr-Commit-Position: refs/heads/master@{#840664}
-
Peter Kasting authored
It makes little sense for GetFontListWithDelta() to also take non-delta values. It will also be convenient for a future CL for callers to be able to use a struct that represents "the meaningful data about a font". For both these reasons, replace the private FontKey struct with a public FontDetails struct that bundles these details, and update callers to use it as appropriate. Also, replace seemingly-unnecessary usage of Font with FontList in l10n_font_util.h (and remove dead method there). This allows removing another ResourceBundle method. Bug: none Change-Id: Ie6b35e7015fe8bdf704fc4111c0e6ca4de522308 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611841 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Auto-Submit: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#840663}
-
Aaron Leventhal authored
An iframe can only have 1 hyperlink. If the iframe is not loaded yet, then it should have 0. When hyperlink 0 is retrieved, it should never be null. However, this is occurring where the url is "empty". It may be occurring during document destruction. There exists code to update the parent hypertext when a BrowserAccessibilityManager's contents are attached, but not on destruction. It's unclear whether a single BrowserAccessibilityManager can be destroyed without the entire tree of them being destroyed. To be safe, update the parent when a BrowserAccessibilityManager is destroyed. Bug: 1157731 Change-Id: I96a6654d3aa2d6934c07a0145e76edad9ee93243 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611665Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Cr-Commit-Position: refs/heads/master@{#840662}
-
Tom Anderson authored
On Linux, the display server closes when the session is ending. In particular, for X11 connections, the server may be remote, so the connection can go down at any time. [1] added a crash on connection IO errors where it would previously gracefully exit. This is desirable on ChromeOS to generate crash reports, but not for desktop Linux, so this CL restores the old behavior on Linux. [1] https://chromium.googlesource.com/chromium/src/+/59a4564762df2b3cb5a6037d5ce97e5ad626eb48 R=rjkroege BUG=1160140 Change-Id: Ie5a5ce762e6b65f533411ce88caa51d3c27366a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2600045 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Auto-Submit: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#840661}
-
Wei Li authored
RoundedOmniboxResultsFrame is the content view for omnibox's autocomplete popup widget. It is in a TYPE_POPUP widget. For this kind of widget, SMALL_SHADOW and BIG_SHADOW has no difference. Since SMALL_SHADOW is the default value for most dialogs, use this type instead. We will eliminate the BIG_SHADOW eventually. Bug: 940027 Change-Id: I4da68c4eeb9eeecb44b0a1a9f6bf610880b90acf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2612150Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: Wei Li <weili@chromium.org> Cr-Commit-Position: refs/heads/master@{#840660}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/cdde253e4082..32d68537a88c 2021-01-06 johnstiles@google.com Add SkVM support for conversion constructors to and from boolean. 2021-01-06 johnstiles@google.com Unify conversion constructor simplification code. 2021-01-06 bsalomon@google.com exclude surfacecontextwritepixels from CK 2021-01-06 bsalomon@google.com Revert "Add more comprehensive test for GPU write pixels." 2021-01-06 egdaniel@google.com Fix inline uploads when using vulkan load resolve. 2021-01-06 bsalomon@google.com Revert "Revert "Reland "Make textures used with alpha-only color-types produce 0s for RGB"""" 2021-01-06 johnstiles@google.com Fix cross-platform deviations in SkStrAppendScalar for inf/nan. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC johnstiles@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-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Cq-Do-Not-Cancel-Tryjobs: true Bug: None Tbr: johnstiles@google.com Change-Id: I487dd4ca6160fb2347a7a4b7681120080edbe6ac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2613120Reviewed-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@{#840659}
-
Miras Myrzakerey authored
Implement encoding part of Save-Page-As-WebBundle feature by introducing a WebBundleBuilder class. This CL will encode a series of snapshots (from a single web page) into a correct WebBundle. Bug: 1040752 Change-Id: I3ab17deb1d658b3098ba32916e8832ea0c829e25 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2215543 Commit-Queue: Miras Myrzakerey <myrzakereyms@google.com> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Kunihiko Sakamoto <ksakamoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#840658}
-
Tibor Goldschwendt authored
Fixed: 1154738 Change-Id: I319269ac2ad02fbf8fb5776e0904ca2fcd4c9b06 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611700 Auto-Submit: Tibor Goldschwendt <tiborg@chromium.org> Commit-Queue: Esmael Elmoslimany <aee@chromium.org> Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#840657}
-
Brian Sheedy authored
Removes chrome_public_test_vr_apk from the two Android L-based builders it was being run on, as we have decided that it's no longer worth the effort to try to keep the tests running on that version of Android. Bug: 1159619 Change-Id: I22722da53fbb288e7ab90f34afffcbe8f463eddb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611900 Auto-Submit: Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Alexander Cooper <alcooper@chromium.org> Reviewed-by:
Alexander Cooper <alcooper@chromium.org> Cr-Commit-Position: refs/heads/master@{#840656}
-
Mihai Sardarescu authored
Methods OnUnconsentedPrimaryAccountChanged is deprecated, so this CL changes the chrome/browser/profiles/* and c/b/safe_browsing/* to override the method OnPrimaryAccountChanged(). Bug: 1163126 Change-Id: I0dbabb38085373cf44de5f7273812d4f44fe7969 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611091Reviewed-by:
David Roger <droger@chromium.org> Reviewed-by:
-
Mihai Sardarescu authored
Methods OnUnconsentedPrimaryAccountChanged is deprecated, so this CL changes the chrome/browser/ui/views/profiles/* to override the method OnPrimaryAccountChanged(). Bug: 1163126 Change-Id: Ia0168bae3070c7c5eefd9919598919b9d0ae4e0d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611711Reviewed-by:
-
John Delaney authored
This is a reland of 53e3903f This now correctly initializes the was_opener_suppressed_ member on NavigationRequest. Original change's description: > [conversions] Route impressions alongside rel="noopener" anchor navs > > What: > Route impressions for the Conversion Measurement API to the browser > process for anchor tags with rel="noopener". > > Why: > We currently drop impressions for anchor clicks which have noopener > set because the impressions do not get forwarded to the browser process. > > Anchor click navigations with the rel="noopener" attribute use a > different navigation path than those without the attribute, in which > the window creation mojo also navigates the frame to the target url. > This navigation is not annotated with an impression. > > How: > Add an Impression member to CreateNewWindowParams which will get > forwarded as a LoadURLParam for the subsequent navigation of the > window. > > This change also fixes a bug in which initiator_frame_tokens are not > set for noopener navigations. > > As of https://chromium-review.googlesource.com/c/chromium/src/+/2327963, > the lack of an initiator frame tokens are used to to indicate whether > an opener was suppressed for a navigation. Because attaching the token > breaks this logic, we route a new |was_opener_suppressed| param through > the NavigationController:LoadUrlWithParams() flow to recover the > behavior. > > Bug: 1155577 > Change-Id: I84739a9f4ec0df9a867f91397f80b46eea1bcee6 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2575223 > Commit-Queue: John Delaney <johnidel@chromium.org> > Reviewed-by: Charlie Harrison <csharrison@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Charlie Reis <creis@chromium.org> > Cr-Commit-Position: refs/heads/master@{#838275} Bug: 1155577 Change-Id: I1ab5bf85e62525e898bd9e9a6986ed844a7a5c39 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2600016 Commit-Queue: John Delaney <johnidel@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Reviewed-by:
-
Yi Gu authored
Bug: 1015645, 1138322 Change-Id: I88a989a5cfad63383dbd47b95bfe665837d57c8a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2595257Reviewed-by:
Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Reviewed-by:
-
Alex Ilin authored
The size of a cr-theme-icon element is computed incorrectly if the element doesn't have `display: block` property. This leads to a subtle bug when displaying a `box-shadow` around a focused element. Fixed: 1161116 Change-Id: I908b5e997c01db0db00574319c49029ef438b3ac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611025 Auto-Submit: Alex Ilin <alexilin@chromium.org> Commit-Queue: Esmael Elmoslimany <aee@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/google/shaderc/+log/4089217d30c1..61649cc4962e 2021-01-06 rharrison@google.com Roll 4 dependencies (#1165) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/shaderc-chromium-autoroll Please CC radial-bots+chrome-roll@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: radial-bots+chrome-roll@google.com Change-Id: I53ea97bc1eda49cdd8cc0ab1a1bc60582eb168e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2613123Reviewed-by:
-
Chris Lu authored
Since this is a client-side driven experiment, the client must turn it on to 45% stable. Bug: 1138603 Change-Id: Ibe97fd2b7f92e19e3461ad635bbce9b5f6ee94a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2588301Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/8a3bba16ba86..3fcecc11721e 2021-01-06 jacktfranklin@chromium.org Add body context menu to data-grid 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:1150793 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I70445696bbb55c006342a76dd27f8f670026bd58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2613226Reviewed-by:
-
Sorin Jianu authored
It uses the idiomatic TestTimeouts::action_max_timeout() instead. Also, it fixes an assert to keep lint happy. Bug: 1163363 Change-Id: I76289b8216e5d8d6dcb729628129835253bcc25c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611641 Commit-Queue: Sorin Jianu <sorin@chromium.org> Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Cr-Commit-Position: refs/heads/master@{#840647}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/de6e3a9d85af..e212b425e90f 2021-01-06 johnchen@chromium.org Revert "devil: make devil_env_test pass under py3." 2021-01-06 jbudorick@chromium.org Revert "devil: make decorators_test pass under py3." 2021-01-06 bsheedy@chromium.org [Telemetry] Disable ResultSink on nested tests 2021-01-05 jbudorick@chromium.org devil: make decorators_test pass under py3. 2021-01-05 jbudorick@chromium.org devil: make devil_env_test pass under py3. 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:1007101,chromium:1159139,chromium:1163340 Tbr: nuskos@google.com Change-Id: Icb4ffd462151ddf111387fe78da12ac0ddf62bbc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2613118Reviewed-by:
-
Egor Pasko authored
Readability of this class suffers from (perhaps premature) optimizations to eliminate locks in a few places where the class is used only on a single thread. I believe avoiding these locks does not affect correctness in practice, but explaining and maintaining the proper ways to use these classes becomes more time consuming. In this change I: * Annotated the fields that are indeed guarded by the main lock. * Expanded the explanation of the use of the secondary lock. * Also added a few short-lived synchronized blocks, where an explanation of why it can be avoided is long and brittle. This adds more locking, but I believe it won't affect performance noticeably. * Moved a few unguarded fields above the lock, so that the latter can guard everything 'below'. * Made a few minor corrections in comments, now that I looked at this code a bit longer. Bug: 1154224 Change-Id: I346d358a4b2a465cecdec1881908026e17fef7d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2602285Reviewed-by:
Benoit L <lizeb@chromium.org> Commit-Queue: Egor Pasko <pasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#840645}
-
Stephen Roe authored
third_party/zstd has been replaced with third_party/zstd-linux-arm64 and third_party/zstd-linux-x64. Bug: 1154022 Change-Id: I87c58d5bbf1bf57a5a64c03255e4731cd1df0eb0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611897Reviewed-by:
Dirk Pranke <dpranke@google.com> Commit-Queue: Stephen Roe <steveroe@google.com> Cr-Commit-Position: refs/heads/master@{#840644}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/4f3fa1b39ec3..875b30eb4cdf 2021-01-06 saklein@chromium.org cros-sdk: Use os.ST_NOSUID constant. 2021-01-06 liaoyuke@chromium.org Lacros: Support --deploy-lacros in cros_run_test 2021-01-06 jintao@google.com usergroup_baseline: Add ap-group-manager to gwifi 2021-01-06 bpastene@chromium.org cros_test: Add args to specify tast sharding parameters. 2021-01-06 liaoyuke@chromium.org Lacros: fix a deployment bug where ash-chrome is not restarted 2021-01-06 saklein@chromium.org dependency_lib: New lib for dependency functionality. 2021-01-05 liaoyuke@chromium.org Support --tast-var in cros_run_tests 2021-01-05 pprabhu@chromium.org Update API bindings 2021-01-05 chrome-bot@chromium.org Update config settings by config-updater. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@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:chromeos-betty-pi-arc-chrome;luci.chromium.try:chromeos-kevin-rel Tbr: chrome-os-gardeners@google.com Change-Id: Ia5d1dfc3ea2798faa5a20ea175997516b16a7f5e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2613116Reviewed-by:
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/4dde23fea599..cee13faaf357 2021-01-06 fmayer@google.com Add end to end test for symbolization. 2021-01-06 primiano@google.com Win port: Introduce //base/ctrl_c_handler.h 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: chromium:174454879 Tbr: perfetto-bugs@google.com Change-Id: I0d2c2b14e4f4828413411e4a0c5320fde2fe5f04 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2613119Reviewed-by:
-
Ben Pastene authored
This removes every reference but the "--master/-m" cmd-line arg and the "masters" mb_config.pyl key, which will be removed in a follow-up once all users of mb.py (ie: recipes) have switched over. "builder_group" is also the term we're using when replacing "master" in recipe code: crbug.com/1109276. So we should conform on using that term going forward. Bug: 1117773 Change-Id: I10a3c7a4bc5fd5146bc88d670564a745cb3fce2a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2605225 Commit-Queue: Ben Pastene <bpastene@chromium.org> Reviewed-by:
-
Daniel Rubery authored
The code to extract DOM features is undergoing a moderate-size refactoring and this metric could be useful in diagnosing a regression. Fixed: 1160884 Change-Id: I9e970610a5b69e2a31d5d4e829cf87b1ec2fccd8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2610956 Auto-Submit: Daniel Rubery <drubery@chromium.org> Reviewed-by:
-
Robbie McElrath authored
This adds missing dependencies and adds nogncheck comments to any includes in //chrome/browser/updates to files that live in //chrome/browser:browser. Bug: 949535 Change-Id: Icdac512e2dc1d4df7620713ae7f912829e214892 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2585665Reviewed-by:
Erik Staab <estaab@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Cr-Commit-Position: refs/heads/master@{#840639}
-
sczs authored
Since diffable data source has been turned off we can remove this workaround. Bug: 1162429, 1162427 Change-Id: I45d786022312bc85fa19ad4e9b96920ce88572a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2612565Reviewed-by:
Adam Trudeau-Arcaro <adamta@google.com> Commit-Queue: Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#840638}
-
David Bokan authored
The feature shipped on Android long ago. It remains disabled on desktop as it doesn't really apply there so the base::Feature remains but we can remove the about:flag. Bug: 903260 Change-Id: I7889b33a52f6a13edf16b1268da5af2cfc29f053 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2605882Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#840637}
-
Mehran Mahmoudi authored
Some URLs are invalid now due to refactorings. This CL updates them. Change-Id: Ibc4350c6797abee3d0c7c4fbb14142142b49233d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2612926 Auto-Submit: Mehran Mahmoudi <mahmoudi@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#840636}
-
Stephen McGruer authored
Bug: 997202 Change-Id: I670d9134fc8770c58f8688366a16830ba9a6435e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2575454 Commit-Queue: Stephen McGruer <smcgruer@chromium.org> Reviewed-by:
Kevin Ellis <kevers@chromium.org> Cr-Commit-Position: refs/heads/master@{#840635}
-
Mihai Sardarescu authored
Methods OnPrimaryAccountSet/Cleared() are deprecated, so this CL changes the signin_profile_attributes_updater.(h|ccc} to override the method OnPrimaryAccountChanged(). Note that is also have a slight change of behavior by updating the auth info even when the unconsent primary account changed. Bug: 1163126 Change-Id: I5f1c739787e3ead98c8dcedc7ca486eeda2bc216 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611717Reviewed-by:
Monica Basta <msalama@chromium.org> Reviewed-by:
-
