- 27 Jul, 2020 40 commits
-
-
Torne (Richard Coles) authored
Apps that target R or later default to setAllowFileAccess(false). Update the WebView UI test code to explicitly enable it before trying to load file:// URLs in the tests, so that it won't break once we build against the R SDK. Bug: 1106471 Change-Id: I863ffefcf2eec947062e2e99f2e40e60ec452fbf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316400Reviewed-by:
Nate Fischer <ntfschr@chromium.org> Commit-Queue: Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#791826}
-
Raymond Toy authored
The audio thread reads the looping parameters (looping, start, and end) possibly many times while rendering the data. The main thread can modify any of these at any time. This is a race. To eliminate the race, modify the setters for looping, loop start, and loop end to add a mutex for the process lock so that the main thread can't modify these until audio thread is finished processing. Bug: 1107500 Change-Id: Ied5cfb35cfb91e7982dc68e694dead09b1c74aa2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2306555 Commit-Queue: Raymond Toy <rtoy@chromium.org> Reviewed-by:
Hongchan Choi <hongchan@chromium.org> Cr-Commit-Position: refs/heads/master@{#791825}
-
Tarun Bansal authored
Also, remove some of the resource loading hints tests that are not needed or are covered by Defer browser tests Change-Id: I9a812a29952ca4c6753627ffedd7b27655e71652 Bug: 1089371,1089208,1089207,1089118,1088701 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316835Reviewed-by:
Sophie Chang <sophiechang@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#791824}
-
Kevin Ellis authored
Change-Id: I99a11b123e46c43fd59ee04b0a8e3c916cdbeae4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317847 Commit-Queue: Kevin Ellis <kevers@chromium.org> Reviewed-by:
Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#791823}
-
Adithya Srinivasan authored
This reverts commit 8b10bc20. Reason for revert: camera.CCAUISanity.fake is failing again: https://ci.chromium.org/p/chromium/builders/ci/chromeos-kevin-rel/27932 Original change's description: > Reland "Do not expose media devices before a successful getUserMedia()" > > This reverts commit 69a60019. > > chrome-prefixed URLs are exempted from the getUserMedia() requirement. > > Original change's description: > > Revert "Do not expose media devices before a successful getUserMedia()" > > > > This reverts commit d64e736a. > > > > Reason for revert: Caused chrome_all_tast_tests / camera.CCAUISanity.fake failures; e.g. https://ci.chromium.org/p/chrome/builders/ci/chromeos-betty-pi-arc-chrome/7680 > > > > Original change's description: > > > Do not expose media devices before a successful getUserMedia() > > > > > > This is a spec-compliance change. > > > > > > This CL also adds some improvements to the blink MediaDevices unit test. > > > > > > Bug: 1101860 > > > Change-Id: If2e7824df3b3dbd3cdb80f4bcf6b5c417d6f0ef2 > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2279808 > > > Commit-Queue: Guido Urdaneta <guidou@chromium.org> > > > Reviewed-by: Harald Alvestrand <hta@chromium.org> > > > Cr-Commit-Position: refs/heads/master@{#786253} > > > > TBR=hta@chromium.org,guidou@chromium.org > > > > Change-Id: I41b676c7d3e437c06d23f2b46270453f1cc81670 > > No-Presubmit: true > > No-Tree-Checks: true > > No-Try: true > > Bug: 1101860 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288282 > > Reviewed-by: Mohsen Izadi <mohsen@chromium.org> > > Commit-Queue: Mohsen Izadi <mohsen@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#786615} > > TBR=mohsen@chromium.org,hta@chromium.org,guidou@chromium.org > > # Not skipping CQ checks because this is a reland. > > Bug: 1101860 > Change-Id: I3fb6d9a620291cda57eb5fe48ce330b45425a5bc > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2289833 > Commit-Queue: Guido Urdaneta <guidou@chromium.org> > Reviewed-by: Guido Urdaneta <guidou@chromium.org> > Reviewed-by: Harald Alvestrand <hta@chromium.org> > Cr-Commit-Position: refs/heads/master@{#791797} TBR=shik@chromium.org,hta@chromium.org,guidou@chromium.org,wtlee@chromium.org Change-Id: I6b18c308de10dc5e5251025fd887892ecfef938b No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1101860 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320081Reviewed-by:
Adithya Srinivasan <adithyas@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#791822}
-
Miguel Casas authored
This CL flips the FeatureFlag to be enabled by default. Bug: 958166 Change-Id: I75edaccca252c3bcb87d8550b6b6350348000022 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317569Reviewed-by:
ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/heads/master@{#791821}
-
Alex Rudenko authored
Previously, `TrackSizesForComputedStyle` was used incorrectly because it didn't account for paddings and borders. This CL changes that to using `ColumnPositions`/`RowPositions` instead. Bug: 1105808 Change-Id: I5bb0994dfb670ae2d43641f8a576a4acf43dd369 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317225 Commit-Queue: Alex Rudenko <alexrudenko@chromium.org> Reviewed-by:
Oriol Brufau <obrufau@igalia.com> Reviewed-by:
Peter Marshall <petermarshall@chromium.org> Reviewed-by:
Brandon Goddard <brgoddar@microsoft.com> Cr-Commit-Position: refs/heads/master@{#791820}
-
Garrett Beaty authored
This reverts commit bb13d209. Reason for revert: Get back to normal Original change's description: > Throttle submits from the CQ. > > There is likely to be a large backlog of CLs to submit that built up > during the network outage. Throttling the submits that land will prevent > large blamelists for CI runs. > > Bug: 1108842 > Change-Id: I117c89082640bb30daf12b2bf5daa2e1b25c323f > No-Try: true > No-Tree-Checks: true > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317760 > Auto-Submit: Garrett Beaty <gbeaty@chromium.org> > Reviewed-by: Erik Staab <estaab@chromium.org> > Reviewed-by: Ben Pastene <bpastene@chromium.org> > Commit-Queue: Erik Staab <estaab@chromium.org> > Cr-Commit-Position: refs/heads/master@{#791308} TBR=estaab@chromium.org,bpastene@chromium.org,gbeaty@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1108842 Change-Id: I17dcb58de486345e71c0551fb9d2501cc259a1dc No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320069 Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Cr-Commit-Position: refs/heads/master@{#791819}
-
Maksim Ivanov authored
Add more browser tests for the CryptohomeKeyDelegate D-Bus service implementation, for various scenarios when of parameters being missing or malformed. Bug: 1033936 Change-Id: Ifa6a2fd337b5ccc1011813814796e7a6b84b2edc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2318065Reviewed-by:
Fabian Sommer <fabiansommer@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#791818}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/ec788be8b577..522fd1e551b2 2020-07-27 fmalita@chromium.org Fix SkottieKit font loading 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 borenet@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: borenet@google.com Change-Id: I6942d4460de029e8a8e34a7d3c7675633d43974c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320186Reviewed-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@{#791817}
-
Roman Sorokin authored
TBR=gambard@chromium.org Bug: 1109202 Change-Id: Ifdf188c159c8ee8a672886b6de3c98c04c3e4a61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320273Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#791816}
-
Oriol Brufau authored
In https://github.com/w3c/csswg-drafts/issues/4206, the CSSWG resolved that the 'text-transform' property should apply to ::marker, and that by default it should be set to 'none' in the UA stylesheet. This patch implements these changes. Note that setting it to 'none' is an observable change in LayoutNG, since it prevents the ::marker from inheriting the 'text-transform' of the list item. This was considered a better behavior by the CSSWG, and it's the behavior that legacy layout already had. LayoutListMarker, used for ::marker with 'content: normal' in legacy layout, doesn't implement the marker text as actual text. Therefore, 'text-transform' has no effect. Bug: 1031667 TEST=external/wpt/css/css-pseudo/marker-default-styles.html TEST=external/wpt/css/css-pseudo/marker-text-transform-default.html TEST=external/wpt/css/css-pseudo/marker-text-transform-uppercase.html TEST=external/wpt/css/css-pseudo/parsing/marker-supported-properties-in-animation.html TEST=external/wpt/css/css-pseudo/parsing/marker-supported-properties.html TEST=http/tests/devtools/elements/styles-2/pseudo-elements.js Change-Id: I2c2b554f521139fabb502856fedc6230aa45c787 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315872Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Oriol Brufau <obrufau@igalia.com> Cr-Commit-Position: refs/heads/master@{#791815}
-
Roman Sorokin [CET] authored
This reverts commit 706a5c51. Reason for revert: Speculative revert for https://ci.chromium.org/p/chromium/builders/ci/Mac10.12%20Tests https://crbug.com/1109738 Original change's description: > [Nearby] Start discovery on discovery page > > This calls the DiscoveryManager interface from JS and listens for > discovered and lost devices to update the device list. The selected > device is then passed through to the confirmation-page. > > Bug: 1103190 > Change-Id: I9296e5449533eb869fd9f20fe442d4ac1bc05a42 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315862 > Commit-Queue: Richard Knoll <knollr@chromium.org> > Reviewed-by: Michael van Ouwerkerk <mvanouwerkerk@chromium.org> > Cr-Commit-Position: refs/heads/master@{#791591} TBR=mvanouwerkerk@chromium.org,knollr@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1103190 Change-Id: I80a3d398897ca3f77b41f0d72012d1039773583c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320272Reviewed-by:
-
Nohemi Fernandez authored
Fixes issue of overlapping buttons in "Turn on Sync?" screen. Bug: 1105853 Change-Id: I876b9e9c5013cd0955eee539b7efdb77f75534c3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315693 Commit-Queue: Nohemi Fernandez <fernandex@chromium.org> Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Cr-Commit-Position: refs/heads/master@{#791813}
-
Anders Hartvoll Ruud authored
Bug: 1108711, 1108686 Change-Id: Ie653dbee0de591f1ea62abc2380db8cd374fea4f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315139 Commit-Queue: Anders Hartvoll Ruud <andruud@chromium.org> Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#791812}
-
Peter Wen authored
Changes to baseline.xml should trigger re-linting. Bug: None Change-Id: I03f8528e0eba540941eff2784533253c7789f9ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2314695 Commit-Queue: Peter Wen <wnwen@chromium.org> Auto-Submit: Peter Wen <wnwen@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#791811}
-
Roman Sorokin authored
TBR=eugenebut@chromium.org No-Try: True Bug: 1109131 Change-Id: I07b8149e0f66af952206de6e0d5756a94df6661e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317963Reviewed-by:
-
Yoshifumi Inoue authored
This patch changes EditingNG runtime enabled feature to "experimental" from "test" to collect real world issue for helping enabling EditingNG. This patch also updates following accessibility test expectations of content_browsertests for differences of AX object tree of list marker between legacy layout and LayoutNG. Legacy layout: LayoutListItem {LI} at (0,0) size 784x20 LayoutListMarker {::marker} at (-1,0) size 7x19: bullet LayoutNG: LayoutNGListItem {LI} at (0,0) size 784x20 LayoutNGInsideListMarker {::marker} at (-1,0) size 7x19 LayoutText (anonymous) at (-1,0) size 7x19 text run at (-1,0) width 7: "\x{2022} " Bug: 707656 Change-Id: I186e5663ad7930e9b41f0dbcb32c0eb9fc8f9f11 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2319947 Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:Kent Tamura <tkent@chromium.org> Auto-Submit: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#791809}
-
Bruce Long authored
The browser language settings page allows a user to add preferred languages, and only the added languages are candidates for toggling on for spellchecking. Several of the accept languages come in both regionally specific form (Brazilian Portuguese, "pt-BR" e.g.) and generic form (Portuguese, "pt" e.g.). However, the current logic in the the SpellcheckService class that maps accept language tags to more complete BCP47 tags specifying Windows spellcheck dictionaries does not map the platform dictionary name to the generic name if an exact match is already found. For example, "pt-BR" gets mapped to its exact match "pt-BR", but not to "pt". Since there is no Hunspell support for generic "pt", you cannot enable generic Portuguese for spellchecking. A more insidious issue is seen with Italian. If you install the Italian (Italy) Windows language pack, you get dictionary support for "it-IT" and "it-CH" (Italian as spoken in Italy and Switzerland respectively). Since there is an exact match with the "it-IT" accept language tag, generic "it" does not get mapped. But since generic "it" does have Hunspell support, the hybrid spellcheck fallback logic will kick in here. A user can enable generic Italian for spellcheck but in fact Hunspell will be used, even with the Italian Windows language pack in place. The fix is to add an entry to the dictionary map for the generic language subtag if a regionally specific platform spellcheck dictionary is available, and pass the generic subtag to the platform API when creating a Windows spellchecker. The platform API is smart enough to recognize the primary language associated with a language pack, so passing the "pt" subtag will in fact create the "pt-BR" spellchecker. Note this fix only works when the Windows platform spellchecking feature is enabled. If using Hunspell exclusively, you still can't toggle generic languages unless they can be mapped to an available Hunspell dictionary Bug: 1105610 Change-Id: Ib995289a55ed8c8c51bf83e3a1ca4118a07814c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2310804Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Commit-Queue: Bruce Long <brlong@microsoft.com> Cr-Commit-Position: refs/heads/master@{#791808}
-
Ben Wagner authored
The new Lato-Medium-Liga font is based on Lato-Medium 3.002 available at https://github.com/latofonts/lato-source/tree/master/fonts/static/TTF It is subset with hb-subset with "fiststfi " and the 'fi' ligatures moved back to 'liga' (version 3.002 moved them to 'dlig'). The crossbar of 't' in the 'st' ligature was also shorted to ensure that it would not cover a normal 't'. The Lato-Medium currently available to the tests is version 2.015 and lacks a mapping from U+fb06 to the 'st' ligature glyph which was only accessible through the 'dlig' feature. Lato-Medium 3.002 added this mapping (required for this new test) but also moved the 'fi' ligatures to 'dlig' and some existing tests depend on the 'fi' ligatures being in 'liga'. This tests https://github.com/w3c/csswg-drafts/pull/3430 Change-Id: Iaf0ff1f99f3ec282bacb0e3cb44ede27d47fac5a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1294874Reviewed-by:
Dominik Röttsches <drott@chromium.org> Commit-Queue: Ben Wagner <bungeman@chromium.org> Cr-Commit-Position: refs/heads/master@{#791807}
-
Nick Harper authored
If the QUIC version uses TLS and the handshake isn't complete, don't try to fill SSLInfo. Bug: 1107327 Change-Id: I2b8d73c3c1185a9dce92ebe0f6abc00bd8b212af Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315435 Commit-Queue: Nick Harper <nharper@chromium.org> Auto-Submit: Nick Harper <nharper@chromium.org> Reviewed-by:
Renjie Tang <renjietang@chromium.org> Cr-Commit-Position: refs/heads/master@{#791806}
-
Roman Sorokin authored
TBR=petermarshall@chromium.org No-Try: True Bug: 1107944 Change-Id: Ib1439afe4f9431ff3855d4db39b7de007a9a5d0d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320173Reviewed-by:
-
Mohamed Heikal authored
This documentation is not very clear and a bit out of date/incorrect. Updated to make it clearer. Change-Id: I125d73884838c388ead20d62de5f0cba093715ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2318426 Auto-Submit: Mohamed Heikal <mheikal@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
-
Sylvain Defresne authored
As https://crrev.com/c/2299189 adds `OS_MAC` & `OS_APPLE`, add a gn variable `is_apple` that is true when `OS_APPLE` is defined. This allow an easy mapping between gn variables and cpp macros. Convert all usages of `is_ios || is_mac` or `is_mac || is_ios` to use the new variable instead. Bug: 1105907 Change-Id: I4cef55496a0ccf0a6162e60409ff3b7996f633dd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315695 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#791803}
-
Adam Ettenberger authored
According to the following spec radio and menuitemradio should expose UIA SelectedItem.IsSelected when aria-checked is set : https://www.w3.org/TR/core-aam-1.1/#mapping_state-property_table This CL ensures that radio / menuitemradio fire appropriate SelectionItem events, and whenever a SelectionItem changes its selection state that we also fire a Property Changed event for SelectionItem.IsSelected. AX-Relnotes: Fire missing events for ARIA radio/menuitemradio when aria-checked changes. Narrator will now announce selected state changes for this scenario. Bug: 1102580 Change-Id: I72f479e25f61b0ddffc28f73d4a9c34e8dc4ecb8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212982Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Reviewed-by:
Kevin Babbitt <kbabbitt@microsoft.com> Commit-Queue: Adam Ettenberger <Adam.Ettenberger@microsoft.com> Cr-Commit-Position: refs/heads/master@{#791802}
-
Caroline Rising authored
Bug: 1107762 Change-Id: I8b5f928ac091811f4b7b6461acfbc1885ffc8093 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2313100Reviewed-by:
Dana Fried <dfried@chromium.org> Commit-Queue: Caroline Rising <corising@chromium.org> Cr-Commit-Position: refs/heads/master@{#791801}
-
Ben Schwartz authored
Bug: 1098883 Change-Id: I52fd87ecd652e7224fd6190f1f58ba36b4f6b148 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2276311Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Natalie Chouinard <chouinard@chromium.org> Commit-Queue: Ben Schwartz <bemasc@chromium.org> Cr-Commit-Position: refs/heads/master@{#791800}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/f4f0170b8694..450afef18d5b 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 rsorokin@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: None Tbr: rsorokin@google.com Change-Id: I6c9a90af068b282d62735d0b05c012e71b112a57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320034Reviewed-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@{#791799}
-
Avi Drissman authored
Bug: none Change-Id: I29a3fb669cbda96ca11c3fd3838347722f1023dd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317993 Auto-Submit: Avi Drissman <avi@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#791798}
-
Guido Urdaneta authored
This reverts commit 69a60019. chrome-prefixed URLs are exempted from the getUserMedia() requirement. Original change's description: > Revert "Do not expose media devices before a successful getUserMedia()" > > This reverts commit d64e736a. > > Reason for revert: Caused chrome_all_tast_tests / camera.CCAUISanity.fake failures; e.g. https://ci.chromium.org/p/chrome/builders/ci/chromeos-betty-pi-arc-chrome/7680 > > Original change's description: > > Do not expose media devices before a successful getUserMedia() > > > > This is a spec-compliance change. > > > > This CL also adds some improvements to the blink MediaDevices unit test. > > > > Bug: 1101860 > > Change-Id: If2e7824df3b3dbd3cdb80f4bcf6b5c417d6f0ef2 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2279808 > > Commit-Queue: Guido Urdaneta <guidou@chromium.org> > > Reviewed-by: Harald Alvestrand <hta@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#786253} > > TBR=hta@chromium.org,guidou@chromium.org > > Change-Id: I41b676c7d3e437c06d23f2b46270453f1cc81670 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 1101860 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288282 > Reviewed-by: Mohsen Izadi <mohsen@chromium.org> > Commit-Queue: Mohsen Izadi <mohsen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#786615} TBR=mohsen@chromium.org,hta@chromium.org,guidou@chromium.org # Not skipping CQ checks because this is a reland. Bug: 1101860 Change-Id: I3fb6d9a620291cda57eb5fe48ce330b45425a5bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2289833 Commit-Queue: Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Harald Alvestrand <hta@chromium.org> Cr-Commit-Position: refs/heads/master@{#791797}
-
Michael Achenbach authored
Fix after https://crrev.com/c/2315691, which references mac_xcode_version without importing gclient_args.gni where it gets defined. It only worked in the Chromium project since the import is in Chromium's build overrides. Bug: 1109624 Change-Id: I575c7b87dbb96c3a79b8243bfe63c33b2427479d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317871Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Reviewed-by:
Hans Wennborg <hans@chromium.org> Reviewed-by:
-
Philip Rogers authored
Compositing SVG exposed a bug where the filter paint property was not used when painting. Before CompositeSVG, this was not needed because filter paint properties were not created for SVG. This will be useful when SVG supports CSS filters (https://crbug.com/109224). Bug: 1106780 Change-Id: I664995c8b81f201e9e30e0a670e9c252829013ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2311114Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#791795}
-
Guido Urdaneta authored
The test was incorrectly removed at https://chromium-review.googlesource.com/c/chromium/src/+/2302193 Bug: 1045846 Change-Id: I8c7f386a245b5a9fc9cc44c8976b220ed5b11b18 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317879 Auto-Submit: Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#791794}
-
Matt Jones authored
This patch reintroduces the visibility flag to the native update prarms in the ToolbarSceneLayer. The reason for this is that the tree becomes stale and keeps some of the layers around (currently no easy alternative to detaching layers). This patch also makes the layer always "visible" according to isSceneOverlayTreeShowing as it would otherwise block the extra frame to make the layer [in]visible. Bug: 1109126 Change-Id: If8aed99d9cd6b7d436b563064be3c33b3deea8b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317291Reviewed-by:
Mei Liang <meiliang@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#791793}
-
David Munro authored
Crostini registers Linux applications it knows about, however many GUI Linux apps don't have .desktop files to register or icons to show but they still have windows that should appear in the shelf. For them we show a default penguin icon as a fallback when we can't find a better icon. This was broken per the linked bug. There are two changes to fix this: 1. We end up in RequestContainerAppIcon, however this method ignores requests for unregistered apps and never calls the callback. Fix that by skipping directly to returning no icon instead of calling RequestContainerAppIcon 2. Pass app_id to LoadIcon in crostini_apps so it can tell if it's being asked for an icon for an unregistered Linux app (as opposed to e.g. an unregistered PluginVM app) and update a check to handle this appropriately. Bug: chromium:1106280 Test: manual - run some Crostini apps and look for penguins Change-Id: Iacf7c889989a4e91db16adf1244f13e56a71e97b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2318891Reviewed-by:
Nancy Wang <nancylingwang@chromium.org> Reviewed-by:
Joel Hockey <joelhockey@chromium.org> Commit-Queue: David Munro <davidmunro@google.com> Cr-Commit-Position: refs/heads/master@{#791792}
-
Viktor Semeniuk authored
This change adds reauthentication mechanism before revealing password on Password Details Screen. Reauth will be requested before showing, editing or copying password. Password editing and copying to be done in the future CLs. Bug: 1075494 Change-Id: Idbebddb9dcdb0928af16e115d69c6fe28860c872 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302753 Commit-Queue: Viktor Semeniuk <vsemeniuk@google.com> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#791791}
-
Maksim Sisov authored
Makes sure initialization of tests is correct and also non-Ozone/X11 specific tests are not run when OzonePlatform feature is used. ----- PS: Please note that this is a temp solution that will help to choose between ozone and non-ozone X11 build. The switch that will be used to choose the path is --enable-features=UseOzonePlatform. Once non-Ozone X11 path is removed (hopefully by Q1 2021 depending on how th finch trial goes), the wrapper will be removed. Please also note that it's impossible to build use_x11 && use_ozone without some hacks in PlatformCursor code. The changes to that are on their way to upstream. ---- Bug: 1085700 Change-Id: If9f7e04b0c39dfe2782622f225de7d1b901575f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315144 Commit-Queue: Maksim Sisov (GMT+3) <msisov@igalia.com> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#791790}
-
Rushan Suleymanov authored
This reverts commit 2c79a900. Reason for revert: tests have stopped to fail and should have low flakiness score now. Original change's description: > Disable MigrationSingleClientTest tests due to flakiness > > Tests disabled on Windows: > - MigrationSingleClientTest.PrefsOnlyTriggerRefresh > - MigrationSingleClientTest.PrefsOnlyModifyBookmark > > Bug: 1094907 > Change-Id: Ifba9e0a15138f4875ed6d2cb1ec1392b29b16c17 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2294859 > Commit-Queue: Rushan Suleymanov <rushans@google.com> > Auto-Submit: Rushan Suleymanov <rushans@google.com> > Reviewed-by: Maksim Moskvitin <mmoskvitin@google.com> > Cr-Commit-Position: refs/heads/master@{#787685} # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1094907 Change-Id: Ic3a2e4e77d49aeaaf3de83aa701f7305b31d76e1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317803Reviewed-by:
Maksim Moskvitin <mmoskvitin@google.com> Commit-Queue: Rushan Suleymanov <rushans@google.com> Cr-Commit-Position: refs/heads/master@{#791789}
-
Jit Yao Yap authored
This CL implements the RemoteApps PublisherBase and RemoteAppsManager which are used to manage Remote Apps. Remote Apps are items which are added to the launcher but are not backed by an app/extension/web app. Instead, the item is managed by an extension via an API. The extension is able to add multiple items and folders to the launcher and observe when the items are launched. The extension can then decide on how to handle the launch event. Currently RemoteApps are only available to Managed Guest Sessions. RemoteAppsManager is the KeyedService which handles the adding and deleting of Remote Apps and folders. RemoteAppsManager also implements RemoteApps::Delegate and is used by RemoteApps to handle the LaunchApp(), GetIcon() and GetMenuModel() methods. RemoteAppsManager is not initialized as a KeyedService so this CL does not change any existing behavior. Bug: 1101208 Change-Id: I664469fa0284c6e65288e59ba7a5d1271b081a04 DD: go/cros-remote-apps Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2299226 Commit-Queue: Jit Yao Yap <jityao@google.com> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#791788}
-
Jan Wilken Dörrie authored
This change adds support for multiple projections to util::ranges by improving the capabilities of the ProjectedBinaryPredicate. The resulting object will now consider several permutations of the provided projections, and choose an appropriate one. This change required the introduction of a few internal helpers. Bug: 1071094 Change-Id: I46b05e1e0ddf0b15b0ed1518fb6de263bb23b8a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2310533 Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#791787}
-
