- 29 Aug, 2019 40 commits
-
-
rbpotter authored
Update tools/polymer/polymer.py so that files with content wrapped in: (() => { ... })() are processed the same way as files with content wrapped in: (function() { ... })() Also add a test. Bug: 965770 Change-Id: I55e7e2bf76e243fbac100ac8e1368fe599df03df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775747Reviewed-by:Dan Beam <dbeam@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#691800}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/3783375c4d41..e26413128902 git log 3783375c4d41..e26413128902 --date=short --no-merges --format='%ad %ae %s' 2019-08-29 senorblanco@chromium.org Dawn: Implement RenderPipeline caching (optimization). 2019-08-29 egdaniel@google.com Fix updating of current task index in GrResourceAllocator 2019-08-29 bsalomon@google.com Make GrSurfaceProxy derive from GrNonAtomicRef and remove GrIORefProxy. 2019-08-29 fmalita@chromium.org [skottie] Refactor/generalize text selector shape generators 2019-08-29 brianosman@google.com Move common particle code to an automatically-injected header 2019-08-29 senorblanco@chromium.org Dawn: submit all command buffers in a single call. Created with: gclient setdep -r src/third_party/skia@e26413128902 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: chromium:996610 Change-Id: I1de2bea4464eaa35a2dbd114922bd4e89efe1f8c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776590Reviewed-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@{#691799}
-
Adam Langley authored
Previously, having the caBLE transport in the list of transports was special and indicated that a caBLE discovery should be created. Also, unlike all over discoveries, the caBLE discovery was created in the GetAssertion request handler, not the base class. This change makes the caBLE transport no longer special because, in the future, we'll want to create a caBLE discovery without the extension. It also regularises the handling of the caBLE transport to match all other transports. As a consequence, the caBLE discovery data becomes the property of the FidoDiscoveryFactory and two tests become inapplicable at the level at which they were being tested. Change-Id: Ibcd758fa9304a7cb8a868b1bcf2c692c15f72221 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1756951 Auto-Submit: Adam Langley <agl@chromium.org> Reviewed-by:
Martin Kreichgauer <martinkr@google.com> Commit-Queue: Adam Langley <agl@chromium.org> Cr-Commit-Position: refs/heads/master@{#691798}
-
John Rummell authored
Rather than including the CDMs in the base install directory, move them into subdirectories (e.g. Widevine would be located in <installdir>/WidevineCdm/_platform_specific/linux_x64/libwidevinecdm.so). This allows the LICENSE and manifest.json file to be bundled with Chrome on Linux. Future CL's will use the manifest.json file to determine the capabilities of the CDM (rather than having them hardcoded as they are today). In addition to the unittests below, ran EME browser_tests (both ECK and Widevine) to check that the library can be loaded from it's new directory, and built "installer" (although I don't know if whatever package it creates is installable). Bug: 971433 Test: CdmPath media_unittests pass Change-Id: If87f97e3200341941438ccbcbb4a918b1e5d0b50 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1767596Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Michael Moss <mmoss@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Commit-Queue: John Rummell <jrummell@chromium.org> Cr-Commit-Position: refs/heads/master@{#691797}
-
Etienne Pierre-doray authored
Bug: 839091 Change-Id: I50f4edc74be0cbb197ed26281744e0c85f46f322 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776227 Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#691796}
-
Matthew Denton authored
Bug: 975708 Change-Id: I01eebebbd0a049ac9f71ef119fe3763a25e0a122 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776679 Commit-Queue: Matthew Denton <mpdenton@chromium.org> Reviewed-by:
Christopher Thompson <cthomp@chromium.org> Cr-Commit-Position: refs/heads/master@{#691795}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@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:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: I9bb41f6167702b8491b68dd87fa867b9255618ff Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775113Reviewed-by:
-
Andrey Kosyakov authored
... to assure DevTools protocol callbacks are always invoked (and destroyed) on the correct thread. Change-Id: Ie94c67d754138161ba22192bb76891f4021ad280 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775501 Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#691793}
-
François Doray authored
Prior to https://chromium-review.googlesource.com/c/chromium/src/+/1706829, PendingTask::delayed_run_time was set by SequenceManager for non-delayed tasks when "set queue time for tasks" was enabled. The jank histograms depend on that behavior, and have regressed since it changed. To ensure that we can track of the evolution of jank over time and detect regressions, this CL restores the previous behavior of setting PendingTask::delayed_run_time for non-delayed tasks when "set queue time for tasks" is enabled. We could explore changing that behavior in a separate CL. Test: Launch Chrome on a fast machine. After 30 seconds, navigate to chrome://histograms. The value recorded for Browser.Responsiveness.JankyIntervalsPerThirtySeconds.Startup should be equal or very close to zero. Bug: 997203 Change-Id: I60649e9415579e62a2fe2206c8307ed7a4e66668 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775660Reviewed-by:
Alexander Timin <altimin@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#691792}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/18b2cb51d7b8..f018c05e44f0 git log 18b2cb51d7b8..f018c05e44f0 --date=short --no-merges --format='%ad %ae %s' 2019-08-29 rharrison@chromium.org Rolling 5 dependencies 2019-08-29 beaufort.francois@gmail.com Make vertex input descriptor optional Created with: gclient setdep -r src/third_party/dawn@f018c05e44f0 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dawn-chromium-autoroll Please CC cwallez@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:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel TBR=cwallez@google.com Bug: None Change-Id: I8c862e36917ef9ce1b099d8ae02cd37931d4626f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776584Reviewed-by:
-
Steven Bennetts authored
This also renames kThirdPartyVPN -> kExtension and kArcVPN -> kArc This is in preparation for merging vpn_list.mojom into cros_network_config and using it in Ash and Settings. Bug: 979314 Change-Id: I2a74156080d59a4ae2f757c8410e01036dedcd68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775319Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#691790}
-
Martin Kreichgauer authored
This changes MakeCredentialTask such that requests returned by GetTouchRequest() get converted into a CTAP2 command if the authenticator supports CTAP2. Currently, all touch-only requests are downgraded to U2F. That breaks with certain types of FIDO2 security keys that require a fingerprint even for U2F operations, but seem to honor that an empty pinAuth parameter implies getting a touch only (even if a fingerprint has been set up). Also change GetTouchRequest() to set the pinProtocol parameter. At least the CTAP2.0 spec doesn't seem to provide any kind of guarantees for pinAuth without pinProtocol (§5.1 only mentions the case of the supplied pinProtocol being "unsupported" by the device). CTAP2.1 has an explicit mention of the "empty pinAuthUvParam" case in 5.1, but doesn't mention whether pinUvAuthProtocol should be set or not. Empirically, it doesn't make a difference. Tested on my collection of authenticators by doing the following: - reset - create() with uv=required (should collect a touch, then set a PIN if CTAP2; or collect the touch, then fail if U2F-only) - if CTAP2, create() uv=required again (should collect a touch, then prompt the PIN). Bug: 998281 Change-Id: Ifaa96ef1ea8f12ee2791967d454cf0e78c438f4f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773723 Commit-Queue: Martin Kreichgauer <martinkr@google.com> Reviewed-by:Adam Langley <agl@chromium.org> Cr-Commit-Position: refs/heads/master@{#691789}
-
Gayane Petrosyan authored
screenshot: https://screenshot.googleplex.com/9q3asZAtYiH.png Bug: 998460 Change-Id: I021aa83013cce477564de5205ce9339172518728 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775056Reviewed-by:
Kyle Milka <kmilka@chromium.org> Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Cr-Commit-Position: refs/heads/master@{#691788}
-
Matt Falkenhagen authored
The thread ServiceWorkerContextCore lives on (the "core thread") will move from the IO thread to the UI thread when the ServiceWorkerOnUI feature is enabled. This CL makes Content Index aware of the core thread instead of assuming the IO thread. This makes ContentIndex* browser_tests pass when the feature is enabled. Bug: 824858 Change-Id: I03e81a564b27ee888a3402fdff11d8b51f3489a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775882Reviewed-by:
Rayan Kanso <rayankans@chromium.org> Commit-Queue: Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#691787}
-
Bailey Berro authored
- Opens a new tab to chrome://settings/ClearBrowserData when clicking "Browser Data" in OS-Settings > Device > Storage. Bug: 998012 Change-Id: I0dd740b08725d8befbef9bd1c1aa5b6b0f0b6814 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775207 Commit-Queue: Bailey Berro <baileyberro@chromium.org> Auto-Submit: Bailey Berro <baileyberro@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#691786}
-
Xianzhu Wang authored
They are first converted with a script in https://chromium-review.googlesource.com/c/chromium/src/+/1772308, then checked, tested and modified manually. Make layer list mode the default mode for LayerTreeHostCommonTest. Tests still in layer tree mode are under LayerTreeHostCommonTestWithLayerTree. Some tests can't be converted into layer list mode because: - They are testing features that for ui compositor; - They are testing PropertyTreeBuilder; or Also haven't converted tests that create layer tree in main-side, and build property trees with the main-side PropertyTreeBuilder. The tests still in layer tree mode using impl-side PropertyTreeBuilder will be converted to use main-side PropertyTreeBuilder. 109 of 167 tests have been converted. Bug: 994361 Change-Id: I04444c861c6f0e8aefa71b6cff2879b54429bc1e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775746Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#691785}
-
Hari Nandagopal authored
- Prevent race condition in CastAudioOutputStream to prevent FillNextBuffer() from calling AudioSourceCallback::OnMoreData() after AudioOutputStream::Close() is called. - FillNextBuffer() will now acquire running_lock_ for the entirety of its scope. - CastAudioOutputStream::Close() will synchronously block on running_lock_ until FillNextBuffer() has completed before setting running_ to false and continuing its close sequence. - Subsequent calls to FillNextBuffer() will early return without calling AudioSourceCallback::OnMoreData(). - Add infrastructure in CastAudioOutputStream unittests to pause and resume the audio thread. - Add a test in CastAudioOutputStream unittests to ensure that a call to Close() will synchronously prevent any further callbacks. mutex synchronization is removed, race condition is difficult to reproduce in real-world testing. Bug: 138695782 Test: Builds, unit tests pass, new unit test passes and fails when the Change-Id: I11cdd64c0987298ba3fc6a9090359bd2715e6f03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761483Reviewed-by:
Olga Sharonova <olka@chromium.org> Reviewed-by:
Kenneth MacKay <kmackay@chromium.org> Reviewed-by:
Yuri Wiitala <miu@chromium.org> Commit-Queue: Hari Nandagopal <hnandagopal@google.com> Cr-Commit-Position: refs/heads/master@{#691784}
-
Chris Harrelson authored
It's only called in unittests. Bug: 992765 Change-Id: I1e5ef83618d089e4ff2d3571a7d26482d8e69974 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775807 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#691783}
-
Yue Li authored
The AssistantStateProxy is only used by Assistant service now, move it from Ash to Service and move pref related logics into the Proxy. Bug: b/138679823 Test: Run existing tests Change-Id: Ife1ded72321c5d0357870a57ffcd15d94b930655 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774833Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Yue Li <updowndota@chromium.org> Cr-Commit-Position: refs/heads/master@{#691782}
-
Eric Stevenson authored
This reverts commit 0817d44b. No changes other than rebasing. The issue with the original CL was fixed by crrev.com/c/1775268 (verified locally running monochrome_public_bundle). Original change's description: JNI refactor: @NativeMethods conversion (//chrome/android/features). To create this change: * cd chrome/android/features * base/android/jni_generator/jni_refactorer.py -R --nonstatic * base/android/jni_generator/jni_refactorer.py -R * Change the native side of VrShellDelegateJni#recordVrStartAction to not pass the Java object TBR=agrieve@chromium.org Binary-Size: More native methods now in base module. See bug/doc. Bug: 929661 Change-Id: I3ce3404bee4516da68028d3946dc8bfc33b8a807 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775754 Commit-Queue: Eric Stevenson <estevenson@chromium.org> Reviewed-by:
Eric Stevenson <estevenson@chromium.org> Reviewed-by:
Ben Mason <benmason@chromium.org> Cr-Commit-Position: refs/heads/master@{#691781}
-
Sophie Yang authored
This change fixes crbug/998959 by setting inSystemMenu_ to false when the system menu (i.e. the right-click or context menu) is removed from the page. Bug: 998959 Change-Id: I391b634217513247c4f4215f5a47ffb028886bc5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1775381Reviewed-by:
Anastasia Helfinstein <anastasi@google.com> Reviewed-by:
Akihiro Ota <akihiroota@chromium.org> Commit-Queue: Sophie Yang <sophyang@google.com> Cr-Commit-Position: refs/heads/master@{#691780}
-
Robert Sesek authored
Also remove support for Mac OS X 10.9, which is no longer supported. Bug: 998881 Change-Id: I8aca9c7e43ec2555f4b19f77c94db8ecabb75f2f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776218Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#691779}
-
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:
-
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:
-
Julian Watson authored
Bug: 992293 Change-Id: Ife84571e87b2b021c4b34f3dffe3b72cce9ceaa3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772795Reviewed-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:
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:
-
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:
-
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:
-
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 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:
-
