- 10 Mar, 2020 40 commits
-
-
Haiyang Pan authored
KitKat CQ builder is short of test resources badly. Removie this test suite from it to improve performance. Bug: 1060245 Change-Id: I9b04d918bfdac894d813ba6f250c6acc9172fde2 No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095354 Commit-Queue: Haiyang Pan <hypan@google.com> Reviewed-by:
Ben Pastene <bpastene@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#748761}
-
Robin Lewis authored
Also add this to future requests when it's available. Bug: 1043199 Change-Id: I466bd42d221aaf7f961f702e12306b12cae1de04 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096108Reviewed-by:
Rakesh Soma <rakeshsoma@google.com> Commit-Queue: Robin Lewis <wrlewis@google.com> Cr-Commit-Position: refs/heads/master@{#748760}
-
Haiyang Pan authored
They are flakily failed in android-pie-x86-fyi-rel Bug: 1059168 Change-Id: I545f2caf7ffb60f50ee4060bb80feabe3cb9feca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096099Reviewed-by:
-
Marijn Kruisselbrink authored
And remove tests for the old/soon to be removed writing API. Bug: 853326 Change-Id: Ief44f98be8b59844976de26c90df9b4933041188 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095564Reviewed-by:
Olivier Yiptong <oyiptong@chromium.org> Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Cr-Commit-Position: refs/heads/master@{#748758}
-
edchin authored
iOS flag owners were asked to claim ownership and state an expiry in this announcement email: https://groups.google.com/a/google.com/g/bling-team/c/7YC_3l6yEzo/m/1-iZg1fPAAAJ For these flags, ownership was claimed, but owners said these are already expired and should be removed. I have put M80 as the default for these flags. Though the flag ownership cleanup process will remind folks to remove expired flags, it is better if folks proactively remove the flag. Would the owners please remove the flag, or create a bug to remind themselves to remove the flag? Please lgtm for those flags where you have been named owner. Thanks! Bug: 1058614 Change-Id: I3200ea515df13bf36a6e7a31a043c91c980b2033 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2093693Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Reviewed-by:
edchin <edchin@chromium.org> Commit-Queue: edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#748757}
-
Ben Pastene authored
Betty ran the full set of CrOS tests, while betty-pi-arc only ran a subset of them. This swaps them since Arc has been uprev'ed to P, and N is on its way out. Bug: b/150926708 Change-Id: I6c5f1349d573e7db106a9b116c95dc3585dfebe1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2091709Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
-
Rouslan Solomakhin authored
Before this patch, the exact constructor for RespondWithCallback that payment_app_provider_impl.cc used depended only on the type of the callback object, which made the code harder to understand. This patch replaces the two constructors with two static methods: CreateForInvoke() and CreateForEvent(). After this patch, it is clear which code path is taken when reading the code. Bug: 1005076 Change-Id: Idc762c79077082088b81d8a564b5acc11e47ae12 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2089616 Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Danyao Wang <danyao@chromium.org> Cr-Commit-Position: refs/heads/master@{#748755}
-
Jonah Ryan-Davis authored
This is a reland of c00d0991 I had added a target to "linux_symbols" that was unnecessary. > Enable GL_TEXTURE_2D for IO-Surface-backed textures on SwANGLE > > GL_ANGLE_rectangle_texture is unavailable on Swiftshader's Vulkan > frontend, so SwANGLE on Mac uses 2D textures instead. > > Bug: chromium:1015454 > Change-Id: Id99850fbfae798b33eaff4f3a285deb043b4ae7c > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2056931 > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Avi Drissman <avi@chromium.org> > Reviewed-by: ccameron <ccameron@chromium.org> > Reviewed-by: Kenneth Russell <kbr@chromium.org> > Commit-Queue: Jonah Ryan-Davis <jonahr@google.com> > Cr-Commit-Position: refs/heads/master@{#748322} Bug: chromium:1015454 Change-Id: Id198cfe72268cfa496e23bea190aea1893c6718c Tbr: dcheng@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095348 Commit-Queue: Jonah Ryan-Davis <jonahr@google.com> Reviewed-by:
Jonah Ryan-Davis <jonahr@google.com> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#748754}
-
Lijin Shen authored
Reland of 76ed93dc Fix failure on TabStripTest suite Reason for revert: breaks org.chromium.chrome.browser.compositor.overlays.strip.TabStripTest suite Original change's description: > Fix chopped text on tab switcher long-press menu when text size is huge > > There exists some unknown issue on the measurement of a text view with > a compound drawable. Using a linear layout containing an imageview and > a textview inside can avoid this issue. > > Bug: 1053448 > Change-Id: Ibdcd220e0eba7218c1d500d870da17d37c7e767c > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2083729 > Commit-Queue: Lijin Shen <lazzzis@chromium.org> > Reviewed-by: Theresa <twellington@chromium.org> > Cr-Commit-Position: refs/heads/master@{#747078} Bug: 1053448 Change-Id: Ib5c48a3010e7074ca9c92e2826ba407ffd76e9fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2090833 Commit-Queue: Lijin Shen <lazzzis@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#748753}
-
Dave Tapuska authored
In order for the renderer to pass associated mojo channels they need to be on the correct thread. If we pass them on the IO thread they will be associated with the wrong channel. So in order to do this in a later CL we need to move these mojo messages first to the UI thread. When legacy IPC is removed this won't need to be a sync message anymore. CreateNewWindow was initially done like this in https://chromium.googlesource.com/chromium/src/+/95f01e92e7147bee4e66ce629712fd1f50808936%5E%21/#F24 Change-Id: I50a135260106b801db4d32527770b4beda3b5b36 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095347 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#748752}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/fae71fb596d0..2e307968fd54 git log fae71fb596d0..2e307968fd54 --date=short --first-parent --format='%ad %ae %s' 2020-03-10 nifong@google.com Clean up Enums in Canvaskit Created with: gclient setdep -r src/third_party/skia@2e307968fd54 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 brianosman@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/+/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 Bug: None Tbr: brianosman@google.com Change-Id: I6429cc965132ff8d724929bcb4621d22c14c8beb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096543Reviewed-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@{#748751}
-
Rahul Arakeri authored
This CL adds mouseMoveTo API to stabilize the test on Mac. Bug: 1058244, 1057355 Change-Id: I0be5ce4ffa508319ed58e37ac3e81f9fa5a88a9a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094816Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: Rahul Arakeri <arakeri@microsoft.com> Cr-Commit-Position: refs/heads/master@{#748750}
-
Robert Ogden authored
Plumbs DRP prefetch proxy hosts to where they will be used in IsolatedPrerender. Bug: 1042829 Change-Id: Id531927075470b37f79af4d7cf42b0352fe785ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2090147Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Robert Ogden <robertogden@chromium.org> Cr-Commit-Position: refs/heads/master@{#748749}
-
Rouslan Solomakhin authored
Before this patch, the salient difference between the two RespondWithCallbacks constructors was not obvious, making development brittle. This patch moves the common code for the two constructors into the common private constructor, which used by the two public constructors. After this patch, the difference between the two constructors for RespondWithCallbacks is more clear. Bug: 1005076 Change-Id: I8b5a80626c4ec2cfffb78439ed44bd159251ef67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2089615 Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
-
Colin Blundell authored
We are in the midst of componentizing ExternalNavigationHandler.java for reuse by WebLayer. The main remaining //chrome-level dependency is on TabRedirectHandler.java. Unlike other dependencies, this dependency can neither be easily componentized (it is essentially //chrome-level functionality) nor can it be easily abstracted through ExternalNavigationDelegate.java. The reason for the latter is that the TabRedirectHandler instance is necessarily accessed through the ExternalNavigationParams instance as it is tied to the navigation in question, and if we were to remove all knowledge of it from ExternalNavigationParams, there would be no straightforward way to plumb it into //chrome's ExternalNavigationDelegateImpl. This CL instead creates a RedirectHandler interface that contains the methods needed by ExternalNavigationHandler. ExternalNavigationParams and ExternalNavigationHandler are changed to talk in terms of this interface, which TabRedirectHandler implements. This approach has the additional benefit that it leaves existing logic coalesced within ExternalNavigationHandler, which will make it easier to adapt this logic in the WebLayer context as needed. For this initial delegate interface, we have preserved the shape of the interface to be the exact current usage by ExternalNavigationHandler.java. Future work can refine the interface to have it be as thin and logical as possible, potentially coalescing some low-level methods into a smaller number of higher-level ones. Bug: 1031465 Change-Id: I5b9568f371c248e8fd2d52429725ed0a44a098a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095719 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Cr-Commit-Position: refs/heads/master@{#748747}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/3ddc82494f77..82e00fd235e4 git log 3ddc82494f77..82e00fd235e4 --date=short --first-parent --format='%ad %ae %s' 2020-03-10 lalitm@google.com Merge "tp: fix dynamic table lifetime scoping" Created with: gclient setdep -r src/third_party/perfetto@82e00fd235e4 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/+/master/autoroll/README.md Bug: None Tbr: perfetto-bugs@google.com Change-Id: Ief1b2e907e7325335575a20c5ce4909d37c778c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096546Reviewed-by:
-
Jesse Schettler authored
Bug: 1059779 Change-Id: Ib345da98f093452a21d575071fdc3872cd52619a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2088403Reviewed-by:
Bailey Berro <baileyberro@chromium.org> Commit-Queue: Jesse Schettler <jschettler@chromium.org> Auto-Submit: Jesse Schettler <jschettler@chromium.org> Cr-Commit-Position: refs/heads/master@{#748745}
-
Shouqun Liu authored
Bug: none Change-Id: I4d4059891719368e04f1fa4391dbe07fdda4e846 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094067Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Cr-Commit-Position: refs/heads/master@{#748744}
-
edchin authored
iOS flag owners were asked to claim ownership and state an expiry in this announcement email: https://groups.google.com/a/google.com/g/bling-team/c/7YC_3l6yEzo/m/1-iZg1fPAAAJ This is one of several CLs that adds iOS flag ownership information into flag-metadata.json. The owner of these flags was identified via git blame. A placeholder expiry milestone of M86 was added. Please either LGTM, or suggest a new owner and/or expiry. Bug: 1058614 Change-Id: I5a3ce15915d122ee6c12d2f5aaf9bc559758c4a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095200Reviewed-by:
Robbie Gibson <rkgibson@google.com> Reviewed-by:
-
Alex Ilin authored
At the end of the remote consent flow, the browser makes the second issueToken request. This request must use a login token of an account that the user has chosen in the consent UI. Gaia returns an id of this account in a consent result. This CL updates IdentityGetAuthTokenFunction to use the id returned in a consent result instead of the one that was originally chosen by Chrome. This CL also moves IdentityTokenCacheValue to use static factory methods instead of multiple constructor overloads. This makes obvious what type of a cache value is created instead of relying on argument types. Bug: 1026237 Change-Id: I59d5eefa7bbca91b59bb3d8cd69deca600580150 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095302 Commit-Queue: Alex Ilin <alexilin@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#748742}
-
Ehimare Okoyomon authored
Add strings for four-state radio buttons that will be used in cookie settings. Designs can be found in the bug. Bug: 1060118 Change-Id: I11ca66811510d93b3f00e730fcac639afaa7fa7c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096637 Commit-Queue: Ehimare Okoyomon <eokoyomon@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Cr-Commit-Position: refs/heads/master@{#748741}
-
Rouslan Solomakhin authored
Before this patch, the RespondWithCallbacks class in payment_app_provider_impl.cc contained a number of public methods that were either not used outside of this class or not used at all. This patch moves a number of RespondWithCallbacks methods from public to private and removes one method that was never used. After this patch, it is more clear which part of RespondWithCallbacks is used outside of the class. Bug: 1005076 Change-Id: I1058137899177454cb786a91e16fe7f88dc8eaee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2089409 Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
-
Jerome Jiang authored
BUG=1055917 Change-Id: I1d8e26b33528b3edb38723a6fd3282401a12b49c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095400Reviewed-by:
James Zern <jzern@google.com> Commit-Queue: Jerome Jiang <jianj@google.com> Cr-Commit-Position: refs/heads/master@{#748739}
-
Hesen Zhang authored
- fix the crash before loading native This is a reland of 4d4e25dd Original change's description: > [Prefetch notification service]: Java side and glue setup, feature flag. > > - Added a feature flag for the migration. > > - Added and hid new code path in OfflineNotificationBackgroundTask. > > - Created JNI bridge for gluing prefetch notification service in native. > > Bug: 1047037 > Change-Id: If7b728609f86665cfe6b4c61f521f7362000b292 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2082087 > Reviewed-by: David Trainor <dtrainor@chromium.org> > Reviewed-by: Xing Liu <xingliu@chromium.org> > Commit-Queue: Hesen Zhang <hesen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#747323} Bug: 1047037 Change-Id: Ifb2d2b97a190c98b00ff2244c2f4d4ae879bdac9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095875Reviewed-by:
Hesen Zhang <hesen@chromium.org> Reviewed-by:
Xing Liu <xingliu@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Hesen Zhang <hesen@chromium.org> Cr-Commit-Position: refs/heads/master@{#748738}
-
Yuki Shiino authored
Bug: 839389 Change-Id: Ibac6eebb9f5c1b4b20d90fa61075c92a45089c99 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096322Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#748737}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/bf9df5292254..eaadca132fce git log bf9df5292254..eaadca132fce --date=short --first-parent --format='%ad %ae %s' 2020-03-10 sigurds@chromium.org [COEP] Add links to future COEP/COOP article Created with: gclient setdep -r src/third_party/devtools-frontend/src@eaadca132fce 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/+/master/autoroll/README.md Bug: chromium:1051473 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I76ddf0000291b43671f3976e04a16447a7fb3edf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096544Reviewed-by:
-
Ken Rockot authored
This is a reland of 1b21e379 Unmodified from the original, which was reverted to revert a preceding change. Original change's description: > Migrate URL and Origin typemaps to GN definitions > > Also moves some longer bits of code out-of-line. > > Bug: 1059389 > Change-Id: Idd1b3606e26504ab92e9aefebc933823919a2431 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2092695 > Commit-Queue: Ken Rockot <rockot@google.com> > Reviewed-by: Dave Tapuska <dtapuska@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#748058} Bug: 1059389 Change-Id: Ifd8edc62b361cd5947e6161758ce310c2c912556 Tbr: dcheng@chromium.org Tbr: dtapuska@chromium.org Nopresubmit: True Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095943 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by:
Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#748735}
-
Jonathan Ross authored
This test has begun flaking regularly on Windows. Marking as RetryOnFailure so that we can hopefully handle the flakes. TBR=ynovikov@chromium.org TEST= webgl2_conformance_gl_passthrough_tests deqp/functional/gles3/shaderloop_do_while.html Bug: 1060024 Change-Id: I5fb22fcf429699839d972dd8a6b512f5dbab48d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096089Reviewed-by:Jonathan Ross <jonross@chromium.org> Reviewed-by:
-
Rouslan Solomakhin authored
Before this patch, the classes in payment_app_provider_impl.h/cc used a combination of the deprecated DISALLOW_COPY_AND_ASSIGN and not specifying copying behavior. This patch removes DISALLOW_COPY_AND_ASSIGN and instead explicitly deletes the copy constructor and assign operator in payment_app_provider_impl.h/cc. After this patch, the classes in payment_app_provider_impl.h/cc use the current style of explicitly deleting their copy constructor and assign operator. Bug: 1005076 Change-Id: Ibae5b145927e2fd8e1e751605caf4aaa49b41ce4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2090491 Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
-
Sam Bowen authored
Support repeated property values of different types. The mojo file change from union to struct isn't compatible with Android so fork this file as well to avoid temporary build breakages in Clank. Also had to change module name to avoid duplicate build definitions. Open to better suggestions on this and the file name. Bug: 1055887 Change-Id: Iad691c45c25a404a4b560b604c63541f7feab89a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2073337 Commit-Queue: Sam Bowen <sgbowen@google.com> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#748732}
-
Corentin Wallez authored
Previously it didn't take a device client id, which meant the map associating shared images to texture reservations in the wire was global to the WebGPUDecoderImpl. This is incorrect because there is one wire per device. This CL adds the missing argument to DissociateMailbox and makes the associated shared image map a per-device thing. It also adds a regression test. Bug: chromium:1059777 Change-Id: Id4db2556dc19f90a13e1af6381f855f7d633e113 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095123 Commit-Queue: Corentin Wallez <cwallez@chromium.org> Reviewed-by:
Austin Eng <enga@chromium.org> Reviewed-by:
-
Oriol Brufau authored
This patch adds LayoutInsideListMarker and LayoutOutsideListMarker classes for ::marker. The logic remains in LayoutListMarker for now, but this is a first step to implementing inside markers with LayoutInline and outside markers with LayoutBlockFlow, which will be necessary in order to support the 'content' property in legacy layout. TEST=blink/web_tests/editing/execCommand/create-list-1.html TEST=editing/pasteboard/drag-selected-image-to-contenteditable.html TEST=editing/pasteboard/input-field-1.html TEST=editing/selection/drag-to-contenteditable-iframe.html TEST=editing/selection/move-by-line-002.html TEST=editing/selection/selectNode.html TEST=editing/selection/selectNodeContents.html TEST=editing/unsupported-content/list-delete-001.html TEST=editing/unsupported-content/list-type-after.html TEST=editing/unsupported-content/list-type-before.html TEST=paint/invalidation/list-marker-2.html BUG=457718 Change-Id: I1eda402ac6e9b137031992257a0542fffa103639 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094991 Commit-Queue: Oriol Brufau <obrufau@igalia.com> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#748730}
-
Yann Dago authored
Screenshot: http://screen/h5v2MCq2nzc Bug: 1050721 Change-Id: If946fd0077aa1db012f44359753f7a01fc6ecb11 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2085753 Commit-Queue: Yann Dago <ydago@chromium.org> Reviewed-by:
Kamila Hasanbega <hkamila@google.com> Reviewed-by:
Andy Paicu <andypaicu@chromium.org> Cr-Commit-Position: refs/heads/master@{#748729}
-
Friedrich Horschig authored
With this CL, the title and subtitle will change based on the progress and the status of the password check. Screenshots at the linked https://crbug.com/1047726#c31 Translation screenshots: IDS_SETTINGS_CHECK_PASSWORDS_ERROR_GENERIC.png: IDS_SETTINGS_CHECK_PASSWORDS_ERROR_NO_PASSWORDS.png: IDS_SETTINGS_CHECK_PASSWORDS_ERROR_OFFLINE.png: IDS_SETTINGS_CHECK_PASSWORDS_ERROR_QUOTA_LIMIT.png: IDS_SETTINGS_CHECK_PASSWORDS_ERROR_SIGNED_OUT.png: IDS_SETTINGS_CHECK_PASSWORDS_ERROR_TOO_MANY_PASSWORDS.png: IDS_SETTINGS_CHECK_PASSWORDS_PROGRESS.png: IDS_SETTINGS_CHECK_PASSWORDS_CANCELED.png: https: //storage.cloud.google.com/chromium-translation-screenshots/4de6e19b4772e8e130b87952ad8fcc10b79054e7 https: //storage.cloud.google.com/chromium-translation-screenshots/ce276f9b31384860c1369b11dd2844ed01d988d6 https: //storage.cloud.google.com/chromium-translation-screenshots/014f118ca023ad004523e7860746aae0a96f981d https: //storage.cloud.google.com/chromium-translation-screenshots/7685361448310ce06f9b704483d1ddd16721ee62 https: //storage.cloud.google.com/chromium-translation-screenshots/25b0c6ed74bd2313c5b7f954a7d34a0ec46cf2d7 https: //storage.cloud.google.com/chromium-translation-screenshots/31e9ccf14e6882a56ffffac5d6ee583e756b7136 https: //storage.cloud.google.com/chromium-translation-screenshots/d5a0c340e5f407e3347eb3323bd9c08b85c09f95 https: //storage.cloud.google.com/chromium-translation-screenshots/1b9d821e5ff2b0fcbf93c120f7e604f61eb6cb63 Bug: 1047726 Change-Id: I265376b6879427c3cbbc318d1d50bc1a0b89d969 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087679 Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#748728}
-
Fergal Daly authored
The current behaviour is not consistent with the name and the two places that call CreateRenderFrameHost already have a call to GetRenderViewHost in their logic. This is a pre-factor for https://crrev.com/c/2084038/. Change-Id: Ib9a16101373f0027cb712c955501360425056a97 Bug: 1059678 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2089569 Commit-Queue: Fergal Daly <fergal@chromium.org> Reviewed-by:
Fergal Daly <fergal@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
-
Rodney Ding authored
See the crbug for more details on the related spec in v15. Bug: 1049698 Change-Id: Iaad730e14630e528a6041ac3a54cec23ded5ee27 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2085696 Commit-Queue: Rodney Ding <rodneyding@google.com> Reviewed-by:
Asanka Herath <asanka@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/heads/master@{#748726}
-
Joshua Pawlicki authored
TBR=andypaicu@chromium.org Bug: 1060175 Change-Id: I524dba2e7ebb9ed1bf1de3060912dfe7d4a66222 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095968Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Commit-Queue: Joshua Pawlicki <waffles@chromium.org> Cr-Commit-Position: refs/heads/master@{#748725}
-
Fergal Daly authored
Currently the ID is used only to signal that a widget exists, however this can be deduced by looking at the frame's parent (or lack of) and site instances. Previously the ID for this case was being generated in RenderFrameHostManager::CreateRenderFrame and passed all the way down. It's simpler to just generate one at the latest point possible. This is a pre-factor for https://crrev.com/c/2084038/. Change-Id: I5627e204c7afeab8b05bcb8a5f09a162e051c1c6 Bug: 1059678 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2089546 Commit-Queue: Fergal Daly <fergal@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Reviewed-by:
-
Mikel Astiz authored
The APIs haven't been used in a long while, presumably after https://crrev.com/1454983004. Change-Id: I1bfbc537663348e0e6c74f0a82e3df77911037d6 Bug: None Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095576Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#748723}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/40d81599..c98aec6a 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,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Iad5f5a4e331c5e4fd04a3ffb3b6b51bc1f8b17d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094665Reviewed-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@{#748722}
-
