- 10 Nov, 2020 40 commits
-
-
danakj authored
Java bitmaps come from many places, but they originate from one of: - mojom::ImageDecoder::DecodeImage() IPC replies from a renderer - Resource bundles from disk - mojom::ImageDownloader::DownloadImage() replies from a renderer - WebContents::DownloadImage() or WebContents::DownloadImageInFrame() which use mojom::ImageDownloader::DownloadImage() - qrcode_generator::mojom::QRCodeGeneratorService::GenerateQRCode() replies from a utility process - mojom::WidgetHost::SetCursor() IPC from a renderer The ImageDecoder is made to verify replies in https://chromium-review.googlesource.com/c/chromium/src/+/2518931 The others are made to made to verify in this CL. For IPC replies, we expect the result to be an N32 32bpp bitmap, and we CHECK() when it differs. We do make an exception for DownloadImage replies as the downloaded image may have an arbitrary type. We would need to have the renderer convert instead otherwise. For IPCs originating from the renderer (aka SetCursor), we convert the sent bitmap to an N32 32bpp bitmap on receipt, or drop the request if that is not possible. R=boliu@chromium.org Bug: 1144462 Change-Id: Ie3b839360fd9098af341a778c0c6ec20ecb72a89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2520236 Commit-Queue: danakj <danakj@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#825973}
-
David Black authored
Bug: 1142572 Change-Id: I67ca2b66c9a10d68e0952bd46ef3cae8ae8865bb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528638Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Commit-Queue: David Black <dmblack@google.com> Cr-Commit-Position: refs/heads/master@{#825972}
-
Patrick Monette authored
This interface is basically a simpler VoteConsumer. It allows to observe votes made through a voting channel without having to deal with AcceptedVotes. One limitation it has is that it is no longer possible to submit multiple votes to the same context through a single VotingChannel. In practice, this was never done anyways. To support multiple votes for one context, it is easy to build multiple voting channels. Of course, AcceptedVotes still exists. Those are managed by the VoteConsumerDefaultImpl, which is a temporary class that handles creating and deleting AcceptedVotes as needed, then notifying the VoteObserver. The goal in the future is to fold this functionality into the VotingChannelFactory and remove VoteConsumer entirely. This will be done after each users of VoteConsumer are migrated one at a time. Bug: 971272 Change-Id: I4e890435cf29d4375ccb41b393b6d80b2d392778 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2519207 Commit-Queue: Patrick Monette <pmonette@chromium.org> Reviewed-by:
Chris Hamilton <chrisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#825971}
-
Ben Mason authored
TBR=govind@chromium.org Change-Id: I96ef90953c873bcd20d3a665e8bac347b54c74ad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529672Reviewed-by:
Ben Mason <benmason@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Commit-Queue: Ben Mason <benmason@chromium.org> Cr-Commit-Position: refs/heads/master@{#825970}
-
Etienne Bergeron authored
This CL is adding unittests for ELIDE_HEAD text eliding. The test coverage of this eliding mode is low and we want to increase it before trying to refactor the code. This CL has no behavior changes. R=gab@chromium.org,msw@chromium.org Bug: 1085014 Change-Id: I7f84fba3c558fcf2fa63a9f07fa0e0270ce7d121 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2522918 Commit-Queue: Etienne Bergeron <etienneb@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#825969}
-
Gordon Seto authored
Add UI for button to skip the profile discovery page and go to the activation code page. Bug: 1093185 Change-Id: I558f2df186829ddb9147dff19a0e34289fa20a34 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528142 Commit-Queue: Gordon Seto <gordonseto@google.com> Reviewed-by:
Azeem Arshad <azeemarshad@chromium.org> Cr-Commit-Position: refs/heads/master@{#825968}
-
Callum May authored
As specified in https://github.com/WICG/trust-token-api/pull/48, the Sec-Signature header now includes the algorithm identifier of the signing algorithm used to sign the header. We add this alg identifier to align with the spec. Change-Id: I90023b3f40fdb18d18aa73edbbecdcc25cf5aacf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2505813Reviewed-by:
Steven Valdez <svaldez@chromium.org> Commit-Queue: Callum May <camay@microsoft.com> Cr-Commit-Position: refs/heads/master@{#825967}
-
Kenneth Russell authored
This reverts commit 674a1af2. Reason for revert: suspected cause of crbug.com/1146687 Original change's description: > Refactor subresource redirect renderer code > > This CL refactors the existing subresource redirect code to make the > robots rules and login based image compression implementation easy. > > * Separate out the url loader throttle that does redirect and timeout > handling from the public image hints decider logic. The robots decider can > be subclassed and used. > > * Make public image hints agent as renderframeobserver. This reduces > cluttter in ResourceLoadingHintsAgent. The upcoming robots checker agent > could also observe renderframeobserver directly. > > No change in behavior is expected. > > Change-Id: Ic555a4402016a645b8820fa87dd5157c7112fb71 > Bug: 1144836 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514068 > Commit-Queue: rajendrant <rajendrant@chromium.org> > Reviewed-by: Michael Crouse <mcrouse@chromium.org> > Cr-Commit-Position: refs/heads/master@{#825115} TBR=rajendrant@chromium.org,mcrouse@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1144836, 1146687 Change-Id: I483882b29ac342d188137690e7ba9f8cbedcd6ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527859Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#825966}
-
Natalie Chouinard authored
Expose ChromeStartupDelegate feature flag to Java (missed in https://crrev.com/c/2523359) and cache flag and variations so they can be accessed before the native FeatureList is initialized. Bug: 1143402 Change-Id: I1f624009e00bc6d40bbef39b6a093cb15a60c50a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529701Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Natalie Chouinard <chouinard@chromium.org> Cr-Commit-Position: refs/heads/master@{#825965}
-
Caroline Rising authored
Allow tab/shift+tab to navigate through all elements in the UI. Let up/down arrows move focus between read later list items. Let right/left arrows cycle focus between a list item and its buttons. This behavior comes from discussion during a11y office hours. Still TODO: add dark mode support for buttons and hover/focus states (crbug/1147457). Bug: 1140166 Change-Id: I3a119e8d940a4307e746c10f3ded7541963f46f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527345 Commit-Queue: Caroline Rising <corising@chromium.org> Reviewed-by:
dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#825964}
-
Lan Wei authored
Use testdriver Action API to simulate mouse actions in html/user-activation/activation-thru-contextmenu-event.html. Bug: 1145677 Change-Id: I8c287cd45d87e809cfa4b5967df38ed0b58d0d37 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2520453Reviewed-by:
Mustaq Ahmed <mustaq@chromium.org> Commit-Queue: Lan Wei <lanwei@chromium.org> Cr-Commit-Position: refs/heads/master@{#825963}
-
Karthika Pai authored
The low entropy source is reported as a variation ID offset from 3320978. For example, a low entropy source value of 10 is reported as variation ID 3320988. The low entropy source is a locally-generated value in the range of 0-7999 that's used by Chrome to derive the variation ids included in the X-Client-Data header. Since other variation ids have been selected based on this value, this does not expose any additional entropy in the header given enough variation ids. Note: 8000 possible values is under 2^13, therefore having 13+ A/B variations would carry the same amount of entropy. A typical Chrome client has more than 13 variation ids reported. Adding this value allows for retrospective A/A tests for validating that there's no existing bias between two randomized groups of clients for a later A/B study. Bug: 1134444 Change-Id: Ia1af76f469cb2e3e25b03884abc4974cec237977 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461906 Commit-Queue: Karthika Pai <karthikapai@google.com> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Caitlin Fischer <caitlinfischer@google.com> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#825962}
-
chinsenj authored
Currently if a user hovers their mouse over a selection in a desk's renaming textfield or a folder's renaming textfield, the default cursor behavior is a normal pointer. Based on UX, we would like to display an ibeam instead. This CL overrides the default cursor behavior of both textfields to allow this. Test: manual Bug: 1133681 Change-Id: I8aef053a7355e8ed873685c0b54f8de96d47da3e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464123Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Jeremy Chinsen <chinsenj@chromium.org> Cr-Commit-Position: refs/heads/master@{#825961}
-
Etienne Bergeron authored
This CL is adding to the RenderText API fuzzer the calls to * GetSubstringBounds(...) * GetCursorSpan(...) These calls were missing to trigger the following bug: http://crbug.com/1146674 R=gab@chromium.org,msw@chromium.org Bug: 1146674, 1146674 Change-Id: I2867262bcc8cba97f666bbc084656dd4e1ac7362 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2530369Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Etienne Bergeron <etienneb@chromium.org> Cr-Commit-Position: refs/heads/master@{#825960}
-
Oksana Zhuravlova authored
Since the background tracing trial names are different for Chrome and WebView, this change makes the trial name a parameter of BackgroundTracingManager::GetBackgroundTracingConfig(). Bug: b/170513316 Change-Id: I963d6d34e87d70633fab9cfda36d96ba8ab770f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523904 Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Michael Bai <michaelbai@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Cr-Commit-Position: refs/heads/master@{#825959}
-
Ryan Hansberry authored
The service controller passed to ServiceControllerProxy was being initialized asynchronously, and could occasionally be passed into ServiceControllerProxy before being correctly initialized, causing a segfault once accessed. This CL changes that behavior to simply immediately initialize the service controller. Fixed: 1147293 Change-Id: Ie4e396706b5477befca47805c8984c697a814bcd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527633Reviewed-by:
James Vecore <vecore@google.com> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Ryan Hansberry <hansberry@chromium.org> Cr-Commit-Position: refs/heads/master@{#825958}
-
Jan Krcal authored
This patch enables rounding of the profile icons in the Mac's people menu to align with the GM2 circular icons used elsewhere in Chrome. This is achieved by applying a circular mask to the image and increasing the resolution of the avatar icons requested. This CL also avoid a call to AvatarMenu::GetImageForMenuButton() and calls GetAvatarIcon() directly (the additional logic of GetImageForMenuButton() is not needed here). Bug: 857064 Change-Id: I1444dc6081f3af6289be7fa6a886153a70c9ecf6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2517059 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Auto-Submit: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
Thomas Lukaszewicz <tluk@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#825957}
-
Devlin Cronin authored
extension_messages.h included a typedef for SubstitutionMap, which mimicked MessageBundle::SubstitutionMap (each a std::map<std::string, std::string>). Instead, just include message_bundle.h in extension_messages.h, and remove the extra typedef. Bug: None Change-Id: I1a076b7dd8f45f774296dcf2cd64fa3ea2960356 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528092 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Auto-Submit: Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#825956}
-
Lukasz Anforowicz authored
The additional crash keys should help determine what happens in ~77% of reports where the "DetermineSiteInstanceForURL / same-site-navigation" branch was picked. In particular, the new crash keys should help understand why the earlier "DetermineSiteInstanceForURL => source_instance" branch was not picked for about:blank navigation when calculating the SiteInstance in RenderFrameHostManager::DetermineSiteInstanceForURL. Bug: 1116320 Change-Id: If54e0a230a5f89b8169fdf2d26022a8267223ea6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527963Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Auto-Submit: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#825955}
-
Jesse Schettler authored
Reorder the file type options to match the spec. Bug: 1059779 Change-Id: I1dc213f2cfb1f1b388288552b8391ef69ef3e38d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529236 Commit-Queue: Jesse Schettler <jschettler@chromium.org> Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Cr-Commit-Position: refs/heads/master@{#825954}
-
Ben Joyce authored
Bug: None Change-Id: I1f436a10c64f9e8ca59dfbe0d811cfc6904ca680 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2530257 Commit-Queue: benjamin joyce <bjoyce@chromium.org> Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#825953}
-
Adam Langley authored
Plumb though better error information (for a future change) and some status updates. Use the status updates to visually indicate the progress of a QR transaction. BUG=1002262 Change-Id: I94ce3abf4accb5b28273fc5ee6a9f07386f64d0c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2511975 Commit-Queue: Adam Langley <agl@chromium.org> Reviewed-by:
Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#825952}
-
Maria Kazinova authored
forms. Bug: 1147363 Change-Id: Ie89fba59a8ea75ae8bed2a150e1cb45af34329e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527127 Commit-Queue: Maria Kazinova <kazinova@google.com> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#825951}
-
Jazz Xu authored
Bug: 1147152 Change-Id: I510df9bbe0ed9a321e0fee7fab7ea15b4fa3248f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527844Reviewed-by:
Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by:
Tommy Steimel <steimel@chromium.org> Commit-Queue: Jazz Xu <jazzhsu@chromium.org> Cr-Commit-Position: refs/heads/master@{#825950}
-
Clark DuVall authored
A branch cut is coming up in a few days, so turning off isolated splits to make sure we have a couple of days with canary in the version it will ship in for M88. This will be re-enabled after the cut. Bug: 1126301 Change-Id: I92ba57d2e3e89a6096a5de041289778099dcf80a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528176Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#825949}
-
David Tseng authored
Fixed: 1146877 TBR=jopalmer@chromium.org Reason for revert: Breaks ChromeVox; found via bisect; see bug. This reverts commit 6cbffa8a. Dependent changes (also reverted): 73bcb673 84c821b5 b197149c Change-Id: I65bc46c273cd64555676073cf6cd172128cfe068 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529695 Commit-Queue: David Tseng <dtseng@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#825948}
-
chromium-autoroll authored
Roll Chrome Linux PGO profile from chrome-linux-master-1605009236-ea4b3e0cf6b6eb37089f063eb3b9762bfdc69494.profdata to chrome-linux-master-1605030922-18cb72684b061f022a5308434494a6442ca4475e.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: I6326b13176739213baa40ebaae88f1ef0e1450f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2530309Reviewed-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@{#825947}
-
Garrett Beaty authored
This reverts commit bce662eb. Reason for revert: Getting pending alerts for builders that use the 16-core SSD machines. Original change's description: > luci: increase the experimental percentage to 100 from 70 for MM-x86-rel > > The pool has plenty of 4-cores for extra builds: http://shortn/_koR1dg8Vvu > Also the host pool also has plenty of rooms: http://shortn/_xqhW5Z67ft > > Bug: 997376, 1127110 > Change-Id: I56b432d960c2f1821934667ac2f38d3fcb4705bf > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2450331 > Reviewed-by: Ben Pastene <bpastene@chromium.org> > Commit-Queue: Haiyang Pan <hypan@google.com> > Cr-Commit-Position: refs/heads/master@{#813888} TBR=liaoyuke@chromium.org,bpastene@chromium.org,hypan@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 997376 Bug: 1127110 Change-Id: Ie3265e71b0604ea6d40a1612b820acffb8c79b8e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2530204Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Cr-Commit-Position: refs/heads/master@{#825946}
-
Moe Ahmadi authored
Repeatable Queries Service now populates the destination URL for the query suggestions which is later used from the MostVisitedSites to delete a repeatable query. Depending on whether the query is provided by the server or the local history, the service issues a deletion request to the server or deletes the suggestion from the in-memory URL DB. In both cases, all eligible search URLs from the default search provider which produce that query also get deleted. Fixed: 1147119 Change-Id: I4fad9eaf209c72e735d2260c7e22b1f0af5db4ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527605 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Nicolas Ouellet-Payeur <nicolaso@chromium.org> Reviewed-by:
Ramya Nagarajan <ramyan@chromium.org> Cr-Commit-Position: refs/heads/master@{#825945}
-
dpapad authored
These images currently reside in shared WebUI code under ui/webui/resources/images/, but they are only used by the Feedback app. Also removing flattenhtml="true" from feedback/html/default.html since it is not actually needed. Bug: 1132403 Change-Id: I76fc062a7e57742faf5ee6068a9fc052e46fd2d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528208Reviewed-by:
Miriam Zimmerman <mutexlox@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Auto-Submit: dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#825944}
-
Chong Gu authored
Bug: 1080853 Change-Id: I0c7fb5677467c9c75fa89d8bcbe0dcb126a6fd2e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527537 Auto-Submit: Chong Gu <chonggu@google.com> Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#825943}
-
Gavin Williams authored
In dark mode, the placeholder text got its color implicitly from the parent toolbar element's dark mode settings. So instead, this change explicitly defines the placeholder text's preferred dark mode color. Before: http://screen/3ogcXSwcRfNLQAG After: http://screen/5jHByt6gXfuTrPd Fixed: 1146343 Change-Id: Ibd8342be6e440d9e828be6ed56dc86847fe332fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529691Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Gavin Williams <gavinwill@chromium.org> Cr-Commit-Position: refs/heads/master@{#825942}
-
Joanmarie Diggs authored
AccessibilityEventRecorderAuraLinux was including the focused detail for AtkObject:state-change, but not AtkObject:focus-event. This made the output of certain content browser tests unclear. Ax-Relnotes: n/a Change-Id: If959a26f9ac1f307b2b74ca360c4b00bae0def62 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529105Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Joanmarie Diggs <jdiggs@igalia.com> Cr-Commit-Position: refs/heads/master@{#825941}
-
Garrett Beaty authored
The CQ testing capacity for marshmallow is oversubscribed, so removing an experimental test and a test that is run against another android version will free up some capacity. Skipping try so that CQ builds will not launch the removed tests. Bug: 1147531 Change-Id: I4a45092226bc5a51c2452540b3747f4f641feb43 No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2530256 Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Reviewed-by:
Haiyang Pan <hypan@google.com> Cr-Commit-Position: refs/heads/master@{#825940}
-
David Dorwin authored
If specified, the content will be displayed when a navigation is canceled due to the SafeSites filter. Bug: 1147231 Change-Id: Id5d77bc6cacae69c3f873404ace8de5cfa898612 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527730 Commit-Queue: David Dorwin <ddorwin@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Auto-Submit: David Dorwin <ddorwin@chromium.org> Reviewed-by:
Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#825939}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/952ef3e06472..8da8bcdf92fb 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: I3098af00991c9e1c01cbc87dde7798eb5711e4d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527631Reviewed-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@{#825938}
-
Wan-Teh Chang authored
The image/avif-sequence MIME type and the associated file extension "avifs" have been removed from the AVIF specification. See https://github.com/AOMediaCodec/av1-avif/issues/59 https://github.com/AOMediaCodec/av1-avif/pull/86 Add "-animated-" to the filenames so it's clear they are animated images. Change-Id: Ifd46d9c40430a5a60ada28fdede0cb232f56f75f Bug: 1147501 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528169 Commit-Queue: Wan-Teh Chang <wtc@google.com> Commit-Queue: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#825937}
-
wutao authored
This patch moves the photo refresh interval to pref. In this way, we can remove the unused interface in ambient_backend_controller and make it easy to change the interval in Settings and tast test. Bug: b/159247243 Test: new tests Change-Id: I45b3192c2d6d3ad06789e6b769a44f14976afc05 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2511753Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Tao Wu <wutao@chromium.org> Cr-Commit-Position: refs/heads/master@{#825936}
-
Noel Gordon authored
Files app removed use of chrome.mediaGalleries a few milestones ago so this code is no longer used. Remove it. Bug: code-health Change-Id: I38fea9943fe8e01530d79e2137ba498f2047c8b8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526824 Commit-Queue: Joel Hockey <joelhockey@chromium.org> Auto-Submit: Noel Gordon <noel@chromium.org> Reviewed-by:
Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#825935}
-
dpapad authored
Removing two entries of developers who have moved on to non-WebUI primary responsibilities. Fixed: 1055481 Change-Id: Iac803a00aeda0da8bf6f49d4d94882846f3ad643 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528175Reviewed-by:
Tommy Li <tommycli@chromium.org> Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Commit-Queue: Esmael Elmoslimany <aee@chromium.org> Auto-Submit: dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#825934}
-