- 02 Oct, 2018 40 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/146eb3bdcf5b..a77bb2e54b35 git log 146eb3bdcf5b..a77bb2e54b35 --date=short --no-merges --format='%ad %ae %s' 2018-10-02 alanbaker@google.com Add validation for execution modes Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@a77bb2e54b35 The AutoRoll server is located here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll 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. TBR=dsinclair@chromium.org Change-Id: Ic098caa47274dd4af11d79272383ab7c95ef2502 Reviewed-on: https://chromium-review.googlesource.com/1256079Reviewed-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@{#595872}
-
Mustaq Ahmed authored
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:win_optional_gpu_tests_rel Change-Id: I81d8e42cfe0b1affe5568c694f3526df6e273fc8 TBR: jbroman@chromium.org Reviewed-on: https://chromium-review.googlesource.com/1252261 Commit-Queue: Mustaq Ahmed <mustaq@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#595871}
-
Andrey Kosyakov authored
Network conditions management is now done in content, so support for this in headless is redundant. This is a re-land of https://chromium-review.googlesource.com/c/chromium/src/+/1255610 fix network_dp.{h,cc} being removed from BUILD.gn. Change-Id: I6903c58f72bfc20588bf795da7b9f2e647dd0801 Reviewed-on: https://chromium-review.googlesource.com/1255726Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#595870}
-
Chromium WPT Sync authored
Using wpt-import in Chromium a2be8456. With Chromium commits locally applied on WPT: 92605c16 "[Unified Plan] Remote MediaStreamTracks should be muted by default." a9d61b8e "Implement RTCIceTransport.onselectedcandidatepairchange" 3e89d0d3 "Revert "Implement RTCIceTransport.onselectedcandidatepairchange"" 8c84af9b "Update beacon WPTs" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/25751 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: bjonesbe@adobe.com: external/wpt/css/css-shapes jsbell@chromium.org: external/wpt/resources NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: Iba3bbb64fe5b0f19b175d6b72ca85372cedd92c1 Reviewed-on: https://chromium-review.googlesource.com/1256627 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#595869}
-
Clark DuVall authored
Browser initiated requests were getting the "main_frame" type in the web request listener details, this gives them the correct "other" type. Bug: 888672 Change-Id: Ie5e24c6f51c21ad57babb710e54cc5af47bb02c2 Reviewed-on: https://chromium-review.googlesource.com/1256562Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#595868}
-
Hans Wennborg authored
The new Clang warning points out that the class can't be copy assigned because the current_ member is const. Copy or move constructing it is fine though, and that's all that's needed. Bug: 890307 Change-Id: I3f4d5e69485b84166ba4dd2356cc7973a5e58da6 Reviewed-on: https://chromium-review.googlesource.com/1255613Reviewed-by:
-
Devlin Cronin authored
Move the mediaGalleries API out of extensions code and into apps code. This includes the API implementation, schema definition, features entries, and API permission info. Bug: 877240 Change-Id: I8b750143cdcd36700f36d9da088ebaec4abd4cfe Reviewed-on: https://chromium-review.googlesource.com/1255584Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#595866}
-
Florin Malita authored
Manual rebaseline for some layout tests left in a bad state after https://chromium-review.googlesource.com/c/chromium/src/+/1254341. TBR=reed Bug: 891230 Change-Id: I8e2c43c972d63f1d3eeb44ecbcaeffea2242fb34 Reviewed-on: https://chromium-review.googlesource.com/1256085Reviewed-by:
Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org> Cr-Commit-Position: refs/heads/master@{#595865}
-
Mugdha Lakhani authored
Bug: 616321 Change-Id: I6a9f570f7f100fa8e1515f7736b6294644483f83 Reviewed-on: https://chromium-review.googlesource.com/1234475 Commit-Queue: Mugdha Lakhani <nator@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Peter Conn <peconn@chromium.org> Cr-Commit-Position: refs/heads/master@{#595864}
-
Torne (Richard Coles) authored
Several places in the code convert between Java byte arrays and C++ std::string, and some had implemented a local helper function for this. Add a generic helper for this to base instead, and convert existing places that do this to use the helper. Test: base_unittests Change-Id: Ia56fb6333bd6c453207814b0bcb5ac58b5488c20 Reviewed-on: https://chromium-review.googlesource.com/1255342 Commit-Queue: Richard Coles <torne@chromium.org> Reviewed-by:
Pavel Yatsuk <pavely@chromium.org> Reviewed-by:
John Rummell <jrummell@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#595863}
-
Fabio Tirelo authored
This was requested during UI review for the Autofill settings toggle split, because the new proposed order (passwords, payment methods, addresses) is more useful. Screenshot linked in the bug. Bug: 891305 Change-Id: Iabd587538b54fd579837c4aec8c762d91e1cfb8d Reviewed-on: https://chromium-review.googlesource.com/1256084Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Fabio Tirelo <ftirelo@chromium.org> Cr-Commit-Position: refs/heads/master@{#595862}
-
Scott Violet authored
And replaces it with a boolean. This makes more sense as EMBED_IN_OWNER is really something that changes depending upon whether the Window has an embedding or not. BUG=834487 TEST=covered by tests Change-Id: If5d46331b1c54e329247996f3fc012448435aabf Reviewed-on: https://chromium-review.googlesource.com/1239371 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Cr-Commit-Position: refs/heads/master@{#595861}
-
Mason Freed authored
There is a Windows-only limit on stack trace depth of 62. It seems to only apply to Server 2003 and XP, though it might exist for later OSes. See https://msdn.microsoft.com/en-us/library/bb204633.aspx for reference. I've left the limit at 62 for all of Windows in this CL. This limit does not (seem to) apply to other operating systems, so I've relaxed the limit to 250. The prior 62 limit does impede some real debugging efforts, as our stack depth can grow significantly past 62 in some cases, for example when performing layout on a deeply nested page. For example, see crbug.com/877093, which quotes the 62/63 limit. In addition, testing seems to show that Android has a problem with larger limits also, so I've left the Android limit at 62. Bug: 877093 Change-Id: Ie6521aa5da5e577cf28a312791fba873df6930cc Reviewed-on: https://chromium-review.googlesource.com/1252002 Commit-Queue: Mason Freed <masonfreed@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#595860}
-
Jordan Demeulenaere authored
Chrome Java style discourages usage of List<> and prefers to use ArrayList<>. Returning an ArrayList explicitly allows callers to pass this list in a method that requires an ArrayList as input. Bug: None Change-Id: Icbb4d3f5b0c9d38b654dc73bb3c2d3dc935427c1 Reviewed-on: https://chromium-review.googlesource.com/1256795 Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#595859}
-
Antonio Gomes authored
InitDataReductionProxySettings constructs DataReductionProxyService, which later on is provides data to DataReductionProxyIOData to construct DataReductionProxyConfig. Ultimately, DataReductionProxyConfig constructs WarmupURLFetcher instances. This CL is a preparation CL for [1], where we migrate WarmupURLFetcher from URLFetcher to SimpleURLLoader. In [1], |url loader factory| is mandatory to be non-null. Passing an actual instance at the beginning of the call chain now has no immediate effect, but is harmless too. [1] https://crrev.com/c/1251641 BUG=879777 Change-Id: I3f3009e6e6d9db9659e7783cbed65a1a82c6a34e Reviewed-on: https://chromium-review.googlesource.com/1256945Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#595858}
-
Etienne Pierre-doray authored
This CL uses ScopedBlockingCall to mark blocking calls in /chrome/browser/apps/app_shim/app_shim_host_manager_mac.mm. This CL was created by replacing calls to AssertBlockingAllowed() with instantiations of ScopedBlockingCall(MAY_BLOCK). I kindly ask the reviewer to make sure of the following: - ScopedBlockingCall is instantiated in a scope with minimal CPU usage. If this is not the case, ScopedBlockingCall should be instantiated closer to the blocking call. See scoped_blocking_call.h for more info. Please let me know when/where the blocking call happens if this needs to be changed. - Parameter |blocking_type| matches expectation (MAY_BLOCK/WILL_BLOCK). See BlockingType for more info. While I assumed MAY_BLOCK by default, that might not be the best fit if we know that this callsite is guaranteed to block. - The ScopedBlockingCall's scope covers the entirety of the blocking operation previously asserted against by the AssertBlockingAllowed(). This CL was uploaded by git cl split. R=thakis@chromium.org Bug: 874080 Change-Id: I449d164d700298ec166451458a5f9b135514156d Reviewed-on: https://chromium-review.googlesource.com/1190933Reviewed-by:ccameron <ccameron@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org> Cr-Commit-Position: refs/heads/master@{#595857}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/73d117f64ebb..1ac95546dd12 git log 73d117f64ebb..1ac95546dd12 --date=short --no-merges --format='%ad %ae %s' 2018-10-02 eladalon@webrtc.org Include optional.h in rtc_event_log_parser_new.cc 2018-10-02 nisse@webrtc.org Delete VideoReceiveStream::EnableEncodedFrameRecording 2018-10-02 danilchap@webrtc.org Add RtpPacket::GetRawExtension function Created with: gclient setdep -r src/third_party/webrtc@1ac95546dd12 The AutoRoll server is located here: https://autoroll.skia.org/r/webrtc-chromium-autoroll 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;luci.chromium.try:mac_chromium_archive_rel_ng BUG=chromium:None TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I3266b39da17f5a3fd64798d6874dc1a39a197049 Reviewed-on: https://chromium-review.googlesource.com/1256904Reviewed-by:
-
Stephane Zermatten authored
Before this patch, scripts were checked for the url from DidFinishLoad. This generally works, because most URLs loaded from a page belong to the same domain, but not all. In some cases, depending on loading order, this resulted in the set of runnable script being cleared unexpectedly. With this patch, the controller always uses the page URL. In most cases, since DidFinishLoad reports loading resources, this reports the same URL, wehich is a no-op for GetOrCheckScripts. The set of runnable scripts should now be stable on a given page. Bug: 806868 Change-Id: Ifb8f5955c00396a9d2165eb0bacf73c8c825f76d Reviewed-on: https://chromium-review.googlesource.com/1257002Reviewed-by:
-
Max Morin authored
Reasonably, nullptr isn't a frame, so it cannot be deleted. This call also isn't necessary, since the destructor of |factory_| is going to clean everything up soon anyways. Adds a DCHECK to ForwardingAudioStreamFactory::FrameDeleted to assert that |render_frame_host| is non-null. Bug: 734672 Change-Id: Ide1679e93923b70182498f57b92eb21f08f8e5f6 Reviewed-on: https://chromium-review.googlesource.com/1256962Reviewed-by:
Olga Sharonova <olka@chromium.org> Commit-Queue: Max Morin <maxmorin@chromium.org> Cr-Commit-Position: refs/heads/master@{#595854}
-
Daniel Bratell authored
In some builds there were duplicate symbols because a cc file was compiled twice and linked with itself. This happened because some code accidentally included the cc file instead of the header. Change-Id: Ifa145ce13190a9fa855e06986ca60605005aa9fc Reviewed-on: https://chromium-review.googlesource.com/1255824 Commit-Queue: Nektarios Paisios <nektar@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#595853}
-
Theresa authored
Rather, wait to record until the user exits the tab switcher to try to record. The user must exist the tab switcher on the tab for which results were returned. Also logs a histogram indicating whether results were returned while the user was in the tab switcher so that we can ballpark estimate adjustments needed to M69 metrics. BUG=890366 Change-Id: Ic5726907911df15d4bc6a5cbd97633fcf7415595 Reviewed-on: https://chromium-review.googlesource.com/1252627Reviewed-by:
Filip Gorski <fgorski@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#595852}
-
Sylvain Defresne authored
Replace them with uses of TestingFactory which is based on base::Callback<> instead of function pointers. This converts uses in /chrome/browser/bookmarks. This CL was uploaded by git cl split. R=sky@chromium.org Bug: 809610 Change-Id: I465d088e3eba92a29a1c03b1e73fb4ceb920a1e3 Reviewed-on: https://chromium-review.googlesource.com/1245730 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#595851}
-
Alexandr Ilin authored
GpuMemoryBufferHandle::offset and GpuMemoryBufferHandle::stride are not initialized in the default constructor. These uninitialized members may be serialized later by mojo that makes MSAN bot unhappy. Bug: 890930 Change-Id: I0663de62997d673b97907a8bd470e9834bdaf638 Reviewed-on: https://chromium-review.googlesource.com/1256786Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Commit-Queue: Alexandr Ilin <alexilin@chromium.org> Cr-Commit-Position: refs/heads/master@{#595850}
-
Sylvain Defresne authored
Replace them with uses of TestingFactory which is based on base::Callback<> instead of function pointers. This converts uses in /chrome/browser/google. This CL was uploaded by git cl split. R=isherman@chromium.org Bug: 809610 Change-Id: I43fdd2d998182d420a2ccce7f9c9bc268f02a779 Reviewed-on: https://chromium-review.googlesource.com/1245714 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#595849}
-
Alex Clarke authored
Bug: 826902 Change-Id: Ic79292edea1d7d92935797103ded169cb114e1cf Reviewed-on: https://chromium-review.googlesource.com/1254542 Commit-Queue: Alex Clarke <alexclarke@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#595848}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/d082819ca2ad..38f24ee666e1 git log d082819ca2ad..38f24ee666e1 --date=short --no-merges --format='%ad %ae %s' 2018-10-02 geofflang@chromium.org Fix missing query for GL_NUM_REQUESTABLE_EXTENSIONS_ANGLE. Created with: gclient setdep -r src/third_party/angle@38f24ee666e1 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-chromium-autoroll 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:890689 TBR=cwallez@chromium.org Change-Id: I74d70fb5e07b1a6a62f67034497f5d2bcc7bc159 Reviewed-on: https://chromium-review.googlesource.com/1256905Reviewed-by:
-
Thomas Tangl authored
This a follow up to crrev.com/c/1251627. The method for showing the cookie exception for signed-in and syncing users has been corrected. The problem was that, when Dice is enabled and the user signs into Gmail, they are also signed into Chrome, so the signed-in/syncing message was shown even though the user is not syncing. Screenshot before: https://drive.google.com/file/d/117tFXEA7WaokpHlghypqWGL3fTQMpC6n/view?usp=sharing Screenshot after: https://drive.google.com/file/d/188Pu7XAHUJm39fpzY9lNqD6CgQAVi24I/view?usp=sharing Bug: 889137 Change-Id: I60ab697e4774842258dfc47ba6a53a2b34765650 Reviewed-on: https://chromium-review.googlesource.com/1256944 Commit-Queue: Thomas Tangl <tangltom@chromium.org> Reviewed-by:
-
stkhapugin@chromium.org authored
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I674ed55abf3313a7e93b5d7404aebcf9609ca621 Reviewed-on: https://chromium-review.googlesource.com/1235933 Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#595845}
-
Andrew Grieve authored
Change-Id: I34e8872d1011d143810a2f6b53a238e517785bf2 Reviewed-on: https://chromium-review.googlesource.com/1255045Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#595844}
-
Tetsui Ohkubo authored
Focus ring around MessageView should be painted on top of all the children. TEST=manual BUG=887366 Change-Id: I5667aff33196a5f3f8792b4e936e7da1cdda1057 Reviewed-on: https://chromium-review.googlesource.com/1255750Reviewed-by:
Yoshiki Iguchi <yoshiki@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#595843}
-
Emily Hanley authored
I have confirmed with the test case in https://github.com/catapult-project/catapult/issues/4490 that the navigation id in document loader is the correct id. Adding an id to the document itself is not the same between navigation and fcp. Bug: 850258,catapult:4490 Change-Id: I4028bcb13d90a5a1a765890fe02e346d496fab7a Reviewed-on: https://chromium-review.googlesource.com/1249533 Commit-Queue: Emily Hanley <eyaich@chromium.org> Reviewed-by:
-
Sylvain Defresne authored
TestingFactory is a base::Callback<> while TestingFactoryFunction is a function pointer. Convert UserCloudPolicyManagerFactory to the callback based type. Bug: 809610 Change-Id: If041d74cf623b29672c90351460f23be9a5fb24d Reviewed-on: https://chromium-review.googlesource.com/1245713 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Cr-Commit-Position: refs/heads/master@{#595841}
-
Hans Wennborg authored
../../third_party/blink/renderer/core/loader/image_loader.h:257:5: warning: explicitly defaulted move constructor is implicitly deleted [-Wdefaulted-function-deleted] DecodeRequest(DecodeRequest&&) = default; ^ ../../third_party/blink/renderer/core/loader/image_loader.h:252:25: note: move constructor of 'DecodeRequest' is implicitly deleted because base class 'GarbageCollected<blink::ImageLoader::DecodeRequest>' has a deleted move constructor class DecodeRequest : public GarbageCollected<DecodeRequest> { ^ ../../third_party/blink/renderer/core/loader/image_loader.h:262:20: warning: explicitly defaulted move assignment operator is implicitly deleted [-Wdefaulted-function-deleted] DecodeRequest& operator=(DecodeRequest&&) = default; ^ ../../third_party/blink/renderer/core/loader/image_loader.h:252:25: note: move assignment operator of 'DecodeRequest' is implicitly deleted because base class 'GarbageCollected<blink::ImageLoader::DecodeRequest>' has a deleted move assignment operator class DecodeRequest : public GarbageCollected<DecodeRequest> { ../../third_party/blink/renderer/modules/peerconnection/adapters/p2p_quic_transport_factory.h:36:27: warning: explicitly defaulted move assignment operator is implicitly deleted [-Wdefaulted-function-deleted] P2PQuicTransportConfig& operator=(P2PQuicTransportConfig&&) = default; ^ ../../third_party/blink/renderer/modules/peerconnection/adapters/p2p_quic_transport_factory.h:41:33: note: move assignment operator of 'P2PQuicTransportConfig' is implicitly deleted because field 'packet_transport' is of const-qualified type 'blink::P2PQuicPacketTransport *const' P2PQuicPacketTransport* const packet_transport; ^ Bug: 890307 Change-Id: I5bcb4e343ae96f26d20f1e772d1442d1edbdb105 Reviewed-on: https://chromium-review.googlesource.com/1256942 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by: -
Eric Orth authored
Moving to a new dns_config.cc/h to separate the public struct from the component-private DnsConfigService. Creates a better place for a new DnsConfigOverrides struct that I'll be adding soon for the servicified DNS config manipulation work. TBR=mef@chromium.org Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;master.tryserver.chromium.android:android_cronet_tester Change-Id: I5e02cc29b47851ea5295d2ec7d17b381b124d97a Reviewed-on: https://chromium-review.googlesource.com/1255783 Commit-Queue: Eric Orth <ericorth@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#595839}
-
Daniel Bratell authored
input_predictor_unittest.cc is used purely as a header file so it should have the .h suffix. Now it looks like some .cc files are trying to include another .cc file. The class was too large to have an implicit inline constructor/destructor so those remain in a cc file. This presubmit check didn't work before because the file was named *.cc. Change-Id: I67740f4181ea60d02cd3149ce57b0d4e8c36aaf2 Reviewed-on: https://chromium-review.googlesource.com/1255638Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Commit-Queue: Daniel Bratell <bratell@opera.com> Cr-Commit-Position: refs/heads/master@{#595838}
-
Sylvain Defresne authored
Replace them with uses of TestingFactory which is based on base::Callback<> instead of function pointers. This converts uses in /chrome/browser/safe_browsing/incident_reporting. This CL was uploaded by git cl split. R=robertshield@chromium.org Bug: 809610 Change-Id: Id58fcdc5a649e33a68effc8e774be09beb930ed0 Reviewed-on: https://chromium-review.googlesource.com/1246169 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#595837}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/16c9216905c1..9e92782016c4 git log 16c9216905c1..9e92782016c4 --date=short --no-merges --format='%ad %ae %s' 2018-10-02 caryclark@skia.org fix path addrrect example in docs Created with: gclient setdep -r src/third_party/skia@9e92782016c4 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll 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-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;master.tryserver.blink:linux_trusty_blink_rel TBR=mtklein@chromium.org Change-Id: I6272354423f082e3a79bd64c4cbb386734ce04de Reviewed-on: https://chromium-review.googlesource.com/1256240Reviewed-by:
-
Pam Greene authored
Eliminating email notifications and code reviews that are no longer relevant. Bug: N/A Change-Id: Ida73132009087c43440fc3e324d8852f096bcf5d Reviewed-on: https://chromium-review.googlesource.com/1255724Reviewed-by:
Dominic Battré <battre@chromium.org> Commit-Queue: Pam Greene (Please send IM for reviews) <pam@chromium.org> Cr-Commit-Position: refs/heads/master@{#595835}
-
Sorin Jianu authored
This reverts commit 46f49dac. Reason for revert: reverting the original change breaks other bots, such as https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win7%20Tests%20%281%29 an it does not fix the problem on WebKit Mac 10.13 Retina anyway. Original change's description: > Revert "remove SK_SUPPORT_LEGACY_SVG_ARC_TO" > > This reverts commit 05ee0393. > > Reason for revert: Sheriff reverting due to several paint-invalidation failing on the bot WebKit Mac 10.13 Retina. > See https://crbug.com/891230 > > Original change's description: > > remove SK_SUPPORT_LEGACY_SVG_ARC_TO > > > > Change-Id: I330bdfe4dcbfed55a03385043e703af6fe070ffd > > Reviewed-on: https://chromium-review.googlesource.com/1254341 > > Commit-Queue: Mike Reed <reed@chromium.org> > > Reviewed-by: Florin Malita <fmalita@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#595627} > > TBR=fmalita@chromium.org,reed@chromium.org > > Change-Id: Ica1ccdd43953772d40c70d22f3fc399df7dd56b1 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 891230 > Reviewed-on: https://chromium-review.googlesource.com/1256825 > Reviewed-by: Camille Lamy <clamy@chromium.org> > Commit-Queue: Camille Lamy <clamy@chromium.org> > Cr-Commit-Position: refs/heads/master@{#595805} TBR=fmalita@chromium.org,clamy@chromium.org,reed@chromium.org Change-Id: If0cd979dc3772397f1631d01efc8e08911ecd98c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 891230 Reviewed-on: https://chromium-review.googlesource.com/1256375Reviewed-by:
Sorin Jianu <sorin@chromium.org> Commit-Queue: Sorin Jianu <sorin@chromium.org> Cr-Commit-Position: refs/heads/master@{#595834}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/59297c6f73a6..48348e91f2d7 git log 59297c6f73a6..48348e91f2d7 --date=short --no-merges --format='%ad %ae %s' 2018-10-02 achuith@chromium.org oobe: Catch the right WebSocketException. Created with: gclient setdep -r src/third_party/catapult@48348e91f2d7 The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-autoroll 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:879353 TBR=sullivan@chromium.org Change-Id: Ie2201501881dfd170f964cf9312fecef7f48e59e Reviewed-on: https://chromium-review.googlesource.com/1256238Reviewed-by:
-
