- 10 Sep, 2019 40 commits
-
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/2ab2ac03d8ee..f57b7cb8f0bc git log 2ab2ac03d8ee..f57b7cb8f0bc --date=short --no-merges --format='%ad %ae %s' 2019-09-10 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/skia@f57b7cb8f0bc 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: I63d7a0786a5d4972a470f42c4655135b097fb25c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795986Reviewed-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@{#695271}
-
Eric Stevenson authored
This change was generated by //base/android/jni_generator/jni_refactorer.py. TBR: tedchoc@chromium.org Bug: 929661 Change-Id: I5a093f1f51876de014a2337e5ce711907109a502 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1782676 Commit-Queue: Eric Stevenson <estevenson@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Auto-Submit: Eric Stevenson <estevenson@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#695270}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/29ab487ea7c1..de5f63910e24 git log 29ab487ea7c1..de5f63910e24 --date=short --no-merges --format='%ad %ae %s' 2019-09-10 srte@webrtc.org Removes decoder thread fallback from VideoReceiveStream. Created with: gclient setdep -r src/third_party/webrtc@de5f63910e24 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: I0a6e401141ca2df371713a06229438d7c7454776 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795988Reviewed-by:
-
Sanket Joshi authored
Per the updated comps, the color picker should use a sans-serif font. Making that change in color_picker.css, and also updating baselines for pixel tests. Bug: 1001576 Change-Id: Iaa2a65ba574ad6f3e46aea82a009301273582723 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792790Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Mason Freed <masonfreed@chromium.org> Commit-Queue: Sanket Joshi <sajos@microsoft.com> Cr-Commit-Position: refs/heads/master@{#695268}
-
Mansi Awasthi authored
This CL adds the logic to enumerate over FPDF_LINKs and populate link annotations' data with its underlying text range in |links_| vector in PDFiumPage. A test is added to validate that the link annotations are correctly populated for a page. Bug: 981448 Change-Id: I6569e939c4a2309cc598f5062dec0e396ca818b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1736886Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Kevin Babbitt <kbabbitt@microsoft.com> Commit-Queue: Mansi Awasthi <maawas@microsoft.com> Cr-Commit-Position: refs/heads/master@{#695267}
-
Miguel Casas-Sanchez authored
This CL follows up on a comment [1], where instead of forcing clients to make the context current and then call either CreateSharedImage or UpdateSharedImage, it is suggested to have public methods bundling those operations in SharedImageStub. This CL does just that, making the current OnCreateGMBSharedImage and OnUpdateSharedImage use the newly introduced ones. [1] https://chromium-review.googlesource.com/c/chromium/src/+/1772642/30/media/gpu/linux/mailbox_video_frame_converter.cc#361 Bug: 998279 Change-Id: If39d94b04d2851e6ed8e248e9af96908dfa27e8e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1794274Reviewed-by:
Eric Karl <ericrk@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#695266}
-
Alex Clarke authored
We are trying to prioritize loading and composting tasks until FCP and when enabled this test fails. Upon investigation it turned out the test script was starting before the link tag resources had loaded. Starting the test in a window.onload handler fixes the issue. Bug: 971191 Change-Id: I9d4f99a5cac86970606dd6c9b4ab2416ccaac2ad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1782814Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Commit-Queue: Alex Clarke <alexclarke@chromium.org> Cr-Commit-Position: refs/heads/master@{#695265}
-
Mitsuru Oshima authored
* Added OnMouseEnter to PlatformWindow{Delegate} * Updated DRM/X11 impl to call it when the mouse location enters it. and chrome for chromeos on linux desktop. Bug: 990589 Test: X11WindowOzoneTest.MouseEnterAndDelete, manually tested on device Change-Id: I452c34ef0019f10d20d1096ef608cf84dd3a73db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761772 Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Reviewed-by:Scott Violet <sky@chromium.org> Reviewed-by:
Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#695264}
-
Yi Gu authored
This function is unused. Bug: None Change-Id: Ic0c39655a6ea8087dbfbf2150e4d1478163b06e5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795756Reviewed-by:
kylechar <kylechar@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#695263}
-
Yue Li authored
Move the status sync to prevent timing issues. Bug: None Test: Manual Test Change-Id: I2e2847c07a4125d830a423ab9e5c4e454f891de3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788176 Commit-Queue: Yue Li <updowndota@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#695262}
-
Dana Fried authored
Details are in the new, improved documentation. But MoveWebContentsAt() will now never leave a group in a non-contiguous state. Also, did some fixup in the unit tests to support testing of the new functionality. Bug: 915956 Change-Id: Ibd83249ebbaeb74e50f6805dc626c26c0291f86e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793544Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Dana Fried <dfried@chromium.org> Cr-Commit-Position: refs/heads/master@{#695261}
-
Eric Karl authored
This reverts commit 797ebfdc. Reason for revert: Fixed minor build issue causing initial revert. Original change's description: > Revert "Cleanup Android video targets" > > This reverts commit 9c44fedd. > > Reason for revert: broke https://ci.chromium.org/p/chromium/builders/ci/android-jumbo-rel/13515 > > Original change's description: > > Cleanup Android video targets > > > > Fixes WebView component build issues. > > > > Previous CLs introduced a number of new targets including > > texture_owner_android and shared_image_video. These were causing build > > problems due to duplicate file compilation which led to DCHECKs. > > > > Moved most of these files into gpu/command_buffer/service:gles2, as > > they all depended on this target (so no new transitive deps), and it's > > probably a good place for them for now and simplifies dependency > > management. > > > > Bug: 996285 > > Change-Id: Ic5a9cced7e67500cfd5fff8809d4ee5e17bbab8b > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787165 > > Reviewed-by: Frank Liberato <liberato@chromium.org> > > Reviewed-by: Nasko Oskov <nasko@chromium.org> > > Reviewed-by: vikas soni <vikassoni@chromium.org> > > Auto-Submit: Eric Karl <ericrk@chromium.org> > > Commit-Queue: Eric Karl <ericrk@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#695006} > > TBR=kinuko@chromium.org,nasko@chromium.org,liberato@chromium.org,ericrk@chromium.org,vikassoni@chromium.org > > Change-Id: Ib51ec09046ecfa51543836755b6b830d76fbce94 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 996285 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793931 > Reviewed-by: Tao Bai <michaelbai@chromium.org> > Commit-Queue: Tao Bai <michaelbai@chromium.org> > Cr-Commit-Position: refs/heads/master@{#695062} TBR=michaelbai@chromium.org,kinuko@chromium.org,nasko@chromium.org,liberato@chromium.org,ericrk@chromium.org,vikassoni@chromium.org Change-Id: I35661eb7691dbbacb063e308513daca68efcff75 Bug: 996285 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796110Reviewed-by:
vikas soni <vikassoni@chromium.org> Reviewed-by:
Tao Bai <michaelbai@chromium.org> Reviewed-by:
Frank Liberato <liberato@chromium.org> Reviewed-by:
-
Andrew Xu authored
In ShelfLayoutManagerTest::ShelfUpdatedWhenStatusAreaChangesSize, it assumes that the shelf view fills the display space. It will be incorrect under new specs. In ShelfLayoutManagerTest::RtlPlacement, the test fails to reset the RTL setting. This CL fixes these two tests. Bug: 1002576 Change-Id: I9dc2a82f5dab18c33e6947e21112af5aa4913f0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796118 Commit-Queue: Andrew Xu <andrewxu@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#695259}
-
Scott Violet authored
We can't use base in the public API, so this makes a copy of ObserverList. BUG=none TEST=none Change-Id: Ia346c8a5c11204c9ee8e77a2a0c2ca861f25543f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793782Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#695258}
-
Tommy Steimel authored
This CL adds a highlight to the GMC toolbar button when the in-product help is shown. Bug: 991585 Change-Id: I6911ba92d323e859bfb95e7d43173ef53a669d71 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790044Reviewed-by:
Collin Baker <collinbaker@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#695257}
-
Ilia Samsonov authored
The purpose of this cl is to prevent the tests from crashing on all platforms when running in a batch. The Task Runner outlives the PooledTaskRunnerDelegate, which it holds a reference too. This causes failure in batch test run. Bug: 1002573 Change-Id: I5fca10116e7b40a2b70bf867d55165b8b5825708 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1794303Reviewed-by:
Derek Schuff <dschuff@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#695256}
-
Vadym Doroshenko authored
This CL removes some leftovers of the old parser. Bug: 949519 Change-Id: I8f94cc9367752da83a20906b238f68eb5ea0a784 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1789539 Commit-Queue: Vadym Doroshenko <dvadym@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Auto-Submit: Vadym Doroshenko <dvadym@chromium.org> Cr-Commit-Position: refs/heads/master@{#695255}
-
David Black authored
The current value is 15ms. We may need to modify this behavior during experimentation if we find that this threshold is insufficient to give the user time to digest the event. Early feedback suggests that may be the case. Bug: b:140765359 Change-Id: I81586347b374342ef836980a69d86e820b1a0886 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793750 Commit-Queue: David Black <dmblack@google.com> Reviewed-by:
Tao Wu <wutao@chromium.org> Cr-Commit-Position: refs/heads/master@{#695254}
-
Kevin Marshall authored
If the HARDWARE_VIDEO_DECODER_ONLY Context flag is set, then only the video codecs which are decodable in hardware are reported as playable. * Adds "enable_software_video_decoder" GN flag for Cast Runner. * Defines new command line switch --disable-software-video-decoders for propagating the flag across ContextProvider/Browser and Browser/Renderer process boundaries. * Bypasses software codec factory registration if the switch is set. * Uses hardcoded temporary logic for defining which codecs are hardware accelerated; will be replaced with a check against the Fuchsia media codec service later. * Adds new browser tests to verify the effect of --disable-software-video-decoders. Bug: 1000858 Change-Id: I780bd38eacf7537bfb1ceb52db2bad5a4ed8a879 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786079 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Auto-Submit: Kevin Marshall <kmarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#695253}
-
Wez authored
Reduce the default Run() timeout set by TaskEnvironment from TestTimeouts::action_max_timeout() to action_timeout(). This typically reduces the timeout from 30s (45s in Debug) down to 10s. Sheriffs! This may result in: - Failures on slow builds, which are not on CQ. Please consider adding an #ifdef to restore those builds to action_max_timeout(), rather than simply reverting. - Flakiness in tests which are too close to the new timeout. Please consider whether such tests should really be taking so long, and whether they can be fixed-forward with a ScopedRunTimeoutForTest rather than reverting (see ThroughputAnalyzerTest in this CL). Bug: 922098 Change-Id: Icd75af79f6d9d3555c91b33ea2566fe7c56cd2f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548382 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Reviewed-by:
Maks Orlovich <morlovich@chromium.org> Auto-Submit: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#695252}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/884c81e1703f..1f562b7ad68c git log 884c81e1703f..1f562b7ad68c --date=short --no-merges --format='%ad %ae %s' 2019-09-10 perezju@chromium.org [Telemetry] Rename story_runner.Run to StorySetRun Created with: gclient setdep -r src/third_party/catapult@1f562b7ad68c 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:866458 Change-Id: I5de6f69bea96d124e27ea1e1968894e353bf70e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795985Reviewed-by:
-
Piotr Bialecki authored
Bug: 992033 Change-Id: I5d79658062e574b7c910af03b0ec0c838c169f05 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774659 Auto-Submit: Piotr Bialecki <bialpio@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Klaus Weidner <klausw@chromium.org> Commit-Queue: Piotr Bialecki <bialpio@chromium.org> Cr-Commit-Position: refs/heads/master@{#695250}
-
David Black authored
Previously the close button in the proactive suggestions view was 16dip as the view bounds matched the icon bounds. Now, the icon is still 16dip but the view bounds are increased to 32dip which is also the standard size across Assistant for buttons. Bug: b:140669269 Change-Id: I89e8fb9e2027553cec8fb6a8d605ce10ec4cdd67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792979Reviewed-by:
-
Chris Blume authored
The SkiaRenderer GL bot was accidentally added outside of alphabetical order. This CL rearranges the order to be alphabetical. Bug: 969855 Change-Id: I3e1183483ba7dcca1f4cccf084b062f41d3c6fe9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795668Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Chris Blume <cblume@chromium.org> Cr-Commit-Position: refs/heads/master@{#695248}
-
chrome-bot authored
Same change as https://crrev.com/c/1793050. This break affects all our UITests: https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=celab_tests Validation: https://cr-buildbucket.appspot.com/build/8902773145814860448 Change-Id: I3dd0efbc1dc797cd51bf587e9fd217385beba06c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791795Reviewed-by:
Owen Min <zmin@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Auto-Submit: Mathieu Binette <mbinette@google.com> Cr-Commit-Position: refs/heads/master@{#695247}
-
John Rummell authored
Moving the code that handles the CDM manifest into a separate file in chrome/common to support manifest parsing outside of the component updater. Bug: 889561,971433 Test: new unit_tests pass Change-Id: I1d6591834f01f1fd01daf5450f5cad08f4af2e26 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764249Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/c1ec0cbb921b..c9800e5adb7a Created with: gclient setdep -r src-internal@c9800e5adb7a 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: I11f333c4ae254ae9af70628c5e52127afcd699cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796109Reviewed-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@{#695245}
-
Alex Clarke authored
We want to prioritize loading and compositing tasks until first contentful paint. That wac casusing fast/replaced/replaced-breaking.html to flakily have image diffs because the test harness is taking screen shots before the plugins have finished loading. This patch makes the test wait until they have both loaded before finishing which fixes the flakes we'd have when turning on the new scheduler policy. Bug: 971191 Change-Id: I08f34cfefa5ffdec14741b7ab1cf8d83fc3e33b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795789Reviewed-by:
-
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 /services/device/public/cpp/power_monitor. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. R=bajones@chromium.org Bug: 891670 Change-Id: Ibb48e3b6a6c219f1c781e719f033d0ee88c33f1c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787099 Auto-Submit: Gabriel Charette <gab@chromium.org> Commit-Queue: Brandon Jones <bajones@chromium.org> Reviewed-by:
Brandon Jones <bajones@chromium.org> Cr-Commit-Position: refs/heads/master@{#695243}
-
dpapad authored
The code was adding a trailing space when the URL ends with "\", to work around https://bugs.webkit.org/show_bug.cgi?id=28885 which no longer seems to be an issue. Bug: None Test: Run https://jsfiddle.net/m7brn0q4/, verify that a network request to "foo/bar/?\" is sent. Change-Id: I351d60350ab43b99270edaef063da76b35955a1f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793789 Commit-Queue: Esmael Elmoslimany <aee@chromium.org> Auto-Submit: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#695242}
-
rbpotter authored
- Autogenerate Polymer 3 cr-toolbar, cr-toolbar-search-field, and cr-toolbar-selection-overlay - Autogenerate Polymer 3 cr-toolbar-search-field test - Add a cr-toolbar to the demo page Bug: 965770 Change-Id: Id51ce14f0a8812be7a858d5362c42f212d199f41 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790435 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
-
Christy Chen authored
Two strings for the checkboxes in Layers panel are not localized. This CL wrap them with ls, and include the strings in the GRDP. https://imgur.com/fbrbvKY Bug: 941561 Change-Id: Iab4e3bfaa791dbb9542133109367284465b59d98 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1779170Reviewed-by:
Erik Luo <luoe@chromium.org> Commit-Queue: Christy Chen <chrche@microsoft.com> Cr-Commit-Position: refs/heads/master@{#695240}
-
Adithya Srinivasan authored
Test flakily fails on TSAN, see bug for more details. Bug: 1002602 Change-Id: I0b68f636f601d2a138297ce0b2da49dc2883304a Tbr: peter@chromium.org No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796385Reviewed-by:
Adithya Srinivasan <adithyas@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#695239}
-
Jenny Zhang authored
Bug: 1001708 Change-Id: I036e5546d84a60fc9248f785c1801bc8d88eef57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793945 Commit-Queue: Jenny Zhang <jennyz@chromium.org> Reviewed-by:
Tony Yeoman <tby@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/e5641be5fe30..d9c1c856c64c git log e5641be5fe30..d9c1c856c64c --date=short --no-merges --format='%ad %ae %s' 2019-09-10 raul@tambre.ee fix_encoding: Decode bytestrings on Python 3 Created with: gclient setdep -r src/third_party/depot_tools@d9c1c856c64c 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:984182 Change-Id: I7518842e1bdcf77ce98f9087790a06e121238964 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796107Reviewed-by:
-
Nico Weber authored
There are a few existing violations. Cleaning these up doesn't look super involved; that's tracked at crbug.com/1002509 Bug: 912946, 1002509 Change-Id: I58d3334040e90967c6614bf17f705cd43d5c5b0b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1780884 Auto-Submit: Nico Weber <thakis@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#695236}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/507f43465b33..29ab487ea7c1 git log 507f43465b33..29ab487ea7c1 --date=short --no-merges --format='%ad %ae %s' 2019-09-10 philipel@webrtc.org Revert "Removes string support in field trial parser." Created with: gclient setdep -r src/third_party/webrtc@29ab487ea7c1 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: I804344bc4330d9a6226ec1b7c4fc01e7cd6d3f0d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795565Reviewed-by:
-
Himanshu Jaju authored
For users who have sync turned off on their current device, we show a button to direct them to the settings page to enable sync. Light mode - https://drive.google.com/open?id=1hYpPBCBQdAfxTsw0a13lS2aBSsvTrqk6 Dark mode - https://drive.google.com/open?id=1FYts7p-9W7duyv9Gqs5XEReJT8j3yYnd Bug: 1001102 Change-Id: I0ad3de82739b74541a06249447ad5c7c72b3ae23 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795426 Commit-Queue: Himanshu Jaju <himanshujaju@chromium.org> Reviewed-by:
Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Cr-Commit-Position: refs/heads/master@{#695234}
-
David Black authored
HTTP headers from proactive suggestion responses may be unescaped. These strings can be shown in both the proactive suggestions UI as well as in Assistant UI. We need to unescape these strings before displaying them to the user. Bug: b:140645078 Change-Id: I967c85a67e230bb86f029f517147b4b1b1fbd3ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790479Reviewed-by:
Asanka Herath <asanka@chromium.org> Reviewed-by:
-
Brian Sheedy authored
Adds the option to run tests using Python3 instead of Python2 to allow for tests to be migrated over in preparation for the Python2 deprecation. This can be achieved by adding a "'use_python3': True" entry to the test's gn_isolate_map.pyl entry. Bug: 898348 Change-Id: I675ec068e18cec6bf59ea9d19802bd712f18e87e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769719Reviewed-by:
Marc-Antoine Ruel <maruel@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
-
