- 29 Aug, 2019 40 commits
-
-
Ioana Stefan authored
The option to schedule tasks at exact times will be added to the BackgroundTaskScheduler. Because these types of tasks will be scheduled with AlarmManager, additional information needs to be added to the BackgroundTaskScheduler storage. This CL makes the necessary changes to the Protocol Buffer that stores a scheduled task. Bug: 970160 Change-Id: Ica396c0fa3ec27b1ffe50f99d56e0f5c83ac38fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765850Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
Mugdha Lakhani <nator@chromium.org> Commit-Queue: Ioana Stefan <ioanastefan@chromium.org> Cr-Commit-Position: refs/heads/master@{#691778}
-
Tricia Crichton authored
When running in mobile emulation mode, SwitchToWindow will apply mobile emulation mode the the new window or tab. Wrap setup of headless_download_dir in a check that the browser is in headless mode, so that headless_download_dir is only set for headless mode. Add is_headless flag for browser_info to enable simpler checking for headless state. Bug: chromedriver:2126 Change-Id: I2134a824c58f1017ab30f2d034e53b7c6d59a7bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774902 Commit-Queue: Tricia Crichton <triciac@chromium.org> Reviewed-by:
John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#691777}
-
Thomas Guilbert authored
This reverts commit 9033c719. Reason for revert: failing on "Linux MSan Tests" bot Original change's description: > Notification scheduler: Add IHNR buttons metrics. > > This CL adds the metrics to track IHNR buttons clicks and show. > > Bug: 963298 > Change-Id: I33ef62df25a9873b4ec814f5730c191289bc7691 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772045 > Reviewed-by: David Trainor <dtrainor@chromium.org> > Reviewed-by: Mark Pearson <mpearson@chromium.org> > Reviewed-by: Hesen Zhang <hesen@chromium.org> > Commit-Queue: Xing Liu <xingliu@chromium.org> > Cr-Commit-Position: refs/heads/master@{#691708} TBR=mpearson@chromium.org,dtrainor@chromium.org,xingliu@chromium.org,hesen@chromium.org Change-Id: I6749f0a947319a11491f5bfaf772936927103d2e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 963298 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776407Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Commit-Queue: Thomas Guilbert <tguilbert@chromium.org> Cr-Commit-Position: refs/heads/master@{#691776}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/c43852aa1834..324b391559e7 git log c43852aa1834..324b391559e7 --date=short --no-merges --format='%ad %ae %s' 2019-08-29 m.maiya@samsung.com Vulkan: Fix compute shader bug affecting 10_10_10_2 formats Created with: gclient setdep -r src/third_party/angle@324b391559e7 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 cwallez@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 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=cwallez@chromium.org Bug: None Change-Id: I73ea19fe9ff95820d13049d1c523a88b4b8f2d7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776328Reviewed-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@{#691775}
-
Xiaocheng Hu authored
After crrev.com/c/1769554, |LengthInterpolationFunctions| has become a trivial wrapper of InterpolableLength. Hence, this patch removes it to reduce code complexity. The patch is basically mechanical, with some manual simplifications and notes added. Bug: 991672 Change-Id: I8892973520d884f1e70b5f9155799c7d771a7276 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772389 Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#691774}
-
ssid authored
Slow telemetry bots seem to be hitting the timeout because startup takes longer. So, increase the timeout. BUG=993799 Change-Id: I6897f78e1d848b944ce355a7e38428e9fb546fed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772391Reviewed-by:
oysteine <oysteine@chromium.org> Commit-Queue: ssid <ssid@chromium.org> Cr-Commit-Position: refs/heads/master@{#691773}
-
ssid authored
The flag could be used along with perfetto-output-file flag. Change-Id: Idd1c7069a0e2e29b3c33d60da02434c8b01af1ad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774039 Commit-Queue: ssid <ssid@chromium.org> Reviewed-by:
oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#691772}
-
Julian Watson authored
Bug: 992293 Change-Id: Ife84571e87b2b021c4b34f3dffe3b72cce9ceaa3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772795Reviewed-by:
Christopher Thompson <cthomp@chromium.org> Reviewed-by:
Joel Hockey <joelhockey@chromium.org> Commit-Queue: Julian Watson <juwa@google.com> Cr-Commit-Position: refs/heads/master@{#691771}
-
JeffreyCA authored
Prior to this change, placeholder text in input fields was set to grey (#757575), but this did not apply to emojis. Now, we use an opacity value to control the placeholder appearance. 0.54 is used because in a white input field, regular placeholder text still appears as #757575. Bug: 932026 Change-Id: Iba02fe3eb31484255332f2f34a67f97b97f50cb8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1754086 Commit-Queue: Jeffrey C <jeffreyca16@gmail.com> Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
Alice Boxhall <aboxhall@chromium.org> Cr-Commit-Position: refs/heads/master@{#691770}
-
Lijin Shen authored
This CL includes instrumentation tests and render tests for the tab switcher long-press menu. Bug: 994364 Change-Id: I47014f00c41005f38517cbe5ed8884cef1ab5daf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769308 Commit-Queue: Lijin Shen <lazzzis@google.com> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#691769}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/022a7c8d4980..25eb47ccf18f git log 022a7c8d4980..25eb47ccf18f --date=short --no-merges --format='%ad %ae %s' 2019-08-29 tommi@webrtc.org Make the RtpHeaderParserImpl available to tests and tools only. Created with: gclient setdep -r src/third_party/webrtc@25eb47ccf18f 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: chromium:none Change-Id: I6f43e6abdfc68afa2898968048b2043c193b583f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776388Reviewed-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@{#691768}
-
Josh Nohle authored
Adds metrics to track the execution times of async function calls during the CryptAuth v2 DeviceSync flow. These values will be used for tuning timeout values. Bug: 951969, 933656 Change-Id: I9669429b0faf845f1f16fb82b53ba884cb4ab7fa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1743096 Commit-Queue: Josh Nohle <nohle@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#691767}
-
Evan Stade authored
This removes the notification from about half of the tests where it's used. Bug: 268984 Change-Id: I9478164ccaac46b07a57f8f80d43f053c2675506 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773845Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#691766}
-
Tarun Bansal authored
Add hardcode denylist for DeferAllScript lite page. URL of the webpage is matched against a pre-compiled regex. The purpose of this change is to land a merge-able change to exclude many login and some banking pages from the preview treatment. Change-Id: I125d3e66d4bb4efbfbfef3e19f8acebd7ce9e01b Bug: 999242 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775103Reviewed-by:
Doug Arnett <dougarnett@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#691765}
-
Max Moroz authored
Some buildbots are executing `ninja all` and fuzz targets should not be attempted to be built there unless certain GN flags are enabled. Bug: 999096 Change-Id: I11addbe7ab00a5ef8c57559bd84a1ec4b287f212 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775756 Commit-Queue: Max Moroz <mmoroz@chromium.org> Reviewed-by:
Matt Mueller <mattm@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#691764}
-
Jordan Demeulenaere authored
This CL fixes a bug introduced in http://crrev/c/1710898: when showing a bottom sheet content that has its own toolbar, the default toolbar visibility was set to gone. After that CL, the default toolbar visibility is not changed anymore. This causes a visual bug in the peek state when the content toolbar height is smaller than 56dp. Before: https://screenshot.googleplex.com/2kHm6NpPZRx.png After: https://screenshot.googleplex.com/AKx4yVYssyz.png Change-Id: I1b35b17fe7c5eb8d257f0ceab4a565f3b4182724 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774451 Auto-Submit: Jordan Demeulenaere <jdemeulenaere@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#691763}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/b1f411ac053d..3783375c4d41 git log b1f411ac053d..3783375c4d41 --date=short --no-merges --format='%ad %ae %s' 2019-08-29 herb@google.com Only send the RemoteStrikes that have pending glyphs. Created with: gclient setdep -r src/third_party/skia@3783375c4d41 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 stani@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=stani@google.com Bug: None Change-Id: I8a62b56d3ecc2e26ca4f57eee9887804f9519340 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776387Reviewed-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@{#691762}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/8dd0eb227a3f..793b6c3d0b6b git log 8dd0eb227a3f..793b6c3d0b6b --date=short --no-merges --format='%ad %ae %s' 2019-08-29 capn@google.com Disable Vulkan projects in Visual Studio solution build Created with: gclient setdep -r src/third_party/swiftshader@793b6c3d0b6b 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/139189696 Change-Id: Ifef23440066867918f6d5ebf17fa3ed3e74f5965 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776386Reviewed-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@{#691761}
-
James Cook authored
When the user manually selects a timezone we keep showing the autodetect method menu, but disable it. This makes the UI shift around less than the old approach, which was to hide the menu. If the user chooses a custom timezone the menu is disabled and it shows the default (IP address) label. This means the label may switch from the WiFi option to the IP address option if the user toggles back and forth between the "Set automatically" and "Choose from list" radio buttoms. Our PM Jesse is OK with this. If policy disables automatic timezones the menu is disabled and the label is 'Automatic time zone detection is disabled'. Before: - automatic: https://screenshot.googleplex.com/TsyNYwjg0HS - manual: https://screenshot.googleplex.com/1eTPHgBqOwF - policy: https://screenshot.googleplex.com/9ZWzTHaawra After: - automatic doesn't change - manual: https://screenshot.googleplex.com/4bf2aG4FwJt - policy: https://screenshot.googleplex.com/KqPh3q7pwQT Bug: 997118 Test: updated browser_tests Change-Id: Ieb5f3d4df887082e981518bddd895ecd8bbf5fd2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769234Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#691760}
-
Andrew Xu authored
Add focus traversal to ScrollableShelfView Bug: 973481 Change-Id: I3beeea53aff1ed32ccb588f2033ab67851590574 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1767484 Commit-Queue: Andrew Xu <andrewxu@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#691759}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/2fe742d3173c..4d747ae4be07 Created with: gclient setdep -r src-internal@4d747ae4be07 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: If6b9a255140e953d20df02b0dc18eaa208eda060 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776385Reviewed-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@{#691758}
-
Hector Carmona authored
No UI change, only behavior on which menu to show. Both menu and selection are pre-existing UI. Bug: 910412 Change-Id: I7335ef5b2e5675b6ff0de5bde14dc5144a4d7311 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772442 Commit-Queue: Hector Carmona <hcarmona@chromium.org> Reviewed-by:
calamity <calamity@chromium.org> Cr-Commit-Position: refs/heads/master@{#691757}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/04fc7733eda6..8aaeffd8edef git log 04fc7733eda6..8aaeffd8edef --date=short --no-merges --format='%ad %ae %s' 2019-08-29 tcwang@google.com afdo: Fix orderfile-generate-toolchain builder. Created with: gclient setdep -r src/third_party/chromite@8aaeffd8edef If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@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:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I6496ccc53c70f422334e448e3719b50ab92df5aa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774932Reviewed-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@{#691756}
-
Kevin Marshall authored
This is a reland of 830d60ac. Fixes a test linker error by adding a export directive. Adds some missing #includes. TBR=nasko@chromium.org TBR=jam@chromium.org Original change's description: > [fuchsia] Add support for loading embedder content via fuchsia-dir://. > > Defines a new scheme for loading resources from named embedder > Directories. Resources from these directories can be loaded using > the URL: > > fuchsia-dir://<directory-name>/<path>/<to>/<file>.html. > > * Browser tests added for testing URLLoaderFactory & URLLoader logic. > * Integration tests added to test ContextProvider changes. > > Bug: 991298 > Change-Id: I5875594d9fe5cc7f9006ccdecda78cc3bdf667b3 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764335 > Commit-Queue: Kevin Marshall <kmarshall@chromium.org> > Reviewed-by: Wez <wez@chromium.org> > Cr-Commit-Position: refs/heads/master@{#691363} Bug: 991298 Change-Id: I1a74d7d3936bcc5666e3c5d814079838cac7afac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775805 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#691755}
-
Ryan Hamilton authored
to force the OS to pick an unused port and hence avoid flakiness in QUIC EndToEndTests. Bug: 997325 Change-Id: I1c6eecac8e599863ab60077f1692d209fe055dc5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776603 Commit-Queue: Ryan Hamilton <rch@chromium.org> Commit-Queue: Zhongyi Shi <zhongyi@chromium.org> Auto-Submit: Ryan Hamilton <rch@chromium.org> Reviewed-by:
Zhongyi Shi <zhongyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#691754}
-
Angel Alvarez authored
Bug: 977713 Change-Id: I54b01d443cf8f4335569e9f741a648bfc3c5cff1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769720Reviewed-by:
Justin DeWitt <dewittj@chromium.org> Reviewed-by:
Cathy Li <chili@chromium.org> Commit-Queue: Angel Alvarez <angelii@google.com> Cr-Commit-Position: refs/heads/master@{#691753}
-
Jeffrey Kardatzke authored
This fixes a bug in the uploader where it would post a task with a callback that had a raw pointer, and if it was during shutdown then the referenced object could be destructed before the call on said pointer was invoked. The simple part of the change is using a WeakPtr in that location instead of a raw pointer, so the callback will be skipped. However, the way the code worked was that it would read the pre-existing feedback report from disk to a string, then it would pass that into the callback which asynchronously generated a new feedback report in a new disk location and then it would delete the old feedback report file. However, if that callback is suppressed (which it would be in the case the new WeakPtr becomes invalidated) then we would end up deleting that feedback report from disk without sending it. So this code also changes things to not recreate the report on disk but instead reuses that existing file for the feedback report. Bug: 994846 Test: components_unittests pass and manually verified report upload Change-Id: Ib64ce1abbfea23e94e45387fa5a296817bf19a30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766766 Commit-Queue: Jeffrey Kardatzke <jkardatzke@google.com> Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Cr-Commit-Position: refs/heads/master@{#691752}
-
Matt Menke authored
https://chromium-review.googlesource.com/c/chromium/src/+/1758546 is the fix for the bug itself. Bug: 994537 Change-Id: I7850325a19fd41e8b08040a69201df9251a78690 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758717 Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by:
Zhongyi Shi <zhongyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#691751}
-
Antonio Gomes authored
... where ever possible. Note that some instances of std::vector still remain, because they are from method signatures' that override //third_party/webrtc APIs, or call out to //media APIs. BUG=787254 R=guidou@chromium.org Change-Id: I358bf8b23109612e3e3f604029c4ad83b06c0297 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775873Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#691750}
-
Josh Nohle authored
TimeTicks are better suited for measuring time durations, specifically for tracking the amount of time a task runs. See comments at the top of https://cs.chromium.org/chromium/src/base/time/time.h. Bug: 899080, 933656 Change-Id: I9892e1690b194b9c91ea091a895555a88a45173a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775209 Commit-Queue: Josh Nohle <nohle@chromium.org> Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Auto-Submit: Josh Nohle <nohle@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#691749}
-
Vasiliy Telezhnikov authored
This CL enables GL_APPLE_sync for Skia when we force it using ES2.0 on ES3.0 context. Bug: 987286 Change-Id: If66d625a548ea486dbb4f376d91a55990d6760e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774931Reviewed-by:
Jonathan Backer <backer@chromium.org> Commit-Queue: Vasiliy Telezhnikov <vasilyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#691748}
-
Avery Musbach authored
Change-Id: Icad584da8b83f653695c22c5417de67c4e0301b5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772425Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Commit-Queue: Avery Musbach <amusbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#691747}
-
chrome://proximity-authJosh Nohle authored
We fix the closure compiler errors on the page. Bug: 993373 Change-Id: If2b884ddb55efa27fd2c2c1c764fe1c6dda80dcd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1752246 Commit-Queue: Josh Nohle <nohle@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#691746}
-
Antonio Gomes authored
... torward its WTF counterparts. BUG=787254 R=guidou@chromium.org, haraken@chromium.org Change-Id: I207a37af8451dd9705c9834c9649e062de68a487 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775866 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#691745}
-
Henrique Nakashima authored
Change-Id: I18976200d45536fc91706a1beaad0c3b4dade11b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775440Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Cr-Commit-Position: refs/heads/master@{#691744}
-
Nico Weber authored
Bug: 961769 Change-Id: Iac5fa643cd60ca8308dd050b7a7f58677f3d824b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776224 Commit-Queue: Nico Weber <thakis@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Auto-Submit: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#691743}
-
Ken Rockot authored
This moves the Resource Coordinator Service off of Service Manager APIs in favor of direct Mojo interface usage. A public browser API is exposed to give content clients access to the service connection. Bug: 977637 Change-Id: I898228ad49e3be405e0bb746d00c23d057658680 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769102 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Yuchen Liu <yucliu@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#691742}
-
Eric Foo authored
This is CL is generated by a script. Data is pulled from review conducted in Q2 2019. See https://tinyurl.com/yxau5sc4 for context. Bug: 977050 Change-Id: Ic56ae0f036156dc160eb72da75b1003585daaca4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1699958Reviewed-by:
Jungshik Shin <jshin@chromium.org> Reviewed-by:
Kyle Milka <kmilka@chromium.org> Commit-Queue: Eric Foo <efoo@chromium.org> Auto-Submit: Eric Foo <efoo@chromium.org> Cr-Commit-Position: refs/heads/master@{#691741}
-
David Black authored
Per UX request, we shouldn't show proactive suggestions during the same login session if we've already shown them to the user. Duplicates are defined as having the same content description. Note that this has been implemented as a FeatureParam to give us a lever to disable this behavior if needed. Bug: b:139199754 Change-Id: I39e78529159ffd697383b98b23b4986f0d3a10ec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774968Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: David Black <dmblack@google.com> Cr-Commit-Position: refs/heads/master@{#691740}
-
Vasiliy Telezhnikov authored
This CL fixes potential inconsistency between backend format we used to create promise image and one in fallback image in case we'll need one. This causes internal checks in Skia to fail. Bug: 998876 Change-Id: I5d7e25472c392c42b5b9f766ca525e965d0f4a4d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776206 Commit-Queue: Vasiliy Telezhnikov <vasilyt@chromium.org> Reviewed-by:
Jonathan Backer <backer@chromium.org> Cr-Commit-Position: refs/heads/master@{#691739}
-