- 06 Aug, 2018 40 commits
-
-
webrtc-chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/0047bce0a943..eb73a7bd16c7 git log 0047bce0a943..eb73a7bd16c7 --date=short --no-merges --format='%ad %ae %s' 2018-08-06 srte@webrtc.org Removes unnecessary webrtc_cc namespaces. 2018-08-06 srte@webrtc.org Adds feedback only mode to GoogCC. 2018-08-06 srte@webrtc.org Removes pause check from RoundRobinPacketQueue. 2018-08-06 nisse@webrtc.org Delete RtpReceiverImpl::CheckPayloadChanged. Created with: gclient setdep -r src/third_party/webrtc@eb73a7bd16c7 The AutoRoll server is located here: https://webrtc-chromium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng BUG=chromium:None TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: Ia5a1c0fae4a6ff6711c30cb20ff970c823cc9fc2 Reviewed-on: https://chromium-review.googlesource.com/1163846Reviewed-by:
webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#580994}
-
Ryan Sleevi authored
Bug: 855690 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I9029b2c3fe66cfb56423ee3fc6e8590547401533 Reviewed-on: https://chromium-review.googlesource.com/1163926Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: Ryan Sleevi <rsleevi@chromium.org> Cr-Commit-Position: refs/heads/master@{#580993}
-
Yuwei Huang authored
HostListService::state_ is uninitialized when the instance is created. For some reason it hasn't caused any flakiness in the current unit test set, but I can see it start failing when I try to add new tests. This CL initializes state_ to State::NOT_FETCHED. Bug: 871443 Change-Id: I7eb3d69483baeb8ae0918122b5ba61300705be87 Reviewed-on: https://chromium-review.googlesource.com/1164181Reviewed-by:
Jamie Walch <jamiewalch@chromium.org> Commit-Queue: Yuwei Huang <yuweih@chromium.org> Cr-Commit-Position: refs/heads/master@{#580992}
-
Artur Khachatryan authored
In case if something went wrong and no tests were run, we should return non zero. Change-Id: I5645ac78bba01ce2f88c3fb88d9a0155c176a79c Reviewed-on: https://chromium-review.googlesource.com/1164078Reviewed-by:
John Chen <johnchen@chromium.org> Commit-Queue: Artur Khachatryan <khachatryan@chromium.org> Cr-Commit-Position: refs/heads/master@{#580991}
-
Mounir Lamouri authored
webkitEnterFullscreen() and webkitEnterFullScreen() are rarely used and could be removed. The exit counter parts are significantly more common. The UKM may help us identify the top sites using these methods and drive their usage down. Bug: 745944 Change-Id: Iec937c4f851049355b49bb59bb4d4e32ab8445f9 Reviewed-on: https://chromium-review.googlesource.com/1163835Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#580990}
-
Muyuan Li authored
Bug: b/111133529 Test: None Change-Id: I4ea3ae74c90eed999eaa6231d283438be0ec65c2 Reviewed-on: https://chromium-review.googlesource.com/1164073Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: Muyuan Li <muyuanli@chromium.org> Cr-Commit-Position: refs/heads/master@{#580989}
-
Lukasz Anforowicz authored
Verifications inside BrowserTest.OtherRedirectsDontForkProcess test were not applicable to site-per-process mode (e.g. they assumed that a cross-site pop-up, or a cross-site top-level navigation would always reuse the same process). This CL tweaks the test, so that it has different expectations with and without site-per-process mode. Bug: 824962 Change-Id: I1b5dfec0c45a8d578d64d4b259e28497d33b1ca9 Reviewed-on: https://chromium-review.googlesource.com/1162894 Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#580988}
-
Fritz Koenig authored
This reverts commit 7a570fae. Re-enable underlays after fixing bugs BUG=854790, 789288 TEST=UnderlayTest Change-Id: I57b42832327924ff568b5dd3da3aadeb780424c2 Reviewed-on: https://chromium-review.googlesource.com/1161485 Commit-Queue: Fritz Koenig <frkoenig@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#580987}
-
Bence Béky authored
This is a lot of work just to avoid using CompletionCallback in HttpCache::PendingOp. Of course one could have used CompletionRepeatingCallback instead, but that is not idiomatic, since the callback is only meant to be called once. No functional change is intended. I wonder why PendingOps cannot be deleted in HttpCache destructor. I guess Backend or WorkItem must outlive HttpCache for whatever reason. I decided it is better not to introduce functional change here. Bug: 807724 Change-Id: Idd7e304702ff31de11073c0a77d041f678fb6ae4 Reviewed-on: https://chromium-review.googlesource.com/1161971 Commit-Queue: Bence Béky <bnc@chromium.org> Reviewed-by:
Josh Karlin <jkarlin@chromium.org> Cr-Commit-Position: refs/heads/master@{#580986}
-
Tessa Nijssen authored
The SuggestedTextTouchBarController was renamed to TextSuggestionsTouchBarController. Any instances of SuggestedTextTouchBarController were updated to the new name. The following were also changed: - File names were modified as follows: */{suggested_text => text_suggestions}_* - BUILD files were updated to reflect the new file names - Tests of the TouchBarController were renamed - Flags relating to the TouchBarController were renamed Bug: 717553 Change-Id: Ie2749ed0942362c7e89fd924441b4eac9397ce53 Reviewed-on: https://chromium-review.googlesource.com/1162895Reviewed-by: -
Ben Smith authored
There are restrictions to sharing SharedArrayBuffer (and WebAssembly Modules) between execution contexts; for example, you cannot share a SharedArrayBuffer between a Window and the Service Worker it created. The rules are described here: https://html.spec.whatwg.org/multipage/webappapis.html#integration-with-the-javascript-agent-cluster-formalism This defines an "agent" which is similar to an execution context, and an "agent cluster" which is a collection of agents that can share between each other. This CL begins to model some of this behavior by using a `base::UnguessableToken` as an agent cluster ID that can be queried from an `ExecutionContext`. This CL also has the correct behavior when a message is sent to an agent that is not part of its agent cluster; in that case a "messageerror" event is sent instead of a "message" event. Bug: chromium:798572, chromium:714842 Change-Id: Ie70cce4cbd0ebd04d8d270d66f59690caf7f616a Reviewed-on: https://chromium-review.googlesource.com/1130505 Commit-Queue: Ben Smith <binji@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#580984}
-
bsheedy authored
Changes "Mac FYI Experimental Release (Intel)" from 10.13.5 to 10.13.6. Bug: 870875 Change-Id: Icb5507246b4e2398ac85eedd8c1e25051a963597 Reviewed-on: https://chromium-review.googlesource.com/1163807Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#580983}
-
Robert Liao authored
Setting #upcoming-ui-features previously locked you into Material Refresh. If you have a touchable device, you probably want Material Refresh Touchable. This change also allows users to choose between Material Refresh and Material Refresh Touchable if #upcoming-ui-features is set. BUG=870002 Change-Id: I8006fcaab8c6f8da299b77fafb9e3418f18bbfa5 Reviewed-on: https://chromium-review.googlesource.com/1159239Reviewed-by:
Allen Bauer <kylixrd@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#580982}
-
Sigurdur Asgeirsson authored
Also drop the relevant RC feature flags and UKM measurement code. Bug: 755840 Change-Id: I055e5e7fa7f316aaaa9d48defa3450148b912700 Reviewed-on: https://chromium-review.googlesource.com/1163729Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Chris Hamilton <chrisha@chromium.org> Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#580981}
-
Clark DuVall authored
These tests needed some extra synchronization due to the changes made in http://crrev.com/c/1139048. These were flaking in component builds. Bug: 871224 Change-Id: Ibe370f9b9c8ebb94ee1dd21ba204c06e91518057 Reviewed-on: https://chromium-review.googlesource.com/1163987 Commit-Queue: Clark DuVall <cduvall@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#580980}
-
David Tseng authored
During touch exploration, we often hit test to unknown nodes. Change-Id: I1f627c59045500c98ad5a63f1524a91d66ad2357 Reviewed-on: https://chromium-review.googlesource.com/1162701 Commit-Queue: David Tseng <dtseng@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#580979}
-
Johannes Henkel authored
InnerEnable does the actual work, but doesn't set the flag, that only happens when enabled() is called (the protocol method). It'd be ok to set it anyway, it doesn't cause traffic on the wire, but I think it's a bit clearer to protect against forgetting to set it with a DCHECK. So this fixes the bug you pointed out when reviewing https://chromium-review.googlesource.com/c/chromium/src/+/1162931 and keeps the change seperate. Bug: 851762 Change-Id: Ibd65a7e69539a760af6499be3f5ff96df6b3912b Reviewed-on: https://chromium-review.googlesource.com/1163808 Commit-Queue: Johannes Henkel <johannes@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#580978}
-
Ken Buchanan authored
Currently all RenderFrameProxys instantiate ChildFrameCompositingHelper objects, even though they are only used for a minority of them -- in particular, only RenderFrameProxys that have RenderFrameImpl parents will ever have to add layers to the layer tree in that process. This CL avoids instantiating them in cases where a RenderFrameProxy is parented under another RenderFrameProxy. Change-Id: Ie789d0331d3aaab17ae531601e00770d786345d7 Reviewed-on: https://chromium-review.googlesource.com/1162694Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: Ken Buchanan <kenrb@chromium.org> Cr-Commit-Position: refs/heads/master@{#580977}
-
Charlie Andrews authored
The story has recently started failing. NOTRY=true TBR=nednguyen@chromium.org CC=kouhei@chromium.org,ksakamoto@chromium.org Bug: 871421 Change-Id: Id7806641d8827f7036ce213296a42e8b8f801e3a Reviewed-on: https://chromium-review.googlesource.com/1164165Reviewed-by:
Charlie Andrews <charliea@chromium.org> Commit-Queue: Charlie Andrews <charliea@chromium.org> Cr-Commit-Position: refs/heads/master@{#580976}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/93f9ef7fc26f..58bf2845eb4d git log 93f9ef7fc26f..58bf2845eb4d --date=short --no-merges --format='%ad %ae %s' 2018-08-06 chiniforooshan@chromium.org Telemetry: fix Gesture IRs Created with: gclient setdep -r src/third_party/catapult@58bf2845eb4d The AutoRoll server is located here: https://catapult-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. 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 BUG=chromium:760553 TBR=sullivan@chromium.org Change-Id: I0efc80b015e459ebf5381d95e49b902b19ca6104 Reviewed-on: https://chromium-review.googlesource.com/1163844Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#580975}
-
Alexey Kozyatinskiy authored
Everywhere on the frontend side, we use both events and process them in the same way. R=lushnikov@chromium.org Bug: none Change-Id: I3ab9b196921ff9e7e8eac86e9217b0869f0c2313 Reviewed-on: https://chromium-review.googlesource.com/1163838 Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org> Reviewed-by:
Andrey Lushnikov <lushnikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#580974}
-
Ahmed Fakhry authored
This reverts commit e4378cb9. Reason for revert: breaks BuildPackages on amd64-generic-goma-canary-chromium-pfq-informational BUG: chromium:871429 Original change's description: > media/gpu: add mojo initialization to unittests. > > Future changes to shared memory introduce an implicit dependency on > mojo, which must be initialized in unittests. > > Bug: 849207 > 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 > Change-Id: I13de882b7f995ac33385d2f6d5f49b5ac44bb2c4 > Reviewed-on: https://chromium-review.googlesource.com/1156588 > Reviewed-by: Miguel Casas <mcasas@chromium.org> > Reviewed-by: Dan Sanders <sandersd@chromium.org> > Commit-Queue: Matthew Cary <mattcary@chromium.org> > Cr-Commit-Position: refs/heads/master@{#580922} TBR=posciak@chromium.org,mcasas@chromium.org,sandersd@chromium.org,mattcary@chromium.org Change-Id: Iaa263cfca7e99b36c609db6b4a81dbd56d1a9be3 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 849207 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 Reviewed-on: https://chromium-review.googlesource.com/1164182Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Cr-Commit-Position: refs/heads/master@{#580973}
-
Nate Chapin authored
This better communicates the responsible party for security checks. You can definitely know which LocalFrame is associated with a DocumentLoader, but a LocalFrame may be associated with multiple DocumentLoaders. Also, remove some dead code and inline a couple of methods. Change-Id: I563a4259981bd61fd5a19b154cdd59f2aed46d24 Reviewed-on: https://chromium-review.googlesource.com/1159604 Commit-Queue: Nate Chapin <japhet@chromium.org> Reviewed-by:
-
Xiaohui Chen authored
Test: build and run locally, stop assistant do not show animation Bug: b:112229074 Change-Id: I7a669f9a6bf2d11d84a6c3a364c7c4f729068eb1 Reviewed-on: https://chromium-review.googlesource.com/1163841Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#580971}
-
angle-chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/a6d34af67286..6ba9754f4a56 git log a6d34af67286..6ba9754f4a56 --date=short --no-merges --format='%ad %ae %s' 2018-08-06 geofflang@chromium.org Refactor index range calculation in DrawCallParams. Created with: gclient setdep -r src/third_party/angle@6ba9754f4a56 The AutoRoll server is located here: https://angle-chromium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. 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=ynovikov@chromium.org Change-Id: Ie123b90795a1951821e966b7606dbd763bc34300 Reviewed-on: https://chromium-review.googlesource.com/1162723Reviewed-by:
angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#580970}
-
Ben Pastene authored
This adds a --deploy option to the vm test runner and binds it to true every time we build the cros_vm_launcher. This assumes that everything that uses that target also builds chrome and also wants it deployed to the VM (which is true atm). Also rebalance telemetry unit test shard count on the FYI bot. Bug: 836447 Change-Id: I012932edac7bf40c369f69378e3ad365dd9e1ab6 Reviewed-on: https://chromium-review.googlesource.com/1155833 Commit-Queue: Ben Pastene <bpastene@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#580969}
-
Ted Choc authored
UrlBar#setAllowFocus was not updating the widget if it had not drawn a single time. This internally was done because tabbed mode wanted to prevent he UrlBar from gaining focus on initial inflation, but tabbed mode never calls setAllowFocus except when manipulating the tab switcher and thus would never happen before drawn a single time. This just updates setAllowFocus to immediately take action and the lazy setting of focus for Tabbed mode still happens in the first draw callback (this method can just be set before that has happened, which is OK for both SearchWidget and Custom Tabs as they both want to set that state as early as possible). BUG=871121 Change-Id: I74ef57890e223a10583ecbb67179705fb2f11caa Reviewed-on: https://chromium-review.googlesource.com/1163754Reviewed-by:
Yusuf Ozuysal <yusufo@chromium.org> Commit-Queue: Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#580968}
-
Charlie Andrews authored
Many similar stories are already disabled on A1. NOTRY=true TBR=panicker@chromium.org Bug: 764868 Change-Id: Iaccf98bdaf53bedbedfbe24169a5d577789ac1ab Reviewed-on: https://chromium-review.googlesource.com/1164028Reviewed-by:
-
Chris Harrelson authored
Previously, code in LayoutBox::StyleDidChange which tried to detect change of border for clips did not take into account changes of border style to hidden or none. Bug: 814439 Change-Id: Ic59985c1b7b877cbbee678930328444d7700274e Reviewed-on: https://chromium-review.googlesource.com/1163220Reviewed-by:
vmpstr <vmpstr@chromium.org> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#580966}
-
Weidong Guo authored
Shelf height is changed unintendedly in https://chromium-review.googlesource.com/c/chromium/src/+/1142441, so change it back. Bug: 865139 Change-Id: Iec2de46cf6e48ed9bde0e5cd736e5d7154c6d9c4 Reviewed-on: https://chromium-review.googlesource.com/1163986Reviewed-by:
-
Maggie Chen authored
This test is disabled due to a regression on Mac FYI Retina Debug (NVIDIA) bot since build 2964 NOTRY=true Bug: 871352 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 Change-Id: Ida3ec7f78ee39c2578f0ee802101559de11c92d9 Reviewed-on: https://chromium-review.googlesource.com/1163993 Commit-Queue: Maggie Chen <magchen@chromium.org> Reviewed-by:
-
Tonko Sabolčec authored
This CL includes: - Add new base::Feature: kDeleteUndecryptableLogins. - Expand metric enum PasswordSyncState with failed decryption & cleanup states. - Call LoginDatabase::DeleteUndecryptableLogins when merging sync data. - Add tests for new feature. Bug: 791541 Change-Id: I5a5f9cea618efdc4b467eee4bb54674be8574b84 Reviewed-on: https://chromium-review.googlesource.com/1160486 Commit-Queue: Tonko Sabolčec <tsabolcec@google.com> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#580963}
-
Eugene But authored
testActivityServiceControllerPrintAfterRedirectionToUnprintablePage is only valid for native error pages, because web-based error pages are not printable. Updated comments, disabling condition and added DCHECK to BVC. testActivityServiceControllerCantPrintUnprintablePages can be rewritten to use Offline Version from Reading List. Updated comments. Bug: 864597 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I6f9ff464abed30024eb10ad9a187b424ccaa2663 Reviewed-on: https://chromium-review.googlesource.com/1163747Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#580962}
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/439da2812f74..97d0970c25e9 git log 439da2812f74..97d0970c25e9 --date=short --no-merges --format='%ad %ae %s' 2018-08-06 brianosman@google.com Fix SkColor4Shader 2018-08-06 benjaminwagner@google.com [infra] Shuffle Mac CPU and xSAN jobs to MacBookPro11.5. 2018-08-06 angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/angle2 a6d34af67286..6ba9754f4a56 (1 commits) 2018-08-06 fmalita@chromium.org [Skottie] Add raw data factory 2018-08-06 recipe-roller@chromium.org Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/skia@97d0970c25e9 The AutoRoll server is located here: https://autoroll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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=djsollen@chromium.org Change-Id: If5269a0f0ba01ef602ba9ccdd88ad0141c733cf1 Reviewed-on: https://chromium-review.googlesource.com/1163842Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#580961}
-
Dale Curtis authored
BUG=871354 TEST=none TBR=tsunghung Change-Id: I8134ea1476c7a25f9dc8d00dc1e62a4cd951ee93 Reviewed-on: https://chromium-review.googlesource.com/1163988Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#580960}
-
pdfium-chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/a7a957f7cec2..f3a3393a2f96 git log a7a957f7cec2..f3a3393a2f96 --date=short --no-merges --format='%ad %ae %s' 2018-08-06 thestig@chromium.org Avoid invalid object numbers in CPDF_Parser::LoadCrossRefV5(). 2018-08-06 tsepez@chromium.org Small optimization in CJBig2_Image::SubImage() 2018-08-06 thestig@chromium.org Check for null object type in CPDF_Parser::LoadCrossRefV5(). 2018-08-06 thestig@chromium.org Do more CPDF_Parser::LoadCrossRefV5() cleanup. 2018-08-06 rharrison@chromium.org Clean up a nit that was identified after CQing an earlier CL Created with: gclient setdep -r src/third_party/pdfium@f3a3393a2f96 The AutoRoll server is located here: https://pdfium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. BUG=chromium:865272,chromium:871042,chromium:863098 TBR=dsinclair@chromium.org Change-Id: I244334140b9abd5f018f052799d278320c51e5fc Reviewed-on: https://chromium-review.googlesource.com/1163845Reviewed-by:
pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#580959}
-
Xiaohan Wang authored
This reverts commit 030c9cd1. We have fixed the issue in the latest Widevine CDM and this test should pass now. Original change's description: > Revert "media: Reenable TestPolicyCheck() on Windows for Widevine" > > This reverts commit 3a127077. > > Reason for revert: The test is still failing on Win64 trunk and Win trunk on official continuous desktop builder. > > Original change's description: > > media: Reenable TestPolicyCheck() on Windows for Widevine > > > > The original problem has been fixed in the latest Widevine CDM. > > > > Bug: 851027 > > Change-Id: I9aced3ea2152e33fcc8a5bcddc8a587c45dee6cc > > Reviewed-on: https://chromium-review.googlesource.com/1154087 > > Reviewed-by: Frank Liberato <liberato@chromium.org> > > Commit-Queue: Xiaohan Wang <xhwang@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#579615} > > TBR=xhwang@chromium.org,liberato@chromium.org > > Change-Id: I1446eb5919ac014420fe7095d7916ee92cbb3d57 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 851027 > Reviewed-on: https://chromium-review.googlesource.com/1158767 > Reviewed-by: Xiaohan Wang <xhwang@chromium.org> > Commit-Queue: Xiaohan Wang <xhwang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#579823} TBR=xhwang@chromium.org,liberato@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 851027 Change-Id: I3ad5c9f79aeab0ed7bfa68ded3a49664c0c295d2 Reviewed-on: https://chromium-review.googlesource.com/1162213 Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#580958}
-
Fritz Koenig authored
Make sure to damage the previous underlay area when switching underlays so that it is repainted. BUG=854790 TEST=UnderlayTest.UpdateDamageWhenChangingUnderlays Change-Id: Idec0e30e0cf65e4937f85dfd3bdfd9ba4120d5fe Reviewed-on: https://chromium-review.googlesource.com/1161393 Commit-Queue: Fritz Koenig <frkoenig@chromium.org> Reviewed-by:
-
depot-tools-chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/82bb756217cd..2ebf9fdade33 git log 82bb756217cd..2ebf9fdade33 --date=short --no-merges --format='%ad %ae %s' 2018-08-06 dgarrett@google.com my_activity: Don't crash if email unset. Created with: gclient setdep -r src/third_party/depot_tools@2ebf9fdade33 The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. BUG=chromium:None TBR=agable@chromium.org Change-Id: I98d7b857515ed74324520306b76d8e79425126be Reviewed-on: https://chromium-review.googlesource.com/1163116Reviewed-by:
depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#580956}
-
yoshiki iguchi authored
This CL renames these classes: - MessageCenterNotificationManager -> NotificationUIManagerImpl - MessageCenterNotificationManagerTest -> NotificationUIManagerTest - MessageCenterNotificationsTest -> NotificationUIManagerBrowserTest - MessageCenterNotificationManagerBrowserTest -> NotificationUIManagerInteractiveUITest MessageCenterNotificationManager was confusing, because this class is not for CrOS (which has message center) but for Win/Mac/Linux (popup only). This CL renames this class into NotificationUIManagerImpl, because this is an implementation of NotificationUIManager. And this also renames the related test classes accordingly. Bug: 869278 Test: trybots pass Change-Id: I296f46d0b8634c95081167d6585f6f42798661f6 Reviewed-on: https://chromium-review.googlesource.com/1158669 Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org> Reviewed-by:Evan Stade <estade@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#580955}
-
