- 05 Oct, 2018 40 commits
-
-
Becca Hughes authored
Add an about://flags flag to enable unified audio focus on ARC++. BUG=883958 Change-Id: Idd0738a116d9eeaf225fab79de5945f73299dca7 Reviewed-on: https://chromium-review.googlesource.com/c/1237406 Commit-Queue: Becca Hughes <beccahughes@chromium.org> Reviewed-by:
Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#597214}
-
Ryan Hansberry authored
Delay showing the 'Chromebook added' notification until the MultiDeviceSetupDialog has been closed. R=jhawkins@chromium.org, khorimoto@chromium.org Bug: 883069 Change-Id: If8ae079f77de89cede94ea840e5335445fb7a0ad Reviewed-on: https://chromium-review.googlesource.com/c/1260304Reviewed-by:
James Hawkins <jhawkins@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Ryan Hansberry <hansberry@chromium.org> Cr-Commit-Position: refs/heads/master@{#597213}
-
Robbie McElrath authored
Bug: 881976 Change-Id: I39b93d0db822c341c7834ee100be957eaecbd15d Reviewed-on: https://chromium-review.googlesource.com/c/1263046Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Cr-Commit-Position: refs/heads/master@{#597212}
-
Ned Nguyen authored
Bug:792928 Change-Id: Ie383f3a5f499eee5e118668b6570800fa78fbdbe NOTRY=true # doesn't affect CQ Change-Id: Ie383f3a5f499eee5e118668b6570800fa78fbdbe Reviewed-on: https://chromium-review.googlesource.com/c/1263336 Commit-Queue: Ned Nguyen <nednguyen@google.com> Reviewed-by:
Stephen Martinis <martiniss@chromium.org> Cr-Commit-Position: refs/heads/master@{#597211}
-
Stephane Zermatten authored
runnable scripts. Before this patch, the controller only run an autostart script if it was the single runnable script. This means that the presence of non-autostart, but runnable script prevents autostart from triggering. This was a misunderstanding. As long as there is exactly one autostart script, it must be started, and other runnable scripts can be ignored. After this patch, as long as there is exactly one autostart script, it is run. Bug: 806868 Change-Id: Id60223064bc3ed8fc364f7f9a8ae1a4ed4ef921b Reviewed-on: https://chromium-review.googlesource.com/c/1264658Reviewed-by:
Ganggui Tang <gogerald@chromium.org> Commit-Queue: Ganggui Tang <gogerald@chromium.org> Cr-Commit-Position: refs/heads/master@{#597210}
-
Mugdha Lakhani authored
"background-fetch" permission was added to the Permissions API with this commit: https://github.com/w3c/permissions/pull/183/commits/096436eb59caaa5b244b0514d84f287392cb069b Chrome uses Automatic Downloads for this permission, so as not to add yet another user facing permission to the UI. This CL: 1. Adds a background_fetch content setting, because that's needed by PermissionContext and allows us to decouple from Automatic Downloads content setting. 2. Adds a BackgroundFetch PermissionContext mapped to this content setting. 3. Makes this PermissionContext decide whether background fetch() is permitted, which we do by: a. Querying DownloadRequestLimiter when there' a top level frame. b. Querying the Automatic Downloads content setting in other cases. For context and details, please see this document: https://docs.google.com/document/d/1rPYSlbzScw_6PLUJ3m96ZLIkxXVAWyBSL75-VDJEMso/edit?usp=sharing Bug: 886896 Change-Id: Id0fcc64d4242290f3782c61ef11babe5717b409f Reviewed-on: https://chromium-review.googlesource.com/c/1233714 Commit-Queue: Mugdha Lakhani <nator@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Reviewed-by:
Rayan Kanso <rayankans@chromium.org> Reviewed-by:
Tobias Sargeant <tobiasjs@chromium.org> Reviewed-by:
Raymes Khoury <raymes@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#597209}
-
chromium-autoroll authored
The AutoRoll server is located here: https://autoroll.skia.org/r/fuchsia-sdk-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:fuchsia_arm64_cast_audio;luci.chromium.try:fuchsia_x64_cast_audio TBR=cr-fuchsia+bot@chromium.org Change-Id: I67564c434bcf88639d7703d207f5a8f874549c9b Reviewed-on: https://chromium-review.googlesource.com/c/1265068Reviewed-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@{#597208}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/39cf6a765ae3..9934cc2f2f1a git log 39cf6a765ae3..9934cc2f2f1a --date=short --no-merges --format='%ad %ae %s' 2018-10-05 tsepez@chromium.org Tidy PWL_FLOATRANGE Created with: gclient setdep -r src/third_party/pdfium@9934cc2f2f1a 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. TBR=dsinclair@chromium.org Change-Id: Ib220b512ee0b339c7f7a0ffc41c945182817ceac Reviewed-on: https://chromium-review.googlesource.com/c/1265063Reviewed-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@{#597207}
-
Kyle Milka authored
This reverts commit 5d112920. Reason for revert: Broke old UI and we probably won't be able to completely remove the old UI in M71. Original change's description: > [NTP] Remove UI MD flag > > Remove flag for ntp md ui. Change the new styling to be the > default and remove the md class, this mainly affects the fakebox > as the icons are styled via md-icons. > > Bug: 864673 > Change-Id: I7eb5da522a657417c527e3aa666bab14f8485bef > Reviewed-on: https://chromium-review.googlesource.com/1255600 > Reviewed-by: Ilya Sherman <isherman@chromium.org> > Reviewed-by: Kristi Park <kristipark@chromium.org> > Commit-Queue: Kyle Milka <kmilka@chromium.org> > Cr-Commit-Position: refs/heads/master@{#595953} TBR=isherman@chromium.org,kristipark@chromium.org,kmilka@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 864673 Change-Id: I5ef13705787953e96b7e4b0ead21e1e8aeb558d1 Reviewed-on: https://chromium-review.googlesource.com/c/1265024Reviewed-by:
Kyle Milka <kmilka@chromium.org> Commit-Queue: Kyle Milka <kmilka@chromium.org> Cr-Commit-Position: refs/heads/master@{#597206}
-
Christian Biesinger authored
R=ikilpatrick@chromium.org,eae@chromium.org Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: I3836a9f162723371650e239918cc43412ab6c6c4 Reviewed-on: https://chromium-review.googlesource.com/c/1265066Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#597205}
-
Rayan Kanso authored
Paused fetch notifications were not displayed on browser start-up. This means that the fetch could not be resumed and is forever paused. The state of the entry needs to be exposed on DS initialization, so that paused items can notify the OfflineContentAggregator and display a notification. Bug: 889423 Change-Id: Ia61c903f6234264821aad20b7bb93d6a3fdeb447 Reviewed-on: https://chromium-review.googlesource.com/c/1251630Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Commit-Queue: Rayan Kanso <rayankans@chromium.org> Cr-Commit-Position: refs/heads/master@{#597204}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/39be1c969580..64626344db73 git log 39be1c969580..64626344db73 --date=short --no-merges --format='%ad %ae %s' 2018-10-05 bhthompson@google.com Make grunt-paladin and grunt pfqs important Created with: gclient setdep -r src/third_party/chromite@64626344db73 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. TBR=chrome-os-gardeners@chromium.org Change-Id: Ide9ce8f3e1cb48798be6d653ce3a277a2fd171b5 Reviewed-on: https://chromium-review.googlesource.com/c/1265038Reviewed-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@{#597203}
-
manuk authored
[Omnibox] When in keyword mode, don't allow shortcut matches of different keywords to be the default match. Prior to this cl, when the user has searched for `searchProvder searchTerm` (not in keyword mode) then typing in `searchProvider `, entering keyword mode, and typing `s` could result in the shortcut match for the earlier search becoming the default match and causing him to leave keyword mode. Bug: 632215 Change-Id: I19ee38e3e5c9ee68f8f7703afccef1276027b521 Reviewed-on: https://chromium-review.googlesource.com/c/1182295 Commit-Queue: manuk hovanesian <manukh@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Cr-Commit-Position: refs/heads/master@{#597202}
-
Tatiana Buldina authored
bug: chromedriver:1945 Change-Id: I3148ef524714e06766db15cf63812bf2ca0d75eb Reviewed-on: https://chromium-review.googlesource.com/c/1265256Reviewed-by:
John Chen <johnchen@chromium.org> Commit-Queue: Tatiana Buldina <buldina@chromium.org> Cr-Commit-Position: refs/heads/master@{#597201}
-
Wei Li authored
UI Devtools use overlay model to support element highlight. However, in remote debugging configuration, the highlight is handled by screencast module which is not supported in current UI Devtools. This CL fixes this by using local debugging on local sessions. BUG=749778 Change-Id: Idc81ff27e05d5b95e72748873b933b1ecdb80ad2 Reviewed-on: https://chromium-review.googlesource.com/c/1262960Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Wei Li <weili@chromium.org> Cr-Commit-Position: refs/heads/master@{#597200}
-
Majid Valipour authored
Worklet animation supports invalidation of its compositing state. This patch makes this invalidation process a no-op for animations that are running on main (i.e., |compositor_animation_| is null). This means we no longer try to read/modify the non-existent |compositor_animation_| which was causing the crash. Note that in some situations, even for currently main thread animaiton, we may actually want to create and start the |compositor_animation_| but this is a separate issue that can be fixed as a follow up. The added DCHECK uncovered a bug where we were attaching the worklet animation to early even if play was failing. This is also fixed. Bug: 887659, 889330 Change-Id: I4a89de6632e053f68beafb810355c25e4bfba0e3 Reviewed-on: https://chromium-review.googlesource.com/c/1249387 Commit-Queue: Majid Valipour <majidvp@chromium.org> Reviewed-by:
Robert Flack <flackr@chromium.org> Cr-Commit-Position: refs/heads/master@{#597199}
-
Robert Ogden authored
This way it can also be used by the new Previews UI Bug: 871839 Change-Id: Iecba63729da87fc6f30cc0f0cceaebf85adc4729 Reviewed-on: https://chromium-review.googlesource.com/c/1265062 Commit-Queue: Robert Ogden <robertogden@chromium.org> Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Cr-Commit-Position: refs/heads/master@{#597198}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/263549ba0d66..4e9584b32dfe Created with: gclient setdep -r src-internal@4e9584b32dfe The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-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=mmoss@chromium.org Change-Id: I85fee9a3d742f47597ade47e850eb1d993769238 Reviewed-on: https://chromium-review.googlesource.com/c/1265037Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#597197}
-
Tommy C. Li authored
This CL takes a section of code used to show the full URL for Steady State Elisions and moves it from the Views UI code to the OmniboxEditModel. This is in preparation for it to be used for Query in Omnibox. It also adds a little bit of specific test coverage for this code. Bug: 874592 Change-Id: I354c17a91751f930bd94d0ce9e02ef3f2846d15f Reviewed-on: https://chromium-review.googlesource.com/c/1262655 Commit-Queue: Tommy Li <tommycli@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#597196}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/a1134509c91d..96a0f6191781 git log a1134509c91d..96a0f6191781 --date=short --no-merges --format='%ad %ae %s' 2018-10-05 oprypin@webrtc.org Revert "[cleanup] Remove useless includes." 2018-10-05 sprang@webrtc.org Turning off a stream should results in target bitrate 0 signal 2018-10-05 yvesg@webrtc.org [cleanup] Remove useless includes. Created with: gclient setdep -r src/third_party/webrtc@96a0f6191781 The AutoRoll server is located here: https://autoroll.skia.org/r/webrtc-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:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: Ib706f62da1f7169d1aea11c1617c76ea2278fe12 Reviewed-on: https://chromium-review.googlesource.com/c/1264629Reviewed-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@{#597195}
-
Brian White authored
Crash reports indicate that checksums are valid even though the bucket boundary values are tested to be incorrect. Try to determine if it is memory corruption or bad calculations. Bug: 836238 Change-Id: Ie89df5fef5093e9dd8c45fe9ff26a0430838769e Reviewed-on: https://chromium-review.googlesource.com/c/1264036Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#597194}
-
Kevin Marshall authored
Net tests broke because test process' /data directories are now isolated and can't be used for sharing data out-of-process. The net test launcher depends on the old behavior for deploying configuration files to test code. This CL designates a special path "/data/shared" into which shared files will be placed. The shared directory will be mounted at "/test-shared" in the test process' namespace. The net test launcher & test code are modified to use the new paths. TBR=mmenke@chromium.org Bug: 891732 Change-Id: Id94bffe05a3b140ed60e42507cdc8d238677f202 Reviewed-on: https://chromium-review.googlesource.com/c/1259829 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#597193}
-
Tommy C. Li authored
Previously, when the user has a disabled PDF plugin (or no PDF plugin) and the website has an IFRAMEed PDF, when Click to Open PDF was enabled, we would inject a fake <object> tag into the IFRAME to force a PDF plugin placeholder to appear. This approach is problematic with CSP, as CSP may forbid OBJECT tags from loading in IFRAMEs. This CL instead injects the placeholder HTML directly into the IFRAME. In the IFRAME case, the button is now a plain link, which should allow the user to click it and download the PDF even if CSP forbids <object> tags or JavaScript. It still may trigger a CSP warning if IFRAME is prohibited from running JavaScript, but that only hurts keyboard-accessibility, and doesn't prevent the mainstream use case from working. This CL also prevents the <object> tag from auto-opening the PDF after download, which was probably a mistake, since that overrides the user configurable "Always open with system viewer" option. Bug: 887752, 879149, 878871 Change-Id: I900c08331d2cfc96ea7cd1cd46ea594445b0920b Reviewed-on: https://chromium-review.googlesource.com/c/1246956Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#597192}
-
Becca Hughes authored
Add an ArcMediaSession bridge that will send an AudioFocusManager mojo pointer to Android. This will allow us to access the media session service in Chrome from Android and in the future allow us to delegate audio focus requests. BUG=883958 Change-Id: I5a00e93ce1a1e4608d3f934522188b6252906cd2 Reviewed-on: https://chromium-review.googlesource.com/c/1225607Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Hidehiko Abe <hidehiko@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#597191}
-
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: I00a5e946eff9d7eeba32bfe0a45a7fd6fc804827 Reviewed-on: https://chromium-review.googlesource.com/c/1265157Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#597190}
-
Paul Jensen authored
Bug: 892666 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;master.tryserver.chromium.android:android_cronet_tester Change-Id: Ic9a5cedb520d339b458c2208a6dbb6e57be4cc34 Reviewed-on: https://chromium-review.googlesource.com/c/1264898Reviewed-by:
Misha Efimov <mef@chromium.org> Commit-Queue: Paul Jensen <pauljensen@chromium.org> Cr-Commit-Position: refs/heads/master@{#597189}
-
Clark DuVall authored
These test the basic proxy usage and headers. We can add more tests here if necessary to help track what's not working with network service. Bug: 721403 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I09c0181ee4cdac97b4c532630f274f2418858d5d Reviewed-on: https://chromium-review.googlesource.com/c/1263038Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#597188}
-
James Wallace-Lee authored
The benchmark was not completely removed in crrev/c/1143595. Bug: 769809 Change-Id: I131b226b7388d97eb380943275b6b16c222d2260 Reviewed-on: https://chromium-review.googlesource.com/c/1257669Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Commit-Queue: James Wallace-Lee <jamwalla@chromium.org> Cr-Commit-Position: refs/heads/master@{#597187}
-
Tom Anderson authored
This CL fixes a bug after [1]. The issue was setting "window_mapped_in_client_ = false" after XIconfiyWindow(). Despite the ICCCM spec requiring the state transition "Iconic -> Normal - The client should map the window" [2], the window is not actually placed in an unmapped state. Instead, the _NET_WM_STATE_HIDDEN property is set. [1] https://chromium.googlesource.com/chromium/src/+/02370c73936726c387f74cfbd5796f1667ae8a65 [2] ftp://www.x.org/pub/current/doc/xorg-docs/icccm/icccm.html BUG=891376 R=sky Change-Id: Ibfcc66a54249bad8926d23cd4eabf1e6979f7d07 Reviewed-on: https://chromium-review.googlesource.com/c/1263431Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#597186}
-
Alexandr Ilin authored
content::PreconnectUrl() is about to be removed and AuthPrewarmer is the last caller. This CL makes AuthPrewarmer to use NetworkContext::PreconnectSockets() for prewarming a connection to the signin server. Bug: 875238 Change-Id: I0865d6d9566b13f71ee0ac59e8a583bc7c401099 Reviewed-on: https://chromium-review.googlesource.com/c/1261717 Commit-Queue: Alexandr Ilin <alexilin@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#597185}
-
Jonathan Metzman authored
This reverts commit 01602d56. Reason for revert: /O1 is not the clang-cl equivalent of -O1 Original change's description: > [libFuzzer][Windows] Enable optimize_for_fuzzing > > Bug: 890639 > Change-Id: Icb2be76982816119a30925fa92121445b5ca5141 > Reviewed-on: https://chromium-review.googlesource.com/c/1253023 > Reviewed-by: Max Moroz <mmoroz@chromium.org> > Reviewed-by: Scott Graham <scottmg@chromium.org> > Commit-Queue: Jonathan Metzman <metzman@chromium.org> > Cr-Commit-Position: refs/heads/master@{#596757} TBR=thakis@chromium.org,scottmg@chromium.org,mmoroz@chromium.org,metzman@chromium.org Change-Id: Ic407400440a3e6c58940b77708ab5334736242bb No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 890639 Reviewed-on: https://chromium-review.googlesource.com/c/1264680Reviewed-by:
Jonathan Metzman <metzman@chromium.org> Commit-Queue: Jonathan Metzman <metzman@chromium.org> Cr-Commit-Position: refs/heads/master@{#597184}
-
Dan Harrington authored
DCHECK(committed_error_page_info_->auto_reload_triggered) was failing. This is possible because when assigning a new value to committed_error_page_info_, the auto load timer can still be active. R=jianli Bug: 881208 Change-Id: I361df3e17bef122f143331107a76d04487b32fea Reviewed-on: https://chromium-review.googlesource.com/c/1240872Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Jian Li <jianli@chromium.org> Commit-Queue: Dan H <harringtond@google.com> Cr-Commit-Position: refs/heads/master@{#597183}
-
Robert Ogden authored
I got a crash earlier today where WillProcessResponse was being called on a non-preview page without response headers, so the DCHECK failed. Moving up the early return makes the navigation throttle more robust against this in the future. Bug: 864635 Change-Id: Ief8e3b2b0ac9179d2ba16ff8f04c950b08f9564d Reviewed-on: https://chromium-review.googlesource.com/c/1265059 Commit-Queue: Robert Ogden <robertogden@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#597182}
-
Tarun Bansal authored
To speed up page loads from CCTs, data reduction proxy enabled check is removed from preconnect. Bug: 850609 Change-Id: I111bc05c453a54c4c6d200186c04876faaec063e TBR: lizeb@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/1264075Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#597181}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/5476e805a321..8efa341839a6 git log 5476e805a321..8efa341839a6 --date=short --no-merges --format='%ad %ae %s' 2018-10-05 geofflang@chromium.org Suppress EXTBlendFuncExtendedDrawTestES3.FragmentArrayOutputLocationsAP on MacOS 2018-10-05 jmadill@chromium.org GL back-end error refactor. 2018-10-05 syoussefi@chromium.org Vulkan: Safer struct initialization Created with: gclient setdep -r src/third_party/angle@8efa341839a6 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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=cwallez@chromium.org Change-Id: Ia6926da32ec25e3a307c14617b00c7bdd36fc295 Reviewed-on: https://chromium-review.googlesource.com/c/1265015Reviewed-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@{#597180}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/032f2756df64..ba651682ae85 git log 032f2756df64..ba651682ae85 --date=short --no-merges --format='%ad %ae %s' 2018-10-05 brianosman@google.com Fix two bugs in GPU special image read-back 2018-10-05 fmalita@chromium.org [skottie] Remove temporary ResourceProvider::load() fallback 2018-10-05 reed@google.com isolate users of SkFindAndPlaceGlyph.h 2018-10-05 robertphillips@google.com Expose the SkYUVAIndex version of the SkImage factories 2018-10-05 brianosman@google.com Remove redundant test Created with: gclient setdep -r src/third_party/skia@ba651682ae85 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-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;master.tryserver.blink:linux_trusty_blink_rel TBR=mtklein@chromium.org Change-Id: I5cb0744e655d18db0a6eea6a421c54f5ba8ab6a6 Reviewed-on: https://chromium-review.googlesource.com/c/1264743Reviewed-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@{#597179}
-
Kyle Horimoto authored
This CL replaces the placeholder setup page with the real page which includes the logic for choosing a multi-device host phone. As part of this CL, WizardController now decides whether or not to show the setup flow based on whether setup is applicable and needed for the GAIA account under question. Bug: 884065 Change-Id: I8f3be643b77890fb249516369d76f5c392a568a9 Reviewed-on: https://chromium-review.googlesource.com/c/1255591 Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#597178}
-
Tatiana Buldina authored
Add support with remote connection for methods: SetWindowRect(), SetWindowPosition(), MaximizeWindow(), MinimizeWindow() and FullScreenWindow() Bug: chromedriver:1855 Change-Id: I452cd54d2fea79763f87a30edb1d034851ed0eea Reviewed-on: https://chromium-review.googlesource.com/c/1263567 Commit-Queue: Tatiana Buldina <buldina@chromium.org> Reviewed-by:
John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#597177}
-
John Chen authored
Group the command endpoints based on the sources that defined the commands, to improve maintainability. Change-Id: I0bdf68bc2bd56703fa87ca8486f1550ca6bb3c86 Reviewed-on: https://chromium-review.googlesource.com/c/1263875Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Commit-Queue: John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#597176}
-
Jaeyong Bae authored
If any of sizes or purpose are missing in ImageResource, provide default values for 'purpose' and 'sizes'. Bug: 868875 Change-Id: I2e88144e1d098a18c740ffeaf9a2fb474db031a1 Reviewed-on: https://chromium-review.googlesource.com/c/1244176 Commit-Queue: Jaeyong Bae <jdragon.bae@gmail.com> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Mugdha Lakhani <nator@chromium.org> Cr-Commit-Position: refs/heads/master@{#597175}
-