- 30 Aug, 2018 40 commits
-
-
Sujie Zhu authored
Add Google Payments account check. We only offer credit card migration when user already has a Google Payments account. We get the PrefService from the Profile and then we can get whether user has a google payments accout from the PrefService. Bug: 852904 Change-Id: I06d326417cdc4fded5a15635f8883ebabbe0a1f5 Reviewed-on: https://chromium-review.googlesource.com/1196013Reviewed-by:
Jared Saul <jsaul@google.com> Reviewed-by:
Scott Chen <scottchen@chromium.org> Commit-Queue: Sujie Zhu <sujiezhu@google.com> Cr-Commit-Position: refs/heads/master@{#587790}
-
Jennifer Apacible authored
Change-Id: I29fd4a91e943ecbda810c1ae4085a5476896e3f8 Reviewed-on: https://chromium-review.googlesource.com/1197288Reviewed-by:
CJ DiMeglio <lethalantidote@chromium.org> Commit-Queue: apacible <apacible@chromium.org> Cr-Commit-Position: refs/heads/master@{#587789}
-
Jennifer Apacible authored
The buttons on other platforms handle the space key as well. This change only adds the handling to toggle play/pause to Mac. The close button is not handled since tabbing is not handled and there are no visual affordances for which button is focused on Mac. These exist on other platforms. Bug: 879139 Change-Id: I40135a7c70fd8d737a2b3b368bfad968d0eeb225 Reviewed-on: https://chromium-review.googlesource.com/1196931Reviewed-by:
-
Tom Anderson authored
The new instruction bubble's recommendation to "press Alt+F" won't work on non-browser windows, so don't show the instructions unless the focused window is a browser window and has a toolbar. BUG=243164 R=sky Change-Id: I1ab550ff57134009a2cbb0a1ee612775c1323e88 Reviewed-on: https://chromium-review.googlesource.com/1197343Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#587787}
-
Tom Anderson authored
The message always used to say "Press Alt+F followed by X to exit". The "X" comes from the menu option "Exit". But the accelerator may be different in non-English languages. This CL internationalizes the accelerator. BUG=243164 R=sky Change-Id: I695979eef5ea15f1764b891611b2c513302c4c0e Reviewed-on: https://chromium-review.googlesource.com/1194976Reviewed-by:
-
Alexei Filippov authored
BUG=878604 Change-Id: I20d281d9b03fbb41c2cac72586a60949fc73a860 Reviewed-on: https://chromium-review.googlesource.com/1197269Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Mike Wittman <wittman@chromium.org> Commit-Queue: Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#587785}
-
Sahel Sharify authored
This reverts commit fa22a78c. Reason for revert: I reverted the initial cl to prove that crash reports in crbug.com/868849 are not caused by it. Relanding it now since the crashes still happened after the revert. Original change's description: > Revert "Clean up DidStopFlinging IPC" > > This reverts commit 7cb247dd. > > Reason for revert: Check to see if the crash report in crbug.com/868849 goes away or not. > > Original change's description: > > Clean up DidStopFlinging IPC > > > > This cl also removes TouchscreenFlingInProgress function that is no > > longer needed. > > > > Bug: 249063 > > Change-Id: Icc939d654cdfe0cf38c843933f2fd469f843b74c > > Reviewed-on: https://chromium-review.googlesource.com/1145784 > > Reviewed-by: Dave Tapuska <dtapuska@chromium.org> > > Reviewed-by: Ken Buchanan <kenrb@chromium.org> > > Reviewed-by: Philip Rogers <pdr@chromium.org> > > Commit-Queue: Sahel Sharify <sahel@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#578410} > > TBR=kenrb@chromium.org,pdr@chromium.org,dtapuska@chromium.org,sahel@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: 249063 > Change-Id: Iee7200d3c4e388fccbc2dc6b86997a07a3a03f0a > Reviewed-on: https://chromium-review.googlesource.com/1155169 > Reviewed-by: Sahel Sharify <sahel@chromium.org> > Commit-Queue: Sahel Sharify <sahel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#579166} TBR=kenrb@chromium.org,pdr@chromium.org,dtapuska@chromium.org,sahel@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 249063 Change-Id: I47b3d00615ebb29b643aa03ef73b523381c092c6 Reviewed-on: https://chromium-review.googlesource.com/1197334Reviewed-by:
Sahel Sharify <sahel@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Commit-Queue: Sahel Sharify <sahel@chromium.org> Cr-Commit-Position: refs/heads/master@{#587784}
-
Vladislav Kaznacheev authored
- show auto-hide related menu option in tablet mode - respect auto-hide bit in tablet mode if it is set in clamshell mode - update test accordingly ash_unittests --gtest_filter=Shelf* Bug: 878039 Test: manual, ash_unittests --gtest_filter=TabletModeWindowManagerTest*, Change-Id: I8a8688e794e243e4b2da026803dff9e136f3c1f1 Reviewed-on: https://chromium-review.googlesource.com/1196142 Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Alex Newcomer <newcomer@chromium.org> Reviewed-by:
Sammie Quon <sammiequon@chromium.org> Cr-Commit-Position: refs/heads/master@{#587783}
-
Kevin Marshall authored
This CL adds support for touch/gesture input (touch, swipe, pinch, etc.) to the ContextProvider service. Also included in this CL: * Adds input focus event handling. * Populate Screen/Display dimensions on window size events. * Miscellaneous IWYU fixes. Bug: 876933, 878439,829980 Change-Id: Ic4e5a8123bac2360e40f2affe6e1ec1e242260d4 Reviewed-on: https://chromium-review.googlesource.com/1194973 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#587782}
-
Wez authored
Temporarily add some values to minidumps, to help diagnose crash reports under SkDraw::drawPaint and related signatures. Bug: 870663 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I84f3aed1c50c8e44bc882e6655688bd26c0e4edc Reviewed-on: https://chromium-review.googlesource.com/1196226Reviewed-by:
Khushal <khushalsagar@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#587781}
-
Johann authored
Cherry pick rtcd-related changes and re-run cmake_update.sh This allows the linker to remove unused C code when an sse2 version is available. See discussion on libvpx: https://chromium-review.googlesource.com/c/chromium/src/+/1195800 R=tomfinegan@chromium.org Change-Id: I419a87e6d0b4f9166754aaaedf6b9c8cd1834dae Reviewed-on: https://chromium-review.googlesource.com/1197466Reviewed-by:
Tom Finegan <tomfinegan@chromium.org> Commit-Queue: Johann Koenig <johannkoenig@google.com> Cr-Commit-Position: refs/heads/master@{#587780}
-
Trent Apted authored
A call to __CFPasteboardDeallocate is triggering "BUG IN CLIENT OF LIBDISPATCH: Semaphore object deallocated while in use" when it invokes _dispatch_semaphore_dispose. This appears to happen under ~UniquePasteboard in Chrome. Try moving ownership of the NSPasteboard the autorelease pool to avoid triggering the call to -[NSPasteboard dealloc] deep in the drag-tracking stack. Bug: 877979 Change-Id: I0f5ff067da31870fd62591ccdd9c3731377d73e2 Reviewed-on: https://chromium-review.googlesource.com/1192865Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#587779}
-
Sujie Zhu authored
We should not include the prompt experiment here because local card migration doesnot have the required messaging to show the shorten tos. Bug: 852904 Change-Id: I4c3c7a9bcee1d24a97dbd23dea2924cd3eef1a3b Reviewed-on: https://chromium-review.googlesource.com/1197370 Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org> Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#587778}
-
Ryan Cui authored
Verified functionality on an Eve. This will need to be revisited for Mash. BUG: 796366 Change-Id: I7b32728ff2440a8f5cda9b5ac0c28ad30182b145 Reviewed-on: https://chromium-review.googlesource.com/1197367Reviewed-by:
-
Thomas Tangl authored
The following methods are renamed to make the difference between opt-in and feature-enabled more clear: * IsUnifiedConsentEnabled() -> IsUnifiedConsentFeatureEnabled() * IsUnifiedConsentBumpEnabled() -> IsUnifiedConsentFeatureWithBumpEnabled() Note: For checking the unified consent opt-in UnifiedConsentService::IsUnifiedConsentGiven must be used. Bug: 878659 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I1ff2342abb0ebb058ac6ba787c95ddba33d53be5 Reviewed-on: https://chromium-review.googlesource.com/1194370Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Thomas Tangl <tangltom@chromium.org> Cr-Commit-Position: refs/heads/master@{#587776}
-
Jeffrey Kardatzke authored
https://chromium.googlesource.com/chromiumos/platform/system_api.git/+log/71adbf6a5686..a71cda985de2 $ git log 71adbf6a5..a71cda985 --date=short --no-merges --format='%ad %ae %s' 2018-08-22 semenzato system_api: dbus: add metrics_event protobuffer 2018-08-16 jkardatzke Add protobuf for mime type mappings in Crostini. 2018-08-22 chirantan seneschal: Update proto comments 2018-08-07 emaxx diagnostics: API for bootstrapping Mojo to browser Created with: roll-dep src/third_party/cros_system_api Bug: N/A Change-Id: I7e5d49f74f31b8c959788cd3edd5a079f87ee315 Reviewed-on: https://chromium-review.googlesource.com/1197362Reviewed-by:
Mike Frysinger <vapier@chromium.org> Commit-Queue: Jeffrey Kardatzke <jkardatzke@google.com> Cr-Commit-Position: refs/heads/master@{#587775}
-
Luna Lu authored
The CL https://chromium.googlesource.com/chromium/src/+/bd4c33b5e81d495ecdac66b0375924d9565330cb did not handle responsive images correctly. This CL resets overridden_intrinsic_size_: When source is chosen using W descriptor, 'sizes' sets the new width of the overridden_intrinsic_size_, and the new height is calculated by the aspect ratio defined by the intrinsicSize attribute. Bug: 874629 Change-Id: I16b2f6c2fb9ae741144ba509c57f1e033c8cdc17 Reviewed-on: https://chromium-review.googlesource.com/1195801 Commit-Queue: Luna Lu <loonybear@chromium.org> Reviewed-by:
Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#587774}
-
Michael Giuffrida authored
Import sample photos from the Demo Mode Resources image into the Downloads directory. This makes browsing File Manager a little more interesting and provides pre-populated images for the Google Photos app. Tested by adding media/photos/{foo,bar,baz}.jpg into the offline resources component and installing on a device in Demo Mode. Bug: 870855 Change-Id: I77b4a664d5714cef1f8aeb524863afe70e891800 Reviewed-on: https://chromium-review.googlesource.com/1196223 Commit-Queue: Michael Giuffrida <michaelpg@chromium.org> Reviewed-by:Aga Wronska <agawronska@chromium.org> Cr-Commit-Position: refs/heads/master@{#587773}
-
Christopher Cameron authored
Create a new component //ui/views_bridge_mac for the Cocoa parts of the Cocoa implementation of views. Move the interface views::BridgedNativeWidgetPublic from being a C++ pure virtual interface to the mojo interface BridgedNativeWidget in views_bridge_mac. Note that the concrete implementation of this interface, views::BridgedNativeWidget is to be renamed to BridgedNativeWidgetImpl, and moved to cocoa_viewer. Bug: 859152 Change-Id: I3d97502285e81d603948672e9789366cd09d6b3d Reviewed-on: https://chromium-review.googlesource.com/1194498 Commit-Queue: ccameron <ccameron@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#587772}
-
Xianzhu Wang authored
BoxModelObjectPainter::PaintTextClipMask() is called from Paint() method of some painter which has already adjusted for paint offset and paint properties, so we should not go back to call box_model_.Paint() again, but should call LineBoxListPainter instead. This fixes an old FIXME which was too bad before we switched to SPv175 paint offsets. Bug: 878685 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I78f3a82a133b14eb56dbeaf2b8b5f81041c0c59e Reviewed-on: https://chromium-review.googlesource.com/1197462Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#587771}
-
Fabio Tirelo authored
This CL replicates on Android the following changes already done on other platforms: - Replace the Autofill toggle with two new toggles, one for credit cards and one for addresses, and add them to the corresponding pages in Settings; - Update the strings for titles, toggles, and toggle descriptions; - Delete deprecated strings. This takes up crrev.com/c/1135514, which was abandoned. Comments initially added there have also been addressed here. A follow-up CL will get rid of the "Autofill and payments" page and move the "Addresses" and "Payments" page to the top Settings page. Screenshots and video (Googlers only): https://drive.google.com/open?id=1ZZr2lbX9q48DAxiTSVq6dZCImchqz-yA Bug: 860526 Change-Id: I613f3badfb655532228ee1b6b6029eac06f1d42f Reviewed-on: https://chromium-review.googlesource.com/1194474 Commit-Queue: Fabio Tirelo <ftirelo@chromium.org> Reviewed-by:
Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Mathieu Perreault <mathp@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#587770}
-
Devlin Cronin authored
Update the context menu strings to the new spec. The submenu title is exceptionally long, but it's the best we have. Bug: 876779 Change-Id: I0d97caa14061b722c958e82e2a189c7b5f6588f2 Reviewed-on: https://chromium-review.googlesource.com/1195940Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#587769}
-
Tommy Steimel authored
This reverts commit 936898b9. Reason for revert: The null checks are not solving the crash. Must be a deeper issue where the style object is garbage Original change's description: > [Media Controls] Reland Prevent ComputedStyleRef crash > > This CL re-adds the null checks for internal_button_ and > GetComputedStyle on the overlay play button. These were added to > prevent a crash, then removed when the crash was not fixed and replaced > with a different null check. However, it looks like we actually need > these too, so we're adding them back. > > Bug: 870490 > Change-Id: I1c6ad4b3f1dc36c7a2149a06c4f42f8ca00fb589 > Reviewed-on: https://chromium-review.googlesource.com/1187710 > Reviewed-by: Becca Hughes <beccahughes@chromium.org> > Commit-Queue: Tommy Steimel <steimel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#585719} TBR=beccahughes@chromium.org,steimel@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 870490 Change-Id: I93d49a3a2071ce5fcca99b68f8cbaf2c3720f889 Reviewed-on: https://chromium-review.googlesource.com/1197346Reviewed-by:
Tommy Steimel <steimel@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#587768}
-
Mustafa Emre Acer authored
The current code uses NavigationEntryCommitted, but it should have used DidNavigationFinish instead. Bug: 843361, 847662 Change-Id: Ie16016844e636a14c2114450a6e3b2a94cd2529b Reviewed-on: https://chromium-review.googlesource.com/1197083Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: Mustafa Emre Acer <meacer@chromium.org> Cr-Commit-Position: refs/heads/master@{#587767}
-
David Black authored
See associated CL and description details in: https://chrome-internal-review.googlesource.com/c/chrome/assistant/+/669327 Bug: b:113541754 Change-Id: Icbab2c753c0e3b0c3f3797a5a85f52fb210e9cf3 Reviewed-on: https://chromium-review.googlesource.com/1196222Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: David Black <dmblack@google.com> Cr-Commit-Position: refs/heads/master@{#587766}
-
Krishna Govind authored
R=abdulsyed@chromium.org Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I69369c19a93e11f408240705c6aa3af803775ef5 Reviewed-on: https://chromium-review.googlesource.com/1197622Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#587765}
-
Meilin Wang authored
- Fix the corner radius of minimized Assistant UI mode. - Paint shadow around the rounded rectangle border while resizing. Test: local compile and manually test. Bug: b/112312304 Change-Id: Ibf07d06564b6fa82790fcc5ac7f656c45e4ac7da Reviewed-on: https://chromium-review.googlesource.com/1188899 Commit-Queue: Meilin Wang <meilinw@chromium.org> Reviewed-by:
-
David Bertoni authored
This CL integrates the new enhanced keepalive data into the JSON output for the chrome://extensions-internals page. Bug: 695711 Change-Id: I7cc3b09dbccd2fef9379f2cb68939f4e029136cf Reviewed-on: https://chromium-review.googlesource.com/1196169 Commit-Queue: David Bertoni <dbertoni@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#587763}
-
Gordana Cmiljanovic authored
This will fix compilation error: ../../third_party/libdrm/src/tests/modetest/modetest.c:388:11: error: format specifies type 'unsigned long long' but the argument has type '__u64' (aka 'unsigned long') [-Werror,-Wformat] prop->enums[i].value); ^~~~~~~~~~~~~~~~~~~~ Change-Id: Id9881f6ae1831812e851f0d1e62998c383355430 Reviewed-on: https://chromium-review.googlesource.com/1194068Reviewed-by:Dirk Pranke <dpranke@chromium.org> Commit-Queue: Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#587762}
-
Peter Kotwicz authored
This CL moves the Android Manifest generation via manifest_processor_template into the webapk_tmpl GN template. This CL is in preparation to adding a new webapk_tmpl() GN target for generating a WebAPK which shows the splash screen within the WebAPK's activity. BUG=817263 Change-Id: I83b1292da2869dc1d8de784d137cba75962633c6 Reviewed-on: https://chromium-review.googlesource.com/1183814 Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org> Reviewed-by:
Xi Han <hanxi@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#587761}
-
Alexey Kozyatinskiy authored
One of the next V8 rolls will contain fix for source location of end of source syntax error. We need to disable to temporary disable ProxyResolverV8Test.ParseError test. R=eroman@chromium.org Bug: v8:8092 Change-Id: I93ad80167c5af4bca8f1d768b42cbea0b7833e55 Reviewed-on: https://chromium-review.googlesource.com/1196863 Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org> Reviewed-by:
Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#587760}
-
Manu Cornet authored
Reland without any modifications since the MSAN issue was fixed separately in CL 1196426 This is a reland of 6442a850 Original change's description: > CrOS shelf: switch to feature mechanism and enable by default > > Fix a lot of tests that hardcode the size of the shelf. > > Bug: 871846 > Change-Id: I586da09835f03beb3e92712fbf32ebefd521a5d5 > Reviewed-on: https://chromium-review.googlesource.com/1188848 > Commit-Queue: Manu Cornet <manucornet@chromium.org> > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > Reviewed-by: Alex Newcomer <newcomer@chromium.org> > Cr-Commit-Position: refs/heads/master@{#587038} Bug: 871846,878952 Change-Id: Iafe30ae18feb8b5dc6f7ba11c367ea8a0651e840 Reviewed-on: https://chromium-review.googlesource.com/1197402Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium 736d6059. With Chromium commits locally applied on WPT: 29a89cda "Import all remaining cookie tests into WPT" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/23663 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: binji@chromium.org: external/wpt/wasm foolip@chromium.org, robertma@chromium.org, smcgruer@chromium.org: external/wpt/infrastructure rouslan@chromium.org, mathp@chromium.org: external/wpt/payment-request NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: Ia125a4d274df950d2fc9d5d23f95cb03792fcc96 Reviewed-on: https://chromium-review.googlesource.com/1197345 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@{#587758}
-
Ian McKellar authored
Called set_data_enabled(true) on net::URLRequestContextBuilder. Added a simple test to validate that a Frame can navigate to a data: URI. Change-Id: I919140b090e71e75a7c6e1137b4267a61ecf177e Reviewed-on: https://chromium-review.googlesource.com/1195843Reviewed-by:
Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
-
Min Chen authored
Bug: 878282 Change-Id: I05461e44d6eb07fb5b6e83882aeb2adc675e3772 Reviewed-on: https://chromium-review.googlesource.com/1197369 Commit-Queue: Min Chen <minch@chromium.org> Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Reviewed-by:
-
Jian Li authored
Bug: 852577 Change-Id: I5d929cd04499a9c694c72ab5607871455ff65169 Reviewed-on: https://chromium-review.googlesource.com/1196011Reviewed-by:
Peter Williamson <petewil@chromium.org> Commit-Queue: Jian Li <jianli@chromium.org> Cr-Commit-Position: refs/heads/master@{#587755}
-
Yuwei Huang authored
We have found a few crashes on Chromoting mobile clients that are related to a refactoring to replace URLFetcher with ChromiumUrlRequest: * Apps crash at launch because mojo core is not initialized. * TransitionalURLLoaderFactoryOwner and the SharedURLLoaderFactory it creates seem to be thread unsafe, so the app crashes because of some thread checker failures. These issues are fixed by: * Calling mojo::core::Init() in the client runtime. * Make url_loader_factory() only being called on the network thread and guard it with DCHECK. Bug: 878919 Change-Id: I3827b5175546aa9845b53e95cbd913043028b59e Reviewed-on: https://chromium-review.googlesource.com/1196026Reviewed-by:
Ken Rockot <rockot@chromium.org> Reviewed-by:
Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Joe Downing <joedow@chromium.org> Commit-Queue: Yuwei Huang <yuweih@chromium.org> Cr-Commit-Position: refs/heads/master@{#587754}
-
Chong Zhang authored
This patch does 3 things to fix the test: 1. Register listeners before loading the actual guest page. See bug for more background. 2. Add |was_fetched_via_cache| to |ResourceResponseInfo|. 3. Call |NetworkContext::ClearHttpCache()| in |WebViewGuest::ClearData()| when Network Service was enabled. We cannot use |BrowsingDataRemover| because it doesn't support non-default |StoragePartition|. Bug: 878060 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I5f0ea8e62ee8f44944428cfb041f9d4e427bd67c Reviewed-on: https://chromium-review.googlesource.com/1195596Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Clark DuVall <cduvall@chromium.org> Commit-Queue: Chong Zhang <chongz@chromium.org> Cr-Commit-Position: refs/heads/master@{#587753}
-
May Lippert authored
This change adds histograms for the total time from when the update is first seen by the client to when the update is downloaded (but before the device has rebooted). There are two histograms, one for the total time in minutes when the client has a time restriction policy for updates, and one for when the client has no policy in place. These two histograms are only reported for enterprise devices. CL where the histogram is used: https://chromium-review.googlesource.com/c/aosp/platform/system/update_engine/+/1179230 BUG=chromium:852860 TEST=Run through scenarios and checked chrome://histograms Change-Id: Ib4b8d0126087d5951b79d7f4d2022c2c776744a7 Reviewed-on: https://chromium-review.googlesource.com/1181649 Commit-Queue: May Lippert <maybelle@chromium.org> Reviewed-by:
-
Bailey Berro authored
The FileSystemProviderApiTests assume that there are no native providers and that the test extension is the only fsp. Once the SMB FSP is enabled, this will no longer be the case so we need to explicitly filter out native fsps in tests. Bug: chromium:757625 Change-Id: I8ba9a8089d5b00aa8bfd3206ced9d60a4f2d05ce Reviewed-on: https://chromium-review.googlesource.com/1195883Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Commit-Queue: Bailey Berro <baileyberro@chromium.org> Cr-Commit-Position: refs/heads/master@{#587751}
-
