- 06 Aug, 2018 40 commits
-
-
Johannes Henkel authored
There's just one enabled_ field here, a bool. Bug: 851762 Change-Id: I25247dc0d0333ea48ce2cf06c3fdf633ef2c1662 Reviewed-on: https://chromium-review.googlesource.com/1163998Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Johannes Henkel <johannes@chromium.org> Cr-Commit-Position: refs/heads/master@{#581033}
-
Takuto Ikuta authored
This fixes exception happening on Windows deterministic builder like below. https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Windows%20deterministic/9666 Change-Id: Iff309dabe5753fa1633366d6bc6716505aaf18c5 Reviewed-on: https://chromium-review.googlesource.com/1164144 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#581032}
-
Ben Pastene authored
And add a few test suites that need that arg to run. Bug: 840967 Change-Id: I9954a08cc22f7237b0e0d2093214b544b6f3d897 Reviewed-on: https://chromium-review.googlesource.com/1162863Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#581031}
-
Alex Newcomer authored
There is a bug where users can keep their clamshell mode context menu open by detaching the keyboard. This allows users to have a side shelf in tablet mode, and they may get stuck in this state. This CL prevents this from happening by ignoring the clamshell options if they are somehow executed in tablet mode. Bug: 870469 Change-Id: Ifb9638c3ae705bd58a2e785edc5ca84d698651d2 Reviewed-on: https://chromium-review.googlesource.com/1161387Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Alex Newcomer <newcomer@chromium.org> Cr-Commit-Position: refs/heads/master@{#581030}
-
Manas Verma authored
This change fixes the Manage Cards button, in the Manage Cards prompt, to link to chrome://settings/payments instead of chrome://settings/autofill bug: 855186 Change-Id: I3d274367deae5d04c7a3029db8e5eec3aa852cc1 Reviewed-on: https://chromium-review.googlesource.com/1163743 Commit-Queue: Manas Verma <manasverma@google.com> Reviewed-by:
Tommy Li <tommycli@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#581029}
-
Jun Cai authored
This CL updates the currently disabled browser tests for setting x-frame-options headers to web store requests. Bug: 852877 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I8e131848a37a8fbf5b9f4bdb53bcbb225440330d Reviewed-on: https://chromium-review.googlesource.com/1155931Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: Jun Cai <juncai@chromium.org> Cr-Commit-Position: refs/heads/master@{#581028}
-
Weidong Guo authored
Background: When screen is rotated to portrait, only half of the background shield is shown because mask layer bounds is incorrectly updated. 1. Remove mask layer used to create corner radius for background shield and, instead, draw rounded corner background on Canvas. 2. As a side effect, this improves dragging smoothness. Bug: 869622 Change-Id: Ia3b8ce30e0f76f27cdc90ab674504915a79cfb99 Reviewed-on: https://chromium-review.googlesource.com/1164178 Commit-Queue: Weidong Guo <weidongg@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#581027}
-
Paul Miller authored
Change-Id: I68ee67d5c0f498e04ca3a892043557b6ce0289f5 Reviewed-on: https://chromium-review.googlesource.com/1164068 Commit-Queue: Paul Miller <paulmiller@chromium.org> Reviewed-by:
Jialiu Lin <jialiul@chromium.org> Cr-Commit-Position: refs/heads/master@{#581026}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/58bf2845eb4d..e19516b87bb1 git log 58bf2845eb4d..e19516b87bb1 --date=short --no-merges --format='%ad %ae %s' 2018-08-06 eakuefner@chromium.org [Dashboard] Halve max concurrent histogram tasks Created with: gclient setdep -r src/third_party/catapult@e19516b87bb1 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:867379 TBR=sullivan@chromium.org Change-Id: Ie7b7d0e7b92b17e8cbae75271929f2e05e0f982b Reviewed-on: https://chromium-review.googlesource.com/1164282Reviewed-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@{#581025}
-
Ross Wang authored
Garnet no longer produces an SDK manifest; switch to topaz. Bug: None Change-Id: I43ef7d7d1cc64a57ec546c046379af381c4dcdb5 Reviewed-on: https://chromium-review.googlesource.com/1164397 Commit-Queue: Ross Wang <rosswang@google.com> Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#581024}
-
Charlie Reis authored
Data URLs are usually put into the process that created them, but this info is not tracked after a tab restore. Ensure that they do not end up in the parent frame's process (or each other's process), in case they are malicious. BUG=863069 Change-Id: Ib391f90c7bdf28a0a9c057c5cc7918c10aed968b Reviewed-on: https://chromium-review.googlesource.com/1150767Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Charlie Reis <creis@chromium.org> Cr-Commit-Position: refs/heads/master@{#581023}
-
Peter Kasting authored
We can't compute contrast ratios on transparent colors. This prevents DCHECKs with Orin's soon-to-be-landed code, and prevents theme authors from setting colors that lead to drawing artifacts. Bug: 856893 Change-Id: I2994cd44b04ea363b20f087295f6b085325f8ec5 Reviewed-on: https://chromium-review.googlesource.com/1164170 Commit-Queue: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Allen Bauer <kylixrd@chromium.org> Cr-Commit-Position: refs/heads/master@{#581022}
-
Peter Boström authored
Force-enables MaterialDesignController::IsSecondaryMaterialUi() and removes the experiment kSecondaryUiMd as a first step of cleanup. Follow-up changes will remove code that only gets called when IsSecondaryMaterialUi() is false. Finally all traces of IsSecondaryMaterialUi will be removed. Bug: chromium:630357, chromium:867557 Change-Id: If1b7f96c100673a8d3861fee712b4f0d7b8ef726 Reviewed-on: https://chromium-review.googlesource.com/1162901Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Commit-Queue: Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#581021}
-
Dominick Ng authored
This is a speculative fix for crashes caused by native methods being called after this object is destroyed. BUG=871129 Change-Id: Id620402711979e6e380dd8e307cca098619e962d Reviewed-on: https://chromium-review.googlesource.com/1163407Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#581020}
-
Siddhartha authored
BUG=855037 Change-Id: Ifb2d3298db6cc368ceb5f7ed4de7cf01f412be81 Reviewed-on: https://chromium-review.googlesource.com/1161381Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Siddhartha S <ssid@chromium.org> Cr-Commit-Position: refs/heads/master@{#581019}
-
Takuto Ikuta authored
Make virtual/mouseevent_fractional/fast/events/touch/touch-latched-scroll-node-removed.html flaky in Mac TBR=sahel@chromium.org Bug: 869726 Change-Id: Ie6610d7bd9b18dedac07444dc65904d30192e945 Reviewed-on: https://chromium-review.googlesource.com/1164143Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#581018}
-
Xiangjun Zhang authored
This CL adds the implementation to create an audio loopback stream through Audio Service as requested. Bug: 734672 Change-Id: Ifd78a1848c7979438401586c03f41758d996978e Reviewed-on: https://chromium-review.googlesource.com/1141284 Commit-Queue: Xiangjun Zhang <xjz@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Yuri Wiitala <miu@chromium.org> Cr-Commit-Position: refs/heads/master@{#581017}
-
Fady Samuel authored
OnSurfaceDiscarded cosists of two operations: 1. Drop tracking of dependencies of the discarded surface. 2. Unblock surfaces blocked ont he discarded surface. 1. is equivalent to listing all activation dependencies as 'removed_dependencies' and calling OnSurfaceDependenciesChanged. This CL does just that, reducing the number of lines of code in SurfaceDependencyTracker further. Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: I8191f1be81d04cce0c2181ac4fc341ebb8e3cf3f Bug: 672962 Reviewed-on: https://chromium-review.googlesource.com/1163632 Commit-Queue: Fady Samuel <fsamuel@chromium.org> Reviewed-by:
vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#581016}
-
Daniel Zhang authored
Fixing a bug that involves SearchResult tile text being cut off on the second line of text. Standardized text style between SearchResults and SearchResultTileItems. Bug: 869632 Change-Id: Ifbcc68c29c17b78a5d2323aa59dc98a759b81b34 Reviewed-on: https://chromium-review.googlesource.com/1162742Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Alex Newcomer <newcomer@chromium.org> Commit-Queue: Daniel Zhang <oxyflush@google.com> Cr-Commit-Position: refs/heads/master@{#581015}
-
David Roger authored
The order of the enum in enums.xml did not match the order from the code, resulting in wrong labels on the histograms. In the code, the type is TokenRevocationRequestProgress, defined in mutable_profile_oauth2_token_service_delegate.cc This CL fixes the label of the histogram. Bug: 834720 Change-Id: I30b039a8f5c9c2181bc1a77d807123eab53272fb Reviewed-on: https://chromium-review.googlesource.com/1163680Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#581014}
-
Scott Violet authored
BUG=842365 TEST=none Change-Id: I3f29c66f1d82c88eaff7da4c8b99ee6aed4902cc Reviewed-on: https://chromium-review.googlesource.com/1164075Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#581013}
-
Xiaoqian Dai authored
1) If the overview ends because a window (either from overview or not) just got snapped, we should not do animation when exiting overview. 2) If the overview ends because of releasing a dragged window, we also should not do animation when exiting overview. Bug: 823769 Test: SplitViewControllerTest.OverviewExitAnimationTest, SplitViewTabDraggingTest.OverviewExitAnimationTest Change-Id: I506120348f55ac7c86ec581ede08473c14f6e128 Reviewed-on: https://chromium-review.googlesource.com/1162587 Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Reviewed-by:Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#581012}
-
Jared Saul authored
Bug: 868552 Change-Id: I6a796cdcc0d4aa27750caa8a93581f205d4e8409 Reviewed-on: https://chromium-review.googlesource.com/1161569Reviewed-by:
Robert Kaplow (slow) <rkaplow@chromium.org> Commit-Queue: Jared Saul <jsaul@google.com> Cr-Commit-Position: refs/heads/master@{#581011}
-
gogerald authored
Bug: 806868 Change-Id: I9290cc5830308134e14b23be0d87eb5b11dd0d6e Reviewed-on: https://chromium-review.googlesource.com/1156480 Commit-Queue: Ganggui Tang <gogerald@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Cr-Commit-Position: refs/heads/master@{#581010}
-
Clark DuVall authored
This reverts commit c9cbab07. Reason for revert: Breaking linux dbg bots: https://luci-milo.appspot.com/buildbot/chromium.linux/Linux%20Tests%20(dbg)(1)/73603 Original change's description: > Fix ExtensionWebRequestApiTest.WebRequestUnloadImmediately with network service > > The error event was not being sent when the client was reset. > > Bug: 721414 > Cq-Include-Trybots: luci.chromium.try:linux_mojo > Change-Id: I951422061022d50644d8453900af14be27bb9eaf > Reviewed-on: https://chromium-review.googlesource.com/1150403 > Commit-Queue: Clark DuVall <cduvall@chromium.org> > Reviewed-by: John Abd-El-Malek <jam@chromium.org> > Cr-Commit-Position: refs/heads/master@{#580911} TBR=jam@chromium.org,cduvall@chromium.org Change-Id: I56a28f0f2ebd8f3ec57aa0af0a94e317d7cffcaa No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 721414 Cq-Include-Trybots: luci.chromium.try:linux_mojo Reviewed-on: https://chromium-review.googlesource.com/1164208Reviewed-by:
Clark DuVall <cduvall@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#581009}
-
Chris Harrelson authored
Previously we unconditionally invalidated both on any style change. Most of the changes in this patch are updates of code that looks like: SetNeedsPaintPropertyUpdate(); to: SetNeedsPaintPropertyUpdate(); if (Layer()) Layer()->SetNeedsCompositingInputsUpdate(); in cases where the LayoutObject in question might have a PaintLayer. The call to SetNeedsCompositingInputsUpdate() cannot be pushed down into SetNeedsPaintPropertyUpdate() because SetNeedsPaintPropertyUpdate() is called from SPv1 compositing code post-inputs, and it's illegal to set lifecycle bits for that phase after it has already completed. Bug: 814439 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I8c2aa3d451ad24e6feb0b14858bdba09d42f4ef5 Reviewed-on: https://chromium-review.googlesource.com/1157651 Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#581008}
-
Yue Li authored
Also update icon for assistant server notifications. Bug: b/111936939 Test: Manual Test Change-Id: I91d013daba51174d9532b6806484f1ed42809f14 Reviewed-on: https://chromium-review.googlesource.com/1162976 Commit-Queue: Yue Li <updowndota@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#581007}
-
Robbie McElrath authored
Currently NetworkConnectionTracker is in //content/public/browser, but it needs to be used by some //components that get used by ios, which can't depend on //content. Bug: 821009 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: Id47eda058f5be58492f510771b3ed6e5bbbaffac Reviewed-on: https://chromium-review.googlesource.com/1162747 Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#581006}
-
Jérôme Lebel authored
metrics::prefs::kMetricsReportingEnabled and prefs::kMetricsReportingWifiOnly are local prefs, and not user prefs. prefs: :kMetricsReportingWifiOnly is part of the local pref service. Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I7a5bfc6d4b7c34e59d911996329b6515190c164c Reviewed-on: https://chromium-review.googlesource.com/1163714Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Cr-Commit-Position: refs/heads/master@{#581005}
-
Azeem Arshad authored
This CL exposes three additional methods: OnVersionActivated, OnVersionRedundant and OnNoControllees. These methods are required for ChromeOS integration for Android Web Messages. Bug: 850823 Change-Id: I2d38e46e28c1f4b2c76fb6410a57fb74c1a3b850 Reviewed-on: https://chromium-review.googlesource.com/1145621Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Jeremy Klein <jlklein@chromium.org> Commit-Queue: Azeem Arshad <azeemarshad@chromium.org> Cr-Commit-Position: refs/heads/master@{#581004}
-
Lukasz Anforowicz authored
WebNavigationApiTest.ServerRedirectSingleProcess from browser_tests has its expectations in serverRedirectSingleProcess/test_serverRedirectSingleProcess.js. The expectations include exact |processId| ("exact" meaning that one cannot use a wildcard - the verification is done via chrome.test.checkDeepEq). Inclusion of |processId| means that the expectation change in site-per-process mode. This CL flips the expectations to match the expected behavior in site-per-process mode and disables the test outside of site-per-process mode. The "disabling" can be removed in a follow-up CL, after we only run //chrome-layer tests in site-per-process mode. Bug: 671734 Change-Id: I984d67342008c3025b84406d99d4798ca46d77f6 Reviewed-on: https://chromium-review.googlesource.com/1162909Reviewed-by:Nasko Oskov <nasko@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#581003}
-
Manas Verma authored
[Autofill] Changes the margins around the sign-in button for the promo shown after a local card save. This change is to fix the UI requirements for the sign-in promo. Should match this mock: https://docs.google.com/presentation/d/1dLbeUlWKkSQzqbDlktot5ldlcVZu32jcjJvIT27vGb8/edit#slide=id.g35839571aa_0_4 (internal) Before: https://screenshot.googleplex.com/6yveL39ovpF After: https://screenshot.googleplex.com/5YhNTQYuHV1 bug: 855186 Change-Id: I8fef17b65091579a5fa6a1f473164c96ff4fd22e Reviewed-on: https://chromium-review.googlesource.com/1163746 Commit-Queue: Manas Verma <manasverma@google.com> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Jared Saul <jsaul@google.com> Cr-Commit-Position: refs/heads/master@{#581002}
-
Francois Doray authored
It is no longer necessary to invoke the DEFINE_WEB_CONTENTS_USER_DATA_KEY macro to define the global variable from which the key of a WebContentsUserData is derived. The key is now derived from a static local variable in WebContentsUserData::UserDataKey(). See https://chromium-review.googlesource.com/c/chromium/src/+/1093015 This CL was uploaded by git cl split. R=avi@chromium.org Bug: 589840 Change-Id: I7cf3b754fbf7452b316e19f25f849017726e3ef6 Reviewed-on: https://chromium-review.googlesource.com/1150284Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#581001}
-
Victor Costan authored
[NSScreen screensHaveSeparateSpaces] is available on MacOS 10.9 and above [1], so selector checking is no longer necessary. [1] https://developer.apple.com/documentation/appkit/nsscreen/1388365-screenshaveseparatespaces Change-Id: I713baf8c10cd0efb510002211aea66b6bb379f99 Reviewed-on: https://chromium-review.googlesource.com/1163214Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#581000}
-
Nicolas Ouellet-Payeur authored
This is the first step towards creating a policy to enforce printing with/without headers and footers. When it is set, the 'Headers and Footers' checkbox is disabled and greyed out. Right now, there is no easy way to set the pref (beyond just editing the Preferences JSON file). Screenshot: https://screenshot.googleplex.com/LWBrvFzkRZ5.png Bug: 785242 Change-Id: I4a8bb8a0ec5455fe8467931f27eb5e6fab43dd2e Reviewed-on: https://chromium-review.googlesource.com/1149994Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org> Cr-Commit-Position: refs/heads/master@{#580999}
-
rajendrant authored
This CL correctly deprecates the UserViewedSavingsPercent histogram for which the code was removed already, and undeprecate the DataReductionProxy.UserViewedSavingsSize histogram for which code still exists. Bug: 870886 Change-Id: I2fa8ab3dec5ad15b87b57afd9d6f9e1df8ba04b5 Reviewed-on: https://chromium-review.googlesource.com/1162822Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: rajendrant <rajendrant@chromium.org> Cr-Commit-Position: refs/heads/master@{#580998}
-
Koji Ishii authored
This patch changes each debug message of js-test.js to produce a block box rather than an inline box. Before this patch, it can create a huge inline formatting context by appending <span>s, which LayoutNG is not very performant to handle at this point. LayoutNG should be able to handle incrementally expanding huge inline formatting context for textarea and some other use cases, but they should have own tests, and not every js-test.js should make this challenge. This change reduces 5 LayoutNG timeout failures. Bug: 636993 Change-Id: I5df429df64bf21dc051fe13600180511ca20ac22 Reviewed-on: https://chromium-review.googlesource.com/1163402Reviewed-by:
Emil A Eklund <eae@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#580997}
-
Matt Menke authored
When there's an error, the ProxyResolver would invoke a method on the ProxyResolverFactoryRequestClient pipe to pass along an error code and close both that pipe and the ProxyResolver pipe. The Job was listening for error on both pipes, so could see the ProxyResolver pipe close before getting a more useful error code on the ProxyResolverFactoryRequestClient pipe. This CL fixes that by just listening for the client pipe to close. Also, this CL closes the client pipe on completion, to protect against unexpected extra messages, if the Job isn't deleted immediately. Bug: 869551 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I2a1942d851e1d2cc4a33b0fb1278d914aac3593f Reviewed-on: https://chromium-review.googlesource.com/1159196Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#580996}
-
Jacob Dufault authored
All of the whitespace views will now shrink to width=0 in views-login, which means that login/lock will function better on extremely small displays. Bug: 839604 Change-Id: I86d67b718e339a7856f9a7c4d2527c8e1e742ee0 Reviewed-on: https://chromium-review.googlesource.com/1147510 Commit-Queue: Jacob Dufault <jdufault@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#580995}
-
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}
-