- 03 Jan, 2020 40 commits
-
-
Samuel Huang authored
Also allow output to be optional again (output written to console, but without SRCDIR modification). Bug: 979886 Change-Id: Ia1e77ec8e73b0be44e3f1f84de4559ad233b169e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986853 Commit-Queue: Samuel Huang <huangs@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#728284}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/de97fb4ac4c6..81ee4d29192d git log de97fb4ac4c6..81ee4d29192d --date=short --first-parent --format='%ad %ae %s' 2020-01-03 shrekshao@google.com Workaround EXT_texture_norm16 for OpenGL ES drivers Created with: gclient setdep -r src/third_party/angle@81ee4d29192d If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC courtneygo@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-asan;luci.chromium.try:win_optional_gpu_tests_rel Bug: chromium:1000354 Tbr: courtneygo@google.com Change-Id: Ifc921d860865b2f832a277e64dc73107c5b3b2a8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986895Reviewed-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@{#728283}
-
Bence Béky authored
Use NoopDecoderStreamErrorDelegate and NoopQpackStreamSenderDelegate implementations from test_tools instead of defining new ones. There is no need to crash on decoder stream errors or encoder stream data, because |qpack_encoder_| is set up not to use the dynamic table and no decoder stream data is fed to it. Also add const qualifier to four methods. Change-Id: Ibd103f3598458f2369e4ff7ce37b3855bfeabfef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986259 Auto-Submit: Bence Béky <bnc@chromium.org> Reviewed-by:
Zhongyi Shi <zhongyi@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#728282}
-
Peter Boström authored
Replaces GetToolbarActionViewForId with GetExtensionsToolbarContainer to be used directly (with GetViewForId). This is intended to be used to pop out and show bubbles which cannot be added to the ExtensionsContainer interface as it is defined outside of Views. This also implements GetDefaultExtensionDialogAnchorView inside WebAppFrameToolbarView and updates BrowserView::GetExtensionsContainer to work with ToolbarButtonProvider. These two combined fix a crash when uninstalling an extension inside a web-app window. Bug: chromium:1021905 Change-Id: I9b9d23adca85600212846fa0fdb8db9d43dcf6ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986305Reviewed-by:
Charlene Yan <cyan@chromium.org> Commit-Queue: Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#728281}
-
Nicolás Peña Moreno authored
Currently, some tests work under the assumption that the height of the viewport is exactly 600px. While this is true for tests running on our bots, it's not true for tests running on WPT.fyi. Thus, this assumption is removed from those tests. Bug: 1036013 Change-Id: I34b057730e22bc208cbb958d179195445def8b77 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1984678Reviewed-by:
Annie Sullivan <sullivan@chromium.org> Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> Cr-Commit-Position: refs/heads/master@{#728280}
-
Mason Freed authored
*** SHERIFFS: if this CL triggers a flaky layout test, PLEASE do not *** revert this CL. Instead, please disable the offending test(s) and *** add them to the crbug.com/1035582 section of TestExpectations. I *** will then take care of them as I fix up the remainder of these *** tests. Feel free to cc me if you create a bug. Thanks. This CL enables the FormControlsRefresh flag by default on all Desktop platforms except Mac. The Mac release is tracked in crbug.com/1012108. Because many layout tests use form controls, including tests that are not directly testing form controls, this CL requires around 1200 tests to be rebaselined. The bug at [1] tracks the effort to re-baseline all of these tests and remove these new lines from TestExpectations. This will be done in pieces, to reduce the code review burden. The bug at [2] tracks cleaning up some of these tests eventually, as there are new tests (see [3]) that directly test the appearance of the new form controls, and those likely supersede many rebaselines in this CL. However, I would like to get this landed first, as it might require several Revert/Reland cycles to get landed for good, and then I can come back and do the cleanup. This CL deletes a handful of form control popup web_tests that are badly broken by this change, but are also superseded by new tests of the new popups. [1] https://crbug.com/1035582 [2] https://crbug.com/1034611 [3] https://source.chromium.org/chromium/chromium/src/+/master:third_party/blink/web_tests/virtual/controls-refresh/ Bug: 1012106, 1035582, 1034611 Change-Id: Ib499da1db0df480dc61efee8231718aee13f24a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1967608Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Mason Freed <masonfreed@chromium.org> Auto-Submit: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#728279}
-
David Bokan authored
Since https://crrev.com/c/1980984 has decoupled latching from scroll animation state, we can now simply use the latched node for sending of scrollend and overscroll events to DOM. We simply require that the latched node be preserved past the GSE so that in the next commit we can pass it along for dispatching the scrollend event. This CL removes |scroll_animating_latched_element_id_| (which was actually unused since the previous CL) and |scroll_animating_overscroll_target_element_id_|. The latter was previously needed since ScrollAnimated would clear the latched node if an animation wasn't created (e.g. at the extent). Since the latch is now maintained, it can be used in its place. This CL renames |last_scroller_element_id_| to |last_latched_scroller_| and makes it match the |CurrentlyScrollingNode|, with the only exception being that it gets cleared only after the first commit after the CurrentlyScrollingNode is cleared. Bug: 940508 Change-Id: I2d0c63bd97c3bc1a9bd0158a08b3352b09bfb1f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1981195 Commit-Queue: David Bokan <bokan@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#728278}
-
Moe Ahmadi authored
Bug: 1034997 Change-Id: If85a4399ecb99257e4733caba20dac70b5686488 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1985169 Auto-Submit: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Dan Beam <dbeam@chromium.org> Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Cr-Commit-Position: refs/heads/master@{#728277}
-
Sophie Chang authored
This makes it easier to refactor CanApply for the URL-keyed hints. The new ShouldTargetNavigation in hints manager is temporary and will be removed once prediction manager is fully live. Bug: 1036490 Change-Id: I5855abc507ff1f12f9c392d234b9bfb71d9acb8c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986250 Commit-Queue: Sophie Chang <sophiechang@chromium.org> Reviewed-by:
Michael Crouse <mcrouse@chromium.org> Cr-Commit-Position: refs/heads/master@{#728276}
-
Andrew Grieve authored
As per discussion: https://groups.google.com/a/chromium.org/d/topic/chromium-dev/RcJgJdkNIdg/discussion Bug: 846432 Change-Id: I0d1abb42c79278d5edf03091a47fce03d5897c25 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1984356 Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#728275}
-
Quinten Yearsley authored
Bug: 1002702 Change-Id: I3100574c1c4a3677c710021d755e91f5db23a87d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1982686Reviewed-by:
Robert Ma <robertma@chromium.org> Commit-Queue: Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#728274}
-
Reilly Grant authored
This change updates //chrome/installer to explicitly choose between run-once callbacks and run-repeatedly callbacks. For the most part this is a no-op. In the case of CallbackWorkItem it has been modified to take separate callbacks for install and rollback tasks. Bug: 1007639 Change-Id: Id4475eec8ad34a057beb1a48442f7e03dc36c368 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1938042 Commit-Queue: Reilly Grant <reillyg@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Auto-Submit: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#728273}
-
Lijin Shen authored
The if check is too strict. Add some tests to prevent from regression. Bug: 1036645 Change-Id: I0902b74fcec616f9a563b1a746b6f7bd09153c54 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986199 Commit-Queue: Lijin Shen <lazzzis@google.com> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#728272}
-
Yusuf Sengul authored
Bug: 1036043 Change-Id: Ia80a822b886b030bc1d7bd6846b810c9e2c20193 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1977392 Commit-Queue: Yusuf Sengul <yusufsn@google.com> Reviewed-by:
Tien Mai <tienmai@chromium.org> Cr-Commit-Position: refs/heads/master@{#728271}
-
Xianzhu Wang authored
It had the following values: kWebScrollbarButtonsPlacementNone, kWebScrollbarButtonsPlacementSingle, kWebScrollbarButtonsPlacementDoubleStart, kWebScrollbarButtonsPlacementDoubleEnd, kWebScrollbarButtonsPlacementDoubleBoth. Now none of the platforms supports double buttons (i.e. two scrollbar buttons at one end of the scrollbar), so the last 3 values were not used. The first two values are used to check whether scrollbars should have buttons. This can be achieved with a simpler API (NativeThemeHasButtons() in this CL). Note that we still support double buttons in custom scrollbars which are used by web sites such as google spreadsheets. (TBR for trivial changes under content/common and third_party/blink/public) TBR=dgozman@chromium.org Change-Id: Iff4d22ec667affb91cb1a98871b7bd3707d5b099 Bug: 1009063 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1966987 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Rahul Arakeri <arakeri@microsoft.com> Cr-Commit-Position: refs/heads/master@{#728270}
-
Boris Sazonov authored
Adds an assert to verify that sign-in is allowed before starting it. To fix tests broken by this assert, this CL also adds an explicit ExternalAuthUtils param to SigninManager. This is necessary for the upcoming removal of AccountIdProvider.canBeUsed. Bug: 914920 Change-Id: Ib2ddd446446539fc45534818d30b5e6dab4fef73 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1985006Reviewed-by:
Tanmoy Mollik <triploblastic@chromium.org> Commit-Queue: Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#728269}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/7426a76a4d80..a6e6c3a30e17 Created with: gclient setdep -r src-internal@a6e6c3a30e17 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: None Tbr: jbudorick@google.com Change-Id: I33fdc0ac545f429409f387e7fe41af774bf5c562 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986335Reviewed-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@{#728268}
-
Reilly Grant authored
This change updates the base::Callbacks and calls to base::Bind() in //google_apis/gcm to explicitly choose betwenn call-once and call-repeatedly semantics. Bug: 1007790 Change-Id: I1f253bd17bca0e5e9975eab429d7a377ced0f962 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1939891 Commit-Queue: Reilly Grant <reillyg@chromium.org> Reviewed-by:
Nicolas Zea <zea@chromium.org> Auto-Submit: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#728267}
-
Josh Simmons authored
Bug: 1030813 Change-Id: I9bc146a28099334ebb1c87e4dce5b65795a8a892 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986255 Commit-Queue: Josh Simmons <jds@google.com> Reviewed-by:
Sophie Chang <sophiechang@chromium.org> Cr-Commit-Position: refs/heads/master@{#728266}
-
Chris Harrelson authored
1. It is no longer possible to cause recursion in RestoreScrollPositionAndViewState, so delete code that protects against that. 2. Remove now-unused parameter. Refactor CL: https://chromium-review.googlesource.com/c/chromium/src/+/1981806 Change-Id: I93d99d1a301f97303186459949963c1da11144d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1982788Reviewed-by:
Stefan Zager <szager@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#728265}
-
kylechar authored
Move DrmOverlayManager and DrmOverlayCandidates out of common and into gpu since they are no longer used by host code. Also finish cleanup in a few places missed in the previous CL. Bug: 936425 Change-Id: I61530dccf431150576c88f6bba1fb52eb37d5e7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986850 Commit-Queue: kylechar <kylechar@chromium.org> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Cr-Commit-Position: refs/heads/master@{#728264}
-
Ben Mason authored
TBR=govind@chromium.org Change-Id: I32361158eb74da2d004ad31d02bfaf1f6a5a2127 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986858Reviewed-by:
Ben Mason <benmason@chromium.org> Commit-Queue: Ben Mason <benmason@chromium.org> Cr-Commit-Position: refs/heads/master@{#728263}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/71b51fa69ee9..430f61e2a241 git log 71b51fa69ee9..430f61e2a241 --date=short --first-parent --format='%ad %ae %s' 2020-01-03 jalyn@microsoft.com Prevent unhandled exception in ModelBreakpoint 2020-01-03 Rob.Paveza@microsoft.com Silent failure in _innerStyleSheetChanged Created with: gclient setdep -r src/third_party/devtools-frontend/src@430f61e2a241 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Bug: chromium:1019538 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I6136d9bff6271eea2b61501085da10bf7b27f1b8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986897Reviewed-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@{#728262}
-
Vladimir Levin authored
This patch does two things: 1. Removes the display locking fuzzer, since its dictionary is testing a now non-existent javascript api. It is also seems to be leaky (see referenced bug as an example) 2. Adds render-subtree keywords to the css fuzzer's dictionary, so that that fuzzer exercises the render-subtree code. R=chrishtr@chromium.org, rakina@chromium.org Bug: 1031946 Change-Id: Ic21b353a38d651c42e9179806f71670ae5e72e5c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986887Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#728261}
-
Sadrul Habib Chowdhury authored
FrameMetrics and related code in ui/latency/ have remain unused for a while. So remove them from the tree. We can bring them back if/when we need to. BUG=790761 Change-Id: Ifc1f6db447cd2e7c5be752b4054cffdb700d0628 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1984718 Auto-Submit: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org> Cr-Commit-Position: refs/heads/master@{#728260}
-
Reilly Grant authored
This change updates //chrome/attestation to use base::OnceCallback rather than base::Callback (and base::Bind/Closure). To do so AttestationFlow has been refactored to no longer bind the completion callback into multiple callbacks and the control flow is now much more linear. Functionality is unaffected. Bug: 1007652 Change-Id: I11e449cc508ea33f997edd865250cabf23664c5d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1935174 Commit-Queue: Reilly Grant <reillyg@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#728259}
-
Clark DuVall authored
This should help figuring out where the duplicate IDs are coming from. Bug: 996940 Change-Id: I3e3611e83ed563cf883729f307bbe2b66ac6b74d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1977186Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#728258}
-
shrekshao authored
Fix the issue where requesting GL_WEBGL_multi_draw_instanced_base_vertex_base_instance returns null when GL_WEBGL_multi_draw is first requested on passthrough. TBR=kbr@chromium.org Bug: 891861 Change-Id: Ib9c15787c8eef79c71d71386a3e1ac4add1f92d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986124Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Shrek Shao <shrekshao@google.com> Cr-Commit-Position: refs/heads/master@{#728257}
-
Tarun Bansal authored
Make it possible to configure loading predictor table size using field trials. The next step is to experiment with different sizes. Change-Id: I607f83e0ca557466d434143fa1894900a135cf7d Bug: 1038703 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1984253Reviewed-by:
Alex Ilin <alexilin@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#728256}
-
John Palmer authored
Change-Id: I8efa1dce401bd3088972a58881c7ff1eeed68fe1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986466 Auto-Submit: John Palmer <jopalmer@chromium.org> Commit-Queue: Brian White <bcwhite@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#728255}
-
Yusuf Sengul authored
Bug: 1036084 Change-Id: If87438015293fcdcfa22eb1d1ccc2c2563884bcd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1981118Reviewed-by:
Tien Mai <tienmai@chromium.org> Commit-Queue: Yusuf Sengul <yusufsn@google.com> Cr-Commit-Position: refs/heads/master@{#728254}
-
Kevin Ellis authored
The animation was often finishing before the screenshot resulting in a failure. Extended the duration of the animation and starting the animation from the midpoint should address the flakiness without needlessly extending the run time of the test. Bug: 1029543 Change-Id: I32e7794ff77a3b26e7d7f260f80d4b00bbc9d6f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986212Reviewed-by:
Stephen McGruer <smcgruer@chromium.org> Commit-Queue: Kevin Ellis <kevers@chromium.org> Cr-Commit-Position: refs/heads/master@{#728253}
-
Andrew Xu authored
Scrollable shelf updates its layout when a shelf icon is added to shelf view as a child. When an external display is connected, shelf view of the external display should add shelf icons during shelf view initialization. However, ShelfView::Init is called before shelf view becomes the grand child of ScrollableShelfView. As a result, scrollable shelf cannot perceive the change in view hierarchy. This CL makes the initialization of shelf view be the last step of ScrollableShelfView::Init. It also fixes the side effect brought by reordering of child views' initialization. Bug: 1035596 Change-Id: Ice08606d34b5af725157602c0038d995df2d36ab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1984725 Commit-Queue: Andrew Xu <andrewxu@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#728252}
-
Alex Turner authored
Adds PageLoad.Clients.Ads.FrameCounts.AdFrames.Total to the set of metrics recorded in the ad_tagging.cluster_telemetry benchmark. This will allow for a more direct measurement of frame coverage changes when filter lists are updated. Bug: 1036451 Change-Id: I31606e97f45867e6e68f9819f24a3ed1da74eae5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986426Reviewed-by:
John Delaney <johnidel@chromium.org> Commit-Queue: Alex Turner <alexmt@chromium.org> Cr-Commit-Position: refs/heads/master@{#728251}
-
Brian Osman authored
This is a refactored (and now public) API to solve the same problem. SkRuntimeColorFilterFactory was just a shim over SkRuntimeEffect, and will be deprecated/deleted soon. Change-Id: I043c82ab38a2ddebfa29680f394c4263973ac42d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1982791Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Cr-Commit-Position: refs/heads/master@{#728250}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/0f14db22decb..d57274888541 git log 0f14db22decb..d57274888541 --date=short --first-parent --format='%ad %ae %s' 2020-01-03 philipel@webrtc.org Run delay tasks on time when using GlobalSimulatedTimeController. 2020-01-03 nisse@webrtc.org Delete unused member BasicPortAllocator::allow_tcp_listen_ 2020-01-03 saza@webrtc.org Merge the preambles of the ProcessStream implementations Created with: gclient setdep -r src/third_party/webrtc@d57274888541 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: I818c0c825514074bc0eff0c200f30ef190b7065c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986864Reviewed-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@{#728249}
-
Kelvin Jiang authored
This CL adds the following functionality: - GetMatchedRules API call, to get all rules an extension has matched - Ability for extension to specify a tab ID to filter rules in GetMatchedRules Bug: 983761 Change-Id: I3672d84cc23be0e21a1387ede52923c3fe6354ed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913611Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Kelvin Jiang <kelvinjiang@chromium.org> Cr-Commit-Position: refs/heads/master@{#728248}
-
Patrick Noland authored
Bug: 1036556 Change-Id: I1f11e34de7915c8b40b141bd5243fbee95688458 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986188Reviewed-by:
Natalie Chouinard <chouinard@chromium.org> Commit-Queue: Patrick Noland <pnoland@chromium.org> Cr-Commit-Position: refs/heads/master@{#728247}
-
gogerald authored
Bug: 1035573 Change-Id: I12f217b6cb01481f3f414f81bcb6c41f14738c9c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974832Reviewed-by:
Yusuf Ozuysal <yusufo@chromium.org> Commit-Queue: Ganggui Tang <gogerald@chromium.org> Auto-Submit: Ganggui Tang <gogerald@chromium.org> Cr-Commit-Position: refs/heads/master@{#728246}
-
Robert Ogden authored
Still wanted to track regressions. Bug: 1037206 Change-Id: I257fe9880e3fe2d6aaadd074ad1c7c67079f7127 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1984817 Commit-Queue: Robert Ogden <robertogden@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#728245}
-