- 22 Aug, 2019 40 commits
-
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/02d4c0d38ea1..243997ac5d09 git log 02d4c0d38ea1..243997ac5d09 --date=short --no-merges --format='%ad %ae %s' 2019-08-22 capn@google.com Don't access sampler handles for sampled image descriptors 2019-08-22 capn@google.com Fix samplerless image fetch Created with: gclient setdep -r src/third_party/swiftshader@243997ac5d09 The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=swiftshader-team+autoroll@google.com Bug: chromium:b/139401791,chromium:b/139401791 Change-Id: I95db1ceb4f2a85e4db040efa6a10695c9168a240 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766168Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#689630}
-
Peng Huang authored
Bug: None Change-Id: I884771b8a060b4001ba997e4b504a9d391b4622b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765990Reviewed-by:
enne <enne@chromium.org> Commit-Queue: Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#689629}
-
Steven Bennetts authored
Bug: 853953 Change-Id: Ie6062b8baa01c76f3ff00c0a88bbe69c9dd74dc1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757171 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#689628}
-
Wei-Yin Chen (陳威尹) authored
Also update the default parameter of zooming-min-sdk-version to M. TBR=twellington@chromium.org Bug: 964406 Change-Id: I3a23e20511f9ecb48ed70fd6952f36b06191817c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1743106 Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Reviewed-by:
Mei Liang <meiliang@chromium.org> Cr-Commit-Position: refs/heads/master@{#689627}
-
Mike Wittman authored
This reverts commit e4fe4ff4. Reason for revert: causes timeouts in IsolateOriginsPrintBrowserTest.PrintIsolatedSubframe Original change's description: > Activate API based fallback by default > > This will prefer performing DWrite API based font fallback on all > Windowses, particularly on Windows 8.0 and below, where API based > fallback was previously not available and hard-coded lookup tables were > used. A content feature is used to flip the switch on this as a > last-resort deactivation method, should this lead to unexpected fallback > problems. > > Bug: 976737 > Change-Id: I65d160a573a625ea60a866bacfd12dd0d8a5935a > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1753006 > Commit-Queue: Dominik Röttsches <drott@chromium.org> > Commit-Queue: Avi Drissman <avi@chromium.org> > Auto-Submit: Dominik Röttsches <drott@chromium.org> > Reviewed-by: Avi Drissman <avi@chromium.org> > Cr-Commit-Position: refs/heads/master@{#688281} TBR=avi@chromium.org,drott@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 976737, 996185 Change-Id: Ifc648163b1414f39f8277b8b963a2d73de3b146b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765993Reviewed-by:
Mike Wittman <wittman@chromium.org> Commit-Queue: Mike Wittman <wittman@chromium.org> Cr-Commit-Position: refs/heads/master@{#689626}
-
David Black authored
Also wires up AssistantUiController to observe changes to the cache in ash in order to show/hide the associated view as necessary. Follow up CLs will wire up the associated view to be in sync with the live data in the cache (currently it stubs in placeholders) as well as to handle the entry point to show content in Assistant UI. Bug: b:139199754 Change-Id: I4bee76004fee1782abc70eab0e9ddb9f8695c648 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762341 Commit-Queue: David Black <dmblack@google.com> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#689625}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/679dc3716bdb..2662d14861da git log 679dc3716bdb..2662d14861da --date=short --no-merges --format='%ad %ae %s' 2019-08-22 achuith@chromium.org Revert "Support minidump symbolization on ChromeOS" Created with: gclient setdep -r src/third_party/catapult@2662d14861da The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=rmhasan@google.com Bug: chromium:978952 Change-Id: I35b79039982a2fc9b71ef28d4c4db01122e40b72 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765832Reviewed-by:
-
Morten Stenshorne authored
This is in preparation for column-span:all support, where we'll need to support multiple column rows within one and the same multicol container fragment. Change-Id: Ia1189ea1526faa37d1a0c51b2dadfc7a9d40b0bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765509 Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#689623}
-
Robbie McElrath authored
Bug: 934152 Test: gn check out/Default //android_webview/public Change-Id: I84a2ef997f65644b6c836b5d501d9da02dd579cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764341 Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Reviewed-by:
Changwan Ryu <changwan@chromium.org> Reviewed-by:
Nate Fischer <ntfschr@chromium.org> Cr-Commit-Position: refs/heads/master@{#689622}
-
Wei-Yin Chen (陳威尹) authored
After switching to another tab from GTS, a thumbnail of the original tab is taken. However, the thumbnail taking would be blocked if the web contents of the original tab is detached. Therefore, when showing the GTS-to-Tab transition animation to another tab, we cannot detach the original tab. The original tab is supposed to be invisible, but the tab decoration was not properly hidden. This CL hides the decoration of that invisible tab. Bug: 995992 Change-Id: Ia55cc21ddcbbc59674fa8c8268faaddc458cd4a3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764660Reviewed-by:
-
Devon O'Brien authored
Adding Google Argon2022, Argon 2023, Xenon 2023 CT Logs to list of Qualified CT Logs in Chrome. Additionally, incorporate recent Qualified --> Usable Log states. Bug: 889033, 833350 Change-Id: Iaca8867da9de61cb5a03e6323e14e8b9df0b5c2f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764834 Commit-Queue: Ryan Sleevi <rsleevi@chromium.org> Reviewed-by:
Ryan Sleevi <rsleevi@chromium.org> Cr-Commit-Position: refs/heads/master@{#689620}
-
Steven Bennetts authored
This CL: * Gets the managed properties via mojo for use within the details page. (Properties from networkingPrivate are still fetched to support sub elements which will be converted in forthcoming CLs). * Uses mojo versions of the policy JS in the details page * Eliminates ManagedThirdPartyVPNProperties, folding the properites into ManagedVPNProperties for consistency with state properties. * Additional cleanup of onc_mojo.js. Note: The result of this CL will temporarily add an extra API call in the details page. However, showing the details page for a network is an infrequent operation, and the extra API call has minimal overhead (O(10) ms on a relatively slow coral device). Bug: 853953 Change-Id: Ifbecea8d55567bec987554657326838ee16315b4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757070 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
-
Raymond Toy authored
WebAudio legacy code often does something like: DCHECK(foo); if (!foo) return; The style guide says the if/return should be removed; DCHECK failures should not be handled. Part 3 of N: Update a few more files and for the most obvious cases where we can remove the if/return. Bug: 957654 Change-Id: Ib9f66273db36dc4b6b2333e15ebb965251927c80 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763259 Commit-Queue: Raymond Toy <rtoy@chromium.org> Reviewed-by:Hongchan Choi <hongchan@chromium.org> Cr-Commit-Position: refs/heads/master@{#689618}
-
Xing Liu authored
This CL applies the custom time window in schedule parameters to select the notification to show. Also add unit test cases in DisplayDeciderTest. Bug: 991010 Change-Id: I843eff284176635bfaf8ec7ed258b1b9793e299b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1755043 Commit-Queue: Xing Liu <xingliu@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#689617}
-
Sadrul Habib Chowdhury authored
Do not allow running chrome with swiftshader for the rendering benchmark, since the produced metrics wouldn't be generally useful. BUG=988045 Change-Id: I58d765ff056ad220912fc6a1a258c706c09a50c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1734190 Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Cr-Commit-Position: refs/heads/master@{#689616}
-
Eric Roman authored
Use 0.0.0.1:1 in place of 0.0.0.0:8000 as the address of the proxy server. The test expects connections to the proxy server to fail - however connecting to the ANY address (0.0.0.0) can succeed. The new literal should fail consistently since: (1) 0.0.0.1 is a reserved IP address. (2) We disallow connections on TCP port 1. Bug: 996367 Change-Id: I54ce278e75983110961da0cc49dd923decdd5ae2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765662Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#689615}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/4c55b355580f..7f90416c08be git log 4c55b355580f..7f90416c08be --date=short --no-merges --format='%ad %ae %s' 2019-08-22 ehmaldonado@chromium.org depot_tools: Use git-checkout instead of git-rebase. Created with: gclient setdep -r src/third_party/depot_tools@7f90416c08be The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=agable@chromium.org Bug: chromium:996359 Change-Id: I31f04d609746e82cdc60e6ecedaba6f87a2bdeed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765989Reviewed-by:
-
Erik Chen authored
Previously, WebView was created by RenderView, but destroyed by RenderWidget. The call to destory the WebView in RenderWidget was shared with a call to destroy other WebWidget subclasses, even though that semantically represents a very different operation. This CL is a refactor with minimal intended behavior change [slight destruction timing change]. It makes the RenderView both create and destroy the WebView. Change-Id: I1d84f2c5cc1eff9558ffcbf4b5384f2f10a8d71b Bug: 995981, 419087 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764757 Auto-Submit: Erik Chen <erikchen@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#689613}
-
Justin Donnelly authored
Both are committers, have been on the omnibox team for over a year, and have demonstrated good knowledge and judgement in code reviews. Change-Id: I88c1aa50cff969a4afb6a958e5ac2c01b69a385f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765658Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#689612}
-
W. James MacLean authored
This CL removes existing code that was intended to prevent OOPIF over-rendering by limiting LayerVisibleRect() for the OOPIF's content layer to the viewport intersection sent from the browser. This is being removed in advance of new work to fix the problem by making |device_viewport_size_| on LayerTreeHost a gfx::Rect instead, and having it define the required intersection. Bug: 852348 Change-Id: I15426c08cfc7eb09a99b30811421b0ab613ab8f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764238Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: James MacLean <wjmaclean@chromium.org> Cr-Commit-Position: refs/heads/master@{#689611}
-
Bettina authored
There was a check for tests where primary account is not syncing and we should not show a modal warning on password reuse. However, this should not be the case since when not syncing, there is no primary account. Bug: 914410 Change-Id: Ia2ece03990aba58501d207f96c6fd1c5ed5b0abc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762829 Commit-Queue: Bettina Dea <bdea@chromium.org> Reviewed-by:
Varun Khaneja <vakh@chromium.org> Cr-Commit-Position: refs/heads/master@{#689610}
-
Patrick To authored
Bug: 989254 Change-Id: I8f2ad0214cc17c324774e08ee34baba87dae7f42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1745502 Commit-Queue: Patrick To <patrto@microsoft.com> Reviewed-by:
Alexander Cooper <alcooper@chromium.org> Cr-Commit-Position: refs/heads/master@{#689609}
-
Synthia Islam authored
Add comment like "// this file is autogenerated, do not edit" at the top of the files generated from .mojom files Bug: 609794 Change-Id: I1b39afc0d7c054510f7446af8795e606f07fdfdb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757406Reviewed-by:
Ken Rockot <rockot@google.com> Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#689608}
-
Jun Cai authored
This CL updates the title icon of the sms infobar based on the mock. I uploaded a screenshot on the issue page Bug: 995434 Change-Id: I428682330936fffc13df19871fc3bf22bfa7e838 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761256Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Ayu Ishii <ayui@chromium.org> Commit-Queue: Jun Cai <juncai@chromium.org> Cr-Commit-Position: refs/heads/master@{#689607}
-
Olivier Yiptong authored
After an atomic write finalizes, this applies Mark-Of-The-Web on the resulting file. This is done via the Quarantine component, which on Mac OS X, simply applies some file metadata. On Windows, the Quarantine component may call the AttachmentServices which could start an Anti-Virus check and that may cause the file to be deleted if malware is found. Bug: 986588 Change-Id: Iebefe25abbc32501b55e29bc8196f8a049dee2c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1737163 Auto-Submit: Olivier Yiptong <oyiptong@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Asanka Herath <asanka@chromium.org> Commit-Queue: Olivier Yiptong <oyiptong@chromium.org> Cr-Commit-Position: refs/heads/master@{#689606}
-
Alex Cooper authored
This adds support to resolve required/optional features against device capabilities as well as sets up the plumbing for appropriately tailored consent. A WPT test for lack of device support was also added, which required updating the tests. Bug: 991605 Change-Id: Ibce6924492311544fd75ca36ed79aa855c943aad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757183 Auto-Submit: Alexander Cooper <alcooper@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Jacob DeWitt <jacde@chromium.org> Reviewed-by:
Brandon Jones <bajones@chromium.org> Cr-Commit-Position: refs/heads/master@{#689605}
-
Shimi Zhang authored
Per AndroidX API design guide, put single abstract method (SAM) to be the last parameter so kotlin could interop with them. Rename removeWebMessageListener() to unsetWebMessageListener(). Bug: 918065 Change-Id: Ief2d7fa80a4cd48185487a39b0463511cc9c6311 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764260Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
-
Jun Cai authored
Per discussion at: https://chromium-review.googlesource.com/c/chromium/src/+/1751966/18/content/browser/sms/sms_service_unittest.cc#458 The SmsServiceTest.SecondRequestTimesOutEarlierThanFirstRequest can be deleted. The test that makes sure there is at most one pending request already exists: SmsServiceTest.AtMostOnePendingSmsRequest https://cs.chromium.org/chromium/src/content/browser/sms/sms_service_unittest.cc?rcl=371f17218c2bd68a8e4ec467f78efb96df8ffa71&l=273 Bug: 996831 Change-Id: I0bf54e959dfb13f81f9ea4ee9f01fde6f5ea52cc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764561Reviewed-by:
-
chrome://omniboxMoe Ahmadi authored
Bug: 996516 Change-Id: I741bdc946d07619afa55c45e1cf2969bbd492fde Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764106Reviewed-by:
Mark Pearson <mpearson@chromium.org> Reviewed-by:
manuk hovanesian <manukh@chromium.org> Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Auto-Submit: Moe Ahmadi <mahmadi@chromium.org> Cr-Commit-Position: refs/heads/master@{#689602}
-
Bailey Berro authored
- Removes bold weight on network name - Removes unneccessary styling for NetworkTitleText font-color - Removes unneccessary styling for NetworkState font-weight Spec: https://screenshot.googleplex.com/zX25XV2M5pE.png Bug: 996496 Change-Id: I720932a1fc86d44fe9f62431827d428671eccdc7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765660 Auto-Submit: Bailey Berro <baileyberro@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Bailey Berro <baileyberro@chromium.org> Cr-Commit-Position: refs/heads/master@{#689601}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/068af9e416cb..2b39cbe8619a git log 068af9e416cb..2b39cbe8619a --date=short --no-merges --format='%ad %ae %s' 2019-08-22 scroggo@google.com Remove SK_LEGACY_HEIF_API 2019-08-22 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/skia@2b39cbe8619a The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=egdaniel@google.com Bug: chromium:b/78868457,chromium:b/120414514 Change-Id: I175b234afb8df2e0fd7b3d1fcb73dfbd7f890729 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765002Reviewed-by:
-
Eric Seckler authored
Enables startup tracing for child processes spawned by the browser while a tracing session is active. This allows us to collect early trace events in the period of time before the child connects to the tracing service. Bug: 968424 Change-Id: I1708cabae77cdd2ab3a33929c28cfab04eb723e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1755746 Commit-Queue: Eric Seckler <eseckler@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
ssid <ssid@chromium.org> Cr-Commit-Position: refs/heads/master@{#689599}
-
Aaron Leventhal authored
Bug: 990455 Change-Id: If3a06c76d08cc6106c91fc22e542464fefdb8b56 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763231 Commit-Queue: Jesse Doherty <jwd@chromium.org> Auto-Submit: Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Cr-Commit-Position: refs/heads/master@{#689598}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/cab59a32b90b..b179babe2c3e git log cab59a32b90b..b179babe2c3e --date=short --no-merges --format='%ad %ae %s' 2019-08-22 saklein@chromium.org uprev_lib: Fix chrome ref/version parsing. Created with: gclient setdep -r src/third_party/chromite@b179babe2c3e The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I8e0a93b874f90c5628f441beba81492cef837cca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765001Reviewed-by:
-
Daniel Rubery authored
This CL adds the code to manage BinaryUploadService instances per-profile, and upload files to them. This is currently behind an off-by-default Feature, since the enterprise policy isn't landed yet. Bug: 980784 Change-Id: Ie3454cd653311bbdc2d009aa5237c32046d38f1d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1742776 Commit-Queue: Daniel Rubery <drubery@chromium.org> Reviewed-by:
-
Brett Kilty authored
PushBuffer had a DCHECK that could be hit in some startup scenarios where there is no data to push (Hit in Bluetooth connect cases here). Zero is the error return from the OnMoreData call, so we should propagate the error and return in the production codepath as well. Bug: internal Test: local build and test against case that was hitting this. Change-Id: Ia9e55bfc9b426399e70753604d0b0118a2070990 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1729055 Commit-Queue: Brett Kilty <brettk@google.com> Reviewed-by:
Kenneth MacKay <kmackay@chromium.org> Auto-Submit: Brett Kilty <brettk@google.com> Cr-Commit-Position: refs/heads/master@{#689595}
-
Ali Juma authored
When a load fails while offline, and an offline version of the page is available, two loads are currently performed: a blank error page is loaded (driven by ErrorRetryStateMachine) and an offline readling list page is loaded (driven by OfflinePageTabHelper::PresentOfflinePageForOnlineUrl). These two loads race with each other. When the second load happens to while the first load is in between didStartProvisionalNavigation and didCommitNavigation, then a didFailProvisionalNavigation is triggered for the first load, and WKWebView's URL is reset to the value it had before that load (which happens to be a placeholder URL). This URL change is incorrectly interpreted as a location.replace call in webViewURLDidChange, which ultimately leads to a placeholder URL being displayed in the omnibox instead of the URL of the offlne page. This CL prevents this race by eliminating the load of the blank error page in this situation. Change-Id: I3e951ed4356b7b1e1efa2b96077f22ad74af60ce Bug: 980912 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1754511 Commit-Queue: Ali Juma <ajuma@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#689594}
-
Christopher Cameron authored
This most notably includes tearing out AppsPageShimHandler, which then allows us to remove much of the complexity from AppShimHostManager. This will allow cleaning up much of the complexity of the AppShimHost and ExtensionAppShimHostHandler classes (which will then allow sharing a single shim process between multiple profiles). Bug: 982024 Change-Id: If8be5323a29502cc447cd3cf4134fac6b53ec834 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757370Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
-
Bettina authored
For non-syncing users, we use the SecurityEventRecorder to record an event when users use to Protect Account on receiving a Password Protection warning. https://screenshot.googleplex.com/Jc9TGRxqCkf Doesn't have any events in the screenshot because the Recorder is WIP still. Bug: 914410 Change-Id: I5a451ca9f2518362a72a858ab78d561c0c0b82df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764790Reviewed-by:
Daniel Rubery <drubery@chromium.org> Commit-Queue: Bettina Dea <bdea@chromium.org> Cr-Commit-Position: refs/heads/master@{#689592}
-
Justin Cohen authored
Bots rolled with the following script: git grep -l '11m382q' | xargs sed -i '' -e 's/11m382q/11m392q/g' This CIPD package only includes iOS 12.2 and iOS 13 to conserve disk space, so cronet (which still tests on iOS 9, 10 and 11) was not rolled to beta 6. CIPD packages: infra_internal/ios/xcode/mac 11m392q infra_internal/ios/xcode/ios 11m392q Bug: 995797 Change-Id: Ie375b0bb61989d87c3ba6cf883bf7bf32f5b6b03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760286 Commit-Queue: Justin Cohen <justincohen@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#689591}
-
