- 30 Jul, 2019 40 commits
-
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/34f8bd8c4850..7b57c5bb770f git log 34f8bd8c4850..7b57c5bb770f --date=short --no-merges --format='%ad %ae %s' 2019-07-30 cwallez@chromium.org Check bindgroup validity first in SetBindGroup Created with: gclient setdep -r src/third_party/dawn@7b57c5bb770f The AutoRoll server is located here: https://autoroll.skia.org/r/dawn-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel TBR=cwallez@google.com Bug: None Change-Id: I534da582a9b1e489af000212d34152cafe2e5d8d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725356Reviewed-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@{#682227}
-
Antonio Gomes authored
BUG=704136,919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: I2cb8fa3240c81253a0324f237b751e5f8f09c896 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724556Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#682226}
-
Leo Zhang authored
Move all the ime-related paths to constants, and share these paths across the different components. Bug: 837156 Change-Id: I067d6b03b81df7cfce8240558c796ab7d1a3edb4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1715678Reviewed-by:
Shu Chen <shuchen@chromium.org> Reviewed-by:
Yuichiro Hanada <yhanada@chromium.org> Commit-Queue: Leo Zhang <googleo@chromium.org> Auto-Submit: Leo Zhang <googleo@chromium.org> Cr-Commit-Position: refs/heads/master@{#682225}
-
Robbie Gibson authored
Bug: 976678 Change-Id: I8ea0b009d1aa8c928215292e4aa377097a66fbcd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1716753Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Commit-Queue: Robbie Gibson <rkgibson@google.com> Cr-Commit-Position: refs/heads/master@{#682224}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/2f01e42cf4f3..1157607f2f54 git log 2f01e42cf4f3..1157607f2f54 --date=short --no-merges --format='%ad %ae %s' 2019-07-30 linxinan@chromium.org chromite: add compiled proto files from test_platform Created with: gclient setdep -r src/third_party/chromite@1157607f2f54 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I1138689246176f23c0dc597750986933ea16cbad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725247Reviewed-by:
-
Dan Elphick authored
Change-Id: I398ff513479ffc26dc1117b4a1a35b37b30c5b71 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1704759 Commit-Queue: Dan Elphick <delphick@chromium.org> Commit-Queue: Peter Beverloo <peter@chromium.org> Auto-Submit: Dan Elphick <delphick@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#682222}
-
Balazs Engedy authored
TL;DR: Only browsing contexts that are same-origin with the top-level context should be able to request the "notifications" permission; but once an origin has obtained the permission, it should see its state as granted, and be permitted to use the related APIs both in first and third party contexts (in the latter case, regardless of the permission state for the top-level origin). In more detail, a cross-origin subframe that had not been granted the NOTIFICATIONS permission previously (in a first-party context) should see the permission as "denied" through all of `Notification.permission`, `pushManager.permissionState`, and `permissions.query`; not be able to prompt for the permission; and not be able to register Push API subscriptions or use the Web Notification API. A cross-origin subframe that had been granted the NOTIFICATIONS permission previously (in a first-party context) should see it as "granted", through all of the above-mentioned API surfaces; and be able to register Push API subscriptions and use the Web Notification API. This CL achieves this by explicitly turning off permission delegation for CONTENT_SETTINGS_TYPE_NOTIFICATIONS. Bug: 987654 Change-Id: Ie7d7dd7da7c9a4630b09a54ba215c883511feb50 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1720831 Commit-Queue: Balazs Engedy <engedy@chromium.org> Reviewed-by:
Raymes Khoury <raymes@chromium.org> Cr-Commit-Position: refs/heads/master@{#682221}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 0624ed94. With Chromium commits locally applied on WPT: 250461b3 "Worker: Register a service worker just once for worker subresource interception tests" bf4ad57e "Experimental CookieStore API: some additional browser-side validation" 14d3a4a1 "Sort tracks by id in MediaStreamAudioSourceNode" 40adbd62 "Rename traits and mojo directory in //skia/public" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: skobes@chromium.org: external/wpt/css/css-scroll-anchoring smcgruer@chromium.org: external/wpt/css/css-position NOAUTOREVERT=true TBR=lpz No-Export: true Change-Id: Ifa298eb5d0107178312f603f8bdd879afc592cd4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724706Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#682220}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/9addf0cb8cee..70cded6ac635 git log 9addf0cb8cee..70cded6ac635 --date=short --no-merges --format='%ad %ae %s' 2019-07-30 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/spirv-tools/src 9559cdbdf011..ac3d131054ac (4 commits) 2019-07-30 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/glslang/src 7fc868349143..f04f1f93a70f (1 commits) Created with: gclient setdep -r src/third_party/angle@70cded6ac635 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=geofflang@google.com Bug: chromium:None,chromium:None Change-Id: I3b823feb0a47766528fc63f8e8376b8df80bc3cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725246Reviewed-by:
-
Marian Fechete authored
This CL enables support for requiring a billing postcode in the Payment Request, based on a flag in the service proto. If the flag is set to false (default), then a postcode is not required to complete a Payment Request. If the flag is set to true, and the billing address doesn't contain a postcode (based on regional settings), an error message is shown and the flow cannot be continued. Bug: b/133731582 Change-Id: Ie6f0199b9dc558d642f1bc4c1dd4faf301d6d512 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1722940 Commit-Queue: Marian Fechete <marianfe@google.com> Reviewed-by:
Clemens Arbesser <arbesser@google.com> Cr-Commit-Position: refs/heads/master@{#682218}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/bfd47a6a..e78b7c05 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: Ic24e0f65d302b032345d06aa9298fe432b349be7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724643Reviewed-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@{#682217}
-
Pauline Leitao authored
BookmarkNodes currently have no GUID or ID consistent across different clients and that remains stable throughout the lifetime of the bookmark. This CL thus introduces such a GUID. This will allow for the insertion of logic geared towards avoiding bookmark duplicates when synced across different devices and also potentially substitute other identifier fields later on. GUIDs are randomly generated for each added bookmark, with the exception of BookmarkPermanentNodes, for whom hard-coded GUIDs are used instead. Objectively, this CL does the following: 1. Introduces an immutable GUID to the BookmarkNode and a corresponding key to the persisted JSON object, in order to allow for encoding/decoding. 2. Changes BookmarkNode constructors to always take in an ID and GUID to be set to the newly created node. 3. Changes the BookmarkPermanentNode constructor to assign hard-coded GUIDs to PermanentNodes (stored in a map) based on their Type. Bug: 978430 Change-Id: I77909f33e24b454f026d5566f64cb731a08f2b68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1676167Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Commit-Queue: Pauline Leitao <psivieroleitao@google.com> Cr-Commit-Position: refs/heads/master@{#682216}
-
Julie Jeongeun Kim authored
This CL is a part of moving access token management to OAuth2AccessTokenManager. It updates the type for scopes with ScopeSet for OnAccessTokenInvalidated from OAuth2AccessTokenManager::Delegate to keep consistency. Bug: 967598 Change-Id: Ic5163c4ce12bc9eba43b46ff13b6bb43b72c7309 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725477Reviewed-by:
Colin Blundell <blundell@chromium.org> Commit-Queue: Julie Jeongeun Kim <jkim@igalia.com> Cr-Commit-Position: refs/heads/master@{#682215}
-
Anders Hartvoll Ruud authored
We changed the name of PropertyDescriptor once @property was added to the spec to avoid confusion with the at-rule descriptors. This CL is not a web-facing change that's strictly necessary, but it's nice to align the naming with the spec. R=futhark@chromium.org Change-Id: I49dbecae5bdccd06e8c75f2c01405d103dbc44f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724522Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Anders Hartvoll Ruud <andruud@chromium.org> Cr-Commit-Position: refs/heads/master@{#682214}
-
Anna Malova authored
The bug was introduced when aw_browser_context prefs were splited up into profile-specific vs global. media_drm_storage is supposed to be profile-specific. Bug: 986965 Change-Id: I29b20f8e77fbe0a3dfa3472ac963bbb096fe3a8d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1720240 Commit-Queue: Anna Malova <amalova@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#682213}
-
Anders Hartvoll Ruud authored
This is a more careful version of rajendrant's CL [1] which doesn't assume that we always have an animating element. [1] https://crrev.com/c/1642949 Change-Id: I931bb55522dcdbf90fa10e2009e57ff1bdb88f2e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724680Reviewed-by:
-
Victor Hugo Vianna Silva authored
Aiming at having an over approximation for the grouping of requests in HistoryUiFaviconRequestHandler, we record whether an icon was already present in the thumbnail database when downloading an on-demand icon. Bug: 985706 Change-Id: Iea71714d81d7f4669b37ec0613e3ffb708e75012 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1709576 Commit-Queue: Victor Vianna <victorvianna@google.com> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Reviewed-by:
-
Idan Raiter authored
When loadValue is a "load", we make a new GPUColor and convert it to a dawn type. However, the conversion tries to access the rgba components of the color, which triggers checks such as has_r_ (since the color hasn't been set). Bug: 877147 Change-Id: Iec24f35a94beb0e17d17d4d7c9f7ac5175c949e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724584 Commit-Queue: Corentin Wallez <cwallez@chromium.org> Reviewed-by:
Kai Ninomiya <kainino@chromium.org> Reviewed-by:
Corentin Wallez <cwallez@chromium.org> Cr-Commit-Position: refs/heads/master@{#682210}
-
Adam Rice authored
Add a callback argument to WebSocket Send() for string and ArrayBuffer. The callback will be called when the send completes, if it happens asynchronously and the callback is non-null. The callback is not used yet. It will be used by the WebSocketStream implementation. The methods return SENT_SYNCHRONOUSLY if they were able to send the data over mojo synchronously, as in that case the callback will not be called. The Blob Send method hasn't been changed, as it won't be used by WebSocketStream. See design doc for WebSocketStream at https://docs.google.com/document/d/1XuxEshh5VYBYm1qRVKordTamCOsR-uGQBCYFcHXP4L0/edit BUG=983030 Change-Id: Ia7f3147a49e8bc55f8ead63e611a395124118959 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1718445 Commit-Queue: Adam Rice <ricea@chromium.org> Reviewed-by:
Yoichi Osato <yoichio@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#682209}
-
Julie Jeongeun Kim authored
This CL is a part of moving access token management to OAuth2AccessTokenManager. It moves InvalidateAccessTokenImpl from OAuth2AccessTokenManager to protected since it's only used from child classes. It also cleans up invalid TODO messages. Bug: 967598 Change-Id: Id4cd6855c4dfc96d5edb668341bf0277ce8ad6fa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1723850Reviewed-by:
-
Lutz Justen authored
Expects timeout on Win 7 for external/wpt/referrer-policy/strict-origin-when-cross-origin/ attr-referrer/same-origin/http-http/img-tag/keep-origin-redirect/ same-insecure.http.html virtual/blink-cors/external/wpt/referrer-policy/ strict-origin-when-cross-origin/attr-referrer/same-origin/http-http/ img-tag/keep-origin-redirect/same-insecure.http.html virtual/not-omt-sw-fetch/external/wpt/referrer-policy/ strict-origin-when-cross-origin/attr-referrer/same-origin/http-http/ img-tag/keep-origin-redirect/same-insecure.http.html TBR=estark@chromium.org BUG=chromium:988890 TEST=None Change-Id: I1f11fa9168933bcf00ce519393631f5009920fbc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725633Reviewed-by:Lutz Justen <ljusten@chromium.org> Commit-Queue: Lutz Justen <ljusten@chromium.org> Cr-Commit-Position: refs/heads/master@{#682207}
-
Makoto Shimazu authored
Bug: 937175 Change-Id: I6f909cbd2bf85d3cbe5192fd13d4438676a6af99 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725552 Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org> Auto-Submit: Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#682206}
-
David Jean authored
A simple change for opening discussion about UMA for Startup time goal: appLaunchTime is used (in metrics_mediator.mm) to fill the following existing metrics: "Startup.ColdStartWithExternalURLTime" "Startup.ColdStartWithoutExternalURLTime" appLaunchTime is set in main_controller, which in the simulator, runs about 0.09 seconds later than IOSChromeMain::InitStartTime(), the first call in main(). I couldn't find any use of IOSChromeMain::StartTime() so I retyped it to a base::TimeTicks and used it as a replacement value in main_controller. If there is something/someone currently tracking the two metrics above, then they will get slightly bigger from now on. If it is not acceptable, then we can look at creating parallel Startup metrics. Bug: 988395 Change-Id: Ie3e7d4d1c0f9931a5f760c6fcd5a06ceaa4a0ceb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1722538 Commit-Queue: David Jean <djean@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#682205}
-
Christos Froussios authored
This reverts commit 103900b9. Reason for reland: The preceding CL was also relanded. Original change's description: > Revert "[Password Manager] Restore UMA metric for how many linux users don't have encryption" > > This reverts commit 7dd2ac40. > > Reason for revert: landed on top of a change that needs to be reverted for https://bugs.chromium.org/p/chromium/issues/detail?id=988010 > > Original change's description: > > [Password Manager] Restore UMA metric for how many linux users don't have encryption > > > > The existing UMA uploads don't separate new profiles from existing > > profiles which couldn't be migrated. > > > > Bug: 950267 > > Change-Id: I0f8f823e843bd2127a71438c347031f6a7b23346 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1720630 > > Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> > > Auto-Submit: Christos Froussios <cfroussios@chromium.org> > > Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#681348} > > TBR=vasilii@chromium.org,cfroussios@chromium.org > > Change-Id: Ib6d74e8400e81f0a62160326273abf76dd74dc6d > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 950267 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1721291 > Reviewed-by: John Budorick <jbudorick@chromium.org> > Commit-Queue: John Budorick <jbudorick@chromium.org> > Cr-Commit-Position: refs/heads/master@{#681351} TBR=vasilii@chromium.org,cfroussios@chromium.org,jbudorick@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 950267 Change-Id: I3c4470089468507b92104e0108f6078b1cef2938 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724759Reviewed-by:
Christos Froussios <cfroussios@chromium.org> Commit-Queue: Christos Froussios <cfroussios@chromium.org> Cr-Commit-Position: refs/heads/master@{#682204}
-
Jérôme Lebel authored
In ChromeSigninViewController, when a swipe to dismiss is done by the user, the secondary button (Cancel) action is triggered. The swipe to dismiss does the exact same behavior than the Cancel button. Swipe to dismiss is only enabled for UnityConsent flag enabled. Bug: 987232 Change-Id: I3dc6f41a49652e3a2c4482628f26e1de5621ab47 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1715292 Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
-
Christos Froussios authored
This reverts commit 341d33c3. Reason for reland: The preceding CL was also relanded. Original change's description: > Revert "[Password Manager] Cleanup feature: Migrate Linux to LoginDB" > > This reverts commit 3a84c395. > > Reason for revert: https://chromium-review.googlesource.com/c/chromium/src/+/1713560 didn't stick, see https://bugs.chromium.org/p/chromium/issues/detail?id=988010 > > Original change's description: > > [Password Manager] Cleanup feature: Migrate Linux to LoginDB > > > > The migration has been completed and the feature is no longer used. > > > > Bug: 950269 > > Change-Id: I91f79f33ccda95e75d2b6156ade5d44f2e27c0ae > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1719008 > > Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org> > > Commit-Queue: Christos Froussios <cfroussios@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#681244} > > TBR=vasilii@chromium.org,cfroussios@chromium.org > > Change-Id: Ic0733620d3ebca563f5fa0be9f60742770eacd45 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 950269 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1721290 > Reviewed-by: John Budorick <jbudorick@chromium.org> > Commit-Queue: John Budorick <jbudorick@chromium.org> > Cr-Commit-Position: refs/heads/master@{#681347} TBR=vasilii@chromium.org,cfroussios@chromium.org,jbudorick@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 950269 Change-Id: Ifb49e8c9a6feeeaaa6e097296dda46540df66f30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724527Reviewed-by:
-
Omer Katz authored
Added a worklist used by MarkingVisitor to record callbacks for backing stores. HeapCompact then reads that worklist to propagate it's inner Fixups() object (used to update references during compaction). Bug: 986235 Change-Id: Idebb79918ab855e768889f3bd4e45476d0ce5c53 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1720643 Commit-Queue: Omer Katz <omerkatz@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#682201}
-
Lowell Manners authored
This adds a NavigationsAreFullyCommitted test which checks that the BFCache code is actually committing navigations, and isn't just swapping in the frame without committing. This CL also defines a separate PageWithDedicatedWorkerNotCached test which focuses on a single disallowed feature. We expect to have many more tests like this, (one per disallowed feature). Change-Id: I5a4c17cd19698cc21dfcaa8dcd7c1d31eecf135d Bug: 976697 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1702021 Commit-Queue: Lowell Manners <lowell@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#682200}
-
Shao-Chuan Lee authored
This reverts commit 56c0a53e. Reason for revert: breaks betty-arcnext-chrome-pfq Original change's description: > viz: Create Separate Type for OutputSurface as Overlay > > When using a BufferQueue, the OutputSurface is treated as an Overlay, > and is made into an OverlayCandidate. That overlay candidate is then > pushed in as the front of the candidate list. This CL changes it into > its own type instead of checking for a bool on OverlayCandidate. > > R=rjkroege > > Bug: 963597 > Change-Id: I05588ba4d3b1ac03ed01ac0fff25d9ae03fc28c5 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1696168 > Commit-Queue: weiliangc <weiliangc@chromium.org> > Reviewed-by: Robert Kroeger <rjkroege@chromium.org> > Reviewed-by: Khushal <khushalsagar@chromium.org> > Cr-Commit-Position: refs/heads/master@{#680431} TBR=rjkroege@chromium.org,weiliangc@chromium.org,khushalsagar@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 963597, 988114 Change-Id: I614aa2c943ea8fdbfca1da40ae622d79070eeda3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725033 Commit-Queue: Shao-Chuan Lee <shaochuan@chromium.org> Reviewed-by:
Shao-Chuan Lee <shaochuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#682199}
-
Carlos Caballero authored
Use ScopedTaskEnvironment instead of MessageLoopForUI in /base/memory/memory_pressure_monitor_win_unittest.cc MessageLoopForUI will go away, eventually. ScopedTaskEnvironment will per default start a ThreadPool, which should be fine in most of the cases. If you believe your test needs to make sure that no ThreadPool runs let me know and I will update the patch. BUG=891670 This CL was uploaded by git cl split. R=brucedawson@chromium.org Change-Id: I212a7534542ecb94ed6414190e285f7eeaff2288 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1715363 Auto-Submit: Carlos Caballero <carlscab@google.com> Reviewed-by:
Bruce Dawson <brucedawson@chromium.org> Commit-Queue: Carlos Caballero <carlscab@google.com> Cr-Commit-Position: refs/heads/master@{#682198}
-
Matt Falkenhagen authored
This is a reland of eb6b4eb4 Original change's description: > Disable flaky test WebRtcMediaStreamTrackAdapterTest.LocalVideoTrack. > > Bug: 982200 > Change-Id: I68547d2d5b097ab0fd816e44ca548a0024acec70 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1692223 > Reviewed-by: Sergey Ulanov <sergeyu@chromium.org> > Commit-Queue: Matt Falkenhagen <falken@chromium.org> > Cr-Commit-Position: refs/heads/master@{#675841} Bug: 982200 Change-Id: Idd6a45b5fc6d6062aca35f6ee15bfe51c44e9eab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1722948Reviewed-by:
Armando Miraglia <armax@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#682197}
-
Stepan Khapugin authored
This reverts the improvements introduced in https://chromium-review.googlesource.com/c/chromium/src/+/1183495 on iOS 13 due to a UIKit bug. This way, swiping on the system keyboard does not hang the app and the keyboard. However, bugs 875918,873544 are regressing. Bug: 988431, 875918, 873544 Change-Id: I91f358d80531d17104ffc6db96d4a488864325a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1722545Reviewed-by:
Justin Cohen <justincohen@chromium.org> Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Auto-Submit: Stepan Khapugin <stkhapugin@chromium.org> Cr-Commit-Position: refs/heads/master@{#682196}
-
Md. Hasanur Rashid authored
As the "overflow" shorthand is divided into two longhands: "overflow-x" and "overflow-y", we add two different counters for both longhands to be counted while parsing the shorthand. Bug: 984177 Change-Id: I5eef66b9cc6bc72a023cfeb8b80ac584e71b433e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724640Reviewed-by:
-
Miyoung Shin authored
|pause_| can be set on the main thread and read on the io thread at the same time. Declaring |pause_| as an atomic variable solves the data race problem. Fuzzer report: https://clusterfuzz.com/testcase?key=5766664522432512 Bug: 988327 Change-Id: Idd43e9d932dc96900b46a09c8b46ba4337505880 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1722952 Commit-Queue: Miyoung Shin <myid.shin@igalia.com> Reviewed-by:
-
Marc Treib authored
It was decided not to launch this for now, so let's reflect that in the client-side default value. Bug: 983940 Change-Id: If6531b697212586e3832d30fea666481c774131d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724749Reviewed-by:
Jan Krcal <jkrcal@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#682193}
-
Lutz Justen authored
Marks org.chromium.chrome.browser.contextmenu.ContextMenuTest #testContextMenuRetrievesImageOptions_NoDefaultSearchEngine as flaky on Android Nexus 5. TBR=tedchoc@chromium.org BUG=chromium:988891 TEST=None Change-Id: Iee2e866aafb57024641a88ac58750aff8a86137d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724524Reviewed-by: -
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/a79c2440f653..9addf0cb8cee git log a79c2440f653..9addf0cb8cee --date=short --no-merges --format='%ad %ae %s' 2019-07-30 timvp@google.com Vulkan: Enable Line Rasterization Emulation Created with: gclient setdep -r src/third_party/angle@9addf0cb8cee The AutoRoll server is located here: https://autoroll.skia.org/r/angle-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=geofflang@google.com Bug: None Change-Id: I493251b1d8831c71855ae7c26a58db220ccf12d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725300Reviewed-by:
-
Eugene But authored
child_window_open_by_dom_egtest.mm file is already EG2 compatible. This CL simply adds a file to an existing EG2 target. Bug: 987646 Change-Id: I763129056ae5ca3f898a3a887eca8c59a0317747 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724821 Auto-Submit: Eugene But <eugenebut@chromium.org> Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by:
Stepan Khapugin <stkhapugin@chromium.org> Cr-Commit-Position: refs/heads/master@{#682190}
-
Makoto Shimazu authored
Previously 404 was just treated as a identical scripts, but it's wrong because if a new worker tries to import the script during the initial script evaluation for a update, it should fail due to the network error. This CL adds code to capture the network failure in ServiceWorkerSingleScriptUpdateChecker and replay the network failure in ServiceWorkerNewScriptLoader. The failure in update-import-scripts.https.html will be fixed once ServiceWorkerImprotedScriptUpdateCheck is enabled. Bug: 648295 Change-Id: Ibcd879cc4c46b9878b1f6df1a6983c110f606fe7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1715414 Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Auto-Submit: Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Kenichi Ishibashi <bashi@chromium.org> Cr-Commit-Position: refs/heads/master@{#682189}
-
Yutaka Hirano authored
Reject invalid use found in https://crrev.com/c/chromium/src/+/1695341/. Bug: 862184 Change-Id: I174d90bec44b8f077168a446b671a76ad38f0317 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1699587 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
David Benjamin <davidben@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Takashi Toyoshima <toyoshim@chromium.org> Cr-Commit-Position: refs/heads/master@{#682188}
-
