- 09 Sep, 2019 40 commits
-
-
Gayane Petrosyan authored
This is a reland of f5a9a694 Original change's description: > [Cleanup] Move complex color calc out of searchbox_extension.. > > In this CL the logo and shortcut color calculation is split into 2 parts. > The color calculation based on the theme is moved to themes, and > defaults are moved to ThemeProperties. > The logo and shortcut colors can be overridden when custom background is > set. This logic is moved to instant_service. > > NTP text and logo_alternate logic is also moved out of > searchbox_extension. > > Bug: 983717 > Change-Id: I12b938ba8a833bd920cf224171163781889989a7 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1759150 > Reviewed-by: Oliver Chang <ochang@chromium.org> > Reviewed-by: Evan Stade <estade@chromium.org> > Commit-Queue: Gayane Petrosyan <gayane@chromium.org> > Cr-Commit-Position: refs/heads/master@{#691869} TBR=ochang@chromium.org Bug: 983717 Change-Id: I1a72f7e4d7c48289ed9cb8b0341b448c03b025f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1778856 Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#694868}
-
Sigurdur Asgeirsson authored
Bug: 910288 Change-Id: I99c2917452ef0f16f391fb0d0592be07480ccafd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791269Reviewed-by:
Patrick Monette <pmonette@chromium.org> Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#694867}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/efce0d1b7657..6a860326887f git log efce0d1b7657..6a860326887f --date=short --no-merges --format='%ad %ae %s' 2019-09-09 brucedawson@chromium.org Handle GOMA_DISABLED more precisely Created with: gclient setdep -r src/third_party/depot_tools@6a860326887f If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC agable@chromium.org 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 TBR=agable@chromium.org Bug: chromium:b/140312943 Change-Id: I0d8d28c96997af61759196d10f65dd814435e9ab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793223Reviewed-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@{#694866}
-
Rahul Singh authored
This CL changes how we overload caption styling on Windows. Going forward, we will overload a caption styling property only when a user explicitly sets it in Settings (set to something other than Default). Previously we always overloaded styling with OS preferences, even when a property was set to Default. This change aligns with changes to macOS behavior that went in recently https://crrev.com/c/1764966 Bug: 1000515 Change-Id: Ibdaaa8a40653543367eb655bd6b0945d78dd516c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786870Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Rahul Singh <rahsin@microsoft.com> Cr-Commit-Position: refs/heads/master@{#694865}
-
Gary Kacmarcik authored
This cl also removes some support for the ICE protocol to simplify the display handling code. Change-Id: I64523c0fc766cd80acdaae832c8f0fa08bb59757 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790274 Auto-Submit: Gary Kacmarcik <garykac@chromium.org> Reviewed-by:
Jamie Walch <jamiewalch@chromium.org> Cr-Commit-Position: refs/heads/master@{#694864}
-
Sidney San Martín authored
Sometimes, a browser window isn't considered "on the current space" during the fullscreen transition at the moment the exclusive access bubble is displayed. Fix by checking again at the end of the transition. Bug: 1000882 Change-Id: Ia2ef081098ca84f975dd26dfa6bf2615a3094af7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790378 Auto-Submit: Sidney San Martín <sdy@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#694863}
-
Kristi Park authored
This reverts commit d4a3f593. Reason for revert: Encountering issues such as http://crbug/1002013 Original change's description: > [NTP] Fix bottom border of focus ring in richer picker > > Before: http://screen/VS5BsT3hGH1.png > After: http://screen/9x05VZFtAkS.png > > Bug: 997665 > Change-Id: If76461a2b26ebad7ad48d9bd157af4d7a76d8070 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772124 > Commit-Queue: Kristi Park <kristipark@chromium.org> > Commit-Queue: Kyle Milka <kmilka@chromium.org> > Auto-Submit: Kristi Park <kristipark@chromium.org> > Reviewed-by: Kyle Milka <kmilka@chromium.org> > Cr-Commit-Position: refs/heads/master@{#690525} TBR=kristipark@chromium.org,kmilka@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 997665,1002013d Change-Id: I4500159c78eca3429528a4aa55071ff3a43d9b81 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790146Reviewed-by:
Kristi Park <kristipark@chromium.org> Commit-Queue: Kristi Park <kristipark@chromium.org> Cr-Commit-Position: refs/heads/master@{#694862}
-
Gyuyoung Kim authored
This CL converts old Mojo types to the new Mojo types. - Apply pending_receiver to CreateGpuMemoryBufferFactory Bug: 955171 Change-Id: I84cb8e73efcfa2d499bc8f568d5586744e8be7dd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792023 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Jonathan Backer <backer@chromium.org> Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#694861}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/28d40b2e7ade..3fdfe2823866 git log 28d40b2e7ade..3fdfe2823866 --date=short --no-merges --format='%ad %ae %s' 2019-09-09 brianosman@google.com Simplify GPUSink constructors by just passing the config object Created with: gclient setdep -r src/third_party/skia@3fdfe2823866 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 mtklein@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=mtklein@google.com Bug: None Change-Id: I6a7eacecb9c1ad1cb2cbb0ea95cf1054b34c7fb3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793226Reviewed-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@{#694860}
-
Saman Sami authored
The tests were disabled due to timeout when native Vulkan is used, so use swiftshader instead. Bug: 940723 Change-Id: Ida3745c7d6d480aebf1a56498b3242a1b77353d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790198 Commit-Queue: Saman Sami <samans@chromium.org> Reviewed-by:
Jonathan Ross <jonross@chromium.org> Reviewed-by:
Stephen Martinis <martiniss@chromium.org> Cr-Commit-Position: refs/heads/master@{#694859}
-
Esmael El-Moslimany authored
Bug: 980856 Change-Id: Ia28c8b4fe90b7fdad5c43660e3a4006a8673c321 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788225 Commit-Queue: Esmael Elmoslimany <aee@chromium.org> Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Cr-Commit-Position: refs/heads/master@{#694858}
-
Adithya Srinivasan authored
Follow up to https://crrev.com/c/1789532, actually marks the tests as disabled. Bug: 997634 Change-Id: I15918a7ce52be12dcb8a1af2ad20affe28a56f67 Tbr: treib@chromium.org, olka@chromium.org No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793245Reviewed-by:
Adithya Srinivasan <adithyas@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#694857}
-
Derek Beckett authored
There are some device protos which have the same field names (e.g. enabled). Currently the policy_server cannot differentiate between them. This will all setting these by: "groupname.fieldname", e.g: "device_power_peak_shift.enabled". TEST=Locally deployed and tested BUG=TBD Change-Id: I9565f3082ed192e787b2a00efec1ae29ebe195f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790534Reviewed-by:
Denis Kuznetsov <antrim@chromium.org> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Commit-Queue: Derek Beckett <dbeckett@chromium.org> Cr-Commit-Position: refs/heads/master@{#694856}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/03038a48..e6d242c0 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: I25eac74326f61f2d300a55890cd830c925485759 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792850Reviewed-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@{#694855}
-
Brian Sheedy authored
Adds several notes about running swarmed tests with extra arguments in docs/workflow/debugging-with-swarming.md. Change-Id: I751e0237399ff96626ef699b9a66e598eb07ab5a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793282 Auto-Submit: Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#694854}
-
Tom Anderson authored
Fixes this error: [ 375s] In file included from ../../base/task/common/intrusive_heap.h:8, [ 375s] from ../../base/task/thread_pool/task_source.h:17, [ 375s] from ../../base/task/thread_pool/task_tracker.h:27, [ 375s] from ./../../base/task/thread_pool/service_thread.cc:14, [ 375s] from gen/base/base_jumbo_28.cc:5: [ 375s] ../../base/containers/intrusive_heap.h:152:36: error: enumerator value '-1' is outside the range of underlying type 'size_t' {aka 'long unsigned int'} [ 375s] 152 | enum : size_t { kInvalidIndex = -1 }; [ 375s] | Change-Id: I6044fb704931b2f8416f6b19a247ae297cd7b0d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792763 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Auto-Submit: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#694853}
-
Livvie Lin authored
This mode will be triggered from PrepareErrorPage in a follow-up CL. Currently, this new code path doesn't change any behavior because a null callback from AllowCertificateError will follow the usual Show() logic. Bug: 987407 Change-Id: I32c2e1bd0fadb7540204e659b218d0138f168260 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769353 Commit-Queue: Livvie Lin <livvielin@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#694852}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/3c6c057c3feb..ec283489193a git log 3c6c057c3feb..ec283489193a --date=short --no-merges --format='%ad %ae %s' 2019-09-09 simonhatch@chromium.org Pinpoint - Create single set of hidden inputs Created with: gclient setdep -r src/third_party/catapult@ec283489193a 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 zhanliang@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=zhanliang@google.com Bug: chromium:1001757 Change-Id: Iebf26a9847474abb27423e097718f85397738ba6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791624Reviewed-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@{#694851}
-
Tommy Li authored
This makes some minor refactors to the usage of AutocompleteController::in_start_. 1. It delays setting it to false until after the UpdateResult call within the synchronous pass. This is more accurate, and also allows in_start_ to be used as a synonoym of in_synchronous_pass_. 2. It also moves in_start_ to be an early exit condition for OnProviderUpdate. It likely doesn't need to be checked there at all, since the comments indicate it's likely ALWAYS false when OnProviderUpdate is called. These refactors help pave the way for the feature to preserve the default match during the asynchronous pass, by clarifying in the code when the synchronous vs. asynchronous pass occurs. Bug: 398135 Change-Id: I8f8c61b8233b1c049338dac448b6ef8f1b0e3b93 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1789823Reviewed-by:
Kevin Bailey <krb@chromium.org> Commit-Queue: Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#694850}
-
Javier Fernandez authored
When calculating the min-content contribution of a grid item of an auto sized grid track we must consider the grid item's margin. When the grid item's area is indefinite, a percent margin is resolved to zero. However, when performing a relayout, the percent margin may be solved against the previously computed grid area, since the grid item has already an OverrideContainingBlockLogicalWidth value. In order to re-compute the percent margin properly, we need to clear the previously override value. It's important be careful of not clearing the override value set during intrinsic size, since we need it for the actual layout phase. Hence, we only reset the 'override' value when we are executing a definite strategy. Bug: 834643 Change-Id: Ib936b26bee1da76afbdc886eb775746e13d40988 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1782840 Commit-Queue: Javier Fernandez <jfernandez@igalia.com> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#694849}
-
Adithya Srinivasan authored
Bug: 1001446 Change-Id: Ia2c182bd2f622ccd07196770b443e92c610e3130 Tbr: mlamouri@chromium.org, msramek@chromium.org No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793232Reviewed-by:
Adithya Srinivasan <adithyas@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#694848}
-
rbpotter authored
- Autogenerate Polymer 3 cr-expand-button and tests - Add to demo page - Autogenerate modulized focus_without_ink, since the button depends on this file. - Update the js_modulizer script to add a way to ignore some lines Bug: 965770 Change-Id: I08b0613b3140b773039855af0105fecb72860987 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788321Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#694847}
-
Henrique Ferreiro authored
Convert the implementation and all users of the ash::mojom::AssistantAlarmTimerController, AssistantNotificationController and AssistantScreenContextController. Bug: 955171 Change-Id: I19142850f1cd445f6d43a4caf4c7bd88415680c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1789386Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#694846}
-
Sadrul Habib Chowdhury authored
Add the checkerboarding metrics in the rendering benchmarks. Adding these extra histograms do not affect the size of the output traces by much. Below are some sample trace-sizes from pinpoint (10 traces for each story): . On windows: - amazon_pinch: 56.32MB -> 56.32MB - twitter_pinch: 58.68MB -> 57.49MB . On pixel 2: - bouncing_balls_shadow: 41.69MB -> 41.74MB - bing_mobile: 57.08MB -> 57.14MB BUG=984592 Change-Id: I3fdb065f8b85814a7b900c46b14f569ec406c243 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1785418Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org> Cr-Commit-Position: refs/heads/master@{#694845}
-
Robert Liao authored
BUG=865318 Change-Id: I03ebbfdc95622f986e6271dbede4d1bfa740c5eb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788198 Commit-Queue: Robert Liao <robliao@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#694844}
-
Leon Han authored
BUG=520391 Change-Id: I5b9a807081657f2d211575a282ae7fa7f45c71ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791927 Auto-Submit: Leon Han <leon.han@intel.com> Commit-Queue: Reilly Grant <reillyg@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#694843}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/44e6523d7ff3..cebb9635a5d5 git log 44e6523d7ff3..cebb9635a5d5 --date=short --no-merges --format='%ad %ae %s' 2019-09-09 jmadill@chromium.org GN: Expose XCB surface extension on Linux. Created with: gclient setdep -r src/third_party/swiftshader@cebb9635a5d5 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Please CC swiftshader-team+autoroll@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_chromium_msan_rel_ng;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=swiftshader-team+autoroll@google.com Bug: chromium:b/140251624 Change-Id: I90a2718722f7687e3d945c6389a6384ef256edb9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792000Reviewed-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@{#694842}
-
Andrew Grieve authored
.part.end is a new section introduced by "partitioned libraries", a part of DFMS. It's a NOBITS section, so needs to be counted the same as .bss. This fixes some CQ runs failing due to: AssertionError: Padding was: $SOME_SMALL_NEGATIVE_NUMBER Bug: 1002085 Change-Id: I49e892d22f8888fd991202d99bc67e342c06db52 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791999 Auto-Submit: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Christopher Grant <cjgrant@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#694841}
-
Gabriel Charette authored
These tests were previously migrated from single-threaded MessageLoop to a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as part of crbug.com/891670. //base OWNERS decided in retrospect that it was better to keep a single-threaded option for TaskEnvironment and introduced SingleThreadTaskEnvironment. This CL retrofits that decision for /components/bookmarks. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. Edit: Revert changes to BookmarkNodeDataTest and BookmarkUtilsTest which rely on base::ThreadPool. R=sky@chromium.org Bug: 891670 Change-Id: I8a37c3ecc469fcbbdc52b80724f5d8b073e30414 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786903 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Auto-Submit: Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#694840}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/a0b94071e6bd..ac68e9c3dd09 git log a0b94071e6bd..ac68e9c3dd09 --date=short --no-merges --format='%ad %ae %s' 2019-09-09 lalitm@google.com trace_processor: migrate track table to new table macros 2019-09-09 lalitm@google.com trace_processor: make inserting into macro tables more explicit 2019-09-09 lalitm@google.com Merge "trace_processor: refactor Column to reduce boilerplate when adding types" 2019-09-09 hjd@google.com Merge "ui: Show full timestamp" Created with: gclient setdep -r src/third_party/perfetto@ac68e9c3dd09 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/+/master/autoroll/README.md TBR=perfetto-bugs@google.com Bug: chromium:135177627,chromium:135177627 Change-Id: If7b064422fdcbcddc16a9798adbc047f3dc9274e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792001Reviewed-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@{#694839}
-
Yun Liu authored
Revert this if capacity is not enough. Bug: 731759 Change-Id: Ia757461849ff14f277b23f3f6869e9d7c4e2bd61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790729Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Yun Liu <yliuyliu@google.com> Cr-Commit-Position: refs/heads/master@{#694838}
-
Mohammad Refaat authored
The problem is the fullscreen VC is presented the ContentView is stashed to another window to keep the renderer from being killed. Then when when that VC is dismissed the contentView is inserted on the top of the container view, and that will cover any other view that was presented on the container view. The solution is to insert the contentView on the back of the container view. Bug: 982845 Change-Id: I65e1cb20cb7cdbb6396678cbc3fc26108a932a2d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788242Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Commit-Queue: Mohammad Refaat <mrefaat@chromium.org> Cr-Commit-Position: refs/heads/master@{#694837}
-
Daniel Rubery authored
This CL marks the following histograms as obsolete: SBClientDownload.DownloadFileHasDetachedSignatures SBClientDownload.ZipFileFailureByType SBClientDownload.ZipFileSuccessByTyep Since they are historically stable and not being used It extends the lifetime of: SBClientDownload.ZipFileSuccess Since it shifted dramatically in M75. Bug: 988809 Change-Id: I3add7ec4972d6be4ac0f4492f91cdfb643c88619 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1784970Reviewed-by:
Bettina Dea <bdea@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Daniel Rubery <drubery@chromium.org> Cr-Commit-Position: refs/heads/master@{#694836}
-
Brandon Wylie authored
This will be helpful in methods such as typeInOmnibox, which rely on the assumption that the omnibox will have focus. Putting it in its own util file because it seems generally useful. Bug: 1000315 Change-Id: I7cb390046462c584ad48785963941fce133da0e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1784882 Commit-Queue: Brandon Wylie <wylieb@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#694835}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/2aaf66e4644a..ddef8d1b6b07 git log 2aaf66e4644a..ddef8d1b6b07 --date=short --no-merges --format='%ad %ae %s' 2019-09-09 titovartem@webrtc.org Add support of displaying video during the PC level test Created with: gclient setdep -r src/third_party/webrtc@ddef8d1b6b07 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 TBR=webrtc-chromium-sheriffs-robots@google.com Bug: None Change-Id: Ib363502b20bc8cf7ee95fb2dea2ec71b7a87cf44 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792447Reviewed-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@{#694834}
-
Brett Kilty authored
In cases where kAudioDecoderLimit is > 1, SFX are getting held to the 1 decoder limit. One decoder is really about video, as that is the platform limitation. Multiple sources may attempt to play sound effects at the same time as these are just audio only sources. Bug: b/139941019 Test: Local build, test race condition and SFX. Change-Id: Ieb7ea18674e58e06dc050f7ff2756e14021ab40d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792451 Commit-Queue: Brett Kilty <brettk@google.com> Reviewed-by:
Kenneth MacKay <kmackay@chromium.org> Cr-Commit-Position: refs/heads/master@{#694833}
-
Danan S authored
The Add Supervision dialog was using OnCloseContents() and OnDialogCloseRequested() to catch attempts to close the dialog, for the purpose of showing the signout confirmation dialog. This failed when OnCloseContents() was called, because by the time that callback was called, the web contents were already disposed of, resulting in dialog contents becoming unusable if the user chose to return to the dialog rather than signing out. Instead, capturing the close event in CanCloseDialog() occurs before the contents are invalidated, therefore it is the correct place to stop the closing of the dialog Bug: 1001732 Change-Id: I50262bd4606e3380700c81f98df925ea3a61f87c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790130 Commit-Queue: Danan S <danan@chromium.org> Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Cr-Commit-Position: refs/heads/master@{#694832}
-
Gabriel Charette authored
These tests were previously migrated from single-threaded MessageLoop to a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as part of crbug.com/891670. //base OWNERS decided in retrospect that it was better to keep a single-threaded option for TaskEnvironment and introduced SingleThreadTaskEnvironment. This CL retrofits that decision for /ui/events/ozone. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. R=spang@chromium.org Bug: 891670 Change-Id: I2d1f3a4f10cf7c57414a04b8a5ebcb9ead9e342a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786939 Auto-Submit: Gabriel Charette <gab@chromium.org> Commit-Queue: Michael Spang <spang@chromium.org> Reviewed-by:
Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#694831}
-
vikassoni authored
Enable AImageReader for both MCVD and MediaPlayer video paths on android by default. Bug: 951832,838725 Change-Id: I0efa453a3a9e0212fd9b9d5d55cb928bfa46a9c1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1789755Reviewed-by:
Frank Liberato <liberato@chromium.org> Reviewed-by:
Eric Karl <ericrk@chromium.org> Commit-Queue: vikas soni <vikassoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#694830}
-
Moe Ahmadi authored
- Queries AutocompleteController when the realbox gets focus and is empty. - Stops AutocompleteController and clears Autocomplete results when realbox loses focus. Otherwise previous non-zero-prefix suggestions remain cached. - Displays the clock icon for the personalized zero-prefix suggestions, if applicable. Bug: 996516 Change-Id: Ib821e0f6f0e5b722f60b6388456c673a7b8b450b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786592 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Dan Beam <dbeam@chromium.org> Reviewed-by:
manuk hovanesian <manukh@chromium.org> Cr-Commit-Position: refs/heads/master@{#694829}
-