- 12 Apr, 2018 40 commits
-
-
Hannes Payer authored
The V8-side landed in: https://chromium-review.googlesource.com/c/v8/v8/+/981146 Bug: chromium:814144 Change-Id: I3ef9523b4a0f371ce0f22400ef3c70bb521117ee Reviewed-on: https://chromium-review.googlesource.com/999472 Commit-Queue: Hannes Payer <hpayer@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Cr-Commit-Position: refs/heads/master@{#550225}
-
arthursonzogni authored
The hard timeout of services_unittests on chromium.android/KitKat Phone Tester (dbg) was 120s. However, it often runs close to 120s, which results in task timeout. This CL increase the timeout to 240s. Bug: 832021 Change-Id: Ibc6a6984de8915a7c7a33349041d15c6cb13e1db Reviewed-on: https://chromium-review.googlesource.com/1009949 Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#550224}
-
Sammie Quon authored
Back button visibility was normally handled on shelf animation finish. When switching from vertical shelf to tablet mode (force bottom shelf), there is no animation, so the back button does not show up. This cl handles that case. Test: ash_unittests BackButtonTest.* Bug: 831469 Change-Id: Ia95e791d30deb5a5c91599039c39094ffa29003b Reviewed-on: https://chromium-review.googlesource.com/1008163 Commit-Queue: Sammie Quon <sammiequon@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#550223}
-
Emily Hanley authored
Bug: 757933 Change-Id: I6d5ec84f38f9ae07e223900476b7198691c43d87 Reviewed-on: https://chromium-review.googlesource.com/1010262Reviewed-by:
Ashley Enstad <ashleymarie@chromium.org> Commit-Queue: Emily Hanley <eyaich@chromium.org> Cr-Commit-Position: refs/heads/master@{#550222}
-
Rob Buis authored
Fix regression allowing text value for -webkit-background-origin that was caused by r514365. Code change implemented by Bugs Nash, testcase by Rob Buis. Bug: 802256 Change-Id: I11e2b1b19a374e9f6b9c1091d82882ef526bc63c Reviewed-on: https://chromium-review.googlesource.com/1008102 Commit-Queue: Rob Buis <rob.buis@samsung.com> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#550221}
-
Jun Choi authored
User verification requirement is an optional paramter for MakeCredential and GetAssertion requests. However, while CTAP spec defines the parameter as a boolean, the WebAuthN spec defines the paramter as a enum with 3 values : Required, Preferred, and Discouraged. Implement logic to convert WebAuthN style user verification requirement parameter to CTAP style parameter. Bug: 780078 Change-Id: If7ebe92a2d5fa4d623348ee52bbd6958c30e1a9e Reviewed-on: https://chromium-review.googlesource.com/1002532 Commit-Queue: Jun Choi <hongjunchoi@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#550220}
-
Troy Hildebrandt authored
Every use of LocationBarButton aside from the omnibox incognito icon were setting their layout_gravity explicitly, so having LocationBarButton have a gravity of end wasn't an issue elsewhere. This brings back the original LocationBarButton and creates the LocationBarActionButton for action buttons instead. TBR=yusufo@chromium.org Bug: 831535 Change-Id: If78e27c32c09d0a462a15503ab73f89788d603bd Reviewed-on: https://chromium-review.googlesource.com/1008169 Commit-Queue: Troy Hildebrandt <thildebr@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#550219}
-
Nico Weber authored
TBR=miu,kbr Bug: 831999,832011 Change-Id: I22bf82f84e0ae874811b1ba3cbac03d8b5f1d49e Reviewed-on: https://chromium-review.googlesource.com/1010382Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#550218}
-
Gustav Sennton authored
Add code to support the Support Library version of WebResourceRequest.isRedirect(). Also add a feature flag to guard that API. Bug: 831315 Change-Id: I528c02cfabdd1a83f66231ed7dd1b78b972b3843 Reviewed-on: https://chromium-review.googlesource.com/1005755 Commit-Queue: Gustav Sennton <gsennton@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#550217}
-
Francois Doray authored
Previously, tasks posted to a shared SingleThreadTaskRunner with the same traits were scheduled on the same thread. If single-threaded task A (BACKGROUND, CONTINUE_ON_SHUTDOWN) was scheduled on the thread and single-threaded task B (BACKGROUND, BLOCK_SHUTDOWN) was posted afterwards, shutdown could not complete until task A finished and allowed task B to run on the thread. This CL fixes the issue by scheduling shared single-threaded tasks that are CONTINUE_ON_SHUTDOWN/non-CONTINUE_ON_SHUTDOWN on different threads. Bug: 829786 Change-Id: Ia0b556971895c2a5799157f8dcf1a1ee6ce650c9 Reviewed-on: https://chromium-review.googlesource.com/1005422 Commit-Queue: François Doray <fdoray@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#550216}
-
Yash Malik authored
The gvr keyboard API requires that gvr_keyboard_advance_frame is called every time after the keyboard is initialized. This regressed in the optimization that doesn't call UiElement::OnBeginFrame for invisible elements. We add a special hook to do certain thing every frame. Bug: 831749 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;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I877299165dde517fab90451788de9680700666d9 Reviewed-on: https://chromium-review.googlesource.com/1009042 Commit-Queue: Yash Malik <ymalik@chromium.org> Reviewed-by:
Ian Vollick <vollick@chromium.org> Cr-Commit-Position: refs/heads/master@{#550215}
-
Krishna Govind authored
R=cmasso@chromium.org Change-Id: I92933136860899b7cdd3e8c3f209febd1c88d9df Reviewed-on: https://chromium-review.googlesource.com/1010471Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#550214}
-
Matt Falkenhagen authored
This makes more sense as SharedWorkerHost's destructor calls WorkerDestroyed to balance it out. It also simplifies the params to Start(). Not strictly needed for the linked bug. Refactoring that will make it easier to fix. Bug: 830288 Change-Id: I3be38ad7cc58c35bef301cd21074987ea9e5c64d Reviewed-on: https://chromium-review.googlesource.com/1004887 Commit-Queue: Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#550213}
-
Alexey Kozyatinskiy authored
Nicer arrow is always better. R=pfeldman@chromium.org Bug: none Change-Id: I38f58ca5adbdaa55516bc215928e99ac0c1f1c2d Reviewed-on: https://chromium-review.googlesource.com/1008630Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org> Cr-Commit-Position: refs/heads/master@{#550212}
-
Jonathan Backer authored
On certain CrOS device, we --enable-native-gpu-memory-buffers. This causes the target for texture to become GL_TEXTURE_EXTERNAL_OES in a RasterDecoder::CreateTexture call. OnceCopyRasterProvider calls TexParameteri(...), which binds gpu::Texture and sets it immutable service side via a call to TextureManager::SetTarget(...). A subsequent call to BindTexImage2DCHROMIUM then fails on the IsImmutable check. 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;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: Ibde88dd449a246573343ac435116e529ce4e88cf Reviewed-on: https://chromium-review.googlesource.com/1008130Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Commit-Queue: Jonathan Backer <backer@chromium.org> Cr-Commit-Position: refs/heads/master@{#550211}
-
Helen Li authored
This is a reland of 53553a99 Original change's description: > Add a net::TransportClientSocket class > > This CL adds a net::TransportClientSocket so that we can have an interface > to expose socket Bind() method. > > This CL changes the following: > - socket_test_util.cc > Changes MockClientSocket to subclass this new net::TransportClientSocket > class instead of net::SSLClientSocket. This new class hierarchy is more > intuitive because not all client sockets are ssl socket. > - client_socket_factory.cc > Makes CreateTransportClientSocket() returns a net::TransportClientSocket > instead of a StreamSocket. > - services/network/* > Makes network::SocketFactory take in a net::ClientSocketFactory. > - cast_socket.cc/h and cast_socket_unittest.cc. > > > Bug: 721401 > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo > Change-Id: Ie258114565e76529555e244364be421e66637960 > Reviewed-on: https://chromium-review.googlesource.com/995554 > Commit-Queue: Helen Li <xunjieli@chromium.org> > Reviewed-by: Derek Cheng <imcheng@chromium.org> > Reviewed-by: Matt Menke <mmenke@chromium.org> > Cr-Commit-Position: refs/heads/master@{#549731} TBR=imcheng@chromium.org Bug: 721401 Change-Id: I93fb736881be6554bdaf17359b881370be1d7fea Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Reviewed-on: https://chromium-review.googlesource.com/1007034Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: Helen Li <xunjieli@chromium.org> Cr-Commit-Position: refs/heads/master@{#550210}
-
Peter Boström authored
Rounded omnibox results apply to touchable and Material refresh. This is not in spec for refresh and touchable doesn't need to diverge here. Bug: chromium:826606 Change-Id: If9ea12ae3912b50fce92c9368452bf342b395e6c Reviewed-on: https://chromium-review.googlesource.com/1007898Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Commit-Queue: Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#550209}
-
Jun Choi authored
Currenly FakeFidoTask used in unit test for FidoRequestHandler categorizes response types depending on whether the response received from the mock device is base::nullopt, empty vector, or a non-empty vector. Change this categorization to use clearly readable enums instead. Bug: 827677 Change-Id: I6817808a6ace85e88bba48039c9cbe054cfbfb81 Reviewed-on: https://chromium-review.googlesource.com/1006233 Commit-Queue: Jun Choi <hongjunchoi@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#550208}
-
Daniel Bratell authored
When building chrome_app_unittests chrome/common/crash_keys.cc are compiled both as a dependency and locally. Enough to compile it as a dependency. (Noticed in jumbo build experiments where the duplicate symbols met in the linker) Change-Id: I517436d21ae17c2ff664afeed43ad8e1b7e529e9 Reviewed-on: https://chromium-review.googlesource.com/1005341Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Commit-Queue: Daniel Bratell <bratell@opera.com> Cr-Commit-Position: refs/heads/master@{#550207}
-
chrome://inspectKen Rockot authored
Bug: 829412 Change-Id: Ia42efe7c749e0eaa44ea9c45a1a4be1a7d43580f Reviewed-on: https://chromium-review.googlesource.com/1007980 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#550206}
-
Ashley Enstad authored
Bug: 816458 Change-Id: Iba29c656f02789baa8d0f9153bfaabe0fc1121e8 Reviewed-on: https://chromium-review.googlesource.com/1005032 Commit-Queue: Ashley Enstad <ashleymarie@chromium.org> Reviewed-by:
Emily Hanley <eyaich@chromium.org> Cr-Commit-Position: refs/heads/master@{#550205}
-
Biao She authored
This CL also did the following three things: 1. refactored the code for checking if we should suppress 2D UI. 2. add kOsStoragePermission enum to represent suppress native storage UI request(currently only Android). 3. add kMediaRouterPresentationRequest enum to represent suppress media router presentation. Bug: 817148 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;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: Id8e0bdc961ffcbcd1f0bc7145be2998ca949602e Reviewed-on: https://chromium-review.googlesource.com/1005842 Commit-Queue: Biao She <bshe@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Amirhossein Simjour <asimjour@chromium.org> Reviewed-by:
Tibor Goldschwendt <tiborg@chromium.org> Cr-Commit-Position: refs/heads/master@{#550204}
-
Emily Hanley authored
Bug: 779634 Change-Id: I613fed97d31045ed7d15bbe17e0d3153fd19812e Reviewed-on: https://chromium-review.googlesource.com/1002922 Commit-Queue: Emily Hanley <eyaich@chromium.org> Reviewed-by:
Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#550203}
-
Bence Béky authored
WebSocketEndpointLockManager being a singleton is causing a number of issues, including test flakiness. This CL makes HttpNetworkSession create and own an WebSocketEndpointLockManager instance instead, as suggested in https://crbug.com/450518#c8. I have verified locally that the following test invokation fails on trunk but passes with this CL: out/gn/net_unittests --gtest_filter=\ HttpStreamFactoryTest.RequestWebSocketBasicHandshakeStreamOverSSL:\ WebSocketEndpointLockManagerTest.LockEndpointReturnsOkOnce This is like the one reported in https://crbug.com/814985, but using LockEndpointReturnsOkOnce instead of GetInstanceWorks, because GetInstanceWorks is removed in this CL. Bug: 450518 Change-Id: Ie882dd7584c8cd3917844e0583370e453e316964 Reviewed-on: https://chromium-review.googlesource.com/1009962Reviewed-by:
Adam Rice <ricea@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#550202}
-
Yuqian Li authored
Bug: chromium:821353 Change-Id: I69a86f539e5e7d3b038fd085abfbba0826eb04bc Reviewed-on: https://chromium-review.googlesource.com/1008043Reviewed-by:
Florin Malita <fmalita@chromium.org> Commit-Queue: Yuqian Li <liyuqian@chromium.org> Cr-Commit-Position: refs/heads/master@{#550201}
-
yiyix authored
In the current implementation, if the new RenderFrameMetadata is same as the initial value, then Browser will not receive the RenderFrameMetadata until it changes. After this patch, Browser will always receive the initial RenderFrameMetadata. Bug:830540 Change-Id: Ic0723d03aff1e4e7fa9eaabe80c27d5dd9bb31d0 Reviewed-on: https://chromium-review.googlesource.com/1006374Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Reviewed-by:
Jonathan Ross <jonross@chromium.org> Reviewed-by:
-
Katie D authored
This uses window.speechSynthesis, which should be using the TTS engine and voice prefs selected by the user. Bug: 823359 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I2b8cd29a2b1fd1da2a5cb002aeb877ed2e615196 Reviewed-on: https://chromium-review.googlesource.com/996764 Commit-Queue: Katie Dektar <katie@chromium.org> Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#550199}
-
Andrew Grieve authored
Change-Id: If11054d64db33bef5f32561950604706d8a55b52 Reviewed-on: https://chromium-review.googlesource.com/1006043Reviewed-by:
Peter Wen <wnwen@chromium.org> Commit-Queue: agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#550198}
-
jdoerrie authored
This change promotes the DCHECKs in base::Optional::value() to CHECKs in order to match std::optional more closely. Furthermore, it also CHECKs when dereferencing a base::Optional via operator* or operator->. Bug: 817982 Change-Id: Ib2bfdd3a863e8bade21fa1b6b1c10c7c4d2ca135 Reviewed-on: https://chromium-review.googlesource.com/997335 Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
-
Istiaque Ahmed authored
Currently calling DictionaryValue::Set(??, bool) and ListValue::Append(bool) would result in calling int variant from the overloaded methods(!). This is confusing and can be a source of bugs. (The current suggested way is to explicitly call SetBoolean and AppendBoolean). There is only once place where this was happening, in extension_printer_handler.cc. Fix that code to use boolean variant, by using SetBoolean() method. This CL will also let us overload the bool variant in future. This CL - Provides an explicit "const char*" param overload. - Declares a bool overload but makes it inaccessible (= delete). This will - Make sure attempting to call DictionaryValue::Set with bool will fail compile (call to deleted member function 'Set'). So accidental introduction of such bugs won't be possible. - The char* variant will make sure we don't attempt to resolve bool to this pointer variant. - Give us the ability to remove GetBoolean/AppendBoolean. Bug: 831839 Change-Id: If61ce8a44799ff8e12957d5206baf74467655894 Test: Expect no behavior change, touches webui/print_preview FYI Reviewed-on: https://chromium-review.googlesource.com/1008870Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
-
Adam Langley authored
The spec suggests aborting an operation when focus[1] is lost, but that's advice for site developers. I can't find anything in the spec about preventing background tabs from triggering operations. The cryptotoken extension refused to start a registration request, or to send a registration response to, anything but the active tab in the focused window. But background tabs could complete an authentication request. This change does something similar: it rejects both authentication and registration requests unless the requesting frame is in a focused window. It also performs that check before returning responses. This is slightly different from the cryptotoken behaviour because cryptotoken could only use what the extensions API exposed. For example, if the omnibox was focused, cryptotoken would complete a registration from the foreground tab but this code will reject it. I think this behaviour is better, and it's certainly far more inline with the content / browser separation. This change has been split from its tests, which will come in a future CL. [1] https://w3c.github.io/webauthn/#abortoperation Bug: 827266 Change-Id: If6e97dd6526e175f40718724eda21e3efd434f7f Reviewed-on: https://chromium-review.googlesource.com/991073 Commit-Queue: Adam Langley <agl@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
-
Lukasz Anforowicz authored
Bug: 831671 Change-Id: Ib0de8b66ebe2963bf25cfde26ab8fef7e1070e56 Reviewed-on: https://chromium-review.googlesource.com/1008172 Commit-Queue: Charlie Reis <creis@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Cr-Commit-Position: refs/heads/master@{#550194}
-
vasilii authored
Bug: 819626 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I4ea56f1d4d7d12184b2fdfbdbc9c44330b72b94e Reviewed-on: https://chromium-review.googlesource.com/1007059 Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Vaclav Brozek <vabr@chromium.org> Cr-Commit-Position: refs/heads/master@{#550193}
-
Min Qin authored
For download service, we are going to pass a class that retrieves URLLoaderFactory. This CL defines a common interface for that purpose. And the implementation can support either a blob url or a network url. BUG=803135 Change-Id: I505e025fc54553753180f05b97fbd3c4b654253a Reviewed-on: https://chromium-review.googlesource.com/1007974 Commit-Queue: Min Qin <qinmin@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#550192}
-
Xi Han authored
This is the precursor step before creating a thread to start the ServiceManager before content_main_runner is running. It is because no thread is expected to be when the zygote process is forked. In this CL, we move the setup to ContentMainRunner::Initialize(). Bug: 740677,729596 Change-Id: I90e95cce8bc32be03c944aaa17f96c79d555e7cb Reviewed-on: https://chromium-review.googlesource.com/999741Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Commit-Queue: Xi Han <hanxi@chromium.org> Cr-Commit-Position: refs/heads/master@{#550191}
-
Henrik Boström authored
This adds RuntimeEnabled feature RTCUnifiedPlanByDefault (off by default even in LayoutTests) which overrides the default SDP semantics of RTCPeerConnection, and adds running external/wpt/webrtc/* as a virtual test suite with this flag enabled. The tests can be run as: virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/ The -expected.txt files are identical to the non-virtual versions. The only problem introduced with Unified Plan is that the following test times out: virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/ RTCPeerConnection-setRemoteDescription-tracks.https.html Bug filed: https://crbug.com/828866 And unlike external/wpt/webrtc/ RTCPeerConnection-setLocalDescription-answer.html, virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/ RTCPeerConnection-setLocalDescription-answer.html does not fail against the -expected.txt file. Bug: 828793, 828866 Change-Id: I51d86fc5a0388ae1d8bf5f8497f31795d205d5d2 Reviewed-on: https://chromium-review.googlesource.com/995278 Commit-Queue: Henrik Boström <hbos@chromium.org> Reviewed-by:Harald Alvestrand <hta@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Cr-Commit-Position: refs/heads/master@{#550190}
-
Francois Doray authored
This CL previously landed as https://chromium-review.googlesource.com/997892. It was reverted because of an access race in ChromeStorageImpl https://crbug.com/829122. This CL fixes the issue by deleting both callbacks on the origin sequence when they can't run. Before, there was always leak when an internal task posted by PostTaskAndReplyImpl didn't run. With this CL, PostTaskAndReplyImpl notices when its internal callbacks aren't scheduled and schedules deletion of all its internal state (including callbacks that didn't run) on the origin sequence via a DeleteSoon(). Note that a deletion scheduled via DeleteSoon() might not happen if Chrome is shutting down. R=gab@chromium.org Bug: 807013 Change-Id: I17abffd3e7afcc861c8c92dce30c14ce8629b125 Reviewed-on: https://chromium-review.googlesource.com/1006045 Commit-Queue: François Doray <fdoray@chromium.org> Reviewed-by:
-
Corentin Wallez authored
Also fixup some documentation links for GPU optional bots. Also change a PRESUBMIT.py's master name for win_angle_rel_ng. TBR=kbr@chromium.org TBR=sky@chromium.org BUG=chromium:815092 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;master.tryserver.chromium.linux:linux_vr Change-Id: Ifda505e121f67f922f19da466a37b7f6266cea05 Reviewed-on: https://chromium-review.googlesource.com/1008182Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Corentin Wallez <cwallez@chromium.org> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Cr-Commit-Position: refs/heads/master@{#550188}
-
Devlin Cronin authored
Update profile_signin_confirmation_helper_unittest.cc and sync_extension_helper.cc to generate extensions with manifest_version 2. Bug: 816679 Change-Id: I302693e92b7925f900cd41b7fa3751a4a299a8cf Reviewed-on: https://chromium-review.googlesource.com/1008624Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#550187}
-
Asanka Herath authored
The generate-quic-chain.sh scripts expects the latter. Also the line endings are not significant. R=jam@chromium.org, rch@chromium.org Change-Id: I6dc0a14bd2f3b7f7e1438ee6207dfbc53deb4eba Reviewed-on: https://chromium-review.googlesource.com/1007642Reviewed-by:
Ryan Hamilton <rch@chromium.org> Reviewed-by:
-
