- 19 Jun, 2020 40 commits
-
-
Devlin Cronin authored
set_icon.js checked the `self` variable to determine whether the script was running in a ServiceWorker (such as a ServiceWorker-backed extension). However, this is unsafe because, unlike `window`, `self` can be overwritten by other script. If an extension specified a value for `self`, that would be used instead - and would subsequently crash if the value didn't support the `in` operator (i.e., wasn't an Object). When this happened, the custom bindings would fail to register any hooks. This, then, would result in API calls to browserAction.setIcon() in passing the arguments directly to the browser-side handling. When this happened, the browser would classify it as a bad message (because it didn't undergo the expected argument modifications), and would kill the renderer. Fix this by adding a native binding to return whether the script is running in a ServiceWorker context. Add a regression test. Bug: 1087948 Change-Id: I1bb593af735dec6d7a2a25050e62677f23c23dce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253284Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Reviewed-by:
David Bertoni <dbertoni@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#780315}
-
Andrew Grieve authored
This reverts commit 44a03014. Reason for reland: Wasn't the culprit (see bug) Original change's description: > Revert "Android: Define paths to expectation files in leaf targets" > > This reverts commit 4a914f21. > > Reason for revert: Build failure in https://ci.chromium.org/p/chrome/builders/ci/arm64-unpublished-builder and https://ci.chromium.org/p/chrome/builders/official/android-arm-official-next > > Possible reason might be renaming //chrome/android:monochrome_public_bundle__base_libs_and_assets_expectations and //chrome/android:trichrome_chrome_bundle__base_libs_and_assets_expectations > > Original change's description: > > Android: Define paths to expectation files in leaf targets > > > > This is easier to follow then deriving it in the templates. > > > > Also changes webview expectations apk->bundle > > > > Bug: 1095654 > > Change-Id: I5952ee9a660e4f32b22ef4100fea2f231d9eb96d > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2245880 > > Commit-Queue: Andrew Grieve <agrieve@chromium.org> > > Reviewed-by: Richard Coles <torne@chromium.org> > > Reviewed-by: Mohamed Heikal <mheikal@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#780157} > > TBR=torne@chromium.org,agrieve@chromium.org,mheikal@chromium.org > > Change-Id: I8ef314098ae3949d21fc885b885b467314f6d695 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 1096987 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252344 > Reviewed-by: Tanmoy Mollik <triploblastic@chromium.org> > Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org> > Cr-Commit-Position: refs/heads/master@{#780271} TBR=torne@chromium.org,agrieve@chromium.org,mheikal@chromium.org,triploblastic@chromium.org No-Presubmit: true Bug: 1096987 Change-Id: I67937a8b09d9ecca0be3356ae17fc517c8be5ce0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2254438Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#780314}
-
Sahel Sharify authored
This cl contains the following changes: 1-PaymentRequestUpdateDetails can have null shippingOptions/modifiers The cl modifies the TypeConverter to check for null arrays before passing them to Arrays.asList 2-changePaymentMethod/changeShipping[Address|Option] from PaymentDetailsUpdateService should get the binder's calling uid before posting the task to the browser's UI thread. Bug: 1026667 Change-Id: I94aaf4caa0d3b08db89993b0450cd3baf7b77a60 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252199Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Commit-Queue: Sahel Sharify <sahel@chromium.org> Cr-Commit-Position: refs/heads/master@{#780313}
-
Hans Wennborg authored
The ASan runtime started calling it after https://github.com/llvm/llvm-project/commit/6b4aeec94afc3626c19ed93dea5d158fbea29732 Bug: 1097067 Change-Id: Ifb37700e5385158cf957c987018f1a9561cc0cef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2254318 Auto-Submit: Hans Wennborg <hans@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#780312}
-
Thomas Tellier authored
This CL reintroduces the submit button on the login / lock screen password textfield and disable the login display password button feature by default, by putting it under a feature flag. Bug: 1090319 Change-Id: I521a19262726f7c0566d824b0efe5aa1c9abb7a1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2247826 Commit-Queue: Thomas Tellier <tellier@google.com> Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#780311}
-
chromium-autoroll authored
Roll Chrome Win64 PGO profile from chrome-win64-master-1592560372-5e1e91918016f52d8aaeea816d2c59deb0aa5d08.profdata to chrome-win64-master-1592568001-645f451bc013d681af722dc2b85e4992534cb0fa.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win64-chromium Please CC jeffyoon@google.com,liaoyuke@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: chrome/try:win64-chrome Tbr: jeffyoon@google.com,liaoyuke@google.com Change-Id: I599d140a53e9aead7b21b3b47fdda93a958f88e1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253891Reviewed-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@{#780310}
-
Andy Paicu authored
TBR=garrick@chromium.org Bug: 1097021 Change-Id: I88623210e697f595ee03e1226abaac743d6b971e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2254139Reviewed-by:
Andy Paicu <andypaicu@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Andy Paicu <andypaicu@chromium.org> Cr-Commit-Position: refs/heads/master@{#780309}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/src/third_party/freetype2.git/+log/62fea391fa99..6e722f2eaa58 2020-06-19 apodtele@gmail.com [base] Rework d1180b5f9598 until further notice. 2020-06-19 sebras@gmail.com [cff, cid] Fix segfaults in case of error (#58621). 2020-06-19 wl@gnu.org [base] Fix UBSAN error. 2020-06-19 wl@gnu.org [woff2] Fix segfault. 2020-06-19 svenevs.dev@gmail.com [sfnt] Support Intel compilers. 2020-06-19 sebras@gmail.com [base] Fix memory leak (#58611). 2020-06-19 sebras@gmail.com [cff] Fix memory leak (#58610). 2020-06-19 wl@gnu.org * src/cff/cffload.c (cff_index_get_pointers): Rename `t' to `tbl'. 2020-06-19 sebras@gmail.com [cff] Free table upon error allocating other data (#58609). 2020-06-17 wl@gnu.org woff2tags.h: Fix copyright year. 2020-06-13 wl@gnu.org Remove redundant inclusion of `ft2build.h'. 2020-06-08 david@freetype.org Make macros for header file names optional. 2020-06-02 wl@gnu.org * src/sfnt/ttcmap.c (tt_face_build_cmaps): Trace number of cmaps. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/freetype-chromium Please CC bungeman@google.com,drott@google.com,thestig@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:linux_chromium_msan_rel_ng;luci.chromium.try:linux-blink-rel;luci.chromium.try:mac10.12-blink-rel;luci.chromium.try:mac10.13_retina-blink-rel;luci.chromium.try:win10-blink-rel;luci.chromium.try:win7-blink-rel Tbr: bungeman@google.com,drott@google.com,thestig@google.com Change-Id: Ida9cedae09810da765b1d8fd71c567c390bb57b7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253887Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Reviewed-by:
Ben Wagner <bungeman@chromium.org> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: Ben Wagner <bungeman@chromium.org> Cr-Commit-Position: refs/heads/master@{#780308}
-
Clemens Arbesser authored
Note that users may enable/disable a11y at any time, which is why the service has to update the state correspondingly. Bug: b/158989900 Change-Id: Ie4e58f3b8ece6df2c176c1b63820dd2a143789f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2245702Reviewed-by:
Luca Hunkeler <hluca@google.com> Reviewed-by:
Mathias Carlen <mcarlen@chromium.org> Commit-Queue: Clemens Arbesser <arbesser@google.com> Cr-Commit-Position: refs/heads/master@{#780307}
-
Majid Valipour authored
Changes: - Fix regex to accept very short otp code: #(.[]+) -> #([]+) - Add tests to ensure we reject forbidden host characters - Add additional tests around number of spaces expected Followup patch will allow host to accept unicode. Change-Id: I7cbeabe31edd9c9ead7ae78425c8760f9c78ce9c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2248071 Commit-Queue: Majid Valipour <majidvp@chromium.org> Reviewed-by:
Sam Goto <goto@chromium.org> Cr-Commit-Position: refs/heads/master@{#780306}
-
Oriol Brufau authored
::marker pseudo-elements are currently behind the CSSMarkerPseudoElement runtime flag. I plan to ship them soon, but without support for nested pseudo-elements (::before::marker and ::after::marker), since that would require more work in the style resolver. That's what Firefox did too. So this patch hides them behind CSSMarkerNestedPseudoElement, which implies CSSMarkerPseudoElement. In order to make it easier to check whether a pseudo-element should be exposed (e.g. in devtools or getComputedStyle), this patch adds a static PseudoElement::IsWebExposed method which takes care of doing the proper checks. Right now it's just for ::marker, but it can also be useful for future tree-abiding pseudo-elements like ::file-selector-button. Bug: 457718 Change-Id: Idf764855cca62e0776463346af04e4f2bcc53040 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2250103 Commit-Queue: Oriol Brufau <obrufau@igalia.com> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#780305}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/d64d998b9f1f..b93c9d1a5342 2020-06-19 lalitm@google.com Merge "ui: revamp quantization for cpu and userspace slices" 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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: I1a514bab55739facd7e84142650f36e459a820ad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253889Reviewed-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@{#780304}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1592560372-e09cb0657078c6c75bd738f91c59baf5eafe0404.profdata to chrome-win32-master-1592568001-0b6b26a9a978d83410a95ed47a0a1a3dcdc59066.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC jeffyoon@google.com,liaoyuke@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: chrome/try:win-chrome Tbr: jeffyoon@google.com,liaoyuke@google.com Change-Id: Ib70cf954f0c3ba7f90459af0e1a0e39a15a19790 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253888Reviewed-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@{#780303}
-
Xi Han authored
Bug: 1097069 Change-Id: I5becd0a8850c8eff6dcbb3bc5e55af656dac35fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2254060Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Commit-Queue: Xi Han <hanxi@chromium.org> Cr-Commit-Position: refs/heads/master@{#780302}
-
Sigurdur Asgeirsson authored
There's no need to represent this as a map on the wire and it creates complications in the mojo bindings for upcoming CLs. Bug: 1080672 Change-Id: I20cfe281d5dbb752bd5b376e45f6d1e1aff98852 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2250790Reviewed-by:
Joe Mason <joenotcharles@chromium.org> Reviewed-by:
Will Harris <wfh@chromium.org> Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#780301}
-
Sigurdur Asgeirsson authored
Bug: 1096543 Change-Id: I87a723e23fe4454eb54704f3bd6b35f5085c63d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252692Reviewed-by:
Joe Mason <joenotcharles@chromium.org> Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#780300}
-
Miguel Casas authored
Hw VP8 Encoding on Intel Gen9.5 devices is disabled for Celeron/Pentium SKUs but sadly it seems that the same GPU hang is observed in Core devices (b/139695266, b/158696760). This CL extends the current blacklisting to all devices with a Gen9.5 GPU, so that said hang can be _temporarily_ avoided. It also refactors a bit the code to identify if a given SoC has a GPU of either Gen9 or Gen9.5. Bug: b:158655609, b:139695266 Change-Id: Id4ebf705e2949fe0715480be0428215d9eae4c75 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253158Reviewed-by:
Andres Calderon Jaramillo <andrescj@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#780299}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/889563cbeb36..75c9c5bd26ae 2020-06-19 johnstiles@google.com Add unit tests for untested SkString APIs. 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 djsollen@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: chromium:1092743 Tbr: djsollen@google.com Change-Id: I21bd96caf32f48344ca4fa4f9fc272f251e99701 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253886Reviewed-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@{#780298}
-
Victor Hugo Vianna Silva authored
This is a reland of a4a3a1a6 Reason for reland: The upstream CL had missing .js extensions in some imports and this caused failures when the new dialog started being used. We also use this CL to add flush() calls after click() calls that make synchronous checks in passwords_section_test, similar to crrev.com/c/2252138. TBR=aee@chromium.org Original change's description: > [b4p/Settings] Display removal dialog > > This CL starts displaying the dialog from crrev.com/c/2248180. Moreover, > RemovePasswordBehavior is deprecated. The more elaborate logic of > removing different copies of a password can live directly in the dialog, > while PasswordsListHandler can use simple logic for the case when a > single copy must be removed. In particular, it felt strange that > PasswordListItem has this Polymer behavior but never used any of its > functionalities (it was PasswordsListHandler that used it instead). > > We also use the occasion to fix an aspect of crrev.com/c/2235850. If > the entry is moved, there's no element to reset the focus to, so the > focusWithoutInk() call is removed. The same is done for removal dialog. > > Bug: 1049141 > Change-Id: I06fe35b8bd14ee27873b6a56849e755c02cbc1bb > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2246157 > Commit-Queue: Victor Vianna <victorvianna@google.com> > Reviewed-by: Esmael Elmoslimany <aee@chromium.org> > Reviewed-by: Friedrich [CET] <fhorschig@chromium.org> > Cr-Commit-Position: refs/heads/master@{#779757} Change-Id: I9ef5016f8f183b36002801663bc91c0d961b7d58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253743 Commit-Queue: Victor Vianna <victorvianna@google.com> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#780297}
-
Jan Krcal authored
This CL removes behavior of profile menu in tests that is not checked by the click tests (i.e. the various actions that happen after clicking the menu items). This CL is a speculative fix for high flakiness of ProfileMenuClickTests. The flakiness has multiple root causes, none of which seems specific to profile menu / the test code. The hope is based on the fact that many of the actions in the user menu are expensive (loading guest mode and opening another browser) and thus increase the risk of flakiness. Bug: 1021930 Change-Id: I0f24a1f18ce8d3029daef44e2bc96b95eef81ac9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2247642 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#780296}
-
Kramer Ge authored
SkiaRenderer SkiaOutputDeviceBufferQueue inserts a GpuFence when scheduling primary plane as an overlay. This fence is inserted after the gl commands for compositing. GbmSurfacelessWayland should use this fence when available. Inserting another fence is necessary. Change-Id: I5ec29019bd8917bdd3ea6e3f10e2d8f1327144f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252467Reviewed-by:
Maksim Sisov <msisov@igalia.com> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Commit-Queue: Kramer Ge <fangzhoug@chromium.org> Commit-Queue: Robert Kroeger <rjkroege@chromium.org> Auto-Submit: Kramer Ge <fangzhoug@chromium.org> Cr-Commit-Position: refs/heads/master@{#780295}
-
Andy Paicu authored
TBR=aee@chromium.org Bug: 1097035 Change-Id: Icf4bb82a640a86f7bfd3cfd1b38512ec791ce8b5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253751Reviewed-by:
Andy Paicu <andypaicu@chromium.org> Commit-Queue: Andy Paicu <andypaicu@chromium.org> Cr-Commit-Position: refs/heads/master@{#780294}
-
Maciej Malinowski authored
GraphProcessor and GlobalDumpGraph will be moved from Chromium to the Perfetto repository. This patch adds some scary notes at the top of the header files to inform that those files will be removed from the Chromium repository. See crbug.com/1095982 for more details. Bug: 1095982 Change-Id: Id4bf3b4d0f7577fe60f9a65089969f639d3e70bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252044Reviewed-by:
Primiano Tucci <primiano@chromium.org> Commit-Queue: Primiano Tucci <primiano@chromium.org> Cr-Commit-Position: refs/heads/master@{#780293}
-
Ricardo Quesada authored
This CL adds timestamps in GamepadDelegate, including a mock test. These timestamps are needed in Wayland-server. Also, this CL replaces the NowInMilliseconds() that is present in wayland-server with the newly passed timestamp. Additionally, this CL adds some missing headers reported by "git cl lint" BUG=1096608 TEST=exo_unittests --gtest_filter=GamingSeatTest.* Change-Id: I41951926b14d6c400902ec77413089c577f23af7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252946 Commit-Queue: Ricardo Quesada <ricardoq@chromium.org> Auto-Submit: Ricardo Quesada <ricardoq@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Cr-Commit-Position: refs/heads/master@{#780292}
-
Stepan Khapugin authored
1. Fix a bug when a blocking UI will show multiple times, but only dismiss once, therefore leaving the scene blocked. 2. Fix a bug when the active scenes are not immediately obscured by the overlay. Ideally we might not want to show this overlay unless the user tries to interact with the scene, and initially I hoped to make it so by observing when the scene becomes active; turns out multiple scenes can be active at once. We might want to revisit it later. Bug: 1096062, 1092824, Change-Id: I978aa855d141e9ba5aa34f7b034a5f284a82ff25 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2249959 Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Auto-Submit: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#780291}
-
Avi Drissman authored
If a WebContents performs a UI-sensitive action (such as showing a dialog or popup), we want to drop fullscreen from all related WebContentses, and to prevent those WebContentses from gaining fullscreen until the UI-sensitive situation is over. Currently the search for related WebContentses is achieved by walking up the opener and outer chains, but that misses related WebContentses that are down those chains. These are one-directional chains that aren't easily walked in the other direction. This is fixed with two changes. First, we keep a list of WebContentses that are in fullscreen, which can then be searched to determine if they are down the chain from the affected WebContents. Second, when a request comes in to go fullscreen, we not only check if the WebContents is prohibited from entering fullscreen, but we now also check if one of the WebContentses up the chain is prohibited. Bug: 1090835 Change-Id: I031e2e0a9ff79b387543a22ec3d643ab468d4d29 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2249090 Commit-Queue: Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Auto-Submit: Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#780290}
-
Andy Paicu authored
TBR=dmurph@chromium.org Bug: 1097050 Change-Id: Ia5c9c5fd02f813b1c01e01a0b05009dd5f969266 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253853Reviewed-by:
Andy Paicu <andypaicu@chromium.org> Commit-Queue: Andy Paicu <andypaicu@chromium.org> Cr-Commit-Position: refs/heads/master@{#780289}
-
Andy Paicu authored
TBR=arthursonzogni@chromium.org Bug: 1097060 Change-Id: Ic873cfd8030d8ef0b25604b638edd895355353be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252808Reviewed-by:
Andy Paicu <andypaicu@chromium.org> Commit-Queue: Andy Paicu <andypaicu@chromium.org> Cr-Commit-Position: refs/heads/master@{#780288}
-
Alexander Surkov authored
Bug: 1096512 Change-Id: I15c797578815ee2d706cf2311cdb2ec722fcb67e AX-Relnotes: n/a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252602 Commit-Queue: Alexander Surkov <asurkov@igalia.com> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#780287}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/3340bd2d130c..23c845754ce8 2020-06-19 wolfi@chromium.org [Issues] Do not show issues banner in console again after interaction 2020-06-19 mathias@chromium.org Fix stylelint-config-standard issues in animation/animationTimeline.css 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/+doc/master/autoroll/README.md Bug: chromium:1083142 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I27fd23df348679619d2dfdd7e2a2e595bdcf8eca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253883Reviewed-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@{#780286}
-
Denis Kuznetsov authored
Bug: 981472 Change-Id: If149ecb5692c7f2e468cf50f91f85552272f1798 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2238835 Commit-Queue: Denis Kuznetsov [CET] <antrim@chromium.org> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#780285}
-
Alice Wang authored
This CL adjusts the peek height and some spacing in the account picker bottom sheet UI used in web sign-in flow. Screenshot: https://crbug.com/1083773#c5 Bug: 1083773 Change-Id: I1d6f19ad07e937174b45200faba598ae91a0d2e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2250045 Commit-Queue: Alice Wang <aliceywang@chromium.org> Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#780284}
-
Ramin Halavati authored
Static |Browser::Create| function calls |CanCreateBrowserForProfile| to make sure the passed profile can be used to create browser. Different items in this function are individually checked again in browser constructor to return proper crash message if constructor is called directly with a wrong profile, but |AllowsBrowserWindows| was missing before and is now added. Bug: 1033903 Change-Id: Iae710ed21243fddea3b54426eccae0077aba56c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1984317 Commit-Queue: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#780283}
-
chromium-autoroll authored
Roll Chrome Win64 PGO profile from chrome-win64-master-1592557014-0fc653e79e8d54496b6e208718c321ae21bb710d.profdata to chrome-win64-master-1592560372-5e1e91918016f52d8aaeea816d2c59deb0aa5d08.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win64-chromium Please CC jeffyoon@google.com,liaoyuke@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: chrome/try:win64-chrome Tbr: jeffyoon@google.com,liaoyuke@google.com Change-Id: Ifd4582ce389f98fb677cf118dd7c679dd1a25d5f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253882Reviewed-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@{#780282}
-
Ioana Pandele authored
Variations of the test are flaking often on Mac10.15 TBR=timvolodine@chromium.org Bug: 1096054 Change-Id: Ib3258a6f78bfa4e86d97089be180517d50ca8e2a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2249782Reviewed-by:
Ioana Pandele <ioanap@chromium.org> Commit-Queue: Ioana Pandele <ioanap@chromium.org> Cr-Commit-Position: refs/heads/master@{#780281}
-
chromium-autoroll authored
Roll Chrome Mac PGO profile from chrome-mac-master-1592524547-91f52ddd745cbca9c170dfb725dd8a14e0e7d1c6.profdata to chrome-mac-master-1592568001-2ce2e0984e1a1c64d6f424e415ec04a2dca940b3.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-mac-chromium Please CC jeffyoon@google.com,liaoyuke@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: chrome/try:mac-chrome Tbr: jeffyoon@google.com,liaoyuke@google.com Change-Id: Icd5dd5fc4db438cea42527c42beb76f9b057a274 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253885Reviewed-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@{#780280}
-
Internal Frameworks Autoroller authored
TBR=bling-team@google.com Change-Id: I5f57419a90b8973ea016a872ba9c9a75773eca51 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253155Reviewed-by:
Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#780279}
-
Roman Sorokin authored
Eventually this class should go away. This CL removes obsolete parts * Gaia screen - flow moved to the Views implementation * RemoveUserDelegate - ditto Bug: 1079199, 1067260 Change-Id: I231154e31b49eb5b645cd01b6e4605edb75f05c7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253900 Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Commit-Queue: Denis Kuznetsov [CET] <antrim@chromium.org> Auto-Submit: Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Cr-Commit-Position: refs/heads/master@{#780278}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1592528104-1ddf0a60ec7fe7a5cf7de6fddb41394c2841acdd.profdata to chrome-win32-master-1592560372-e09cb0657078c6c75bd738f91c59baf5eafe0404.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC jeffyoon@google.com,liaoyuke@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: chrome/try:win-chrome Tbr: jeffyoon@google.com,liaoyuke@google.com Change-Id: Icec68f40a757edb8ffdefd9bd54fdaf82d8502c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253880Reviewed-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@{#780277}
-
Etienne Pierre-Doray authored
While keeping FixedMaxBestEffortTasks enabled. MayBlockWithoutDelay is suspected to cause too many thread creation in blink_perf which tracing can't handle. Bug: 1095596 Change-Id: Ic8dd5a12cb729b1c4e5a5ce6bc3087eb81cd050a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2248726Reviewed-by:
Brian White <bcwhite@chromium.org> Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org> Cr-Commit-Position: refs/heads/master@{#780276}
-