- 24 Sep, 2019 40 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/e18bf0acc0be..410370705505 git log e18bf0acc0be..410370705505 --date=short --no-merges --format='%ad %ae %s' 2019-09-24 dberris@chromium.org Pinpoint: Leverage cache for high-traffic entities Created with: gclient setdep -r src/third_party/catapult@410370705505 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC simonhatch@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=simonhatch@google.com Bug: chromium:1006888 Change-Id: I3c819204f425f0004743deacdfe3c0edd48d0c00 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820557Reviewed-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@{#699240}
-
Xiaocheng Hu authored
This patch implements CSS clamp() as a combination of min() and max() according to how it's defined: https://www.w3.org/TR/css-values-4/#calc-notation Note: clamp() in 'sizes' attribute uses a different independent code path, which will be covered by a separate CL. Bug: 825895 Change-Id: Ifd3cdaf1f3e31e10594d95158c2c730f34689924 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1815850Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#699239}
-
Mark Cogan authored
Per comments on the CL to change the ownership of WebStateLists (crrev.com/c/1796358), there are a couple of nuances to behavior in TabModel that aren't currently covered by unit tests. First, multiple calls to -browserStateDestroyed must be safe. Since possible failure modes are DCHECKS or outright segfaults, the test just creates and populates a tab model, then calls -browserStateDestroyed twice. Second, empty sessions should be saved (meaning that when the user has no open tabs, restoring the session should not show any of the previously open tabs). The test here is also straightforward: force-save a non-empty session, close all of the tabs, force-save again, restore, and verify that there are no open tabs. Change-Id: I3b40d8fc6f60ca51bfaebeb704324a04d60bd7d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819239Reviewed-by:
Mohammad Refaat <mrefaat@chromium.org> Commit-Queue: Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#699238}
-
Mihai Sardarescu authored
This reverts commit 71e64b81. Reason for revert: This CL is only needed to be tested on canary and then merged to M77. CL https://chromium-review.googlesource.com/c/chromium/src/+/1819247 is the fix intended for M78+. Original change's description: > [iOS] Disable signed-in account view controller > > Disable the signed-in account view controller dialog. > This patch is to disable it for M-77 and M-78. The right fix will > come later with crbug.com/1006717 > > Bug: 1006744 > Change-Id: I537969bb221e165a7a0ca66d4324f8d87580916b > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819304 > Commit-Queue: Jérôme Lebel <jlebel@chromium.org> > Reviewed-by: Mihai Sardarescu <msarda@chromium.org> > Cr-Commit-Position: refs/heads/master@{#698946} TBR=msarda@chromium.org,justincohen@chromium.org,jlebel@chromium.org Change-Id: Ie6e9b97321b54d00b8ef3c2e600934e365fd9dc4 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1006744 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1818528Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#699237}
-
Julie Jeongeun Kim authored
This CL converts ThreatReporterPtr and ThreatReporterRequest to new Mojo types using Remote, PendingRemote, PendingReceiver, Receiver, and ReceiverSet. Bug: 955171 Change-Id: I2bc837a87621763a7ee6604265296e63c0eef339 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816200Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Daniel Rubery <drubery@chromium.org> Commit-Queue: Julie Kim <jkim@igalia.com> Cr-Commit-Position: refs/heads/master@{#699236}
-
Alan Cutter authored
This test is flakily failing on Windows bots. https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=BookmarkBarNavigationTest.SecFetchSiteNoneFromNonEmptyTab TBR=mkwst@chromium.org Bug: 1006033 Change-Id: I72fa0c6baf009873850b7043d36d009bb6020e03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1818014Reviewed-by:
Alan Cutter <alancutter@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#699235}
-
Robert Liao authored
BUG=1005568 Change-Id: I36dd422d65620ac945bd139b41afd274f2837f0f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820045 Auto-Submit: Robert Liao <robliao@chromium.org> Commit-Queue: Greg Thompson <grt@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#699234}
-
Robert Liao authored
BUG=1005568 Change-Id: Ibbf77ef3de28e836d85afd2f2a9a1fa87cce1b13 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820000 Commit-Queue: Greg Thompson <grt@chromium.org> Auto-Submit: Robert Liao <robliao@chromium.org> Reviewed-by:
-
Eugene But authored
This change is made because all subdirectories of ios/web/public represent features, while web_state subdirectory is class name. The move proposal was discussed here: https://docs.google.com/document/d/1C2YIU0rDbH5wz2VI6nk0COrktwY3DbF-e7QbMIAKEJc/edit Bug: 942244 Change-Id: Ia3c5cf1a6e0a0a58ba3f549e3619a6ada707d1a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819355 Commit-Queue: Yi Su <mrsuyi@chromium.org> Auto-Submit: Eugene But <eugenebut@chromium.org> Reviewed-by:
Yi Su <mrsuyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#699232}
-
Ken Rockot authored
This is a reland of 749dcc3b PS1->PS4 shows the exclusive delta from the original patch, fixing a subtle lifetime issue in FileService. This turns out to have always been wrong: LevelDBServiceImpl can be destroyed before its created LevelDBDatabaseImpl objects have received all messages, and the service impl owns the shared message pipe for the associated database interfaces; thus messages can be dropped and data lost. This never mattered before because FileService instances effectively lived forever anyway. The fix here makes LevelDBServiceImpl lifetime independent of FileService lifetime, tying it instead to client lifetime. Original change's description: > Move File Service off Service Manager > > This is a first step towards eliminating the File Service and migrating > DOM Storage code into the Storage Service. > > File Service and its only clients (DOM storage objects) are moved off of > Service Manager APIs. LocalStorageContextMojo and > SessionStorageContextMojo are changed to own their own FileService > instance, now constructed directly over a directory path. > > No functional changes intended in this CL. > > Bug: 994911,977637 > Change-Id: I2ffd0bef670304a0dc3c103981139603521a14ac > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1778801 > Commit-Queue: Ken Rockot <rockot@google.com> > Reviewed-by: Avi Drissman <avi@chromium.org> > Reviewed-by: Robert Sesek <rsesek@chromium.org> > Reviewed-by: Marijn Kruisselbrink <mek@chromium.org> > Cr-Commit-Position: refs/heads/master@{#698487} NOPRESUBMIT=true Bug: 994911, 977637 Change-Id: I12658247e428f2e5eee32a2e3eabe3eb35381238 Tbr: avi@chromium.org Tbr: mek@chromium.org Tbr: rsesek@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816582 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by:
Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#699231}
-
Mario Sanchez Prada authored
Convert both the implementation and clients in the browser and renderer processes for the media_router.mojom.MediaController interface, and adapt unit tests. Bug: 955171 Change-Id: I59ab6af047fe6b87ae470e0379f7f607e12a427c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810998Reviewed-by:
Brandon Tolsch <btolsch@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Cr-Commit-Position: refs/heads/master@{#699230}
-
Tomasz Moniuszko authored
Bug: 1006749 Change-Id: I74334514acd6fd195e6cc435e37e5b46731ea4a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819306Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Commit-Queue: Tomasz Moniuszko <tmoniuszko@opera.com> Cr-Commit-Position: refs/heads/master@{#699229}
-
Alan Cutter authored
Mark virtual/omt-service-worker-startup/external/wpt/service-workers/service-worker/update-bytecheck.https.html as timing out https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=virtual%2Fomt-service-worker-startup%2Fexternal%2Fwpt%2Fservice-workers%2Fservice-worker%2Fupdate-bytecheck.https.html This test recently started timing out frequently on Windows 7. TBR=knollr@chromium.org Bug: 1002377 Change-Id: I5c54b1c0b683b214f11beaddd2259508dd71ba8c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820405Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/e712395b0e52..ca2ef4313b40 Created with: gclient setdep -r src-internal@ca2ef4313b40 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None Change-Id: I6d62654e2e88426b048b9ccb323a9906b3bcb9d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819854Reviewed-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@{#699227}
-
Yuta Kitamura authored
Originally WindowAgent's constructor's comment suggested it should not be used, but nevertheless Document was using it. This CL updates the comment and adds information about when it should be used. Change-Id: Ia960af496bdea3fe7dbb0e07dc1f6a97cf64b760 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809035Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Yuta Kitamura <yutak@chromium.org> Auto-Submit: Yuta Kitamura <yutak@chromium.org> Cr-Commit-Position: refs/heads/master@{#699226}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/017e8357..778af2fd Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I8d6e0ec38b34b1e956c861e77606313aa469d0fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820443Reviewed-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@{#699225}
-
Jeevan Shikaram authored
This CL corrects a regression which hides the more settings and permission linkout. Bug: 979531 Change-Id: I468734a201836b23c1d7943f139ff9090af45390 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820401Reviewed-by:
-
John Z Wu authored
This makes it easier to output debug menus. I had to change the private property into a method because C++ things do not seem to work well with valueForKey: Change-Id: I62b08be230b968449a6a002a1c7e62a4c0026a03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816288Reviewed-by:
Hiroshi Ichikawa <ichikawa@chromium.org> Commit-Queue: John Wu <jzw@chromium.org> Cr-Commit-Position: refs/heads/master@{#699223}
-
Jeevan Shikaram authored
This CL deletes the Google Play Store section and adds it's children into the new Apps settings section. Before: https://bugs.chromium.org/p/chromium/issues/attachment?aid=413184&signed_aid=VDO2a7j5Bu_Via8i0H_BzA==&inline=1 After: https://bugs.chromium.org/p/chromium/issues/attachment?aid=413183&signed_aid=sQOsHwyPDGWKiaC2zZlgQQ==&inline=1 Bug: 1004583 Change-Id: If9d307de4240ab8246f2b7ebb61e38b2530f792a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1815957 Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#699222}
-
Peter Kasting authored
Bug: 82078 Change-Id: Id6acded56ab6c6e4133a274875db985e963fcf8a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820539 Commit-Queue: Peter Kasting <pkasting@chromium.org> Commit-Queue: Jonathan Metzman <metzman@chromium.org> Auto-Submit: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Jonathan Metzman <metzman@chromium.org> Cr-Commit-Position: refs/heads/master@{#699221}
-
François Beaufort authored
This CL adds missing attribute error to GPUUncapturedErrorEvent. It was forgotten in http://crrev.com/1787504 Bug: 877147 Change-Id: Ia135b7b79a4215c0c90a4704516c3b70f3658072 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1814827 Auto-Submit: François Beaufort <beaufort.francois@gmail.com> Reviewed-by:
Corentin Wallez <cwallez@chromium.org> Reviewed-by:
Kai Ninomiya <kainino@chromium.org> Commit-Queue: François Beaufort <beaufort.francois@gmail.com> Cr-Commit-Position: refs/heads/master@{#699220}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/984536bb45f4..e18bf0acc0be git log 984536bb45f4..e18bf0acc0be --date=short --no-merges --format='%ad %ae %s' 2019-09-24 dberris@chromium.org PinpointScheduler: A Validator Implementation 2019-09-24 dberris@chromium.org Pinpoint: Allow for partial job instance migration Created with: gclient setdep -r src/third_party/catapult@e18bf0acc0be If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC simonhatch@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=simonhatch@google.com Bug: chromium:988912,chromium:1006658 Change-Id: I6bc3f39f8d0124e31726b702551b7a6142468a89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819851Reviewed-by:
-
Mario Sanchez Prada authored
This CL migrates code to the new BrowserInterfaceBroker class, allowing to retrieve the remote implementation from the renderer process without relying on FrameClient's InterfaceProvider. Additionally, this CL also migrates the few usages of the old Mojo types that were still around in the codebase. TBR=clamy@chromium.org Bug: 955171, 978694, 936482 Change-Id: I95d2d9ef39838a09748542c847440b060190e2f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819238 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/src/third_party/freetype2.git/+log/5327092bb28b..4881f75b7ca0 git log 5327092bb28b..4881f75b7ca0 --date=short --no-merges --format='%ad %ae %s' 2019-09-24 apodtele@gmail.com * src/base/ftstroke.c (ft_stroke_border_arcto): Speed up calculations. Created with: gclient setdep -r src/third_party/freetype/src@4881f75b7ca0 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/+/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 Bug: None Change-Id: I010c59395b4b818f5ecd45d26cf5230010a5f727 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816426Reviewed-by:
-
Mario Sanchez Prada authored
This CL migrates code to the new BrowserInterfaceBroker class. This allows retrieving the remote implementation from the renderer process without relying on FrameClient's InterfaceProvider. TBR=clamy@chromium.org Bug: 955171, 978694, 936482 Change-Id: I38672e3a40864aef9a179afc5463ab241491d096 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816552 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Eric Stevenson <estevenson@chromium.org> Reviewed-by:
-
Mario Sanchez Prada authored
Convert the remaining bits in both the implementation and clients from the browser and renderer processes for the blink::mojom::DevToolsAgent[Host] and blink::mojom::DevToolsSession[Host] interfaces. Also, rename blink::DevToolsAgent's BindRequest() methods to BindReceiver, to make them consistent with the usual pattern followed on this migration. Bug: 955171, 978694 Change-Id: Ide73670eb7d3d3fafe30f09a4d94a0e98d63b552 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819309Reviewed-by:
-
Mario Sanchez Prada authored
Convert both the implementation and clients in the browser and renderer processes for the media_router.mojom.MediaRouterTraitsTestService interface, and adapt unit tests. Bug: 955171 Change-Id: I4856427d46032c66a0dec395cd0711e93ba93700 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816539Reviewed-by:
-
Julie Jeongeun Kim authored
This CL converts SpellCheckHost and SpellChecker to new Mojo types using Remote, PendingRemote, PendingReceiver, Receiver, and ReceiverSet. It introduces a new template for GetInterface from LocalInterfaceProvider. Bug: 955171 Change-Id: I459d2ffc50242f8f2159c31212fd2346e571544f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813125 Commit-Queue: Julie Kim <jkim@igalia.com> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Tim Volodine <timvolodine@chromium.org> Cr-Commit-Position: refs/heads/master@{#699213}
-
Julie Jeongeun Kim authored
This CL converts apps::mojom::PublisherPtr to new Mojo types using PendingRemote or Remote, PendingReceiver, ReceiverSet, and Receiver. It also updates RegisterPublisher from app_service.mojom Bug: 955171 Change-Id: Ic4001997764a9872a0df5f4931368312c263d27f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1818886 Commit-Queue: Julie Kim <jkim@igalia.com> Reviewed-by:
-
Roman Sorokin authored
I've left it a separate polymer element to make the CL as simple as possible Before: https://screenshot.googleplex.com/O4r365FLMzJ.png After: https://screenshot.googleplex.com/2uf0mVpH5ck.png Bug: 999090 Test: manual Change-Id: If815a9b5e45334cde63b615d60d72be32f9d9021 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819299Reviewed-by:
Denis Kuznetsov <antrim@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#699211}
-
Marina Sakai authored
V8_PRIVATE_PROPERTY_GETTER_NAME: a macro-generated function can be cleaned by using the new API which was created in CL1810511. Two macros variable can be removed with this CL. Bug: 715418, 1005601 Change-Id: I1da3aa583d95700ce305c237e4a4ee4ce2aa01ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813117Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Reviewed-by:
-
Jeevan Shikaram authored
This CL follows up crrev.com/c/1784238 with few naming and console error fixes. Bug: 999896 Change-Id: Ic053db94e250aebd38105ace7e895c468d3d30c7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1798255Reviewed-by:
-
Ricky Liang authored
In crrev.com/c/1687018 we added a local call site to RegisterClient, so we can no longer assume the method is always called through Mojo IPC. enabled. Bug: 1000842 Test: Make sure Chrome debug build starts on Nocturne with ARC++ Change-Id: Ibe546396646bab01c7caec02ff72f8122ac80181 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1817875Reviewed-by:Wei Lee <wtlee@chromium.org> Reviewed-by:
Shik Chen <shik@chromium.org> Commit-Queue: Ricky Liang <jcliang@chromium.org> Cr-Commit-Position: refs/heads/master@{#699208}
-
Alex Keng authored
This change updates forced_color.css to enable <select multiple> to render high contrast color. Bug: 994219 Change-Id: If9814a11544232cfe8ee4de219f0ddaa83f70204 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816448 Commit-Queue: Alex Keng <shihken@microsoft.com> Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#699207}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/b4b1005d485f..c096654fa7c6 git log b4b1005d485f..c096654fa7c6 --date=short --no-merges --format='%ad %ae %s' 2019-09-24 csmartdalton@google.com ccpr: Don't assign inverse fill types to fan tessellations Created with: gclient setdep -r src/third_party/skia@c096654fa7c6 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 bungeman@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_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@google.com Bug: None Change-Id: Ida0d7f3e7aa6a2e18927c488f476f1e705b06bd4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820012Reviewed-by:
-
Xiaocheng Hu authored
The current parser allows `/ <bg-size>` as long as there's a <bg-position> in the current or previous layers, which is wrong. This patch ensures that it's allowed only after a <bg-position> in the same layer as per spec: https://drafts.csswg.org/css-backgrounds-3/#the-background Bug: 1002568 Change-Id: I79630a1331d880248d27205788a65cd5b4032a0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819562 Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
-
Chih-Yu Huang authored
Originally we assume that VD::CreateSurface() will be called after AVD::SetStream() is called, and the returned surface is used for the current bitstream buffer. However, the description of AVD doesn't guarantee this fact. This CL changes to set the timestamp that is mapped from bitstream id passed from SurfaceReady(). This timestamp is actually reliable value. BUG=b:140842667 TEST=run video_decode_accelerator_tests on Kevin and Eve Change-Id: I255870c6220a4bd771e1f23b54c309fe8ca42246 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810452Reviewed-by:
Hirokazu Honda <hiroh@chromium.org> Commit-Queue: Chih-Yu Huang <akahuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#699204}
-
Gyuyoung Kim authored
As a step to remove all uses of WebLocalizedString::kFooBar, this CL replaces below ones with matched IDS_FOO. - WebLocalizedString::kAXMillisecondFieldText - WebLocalizedString::kAXMinuteFieldText - WebLocalizedString::kAXMonthFieldText - WebLocalizedString::kAXSecondFieldText - WebLocalizedString::kAXWeekOfYearFieldText - WebLocalizedString::kAXYearFieldText Bug: 995644 Change-Id: I09495c8f2737a398e1b553b2b006f44e424f6686 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820398Reviewed-by:
-
Raul Tambre authored
Initial conversion performed using '2to3 -f print .'. Imports added and duplicate parentheses removed manually. Manually converted files, comments and inline code that 2to3 missed. Presubmit disabled due to an unrelated error in find_runtime_symbols/find_runtime_symbols.py. Afterwards ran "git cl format --python" and cherry-picked the formatting changes. There are no intended behavioural changes. NOPRESUBMIT=true Bug: 941669 Change-Id: I3174ed0eb7005d493c6bc44751353b8fae18a1f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1818478 Commit-Queue: Raul Tambre <raul@tambre.ee> Reviewed-by:
Nico Weber <thakis@chromium.org> Auto-Submit: Raul Tambre <raul@tambre.ee> Cr-Commit-Position: refs/heads/master@{#699202}
-
Kenichi Ishibashi authored
We should terminate a service worker thread when the top-level script fetch fails. Otherwise the thread would stay running and leak a V8 isolate. WorkerGlobalScope::close() schedules thread termination. I manually confirmed that this CL fixes the leak. I'll add a test for this fix in a follow-up CL. Bug: 1005650 Change-Id: If3fd2f7f0f746c4e240b9fdebd4cf242d87a0943 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819282 Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#699201}
-
