- 04 Jan, 2019 40 commits
-
-
Arthur Sonzogni authored
URLLoader::FollowRedirect() takes two arguments: |removed_headers| and |modified_headers|. This is partially implemented. This CL is essentially plumbing to improve support for this two arguments. Notable changes: 1) |modified_headers| was exposed from content/public, now |removed_headers| is exposed as well. Tests are added. 2) In particular, NavigationURLLoader and ThrottlingURLLoader now supports |removed_headers|. TBR=mef@chromium.org,scottmg@chromium.org Bug: 911105 Change-Id: I46916d98a3900c331b25bd75eb51c21aef3e1533 Reviewed-on: https://chromium-review.googlesource.com/c/1369856 Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Misha Efimov <mef@chromium.org> Reviewed-by:
Camille Lamy <clamy@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#620011}
-
Emil A Eklund authored
Update NGInlineBoxFragmentPainter::PaintBackgroundBorderShadow to safely handle cases where the FragmentRange is empty. This shouldn't happen but assuming the layout object has a single fragment in those cases is safe. Bug: 916948 Change-Id: I2e57b468a6d5137f843a61dbd9a02de117af3c20 Reviewed-on: https://chromium-review.googlesource.com/c/1396323Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#620010}
-
Sami Kyostila authored
Some message pumps -- most notably the Mac ones -- don't guarantee that returning true from DoDelayedWork causes a future call to DoWork to be scheduled. This patch ensures the optimization to prevent a redundant call to ScheduleWork from a running task isn't applied for delayed tasks. This is a speculative fix to some of the Mac hangs that we are observing with the SequenceManager. Bug: 891670 Change-Id: Ib457a66afc68f9aa41b022a812fc433cd991d124 Reviewed-on: https://chromium-review.googlesource.com/c/1396317 Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Alex Clarke <alexclarke@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#620009}
-
Kelvin Jiang authored
This adds a new method for the activityLogPrivate API: deleteActivitiesByExtension, which takes in an extension id and an optional callback for when the operation has finished. Callback is used to ensure that the activities are removed from the UI side when the delete operation is guaranteed to have successfully completed. Bug: 832354 Change-Id: I8e2df25d39f9879efc6979d949999b7e35be299b Reviewed-on: https://chromium-review.googlesource.com/c/1389165Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Kelvin Jiang <kelvinjiang@chromium.org> Cr-Commit-Position: refs/heads/master@{#620008}
-
John Rummell authored
Clearing browser data can be done with filters that specify origin white lists or black lists, so update the code that clears media licenses to handle a filter if one provided. BUG=901993 TEST=new browser_test passes Change-Id: I413703b456479fc8b85916c96eda6df7cbea2550 Reviewed-on: https://chromium-review.googlesource.com/c/1383308Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Commit-Queue: John Rummell <jrummell@chromium.org> Cr-Commit-Position: refs/heads/master@{#620007}
-
Xida Chen authored
This reverts commit fbafdddd. Reason for revert: causing failure to the unit test here: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20ChromiumOS%20MSan%20Tests/10553 Original change's description: > Added toolbar color processing to BrowserThemePack > > To fix a class of theme-related contrast bugs, BrowserThemePack now does processing for the toolbar to generate COLOR_TOOLBAR by compositing the provided image with the color it will be drawn on top of, and then samples that image into COLOR_TOOLBAR - this solves some issues with contrast checking not working properly if a toolbar image is specified alongside an explicit toolbar color that doesn't represent the color of the image. Additionally, COLOR_FRAME (and its variants) now similarly get replaced with the values computed from their composited images regardless of whether an explicit color was provided (previously, the colors only got set as such if there was no explicit color provided). > > To preserve some existing behavior, any explicit color provided for COLOR_TOOLBAR is copied into COLOR_INFOBAR (new), and COLOR_DETACHED_BOOKMARK_BAR_BACKGROUND (existing), and several UI elements have been updated to draw using those colors instead of COLOR_TOOLBAR. > > Bug: 913227 > Change-Id: Idc4166f3f4e4ee75f178fddc40458b03883e85cc > Reviewed-on: https://chromium-review.googlesource.com/c/1387154 > Commit-Queue: Ryan Meier <rameier@chromium.org> > Reviewed-by: Peter Kasting <pkasting@chromium.org> > Cr-Commit-Position: refs/heads/master@{#619961} TBR=pkasting@chromium.org,rameier@chromium.org Change-Id: I1b5cf5180ad970195a753ef51a9c1bd948bc29b5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 913227 Reviewed-on: https://chromium-review.googlesource.com/c/1396169Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#620006}
-
Avi Drissman authored
This is purely a mechanical change; there is no intended behavior change. BUG=837308 R=rouslan@chromium.org Change-Id: I7d201f56e78f506f9f23da9cdb6944cb788ac62c Reviewed-on: https://chromium-review.googlesource.com/c/1392746Reviewed-by:
Mathieu Perreault <mathp@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#620005}
-
Nasko Oskov authored
As of https://crrev.com/617708 the expectation is that all renderer initiated navigations must specify the origin of the document that started the navigation. This CL fixes PopupBlockerBrowserTest to pass when DCHECK for initiator origin on renderer-initiated navigations is enabled. It also changes the parameter type for CanCreateWindow from GURL to url::Origin. Bug: 882053 Change-Id: Id75c955885bc3c3c4c0e8e5c874ae9a24bcbc8ee Reviewed-on: https://chromium-review.googlesource.com/c/1395066Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Nasko Oskov <nasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#620004}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/be7fc466271b..d4c7458f6427 git log be7fc466271b..d4c7458f6427 --date=short --no-merges --format='%ad %ae %s' 2019-01-04 brianosman@google.com Revert "Add SkColorSpace factory from 3x3 row-major gamut and transfer function" 2019-01-04 benjaminwagner@google.com Reland "Remove win_vulkan_sdk." 2019-01-04 brianosman@google.com Fix bugs and add features to imgcvt 2019-01-04 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2019-01-04 reed@google.com remove blob iterator::applyFontToPaint 2019-01-04 egdaniel@google.com Set skia_use_vulkan on our vulkan bots. 2019-01-04 reed@google.com start to change Viewer's filtering to know about fonts 2019-01-04 kjlubick@google.com more extra semis 2019-01-04 egdaniel@google.com Add support to gpu command buffers to wrap an external command buffer. 2019-01-04 jcgregorio@google.com Fix link in /dev/contrib. 2019-01-04 michaelludwig@google.com Revert "Use specialized quad lists in rectangle ops" 2019-01-04 brianosman@google.com Add SkColorSpace factory from 3x3 row-major gamut and transfer function 2019-01-04 kjlubick@google.com remove more extra semicolons Created with: gclient setdep -r src/third_party/skia@d4c7458f6427 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-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:915025,chromium:917242 TBR=bungeman@chromium.org Change-Id: Ifd835cf3849d79171c460befaa3f09ea6a9c192f Reviewed-on: https://chromium-review.googlesource.com/c/1396189Reviewed-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@{#620003}
-
Erik Luo authored
When highlighting the active property, we used to always hide the overlay highlight, even if no active property was being highlighted. This CL checks that we do not hide unrelated highlights. Screenshot: https://imgur.com/a/W0zFEqq Bug: 918957 Change-Id: I118369e64af4062994221a5a79e8b0e16989b665 Reviewed-on: https://chromium-review.googlesource.com/c/1394986Reviewed-by:
Joel Einbinder <einbinder@chromium.org> Commit-Queue: Erik Luo <luoe@chromium.org> Cr-Commit-Position: refs/heads/master@{#620002}
-
Bruce Dawson authored
This is a reland of dd8c7b26 Original change's description: > ARM64 capable toolchain hash > > This updates Chrome to build with a toolchain based on VS 2017 15.9.3 > and the Windows 10.0.17763.132 SDK, with the ARM64 components included to > support building for ARM64 for Win32. This adds about 400 MB to the package > size, a bit less than a 50% increase. > > Packaging was done on a Windows Server 2016 VM, cleanly created for this > purpose. > > The package was created by downloading VS 2017 Update 9.3, from > https://www.visualstudio.com/vs/, and then running the installer like this: > > $ PATH_TO_INSTALLER.EXE ^ > --add Microsoft.VisualStudio.Workload.NativeDesktop ^ > --add Microsoft.VisualStudio.Component.VC.ATLMFC ^ > --add Microsoft.VisualStudio.Component.VC.Tools.ARM64 ^ > --add Microsoft.VisualStudio.Component.VC.MFC.ARM64 ^ > --includeRecommended --passive > > Then Add or Remove Programs was used to modify the SDK to add the > Debuggers package. > > Then the packaging script was run like this: > > python depot_tools\win_toolchain\package_from_installed.py 2017 -w 10.0.17763.0 > > This was tested on two VMs to ensure that the results are consistent. > > This should not significantly affect the x86 and x64 builds, although > there may have been some STL and CRT changes that could cause changes. > > Bug: 893460 > Change-Id: I33dd5b915ffaf17f306d804f561d26bf82060b03 > Reviewed-on: https://chromium-review.googlesource.com/c/1342814 > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Reviewed-by: Nico Weber <thakis@chromium.org> > Commit-Queue: Bruce Dawson <brucedawson@chromium.org> > Cr-Commit-Position: refs/heads/master@{#616123} Bug: 893460,915046 Change-Id: I1f4f8fb7f37785ba935ef698964fa87f9a91e313 Reviewed-on: https://chromium-review.googlesource.com/c/1377510Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Commit-Queue: Bruce Dawson <brucedawson@chromium.org> Cr-Commit-Position: refs/heads/master@{#620001}
-
A Olsen authored
Use ScopedTestingCrosSettings instead. See bug for more details. Bug: 909635 Change-Id: I37daf492df8fe0be9f2b67ac634a2e68ed886b6f Reviewed-on: https://chromium-review.googlesource.com/c/1394308Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: A Olsen <olsen@chromium.org> Cr-Commit-Position: refs/heads/master@{#620000}
-
Peter Boström authored
This makes ToolbarActionView the same size as other toolbar buttons instead of increasing its internal size with 4dp to accomodate for padding between items. This reduces the BrowserActionsContainer height by 4dp (matching its current visual height) so it should no longer be able to push the toolbar height. It also fixes some polish where the spacing left and right of the browser actions were inadvertedly 10dp instead of 8dp. Bug: chromium:831393, chromium:916241, chromium:916462 Change-Id: I63e5413ce187f173a5e35b40a3ec8f02a68fa09c Reviewed-on: https://chromium-review.googlesource.com/c/1394870Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#619999}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/5d262aa379e9..c25b1e4af00e Created with: gclient setdep -r src-internal@c25b1e4af00e The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=mmoss@chromium.org Change-Id: I3e7198f71f1116aa6401dcfe1b71fc173884b521 Reviewed-on: https://chromium-review.googlesource.com/c/1396188Reviewed-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@{#619998}
-
Dan Harrington authored
Before this change, URLLoaderInterceptor required using a nested RunLoop, which isn't allowed on Android instrumentation tests ( DCHECK(IsTestImplementation()) fails in message_pump_android.cc). Now URLLoaderInterceptor can optionally avoid the RunLoop by using asynchronous initialization. To trigger this new behavior, the caller must pass in a callback to be invoked after the interceptor is installed. Change-Id: Ib76c66d9639d8a48dcf5d75fe1f9a6042e9eeda1 Reviewed-on: https://chromium-review.googlesource.com/c/1385121Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Dan H <harringtond@google.com> Cr-Commit-Position: refs/heads/master@{#619997}
-
Jamie Madill authored
The passthrough command decoder uses robust buffer access behaviour. This makes OOB error checks unimportant because OOB accesses will not cause errors. OOB accesses will also return implementation-dependent values. See the KHR_robust_buffer_access_behavior spec for more information. This will allow ANGLE to selectively disable range checking when robust buffer access behaviour is enabled. This applies to DrawArrays calls only. DrawElements already had the robust buffer access check instated. Bug: angleproject:3000 Change-Id: I3bf5f694d740025a9c40142bb5b0a7801489a70a Reviewed-on: https://chromium-review.googlesource.com/c/1396298Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org> Cr-Commit-Position: refs/heads/master@{#619996}
-
Steven Bennetts authored
This moves //chromeos/cryptohome from //chromeos public_deps to deps so that other components (chrome, ash, extensions, arc) must depend on it explicitly. This will also allow the dependency to be removed from src/ash. For explicit dependency add to components/arc/BUILD.gn: TBR=hidehiko@chromium.org Bug: 918682 Change-Id: I660aedbdb04b4edd9f6308d27135be9d5aab8276 Reviewed-on: https://chromium-review.googlesource.com/c/1394865 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#619995}
-
Jérôme Lebel authored
Setting "Sign in to Chrome" as the AccountSignInItem title. Related to crrev.com/c/1392962 Change-Id: I2b2eef3320dd1455e0b1db6a564699ee9e0596a1 Reviewed-on: https://chromium-review.googlesource.com/c/1396125 Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#619994}
-
Nicolas authored
This class can listen for prefs changes, and pre-compute/convert types for the BrowserSwitcher prefs. It also checks whether the prefs are managed or not, and offers getters to conveniently access the prefs. Change-Id: I64a218f10e30193b8a322c49f1ae08feaf80a128 Reviewed-on: https://chromium-review.googlesource.com/c/1393450 Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Cr-Commit-Position: refs/heads/master@{#619993}
-
Manu Cornet authored
This is the first step of an effort to de-duplicate some code between ShelfButton, AppListButton, BackButton, ShelfControlButton. This makes the class inheritance of ShelfControlButton a lot closer to ShelfButton. None of the functionality of ImageButton is used anyway, so inheriting from plain Button directly makes more sense. Bug: 714781 Change-Id: I20cdc87b39a82ea70813ca7492df79890a255126 Reviewed-on: https://chromium-review.googlesource.com/c/1395565Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Manu Cornet <manucornet@chromium.org> Cr-Commit-Position: refs/heads/master@{#619992}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/d16e9a9b..6bff88d7 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I012c9e2d1fd4805463f13b7b30239491e720f348 Reviewed-on: https://chromium-review.googlesource.com/c/1396228Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#619991}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/43f09d829d87..b428ef224ff5 git log 43f09d829d87..b428ef224ff5 --date=short --no-merges --format='%ad %ae %s' 2019-01-04 fmayer@google.com Bump flush timeout in client and service to 5s. Created with: gclient setdep -r src/third_party/perfetto@b428ef224ff5 The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. BUG=chromium:121352331 TBR=perfetto-bugs@google.com Change-Id: I8f8c97ede843eb0e0f14a5e2f4ac3c98486bb1d6 Reviewed-on: https://chromium-review.googlesource.com/c/1396185Reviewed-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@{#619990}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=kariah@chromium.org Change-Id: I3f2f3a9a1b0e7f593373c0a9c9d8bcfaebcd07da Reviewed-on: https://chromium-review.googlesource.com/c/1396338Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#619989}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/39a48a8a5d7d..6551d805f7ca git log 39a48a8a5d7d..6551d805f7ca --date=short --no-merges --format='%ad %ae %s' 2019-01-04 jclinton@chromium.org Reland "cbuildbot/manifest_version.py: Use Gerrit autosubmit feature throughout" Created with: gclient setdep -r src/third_party/chromite@6551d805f7ca The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=chrome-os-gardeners@chromium.org Change-Id: Id9afa1ade12492313d9d152cec8e0b3cb035c686 Reviewed-on: https://chromium-review.googlesource.com/c/1396183Reviewed-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@{#619988}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/93158ebede89..110bc7918fc2 git log 93158ebede89..110bc7918fc2 --date=short --no-merges --format='%ad %ae %s' 2019-01-04 cwallez@chromium.org Validate EndPass isn't called more than once. 2019-01-04 cwallez@chromium.org dawn_native: Add Instance and Adapters 2019-01-04 cwallez@chromium.org Vulkan: Print the VkResult value on device creation failure. 2019-01-04 cwallez@chromium.org WireServer: check buffer exists before sending the map callback 2019-01-04 cwallez@chromium.org WireCmd: guard against overflows when computing array sizes 2019-01-04 yunchao.he@intel.com Unify the compare function for sampler and depth stencil Created with: gclient setdep -r src/third_party/dawn@110bc7918fc2 The AutoRoll server is located here: https://autoroll.skia.org/r/dawn-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG=chromium:918254,chromium:917555,chromium:918254,chromium:918094,chromium:918348,chromium:918260 TBR=cwallez@chromium.org Change-Id: I4fe10af6d77616459112369b6414dc5b18b3cee4 Reviewed-on: https://chromium-review.googlesource.com/c/1396182Reviewed-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@{#619987}
-
Mason Freed authored
With this CL, there are more early-outs in UpdateCompositorFilterOperationsForBackdropFilter and CreateCompositorFilterOperationsForBackdropFilter in the case where there isn't a backdrop filter present. Also combined BackdropFilterBounds and BackdropFilterReferenceBox, as the latter was not used anywhere else. And added a backdrop-filter test that uses an SVG reference filter. Bug: 497522 Change-Id: If6807f774fa0a552ee57b584839ad960fdc27189 Reviewed-on: https://chromium-review.googlesource.com/c/1393943Reviewed-by:
Stefan Zager <szager@chromium.org> Commit-Queue: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#619986}
-
Miguel Casas authored
As explained in crbug.com/878255#c13, MR should not take into account MediaStreamTrack::IsEnabled() to start recording a given MediaStreamTrack: when they are disabled, they produce either black VideoFrames or all zero Audio -- but that's a legit content and should be recorded. This CL fixes that and adds a LayoutTest. TBR=foolip@chromium.org Bug: 878255 Change-Id: I4be380e838d3287acae7737efb7075e195ac65fd Reviewed-on: https://chromium-review.googlesource.com/c/1393538Reviewed-by:
Miguel Casas <mcasas@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#619985}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/5f1692c60a43..be7fc466271b git log 5f1692c60a43..be7fc466271b --date=short --no-merges --format='%ad %ae %s' 2019-01-04 egdaniel@google.com Restrict ops that can be executed when we have a wrapped vulkan secondary command buffer. Created with: gclient setdep -r src/third_party/skia@be7fc466271b The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-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 TBR=bungeman@chromium.org Change-Id: I06b34b6a2f7fa128910450cc0368df8201d12445 Reviewed-on: https://chromium-review.googlesource.com/c/1396179Reviewed-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@{#619984}
-
Ted Choc authored
This can be handled w/o a server side config now. BUG=918895 Change-Id: I47393da7efd42259f170e19f37a4feee24fc83b0 Reviewed-on: https://chromium-review.googlesource.com/c/1394863Reviewed-by:
Yusuf Ozuysal <yusufo@chromium.org> Commit-Queue: Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#619983}
-
Bo Liu authored
In preparation for new draw functor which does not have synchronous invoke to free render thread resources, change all resource return to posting to UI thread instead. Correctness is relatively simple: if BrowserViewRenderer is destroyed then there is no need to return any resources. Otherwise resources are returned eventually. Detach/shutdown has the same logic. RTM just neends to ensure all resource returns are posted. There are some potential perf downsides: There will be more tasks on the UI thread. This is not really avoidable. Can no longer synchronous return resources before starting a new frame. This may cause higher resource usage. This is technically avoidable but is technically complicated, so go with the simple solution first. Bug: 900965 Change-Id: Id34d05a575799f2cf22166da595ba5e062c0078c Reviewed-on: https://chromium-review.googlesource.com/c/1385854Reviewed-by:
Tobias Sargeant <tobiasjs@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#619982}
-
Dominik Röttsches authored
As this helper function will be used by font_cache_skia_win.cc as well, move it out to FontCache and share the implementation between backends. Bug: 889864 Change-Id: Ic0d6c3eec1cd7689af7f2e394b9f83c8487dd22a Reviewed-on: https://chromium-review.googlesource.com/c/1396020 Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#619981}
-
Antoine Labour authored
PPB_Compositor is going away, so replace with PPB_Graphics3D. Also fix logic related to submitting frames while callbacks are pending (might be related to test flakiness). Bug: 918951 Change-Id: I232c87525f39e268c6fd26072cb465c07fc4256b Reviewed-on: https://chromium-review.googlesource.com/c/1395144Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#619980}
-
Steve Kobes authored
Also use std::unordered_map instead of std::map in BasicIntervals, which is much faster. Bug: 581518 Change-Id: If36a6a70bdf5a4d02d87c66a2ee8264441602388 Reviewed-on: https://chromium-review.googlesource.com/c/1394402Reviewed-by:
Timothy Dresser <tdresser@chromium.org> Commit-Queue: Steve Kobes <skobes@chromium.org> Cr-Commit-Position: refs/heads/master@{#619979}
-
Eric Karl authored
GL uses triple buffering, but Vk was set to use double in most cases. Bump Vk to triple buffering for now. Change-Id: I8e68106b5f0306eb6867382d58d1195341d62a07 Reviewed-on: https://chromium-review.googlesource.com/c/1388223Reviewed-by:
Sunny Sachanandani <sunnyps@chromium.org> Commit-Queue: Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#619978}
-
Xida Chen authored
leak TBR=maxmorin@chromium.org NOTRY=true Bug: 919117 Change-Id: I74df3710087857eaab2a44e8d1751b4bc7df4a19 Reviewed-on: https://chromium-review.googlesource.com/c/1396235Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#619977}
-
Steven Bennetts authored
This forces explicit dependencies for code relying on //chromeos/login. Only code that is part of the Chrome process should have these deps. For minor BUILD.gn changes: TBR=rdevlin.cronin@chromium.org,hidehiko@chromium.org Bug: 918682 Change-Id: Ic443725f1f6763e36a7214f569542822db94482c Reviewed-on: https://chromium-review.googlesource.com/c/1394800 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#619976}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/936a99460834..f3d4ab343082 git log 936a99460834..f3d4ab343082 --date=short --no-merges --format='%ad %ae %s' 2019-01-04 recipe-roller@chromium.org Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@f3d4ab343082 The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=agable@chromium.org Change-Id: I5972a30c28af89df8c6b29da5a70ce002ef04746 Reviewed-on: https://chromium-review.googlesource.com/c/1396186Reviewed-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@{#619975}
-
Roger McFarlane authored
This CL annotates as obsolete and removes the code which generates the following histograms: Autofill.Unknown.HttpResponseOrErrorCode Autofill.Unknown.RequestDuration Autofill.Unknown.BackoffDelay Autofill.Unknown.FailingPayloadSize Samples for these histograms have never been generated in the wild. Bug: 915891 Change-Id: Ic74997a0326c22dee3dc95267c3157c4d646573f Reviewed-on: https://chromium-review.googlesource.com/c/1396233Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Roger McFarlane <rogerm@chromium.org> Cr-Commit-Position: refs/heads/master@{#619974}
-
Jungshik Shin authored
It has one change to update the IANA timezone database to 2018i. See https://chromium-review.googlesource.com/c/1395498 for details. TBR=gsathya@chromium.org Bug: 473288 Test: See the CL description at https://crrev.com/c/1395498 Change-Id: Idefb26be5852a4839a0a6d8cf062b09815b626a4 Reviewed-on: https://chromium-review.googlesource.com/c/1396231Reviewed-by:
Jungshik Shin <jshin@chromium.org> Commit-Queue: Jungshik Shin <jshin@chromium.org> Cr-Commit-Position: refs/heads/master@{#619973}
-
edchin authored
This CL works towards reducing duplicate code in snapshot_generator.mm. An improvement made in an earlier CL was only made to web view snapshots and not to native views. This CL fixes native view snapshots. Bug: 918032 Change-Id: I37d0a9bdead31ccce8e467846c2927105c0a76c7 Reviewed-on: https://chromium-review.googlesource.com/c/1395878Reviewed-by:
edchin <edchin@chromium.org> Reviewed-by:
Justin Cohen <justincohen@chromium.org> Commit-Queue: edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#619972}
-