- 03 Dec, 2019 40 commits
-
-
Jan Wilken Dörrie authored
This change removes password_manager::BlacklistedCredentialsCleaner and the corresponding kBlacklistedCredentialsNormalized pref. The clean-up logic was introduced in M76 and has run its course. The corresponding histogram will be marked as obsolete. Bug: 952336, 988696 Change-Id: I590acfa828f03ec851480c7e0369af2fe3e1c927 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1936488Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#720919}
-
Shuhei Takahashi authored
This reverts commit a47f7784. Reason for revert: breaks graphics on some Chrome OS devices. Original change's description: > Delete bind_interface_helpers.h > > This header provides a bunch of helper functions for Content processes > modeled as Service Manager services. Since those are no longer a thing, > this no longer needs to be a thing. > > Bug: 977637 > Change-Id: Ia1ae177b4a9665b727618b12a35be905d5605ce9 > Tbr: jam@chromium.org > Tbr: erikchen@chromium.org > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1929396 > Commit-Queue: Ken Rockot <rockot@google.com> > Reviewed-by: Ken Rockot <rockot@google.com> > Cr-Commit-Position: refs/heads/master@{#718367} Bug: 1028852 Bug: 977637 Tbr: rockot@google.com Tbr: jam@chromium.org Tbr: erikchen@chromium.org Change-Id: Ie621804e4a6c03c26bf64130bab49f6a9380dea6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948401Reviewed-by:
Shuhei Takahashi <nya@chromium.org> Commit-Queue: Shuhei Takahashi <nya@chromium.org> Cr-Commit-Position: refs/heads/master@{#720918}
-
Balazs Engedy authored
Now that we have settled on using an animated icon on desktop, and a mini infobar on Android, remove the ability to specify the UI flavor using either variation parameters or FeatureEntry::FeatureParams. The CL also removes now-obsolete string literals, and cleans up tests: -- PermissionRequestNotificationAndroidTest.ShouldShowAsNotification already codified the expectation that the UI flavor variation parameters are ignored, this test is now removed. -- PermissionRequestManagerBrowserTest_StaticIcon.* verified behavior that was orthogonal to whether the quiet prompt is shown as static or animated icon, and the test for the latter still exists, so the former is now removed. Bug: 1030216 Change-Id: I32c27d327a8704572bb617cd0800338587f58dab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948428Reviewed-by:
Kamila Hasanbega <hkamila@chromium.org> Commit-Queue: Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#720917}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/6b4f6d08b82f..449ac187cb55 git log 6b4f6d08b82f..449ac187cb55 --date=short --first-parent --format='%ad %ae %s' 2019-12-03 aerotwist@chromium.org Adds manual boot perf script Created with: gclient setdep -r src/third_party/devtools-frontend/src@449ac187cb55 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:1027519 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I1be4bea654ca97f4477a1f1234c58fdf88c91576 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948613Reviewed-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@{#720916}
-
Omar Morsi authored
This reverts commit 5d5dad5b. Reason for revert: This feature is targeting M-81 as shown in Bug: 1026612 and it will be included in M-80 if not reverted. Original change's description: > Turn on enterprise network state reporting for affiliated users. > > Network state should be reported in three conditions. > 1- Primary user signed in is affiliated. > 2- Managed device is in kiosk mode. > 3- Managed guest session is running. > > Bug: 1027086 > Test: browser_tests --gtest_filter=DeviceStatusCollectorNetwork* > Change-Id: I29aaad9f5e442ea164bd1afd28a3bca72c66e459 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1928702 > Commit-Queue: Omar Morsi <omorsi@google.com> > Reviewed-by: Maksim Ivanov <emaxx@chromium.org> > Cr-Commit-Position: refs/heads/master@{#720225} TBR=emaxx@chromium.org,omorsi@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1027086 Change-Id: Id16de075fa98990d85f390f1c791917f4855fbe4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947651Reviewed-by:
Pavol Marko <pmarko@chromium.org> Commit-Queue: Omar Morsi <omorsi@google.com> Cr-Commit-Position: refs/heads/master@{#720915}
-
Jan Scheffler authored
This cl reenables test that have previously failed because of an issue with the sourcemaps. Bug: chromium:1936249 Change-Id: I8b3d8d2a79b6fa97fa64743523ffc001eabe4b19 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1943246Reviewed-by:
Yang Guo <yangguo@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Auto-Submit: Jan Scheffler <janscheffler@chromium.org> Cr-Commit-Position: refs/heads/master@{#720914}
-
Alexandre Courbot authored
CL crrev.com/c/1947511 improved support for V4L2 requests, but introduced a bug in the SVDA where devices using the config store will also try to acquire the requests queue (and fail in doing so). This CL fixes this. BUG=1009921 BUG=1029856 TEST=vdatests passing with and without --use_vd on Kevin. TBR=fbuergisser@chromium.org TBR=hiroh@chromium.org Change-Id: I9ee512ec1476170565d829704e33c128abbff207 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948579Reviewed-by:
Alexandre Courbot <acourbot@chromium.org> Commit-Queue: Alexandre Courbot <acourbot@chromium.org> Cr-Commit-Position: refs/heads/master@{#720913}
-
Vitaly Buka authored
This is a reland of 4c424ef5 Windows bot is fixed with crrev.com/c/1946985 Original change's description: > Enable init_stack_vars for non-official Windows builds > > With crrev.com/c/1914359 merged so we can enable the flag on Windows. > Patch is similar to crrev.com/c/1795632 and crrev.com/c/1838383. > > Bug: 977230 > Change-Id: I166392ee0f65ed7fd7f68cbdd5b7a3b7d589539b > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854805 > Commit-Queue: Vitaly Buka <vitalybuka@chromium.org> > Auto-Submit: Vitaly Buka <vitalybuka@chromium.org> > Reviewed-by: Hans Wennborg <hans@chromium.org> > Cr-Commit-Position: refs/heads/master@{#719737} Bug: 977230 Change-Id: I7fa02038503e83eb5ccdbba8c07dc08010c2722f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1941278Reviewed-by:
Hans Wennborg <hans@chromium.org> Commit-Queue: Vitaly Buka <vitalybuka@chromium.org> Cr-Commit-Position: refs/heads/master@{#720912}
-
Himanshu Jaju authored
To avoid potential memory leak in the future, we make the destructor overridable. Bug: 1029505 Change-Id: I75ac34ceface5a09404bb720528faab5e63b0f14 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1945819Reviewed-by:
Richard Knoll <knollr@chromium.org> Commit-Queue: Himanshu Jaju <himanshujaju@chromium.org> Cr-Commit-Position: refs/heads/master@{#720911}
-
Mikhail Khokhlov authored
This CL implements computing TBMv3 metrics using the Perfetto trace processor. Metrics to compute should be passed as tags with key "tbmv3". Bug: 990304 Change-Id: I514396861f801d2d9889e5866bf95f2cae21be4e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1942651 Commit-Queue: Mikhail Khokhlov <khokhlov@google.com> Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Cr-Commit-Position: refs/heads/master@{#720910}
-
Vitaly Buka authored
mini_installer uses unusual linking configuration to avoid linking CRT. Bug: 977230 Change-Id: I143a693aebc333c26c56c809038b5de3fab6ef91 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946985Reviewed-by:
Hans Wennborg <hans@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Commit-Queue: Vitaly Buka <vitalybuka@chromium.org> Cr-Commit-Position: refs/heads/master@{#720909}
-
Alex Clarke authored
During startup lots of tiny tasks are posted by FieldTrialSynchronizer. Individually they are not a problem but on low end devices it can take 60+ ms to process all of these. This patch significantly reduces the number of tasks posted. Change-Id: I1d0bff3386cfb308c27641f90189aae2071a5fe0 BUG: 1013535 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849852Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Alex Clarke <alexclarke@chromium.org> Auto-Submit: Alex Clarke <alexclarke@chromium.org> Cr-Commit-Position: refs/heads/master@{#720908}
-
Fabian Sommer authored
Fix l10n by no longer appending strings programmatically. R=emaxx (e.g. smartcard), enter wrong PIN. Bug: 1023360 Test: Enable RTL language, attempt to log in with PIN dialog Change-Id: I9d4ffd3d1acef532d4899ef95a99d31af6fb682e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1926483Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Fabian Sommer <fabiansommer@google.com> Cr-Commit-Position: refs/heads/master@{#720907}
-
Mikel Astiz authored
This patch is similar in spirit to the Android equivalent in https://chromium-review.googlesource.com/c/chromium/src/+/1930796. The overall idea is that, if encryption keys are missing, the UI needs to display this state as an error, and the user should be able to go through a reauth flow to retrieve the keys. The actual logic to trigger the key retrieval process is deferred to future patches and reflected in TODOs. Bug: 1019685 Change-Id: Iad250aaca3519b7af50c0d8bfc038009a7d0f30d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1937368 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#720906}
-
Peter E Conn authored
This adds a histogram similar to CustomTabs.Visible which can be used to separate metrics between those triggered in a Custom Tab and those triggered in Tabbed Chrome. This metric splits the use cases of Custom Tabs into Plain, Trusted Web Activity, WebAPK and Webapp. Bug: 1029500 Change-Id: I26fbbdb9eff1973ec660288a08abe902258dcd06 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1944170 Commit-Queue: Peter Conn <peconn@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#720905}
-
Richard Knoll authored
This adds metrics for the new phone number regex compared to the previous one. These are logged when we show the context menu and when a user selects a device from the menu. With those we can determine the false positive and false negative rate compared to the current regex. Bug: 1001033 Change-Id: Ibacfb095305eb106ec3dd21bce7c0e161172824c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1942655Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Commit-Queue: Richard Knoll <knollr@chromium.org> Cr-Commit-Position: refs/heads/master@{#720904}
-
Alice Wang authored
This CL enables the disabled signin helper test. Bug: 577880 Change-Id: I14d5f818a1b091b6392dca8083c217e179993239 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1942089Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Commit-Queue: Alice Wang <aliceywang@chromium.org> Cr-Commit-Position: refs/heads/master@{#720903}
-
Hitoshi Yoshida authored
Defines bindings::DictionaryBase as a base class for IDL dictionaries generated by the new code generator. This CL also prepares some basic APIs; NativeValueTraits and ToV8. Bug: 839389 Change-Id: I47012911933c9bd6a8934f4d18f27e4a86e80974 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898893 Auto-Submit: Hitoshi Yoshida <peria@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#720902}
-
Shuhei Takahashi authored
This reverts commit a9700aba. Reason for revert: Turned out this workaround does not work. Original change's description: > Prevent racy behavior breaking OOBE welcome dialog > > It seems that the dialog's show() can be invoked prior to attached(), > resulting in a null object being indexed. This adds a null > check to prevent the race from completely breaking login flow. > > Bug: 1028852 > Change-Id: I9bfae2e420b1a694a77479f49aa4633089689a54 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948185 > Auto-Submit: Ken Rockot <rockot@google.com> > Reviewed-by: Roman Sorokin [CET] <rsorokin@chromium.org> > Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> > Commit-Queue: Ken Rockot <rockot@google.com> > Cr-Commit-Position: refs/heads/master@{#720873} TBR=rockot@google.com,rsorokin@chromium.org Change-Id: I5220764c388c62240b42008e1caa25202d22f6f5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1028852 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948581Reviewed-by:
Shuhei Takahashi <nya@chromium.org> Commit-Queue: Shuhei Takahashi <nya@chromium.org> Cr-Commit-Position: refs/heads/master@{#720901}
-
Antonio Gomes authored
BUG=787254, 919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: I1125698a24e20fda4e6fbfecc0b2d52b5272d0c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946633Reviewed-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@{#720900}
-
Friedrich [CET] authored
This reverts commit b5bf966d. Reason for revert: Most likely culprit to break All/ShelfLayoutManagerTest.NoShelfUpdateDuringOverviewAnimation/1 (more details in the linked bug) Original change's description: > split view: Replace CanSnapInSplitview with SplitViewController function > > Instead of a CanSnapInSplitview function referring to the root window of > the argument window, there shall be a SplitViewController::CanSnapWindow > function referring to the root window of the SplitViewController. > Leveraging that, the split view drag indicators on each root window > shall account for whether the dragged window can be snapped in split > view on that root window. The present CL is also a step toward properly > implementing functionality where you drag from one display to another > and snap (see the TODO in OverviewWindowDragController::SnapWindow). > > Bug: 970013, 995413 > Change-Id: I0e0b6d34ec8b3b3856d26fc5750bb062405f8dc6 > Fixed: 961171 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906931 > Reviewed-by: Mitsuru Oshima <oshima@chromium.org> > Commit-Queue: Avery Musbach <amusbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#720754} TBR=oshima@chromium.org,amusbach@chromium.org Change-Id: I1398c2eefc9d275d90708229521ff906ba667493 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 970013, 995413, 1030222 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947726Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#720899}
-
Friedrich [CET] authored
This reverts commit 2f4e14b2. Reason for revert: Most likely culprit for the consistently failing Windows devices tests (also bluetooth-related). Original change's description: > Track WinRT device connection status > > Windows sometimes sends OnConnectionStatusChange events when the status > has not changed. Previously we had assumed the status changed and acted > accordingly. This change keeps track of the connection status and if > OnConnctionStatusChange fires with the same status it is now ignored. > > Bug: 127810 > Change-Id: I64c5b3d99e0ac827f05ca1618d570ff83296a053 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1939827 > Reviewed-by: Ovidio de Jesús Ruiz-Henríquez <odejesush@chromium.org> > Commit-Queue: James Hollyer <jameshollyer@chromium.org> > Cr-Commit-Position: refs/heads/master@{#720876} TBR=odejesush@chromium.org,jameshollyer@chromium.org Change-Id: I8add21a3922e2a2c56ad72cd8e986e4a7b67a228 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 127810 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947648Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#720898}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/1e3659a4633a..6b4f6d08b82f git log 1e3659a4633a..6b4f6d08b82f --date=short --first-parent --format='%ad %ae %s' 2019-12-03 mathias@chromium.org Update Device Toolbar user-agent strings for iPhone 6+ Created with: gclient setdep -r src/third_party/devtools-frontend/src@6b4f6d08b82f 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:1029031 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I51cc826d01db3b605bc795b098e0e1a5ad788279 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947762Reviewed-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@{#720897}
-
Mario Sanchez Prada authored
This CL migrates code to the new BrowserInterfaceBroker class. This allows retrieving the remote implementation in the browser process without relying on the InterfaceProvider API. Bug: 936482 Change-Id: I6f93bf8e1004234dc17d7aec7ab9f749581500d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1942696Reviewed-by:
anthonyvd <anthonyvd@chromium.org> Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Cr-Commit-Position: refs/heads/master@{#720896}
-
Chromium WPT Sync authored
Using wpt-import in Chromium de587506. With Chromium commits locally applied on WPT: 500a74f5 "Add a runtime flag to enforce strict MIME type checks for workers." a2a91a99 "Allow another touch start happen when there are active touch points" 85b61e6b "[resource-timing] Ensure workerStart is protected by TAO" a75430aa "Fix appcache tests." a50e7e60 "Move `Sec-Fetch-User` to SetSecFetchMetadataHeaders." 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: yhirano@chromium.org, mkwst@chromium.org, japhet@chromium.org: external/wpt/fetch NOAUTOREVERT=true TBR=raphael.kubo.da.costa No-Export: true Change-Id: I3127da76be08e729cade0b3586598f5110539436 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948324Reviewed-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@{#720895}
-
Alexandre Courbot authored
Imported output frames must have the EGL format and size, otherwise the image processor will complain of the mismatch since crrev.com/c/1939078. BUG=chromium:1030190 TEST=vdatests passing on Hana. Change-Id: I75356aff8c4b0fefb0d60bdce6d4eda889cc25fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948372 Commit-Queue: Alexandre Courbot <acourbot@chromium.org> Commit-Queue: Chih-Yu Huang <akahuang@chromium.org> Auto-Submit: Alexandre Courbot <acourbot@chromium.org> Reviewed-by:
Chih-Yu Huang <akahuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#720894}
-
Oleg Davydov authored
The DeviceLoginScreenFullScreenMagnifier used to check that user cannot change preference value while it's set by recommended policy. This worked (unreliable, but still) because of PolicyRecommendationRestorer class, which switches preference value back to the recommended policy (under some conditions, because of them it actually worked on real devices, this is test-only issue). This commit turns off PolicyRecommendationRestorer behaviour and makes tests to verify that user has better priority than recommended policy, as it should be. Bug: 1015763 Change-Id: Ia6714fe58cedebb097d5eea24b70c590fbd04912 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1930743Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Commit-Queue: Oleg Davydov <burunduk@chromium.org> Cr-Commit-Position: refs/heads/master@{#720893}
-
Fredrik Söderquist authored
Make FindInstanceTime(...) only handle the "strictly greater than" case and rename it to NextAfter(...), adjusting the implementation accordingly. This can be used for all the callers of FindInstanceTime(...) except ResolveInterval(...). Fold FindInstanceTime(...) with the "greater or equal" case into ResolveInterval(...). Bug: 1029573 Change-Id: Id727514e10586576796ecfacc5a045114217050c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1944473 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:
Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#720892}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/9af0784b509e..bfcc2cabed77 git log 9af0784b509e..bfcc2cabed77 --date=short --first-parent --format='%ad %ae %s' 2019-12-03 rslawik@google.com Merge "Find process sizes during LMK" Created with: gclient setdep -r src/third_party/perfetto@bfcc2cabed77 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: Ic325a9ae7916863fa16e2241ed8630e0722a741c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947761Reviewed-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@{#720891}
-
arthursonzogni authored
This test doesn't pass, because there is a problem with Chrome's implementation (See https://crbug.com/980127) and because there are two problems in the test itself. 1) t_spv.done() is never called. 2) A typing mistake in "tsv_unreached_func" Bug: 980127 Change-Id: I91949f64db3f3d7d3033d559d8e277b2cb980b13 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946482Reviewed-by:
Daniel Vogelheim <vogelheim@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#720890}
-
Friedrich Horschig authored
This CL disables some KioskUpdateTests which are incredibly flaky on debug builds and have been disabled on release builds anyway. In the last 200 builds, 30 failures were due to KioskUpdateTests, so I consider disabling the test on debug in general (since they seem more flaky there). TBR=xiyuan@chromium.org Bug: 949490 Change-Id: I09a31f794339bc55a2acba2b12d01f048454f597 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948426Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#720889}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/db299863e0c5..d405652ad675 Created with: gclient setdep -r src-internal@d405652ad675 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@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.chrome.try:linux-chromeos-chrome Bug: None Tbr: jbudorick@google.com Change-Id: I9dfcc82f1684785ef89289594210ca1fddc8e013 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947760Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#720888}
-
Pawel Baran authored
TabRendererData is moved moving_tab->SetData(std::move(data)); then is used in layout_helper_->SetTabPinned(...data.pinned ?...) Change-Id: I5b9ac8122b942985cb354f90726e78270113e1f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1944676Reviewed-by:
Allen Bauer <kylixrd@chromium.org> Reviewed-by:
Taylor Bergquist <tbergquist@chromium.org> Commit-Queue: Wojciech Dzierżanowski <wdzierzanowski@opera.com> Cr-Commit-Position: refs/heads/master@{#720887}
-
Kalpak Tapas authored
This CL adds support for passing text highlight information from PDFiumEngine to mimehandler process using appropriate AccessibilityHighlightInfo structs. Add GetAccessibilityHighlightInfo() which works similar to GetAccessibilityLinkInfo() called from within GetAccessibilityInfo() in pdf/accessibility.cc. Also includes corresponding unit test. Bug: 1008775 Change-Id: I7916177fef5e864f0e3b4b0a0b8c1693bf09f19c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1871492Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Kalpak Tapas <katapas@microsoft.com> Cr-Commit-Position: refs/heads/master@{#720886}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/47f3eb42641b..3ed4586044f2 git log 47f3eb42641b..3ed4586044f2 --date=short --first-parent --format='%ad %ae %s' 2019-12-02 stevenperron@google.com Folding: perform add and sub on mismatched integer types (#3084) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@3ed4586044f2 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,geofflang@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_angle_vk32_deqp_rel_ng;luci.chromium.try:android_angle_vk32_rel_ng;luci.chromium.try:android_angle_vk64_deqp_rel_ng;luci.chromium.try:android_angle_vk64_rel_ng;luci.chromium.try:linux_angle_deqp_rel_ng;luci.chromium.try:linux-angle-rel;luci.chromium.try:win-angle-rel-32;luci.chromium.try:win-angle-rel-64;luci.chromium.try:win-angle-deqp-rel-32;luci.chromium.try:win-angle-deqp-rel-64 Bug: None Tbr: radial-bots+chrome-roll@google.com,geofflang@google.com Change-Id: I8ceda5cb6c6253d03ce360b5fc18473190483d9f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947757Reviewed-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@{#720885}
-
Koji Ishii authored
The offset in |NGFragmentItem| is relative to the inline formatting context even when they are in inline boxes. When painting self-painting inline boxes, |paint_offset| is relative to the box itself. This patch adjusts it to relative to the containing block. Bug: 982194 Change-Id: Ie9b9233f1f78a4c79fdfb73932298701be2a19c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947608Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#720884}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/75ba8e350e55..1e3659a4633a git log 75ba8e350e55..1e3659a4633a --date=short --first-parent --format='%ad %ae %s' 2019-12-03 szuend@chromium.org Reland "Remove top-level await pre-processing" 2019-12-03 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update Devtools DEPS. 2019-12-02 etiennep@chromium.org Revert "Remove top-level await pre-processing" Created with: gclient setdep -r src/third_party/devtools-frontend/src@1e3659a4633a 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:1021921,chromium:1029863 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I5466dd4d5508c5102bd395e55313b227136bea3e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947755Reviewed-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@{#720883}
-
Abhijeet Kandalkar authored
This CL has below goals, 1. Use To<SVGDefsElement> and DynamicTo<SVGDefsElement> as new downcast helper 2. Use IsA<SVGDefsElement>(element) in place of IsSVGDefsElement(element) 3. Use To<SVGCircleElement> and DynamicTo<SVGCircleElement> as new downcast helper 4. Use IsA<SVGCircleElement>(element) in place of IsSVGCircleElement(element) 5. Use DynamicTo<SVGAElement> as new downcast helper 6. Use IsA<SVGAElement>(element) in place of IsSVGAElement(element) Bug: 891908 Change-Id: If2c1706899083e924a7efc762e403c81a6c6c211 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946193 Commit-Queue: Abhijeet | Igalia <abhijeet@igalia.com> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#720882}
-
Ella Ge authored
Removed unused a few functions and variables in MainThreadEventQueue They were used for recording metrics and metrics were removed a while ago. Change-Id: I724ff6a4b80d34f7a0b9de0db2376f5fa35eb6be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946480Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Commit-Queue: Ella Ge <eirage@chromium.org> Cr-Commit-Position: refs/heads/master@{#720881}
-
Rune Lillesveen authored
TBR=ericwilligers@chromium.org Bug: 1029640, 1029997 Change-Id: Ie1bae0880e4ee1610bc03e88f73237ddc6501a19 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947263Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#720880}
-