- 30 Aug, 2018 40 commits
-
-
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:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-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:
Scott Violet <sky@chromium.org> Reviewed-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:
Wez <wez@chromium.org> Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#587757}
-
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:
Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#587752}
-
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}
-
Clark DuVall authored
This is a reland of ac880b3a Switched to using ChromeDevToolsManagerDelegate::GetDefaultBrowserContext() which outlives TCPDeviceProvider. The original was reverted because of a crash when TCPDeviceProvider tried to access a destroyed BrowserContext, see http://crbug.com/879060. Original change's description: > Convert tcp_device_provider.cc to mojo host resolver > > Bug: 874653 > Change-Id: I95b968427344a4c9821c0985ae1f89c153036bcd > Reviewed-on: https://chromium-review.googlesource.com/1185626 > Commit-Queue: Clark DuVall <cduvall@chromium.org> > Reviewed-by: Andrey Kosyakov <caseq@chromium.org> > Cr-Commit-Position: refs/heads/master@{#586738} Bug: 874653 Change-Id: Ia6c00ba51020473f820add8c4257f3bf16fd5b24 Reviewed-on: https://chromium-review.googlesource.com/1197265Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#587750}
-
Clark DuVall authored
When WebRequestAPI is destroyed, it posts a task to the IO thread to shutdown the proxies. This crash can happen when the WebRequestAPI is destroyed, and a connection error happens on one of the proxies (which accesses data destroyed by the profile) before the ProxySet can be fully shutdown on the IO thread. This change adds an atomic flag to check to make sure the WebRequestAPI has not been destroyed before running error handlers. Bug: 878366 Change-Id: I2f7b98c1d6df7c53a5917444ac996e9b5ef4e794 Reviewed-on: https://chromium-review.googlesource.com/1196031Reviewed-by:
-
James Zern authored
https://chromium.googlesource.com/webm/libvpx.git/+log/0bfab06084e4..753fd86e86ac $ git log 0bfab0608..753fd86e8 --date=short --no-merges --format='%ad %ae %s' 2018-08-30 jzern Revert "Loopfilter MultiThread Optimization" Created with: roll-dep src/third_party/libvpx/source/libvpx R=johannkoenig@chromium.org Bug: webm:1558 Change-Id: I356a6b038ad5e77adf011dcef80e1cc1d62eb4fe Reviewed-on: https://chromium-review.googlesource.com/1197368Reviewed-by:
Johann Koenig <johannkoenig@google.com> Commit-Queue: James Zern <jzern@google.com> Cr-Commit-Position: refs/heads/master@{#587748}
-
Devlin Cronin authored
Reorder the context menu items for runtime host permissions, to be in ascending order of privilege. The new order is: - On click - On this site - On all sites Bug: 876779 Change-Id: Ib557668ec3aebb94c487896598065f3f4eee35a9 Reviewed-on: https://chromium-review.googlesource.com/1196106Reviewed-by:
-
Eric Orth authored
In order to avoid incompatible changes to RulesBasedHostResolverProc, which is used all over the place (see crbug.com/878838), created separate internal rules procs to handle those requests specifying different sources. Updated tests for the one piece of code currently specifying SYSTEM source (via flag through the old API). Bug: 846423 Change-Id: I1b92384d831579fe119817afc3e0cb92cfdf4565 Reviewed-on: https://chromium-review.googlesource.com/1195547 Commit-Queue: Eric Orth <ericorth@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#587746}
-
Tien-Ren Chen authored
Consider the following case: <span style="position:relative; left:-100px;"> <div style="display:inline-block; position:absolute;"></div> </span> In our implementation the absolute positioned element will have a counter offset applied, to ignore the negative offset on its rel-pos parent. The compensation was added in https://codereview.chromium.org/423173003/ which wasn't covered by the added test, and isn't consistent with other browser implementations. (Checked with Safari, IE, and Firefox.) BUG=398164 Change-Id: I44127ff42446a7483d7e4ebb3197ecdc915db39d Reviewed-on: https://chromium-review.googlesource.com/1195084Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Commit-Queue: Tien-Ren Chen <trchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#587745}
-
Leonard Grey authored
Bug: 543891 Change-Id: Idd78cbc45d2ab443437165d8511fec28b847e69c Reviewed-on: https://chromium-review.googlesource.com/1195828Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#587744}
-
Fredrik Söderquist authored
When a cubic or quadratic curve have a point that coincide with another of its point such that either the start or end tangent vector is degenerate, the orientation in that point will be 0. In such cases try to use the next/previous non-degenerate tangent vector (if any) instead. Bug: 450368 Change-Id: I35db404b3f82cf61b10427678feecc2157eeeb26 Reviewed-on: https://chromium-review.googlesource.com/1196503Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#587743}
-
Yue Cen authored
Specs based on b/112928364 and b/112934458. Bug: 835029 Change-Id: I58deccbbbd83a385407d02e208db3f112bed1fc6 Reviewed-on: https://chromium-review.googlesource.com/1188594 Commit-Queue: Yue Cen <rsgingerrs@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Reviewed-by:
Wenzhao (Colin) Zang <wzang@chromium.org> Cr-Commit-Position: refs/heads/master@{#587742}
-
Mohsen Izadi authored
This CL moves portions of GpuProcessHost that implement viz::mojom::GpuHost out of content into //components/viz/host/ as GpuHostImpl. GpuHostImpl also owns GpuServicePtr, so parts of GpuProcessHost that work directly with GPU service are also moved. Whenever needed, GpuHostImpl accesses content through its Delegate interface implemented by GpuProcessHost. This is mostly needed to access content::GpuDataManager. BUG=709332 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Change-Id: I5f27805aff16fe7db753c4700dbab5bf5ee0812d Reviewed-on: https://chromium-review.googlesource.com/1180658Reviewed-by:
Jonathan Backer <backer@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
David Reveman <reveman@chromium.org> Commit-Queue: Mohsen Izadi <mohsen@chromium.org> Cr-Commit-Position: refs/heads/master@{#587741}
-
Jan Wilken Doerrie authored
This change adds work arounds for platform limitations on Windows to the FidoCableDiscovery. In particular, multiple EIDs are not supported. Bug: 837008 Change-Id: Ic84fac8abef076697ea73c191a43a22045986bcf Reviewed-on: https://chromium-review.googlesource.com/1197365Reviewed-by:
Balazs Engedy <engedy@chromium.org> Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#587740}
-
Esmael El-Moslimany authored
Bug: 875461 Change-Id: I4198baa1461241de0f869cb63fb688a74ba5e337 Reviewed-on: https://chromium-review.googlesource.com/1194338Reviewed-by:
Scott Chen <scottchen@chromium.org> Commit-Queue: Esmael El-Moslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#587739}
-
Troy Hildebrandt authored
Adds the ability to record CPU frequency on Android devices, viewable in the Chrome trace viewer. Records the CPU frequency on a per-cluster basis, meaning for a Pixel/Pixel 2 we have two separate trace counters for CPU0-3, and CPU4-7. The sampler is a TraceLog::EnabledStateObserver, so on Android devices it will start and stop when tracing is enabled and disabled. Bug:873303 Change-Id: I1702fa184f16c53f200667528bb88e96957b0e65 Reviewed-on: https://chromium-review.googlesource.com/1123029 Commit-Queue: Troy Hildebrandt <thildebr@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
Siddhartha S <ssid@chromium.org> Cr-Commit-Position: refs/heads/master@{#587738}
-
Christopher Thompson authored
This adds the DownloadConnectionSecurity of the download and a boolean for whether the final download URL is a different host than the originating URL to the existing Download.Started UKM. This also refactors out the logic for checking the DownloadConnectionSecurity into a new public function (CheckDownloadConnectionSecurity) in download_stats.h and exposes the underlying DownloadConnectionSecurity enum. This will help us identify sites that have a lot of downloads over HTTP (or go over HTTP in their redirect chain) for outreach efforts. Bug: 839986 Change-Id: I9a1bb78b8c62547e322ec1b85b5acfaa7acc813a Reviewed-on: https://chromium-review.googlesource.com/1195854 Commit-Queue: Christopher Thompson <cthomp@chromium.org> Reviewed-by:
Robert Kaplow (slow) <rkaplow@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#587737}
-
Dale Curtis authored
8-bit YUV is no longer stored in 16-bit components with recent libaom builds. So we don't need this hack anymore. I've also cleaned up some TODOs and code structure. BUG=867613 TEST=unittests still pass. Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Change-Id: I7093240e4475ff4367e5f49681c0172e7ad11152 Reviewed-on: https://chromium-review.googlesource.com/1178938Reviewed-by:
Tom Finegan <tomfinegan@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#587736}
-
Nathan Parker authored
Bug: 849781 Change-Id: I09a37db757ea0a53c2e9bdddd6c0fdb23250ae6f Reviewed-on: https://chromium-review.googlesource.com/1180725 Commit-Queue: Nathan Parker <nparker@chromium.org> Reviewed-by:
Markus Heintz <markusheintz@chromium.org> Reviewed-by:
Jialiu Lin <jialiul@chromium.org> Cr-Commit-Position: refs/heads/master@{#587735}
-
pdfium-chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/e70aff80976e..678f5418d36f git log e70aff80976e..678f5418d36f --date=short --no-merges --format='%ad %ae %s' 2018-08-30 tsepez@chromium.org Fix lifetime issue in CCodec_ProgressiveDecoder. Created with: gclient setdep -r src/third_party/pdfium@678f5418d36f The AutoRoll server is located here: https://autoroll.skia.org/r/pdfium-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. BUG=chromium:879025 TBR=dsinclair@chromium.org Change-Id: Ib8c406eeae805bdb9a4e27c0fbe551a3a2368759 Reviewed-on: https://chromium-review.googlesource.com/1197287Reviewed-by:
pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#587734}
-
