- 14 Nov, 2019 40 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/bacb84b99758..966c82596b15 git log bacb84b99758..966c82596b15 --date=short --no-merges --format='%ad %ae %s' 2019-11-14 jrn@google.com Revert "List which tests are googlesource.com qualification tests" Created with: gclient setdep -r src/third_party/chromite@966c82596b15 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I440de2085d28be01573ff0be93540518951e2e43 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917985Reviewed-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@{#715491}
-
Ryan Hansberry authored
crrev.com/c/1758837 intended to restrict OOBE HID detection logic to only run on devices that it should (e.g., Chromeboxes), but overlooked the instantiation of HidDetectionScreen, which once created, goes ahead and tries to connect to all HID devices even when the actual screen is not shown. This CL corrects that oversight and only creates HidDetectionScreen when it is known to be needed. This CL also adds a WizardController::HasScreen() method to facilitate testing. Using WizardController::GetScreen() to test the absence of the HIDDetectionScreen does not work because it triggers a DCHECK in SCreenManager [1]. 1) https://cs.chromium.org/chromium/src/chrome/browser/chromeos/login/screen_manager.cc?l=25 Bug: 1007500, 1024479 Change-Id: I362a05bad73a2876d4b3a937b99823a90fa3e3a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915080 Commit-Queue: Ryan Hansberry <hansberry@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#715490}
-
kylechar authored
Update code where NULL or 0 is assigned to a scoped_refptr to reset it. Just call reset() instead. If operator=(std::nullptr_t) is added these assignments become ambiguous. This CL was uploaded by git cl split. R=mseaborn@chromium.org Bug: 1024981 Change-Id: I2d6317c780f849bb47e943d48868fe24283120f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918157 Auto-Submit: kylechar <kylechar@chromium.org> Reviewed-by:
Mark Seaborn <mseaborn@chromium.org> Commit-Queue: Mark Seaborn <mseaborn@chromium.org> Cr-Commit-Position: refs/heads/master@{#715489}
-
Tommy Steimel authored
This test was recently marked as flaky on debug builds, but it is also flaking on release builds. Example: https://ci.chromium.org/p/chromium/builders/try/win10_chromium_x64_rel_ng/419689 TBR=cmp@chromium.org Bug: 1022915 Change-Id: Ieca4c3b5b0f435862c28aa818f237ade814e1389 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918087Reviewed-by:
Tommy Steimel <steimel@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#715488}
-
kylechar authored
Update code where NULL or 0 is assigned to a scoped_refptr to reset it. Just call reset() instead. If operator=(std::nullptr_t) is added these assignments become ambiguous. This CL was uploaded by git cl split. R=tguilbert@chromium.org Bug: 1024981 Change-Id: I3b28a9c786a9e58707716ce0b2162c9e18dad02d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918150 Auto-Submit: kylechar <kylechar@chromium.org> Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Commit-Queue: kylechar <kylechar@chromium.org> Commit-Queue: Thomas Guilbert <tguilbert@chromium.org> Cr-Commit-Position: refs/heads/master@{#715487}
-
Tommy Steimel authored
This reverts commit 6a44cbfb. Reason for revert: Causing ozone_unittests failures on msan bots: https://ci.chromium.org/p/chromium/builders/ci/Linux%20ChromiumOS%20MSan%20Tests/16285 Original change's description: > Disable Render Buffer Compression on external displays. > > When Getting format modifiers, exclude I915_FORMAT_MOD_Y*_TILED_CCS > (Intel color control surface (CCS) for render compression) if it's not > an internal display. > This solves the bandwidth issue when driving more than 1 4K monitor with > rbc enabled. > > BUG=996011,996036,994341,979736 > TEST=External display doesn't pick I915...CSS modifier > > Change-Id: I9d7aa06f5f9331d232a3cbd7e74bd41771bc4030 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912460 > Commit-Queue: Mark Yacoub <markyacoub@google.com> > Reviewed-by: Michael Spang <spang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#715332} TBR=spang@chromium.org,dcastagna@chromium.org,hoegsberg@chromium.org,tutankhamen@chromium.org,markyacoub@google.com Change-Id: Ie9495bbd892a5b736f1f70cb60edbf1b14c21eed No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 996011, 996036, 994341, 979736 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918364Reviewed-by:
-
kylechar authored
Update code where NULL or 0 is assigned to a scoped_refptr to reset it. Just call reset() instead. If operator=(std::nullptr_t) is added these assignments become ambiguous. This CL was uploaded by git cl split. R=reillyg@chromium.org Bug: 1024981 Change-Id: I11e16eb6457e3de133bf6528b437f0753e73b5ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918322 Auto-Submit: kylechar <kylechar@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#715485}
-
Reilly Grant authored
Bug: 1007783 Change-Id: I0f7fd63da98bfc9ea6d869cea97072c7c94a30be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917741Reviewed-by:
Matt Reynolds <mattreynolds@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Auto-Submit: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#715484}
-
Roger Tawa authored
SafeBrowsingPrivateEventRouter should have tests that verify the feature flag works as expected. Bug: 1021935 Change-Id: I1dd181b2c68b319cb3193ed021e9633dd7c0cbdd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912141Reviewed-by:
Daniel Rubery <drubery@chromium.org> Commit-Queue: Roger Tawa <rogerta@chromium.org> Cr-Commit-Position: refs/heads/master@{#715483}
-
Brian Sheedy authored
This reverts commit 4430c28a. Reason for revert: https://crbug.com/1024954 Original change's description: > Use grey triangle icon for HTTP when experiment enabled (Android) > > Bug: 1008219 > Change-Id: If02a36539fe61e71d1434528ec5af31d3914439b > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1889116 > Commit-Queue: Livvie Lin <livvielin@chromium.org> > Reviewed-by: Michael Thiessen <mthiesse@chromium.org> > Cr-Commit-Position: refs/heads/master@{#714377} TBR=mthiesse@chromium.org,livvielin@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1008219 Change-Id: I3f63150118c21da153c995529848fc92cd18bd52 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918103Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Livvie Lin <livvielin@chromium.org> Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#715482}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/3da78c4bb7dc..336e51f68564 git log 3da78c4bb7dc..336e51f68564 --date=short --no-merges --format='%ad %ae %s' 2019-11-14 ehmaldonado@chromium.org depot_tools: Execute tests using bot's vpython instead of depot_tools' vpython Created with: gclient setdep -r src/third_party/depot_tools@336e51f68564 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC agable@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=agable@chromium.org Bug: None Change-Id: Ie966f0772192a856bd3803dd8b80a2dd5428949d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918043Reviewed-by:
-
zhaoyangli authored
Bug: 1024699 Change-Id: Iffaf410380e85376abbfcffad44a56952c905fbe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917602Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Commit-Queue: Zhaoyang Li <zhaoyangli@chromium.org> Cr-Commit-Position: refs/heads/master@{#715480}
-
Ian Clelland authored
This aligns the end date with other FP histograms. Bug: 1021139 Change-Id: Ic132a07c1dfe804d8e7d0456b8f641e8c15cd10f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913869 Auto-Submit: Ian Clelland <iclelland@chromium.org> Reviewed-by:
Jason Chase <chasej@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#715479}
-
Robert Ogden authored
Launching feature Bug: 914577 Change-Id: I4e050a6e2f3be795ee4402d5a8f87876122a9ef8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913524Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Robert Ogden <robertogden@chromium.org> Cr-Commit-Position: refs/heads/master@{#715478}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/8fccd236..ef60c912 Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I4779f85a8cc5208b2836c89082f36a9756268a58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917086Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#715477}
-
kylechar authored
Update code where NULL or 0 is assigned to a scoped_refptr to reset it. Just call reset() instead. If operator=(std::nullptr_t) is added these assignments become ambiguous. This CL was uploaded by git cl split. R=lambroslambrou@chromium.org Bug: 1024981 Change-Id: If554e4d375338cb2a814196a4436694cf697f8da Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918223 Auto-Submit: kylechar <kylechar@chromium.org> Reviewed-by:
Lambros Lambrou <lambroslambrou@chromium.org> Commit-Queue: Lambros Lambrou <lambroslambrou@chromium.org> Cr-Commit-Position: refs/heads/master@{#715476}
-
Garrett Beaty authored
Change-Id: I269cf8994e966eb9dae19e7c6c7c86e66e2f8a89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918085 Auto-Submit: Garrett Beaty <gbeaty@chromium.org> Reviewed-by:
-
Mustaq Ahmed authored
The flag was removed through https://crrev.com/c/1873384. Bug: 959850 Change-Id: I49916760f106bb5e07f468eb9bb04e547b52d231 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918101Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Commit-Queue: Mustaq Ahmed <mustaq@chromium.org> Cr-Commit-Position: refs/heads/master@{#715474}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/3120d0337382..c95e54b558d9 git log 3120d0337382..c95e54b558d9 --date=short --no-merges --format='%ad %ae %s' 2019-11-14 rmistry@google.com Specify only luci buckets instead of hardcoding trybot names 2019-11-14 ahujabharat93@gmail.com Issue- >Skia tiled bitmap rendering is too slow when tildmodeX is not same as tileModeY. 2019-11-14 jvanverth@google.com Reduce scaling of large SDF glyphs on Mac. 2019-11-14 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/skia@c95e54b558d9 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC halcanary@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md 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=halcanary@google.com Bug: chromium:1003270 Change-Id: I8bd17b0fcf9ab00c77c458821bd208de023d83dc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918041Reviewed-by:
-
Ovidio Henriquez authored
This changes fixes an OOB access that may occur in WebBluetoothServiceImpl::OnBluetoothScanningPromptEvent(). The method assumes that |scanning_clients_| will be populated when the method is called, however it can be cleared if a Mojo connection error is triggered. The method now returns if |scanning_clients_| is empty, and it uses the back() and pop() methods of vector to further prevent accidental OOB access. Additionally, in BluetoothDeviceScanningPromptController, the EventHandler binding is updated so that the lifetime of the class is associated with the binding. Bug: 1024116 Change-Id: I2008f7bc1ce65be1d94d39370ac8593f5ff418e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916686 Commit-Queue: Ovidio de Jesús Ruiz-Henríquez <odejesush@chromium.org> Reviewed-by:
-
Dale Curtis authored
Basic unit test, we should still see if there's a way to create an integration test which verifies this. BUG=1006092 R=liberato Change-Id: Ic92871fd25964a3d23f9a015014bab1994d5e79d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918333 Auto-Submit: Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Frank Liberato <liberato@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#715471}
-
Min Qin authored
Here is how the crash can happen: 1. DragObjectImpl::GetData() calls DragDownloadFile::Start() to start a download. 2. GetData() then calls DragDownloadFile::Wait(). 3. DragDownloadFile::Wait() runs the nested_loop_. 4. DragDownloadFile::DownloadCompleted() gets called, which calls DataObjectImpl::OnDownloadCompleted(). Inside that method, it creates a new StoredDataInfo objects and deletes the downloader pointer, which is the DragDownloadFile instance. 5. DragDownloadFile dtor is invoked, it will destroy the nested_loop_ member variable. However, because RunLoop::Run() is called earlier and Wait() is still pending, deleting nested_loop_ will hit the DCHECK(!running_) in RunLoop dtor and fail. The issue happens mainly due to step 4, as DragDownloadFile::DownloadCompleted() could delete itself. This CL fixes the issue by not deleting the StoredDataInfo object in step 4, instead, the DragDownloadFile instance is deleted after the newly constructed StoredDataInfo object in 4 is destroyed. BUG=968303 Change-Id: I57dbe275f76fd8e566185538a8f13def476ff9f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904950Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Commit-Queue: Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#715470}
-
Tommy Steimel authored
This reverts commit 5848123a. Reason for revert: compile failure: https://ci.chromium.org/p/chromium/builders/ci/win32-archive-rel/7123 Original change's description: > Move dependencies on cast_channel protos > > This patch rectifies a duplicate definition error, since certain proto > definitions are defined both in components/cast_channel and in > third_party/openscreen, by moving all consumers of these protos to using > the Open Screen version. > > Change-Id: I0347424c019b1489eb09b3822fd4a97d062e0e0f > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902718 > Reviewed-by: Luke Halliwell (slow) <halliwell@chromium.org> > Reviewed-by: Karan Bhatia <karandeepb@chromium.org> > Reviewed-by: mark a. foltz <mfoltz@chromium.org> > Reviewed-by: Yuri Wiitala <miu@chromium.org> > Commit-Queue: Jordan Bayles <jophba@chromium.org> > Cr-Commit-Position: refs/heads/master@{#715462} TBR=mfoltz@chromium.org,alexst@chromium.org,miu@chromium.org,benwells@chromium.org,rockot@google.com,dnicoara@google.com,dougsteed@chromium.org,halliwell@chromium.org,karandeepb@chromium.org,seantopping@chromium.org,ryanchung@chromium.org,jophba@chromium.org Change-Id: I0df964d6a4300246b1143965be6c73751952b879 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918089Reviewed-by:
-
Jon Mann authored
Bug: 966270 Change-Id: I0261264a0e7b1c4128329902722f33022f842078 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917756Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Jon Mann <jonmann@chromium.org> Cr-Commit-Position: refs/heads/master@{#715468}
-
kylechar authored
Update code where NULL or 0 is assigned to a scoped_refptr to reset it. Just call reset() instead. If operator=(std::nullptr_t) is added these assignments become ambiguous. This CL was uploaded by git cl split. R=michaeldo@chromium.org Bug: 1024981 Change-Id: I2d8b8c4b572199f76d40466074343c4fc6885870 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918151 Auto-Submit: kylechar <kylechar@chromium.org> Reviewed-by:
Mike Dougherty <michaeldo@chromium.org> Commit-Queue: Mike Dougherty <michaeldo@chromium.org> Cr-Commit-Position: refs/heads/master@{#715467}
-
Wez authored
This reverts commit ee5f0d37. Reason for revert: This CL has no effect because the |thread_checker_| in the class is a no-op implementation in Release builds. Will re-land a version of this CL that fixes that, in a single CL, so it's easy to revert. Original change's description: > history service: Turn DCHECKs to CHECKs to try to diagnose crash bug. > > Bug: 1009795 > Change-Id: Ib33c1242228e674442f21dea8a56ef456d84b886 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899231 > Reviewed-by: Scott Violet <sky@chromium.org> > Commit-Queue: Matt Falkenhagen <falken@chromium.org> > Cr-Commit-Position: refs/heads/master@{#712948} TBR=falken@chromium.org,sky@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1009795 Change-Id: I24a8c7ff548d43c315528e85556967800336aea9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918120Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#715466}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/5bd6ca022650..935c827ec93c git log 5bd6ca022650..935c827ec93c --date=short --no-merges --format='%ad %ae %s' 2019-11-14 chshrest@microsoft.com Revert: Make add button in color-picker accessible https://chromium-review.googlesource.com/c/chromium/src/+/1688337 Created with: gclient setdep -r src/third_party/devtools-frontend/src@935c827ec93c If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC v8-waterfall-sheriff@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=v8-waterfall-sheriff@grotations.appspotmail.com Bug: chromium:1023748 Change-Id: I0675cfc734615754aedc461b9467779e2f4af2ab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918040Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/968df09eed92..1a01b4b322a5 git log 968df09eed92..1a01b4b322a5 --date=short --no-merges --format='%ad %ae %s' 2019-11-14 tobine@google.com Refactor end2end test macros Created with: gclient setdep -r src/third_party/angle@1a01b4b322a5 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC cnorthrop@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md 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=cnorthrop@google.com Bug: None Change-Id: I6a6a07036594a482d6bb53f016da1dbfc8a34949 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918039Reviewed-by:
-
Michael Crouse authored
This change moves the metric for fetch coverage at commit into the navigation data so that the page loads recorded match across all of the client hint coverage metrics. This also adds an additional metric of whether a hint was available on the client or had been covered by a fetch at commit. Bug: 1022625 Change-Id: I6fcacba86df635d0f0ab22ae543031a9ca0d3d81 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915293Reviewed-by:
Sophie Chang <sophiechang@chromium.org> Reviewed-by:
-
Jordan Bayles authored
This patch rectifies a duplicate definition error, since certain proto definitions are defined both in components/cast_channel and in third_party/openscreen, by moving all consumers of these protos to using the Open Screen version. Change-Id: I0347424c019b1489eb09b3822fd4a97d062e0e0f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902718Reviewed-by:
Luke Halliwell (slow) <halliwell@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Reviewed-by:
Yuri Wiitala <miu@chromium.org> Commit-Queue: Jordan Bayles <jophba@chromium.org> Cr-Commit-Position: refs/heads/master@{#715462}
-
Xianzhu Wang authored
The rebaseline was incomplete in the previous https://chromium-review.googlesource.com/c/chromium/src/+/1915461. TBR=steimel@chromium.org Bug: 377696 Change-Id: I2455acef8a0c628744bd7360de51f77511278da9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918442Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#715461}
-
Darin Fisher authored
This change postpones calling RenderFrameObserver::DidCommitProvisionalLoad until after RenderFrameImpl is done updating its own state and performing its own work. This avoids running arbitrary other code from within the middle of those state updates. In particular, this ensures that observers that wish to execute script from within DidCommitProvisionalLoad do not do so until after we have setup the new BrowserInterfaceBrokerProxy and finished other critical steps associated with committing a navigation in the renderer. See ChromeRenderFrameObserver, which uses this signal to inject scripts for WebUI pages. (We might decide that is non-ideal for other reasons and design a better injection point, but that can be done in a subsequent CL.) Injected script can call into local storage, which is a synchronous operation. That operation requires a synchronous check of the ContentSettingsManager to determine if the script can access local storage. That interface is acquired through the BrowserInterfaceBrokerProxy. Hence, there is opportunity for a dead-lock if the BrowserInterfaceBrokerProxy has not already been set-up (meaning, not already had its receiver handle sent to the browser process) by the time script tries to access local storage. In practice, this has not been an issue, but in another change I am making, the ContentSettingsManager will be lazily acquired causing this potential dead-lock to more easily occur and to definitely occur in some of our WebUI browser tests. As part of this change, the page_load_metrics/ module had to be revised to not flush existing metrics from DidCommitProvisionalLoad. Doing so could attribute metrics from the previous page to the newly committed page including especially the navigation_start time which is used as a key to identify page loads by the metrics system. That in turn could cause subsequent metrics on the newly loaded page to simply be dropped. Fortunately, tests helped root out this issue. The PageTimingMetricsSender grows a SendLatest method that can be used to trigger when metrics should be flushed rather than having that be a side-effect of the destructor running. SendLatest is now called from ReadyToCommitNavigation to send any accumulated metrics for the previous page before the new load commits. This may result in dropping some metrics data between the calls to ReadyToCommitNavigation and DidCommitProvisionalLoad, but that is probably reasonable. Also, a comment from dgozman indicates that ReadyToCommitNavigation is something we want to get rid of but MetricsRenderFrameObserver already implements this method so whatever substitute there will be in the future for this method can hopefully account for both this new use case as well as the existing use case. Files w/ implementations potentially impacted: chrome/renderer/chrome_render_frame_observer.cc chrome/renderer/content_settings_agent_impl.cc chrome/renderer/content_settings_agent_impl_browsertest.cc chrome/renderer/net/net_error_helper.cc chrome/renderer/safe_browsing/phishing_classifier_delegate.cc chrome/renderer/searchbox/searchbox.cc components/autofill/content/renderer/autofill_agent.cc components/autofill/content/renderer/form_tracker.cc components/autofill/content/renderer/password_autofill_agent.cc components/autofill/content/renderer/password_generation_agent.cc components/data_reduction_proxy/content/renderer/content_previews_render_frame_observer.cc components/page_load_metrics/renderer/metrics_render_frame_observer.cc content/renderer/accessibility/render_accessibility_impl.cc content/renderer/media/audio/audio_renderer_sink_cache_impl.cc content/renderer/pepper/plugin_power_saver_helper.cc extensions/renderer/extension_frame_helper.cc Change-Id: Ia6627b2db06ea51e769ed6c8889e37e554fcc1d1 Bug: 1023519 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898769Reviewed-by:
Bryan McQuade <bmcquade@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Darin Fisher <darin@chromium.org> Cr-Commit-Position: refs/heads/master@{#715460}
-
Dan Beam authored
This is a reland of f5946a0a It was reverted as 9948fd01 Fix: just exclude ASAN/LSAN bots from running this test. Context: I was able to produce flaky failures locally, and I believe these bots are simply timing out because there's now 30 realbox tests. Because these tests are basically just Javascript, there's really no benefit from running them under leak/address sanitizer bots (generally only helpful to C++). Original change's description: > Local NTP, Realbox: fix navigation to chrome:// URLs > > Partially borrowed from https://crrev.com/c/1836021 by Archana Simha <archanasihma@chromium.org> > > Bug: 1020025 > Change-Id: I7de22a41d872ec624f25d1633782d27448988c58 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1893824 > Commit-Queue: David Bokan <bokan@chromium.org> > Reviewed-by: David Bokan <bokan@chromium.org> > Reviewed-by: Moe Ahmadi <mahmadi@chromium.org> > Reviewed-by: Nasko Oskov <nasko@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Auto-Submit: Dan Beam <dbeam@chromium.org> > Cr-Commit-Position: refs/heads/master@{#715106} TBR=dcheng@chromium.org,bokan@chromium.org,nasko@chromium.org,mahmadi@chromium.org Bug: 1020025 Change-Id: I666c372b76a058702c29b88eff765c2cd90c0bb2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917982Reviewed-by:
Dan Beam <dbeam@chromium.org> Reviewed-by:
Moe Ahmadi <mahmadi@chromium.org> Commit-Queue: Dan Beam <dbeam@chromium.org> Cr-Commit-Position: refs/heads/master@{#715459}
-
tby authored
As discussed in crrev.com/c/1911067, this is where the majority of the code for structured metrics will live, along with some initialization in /chrome/browser/metrics. This CL just creates the directory and adds me as OWNER. Same as for crrev.com/c/1911067, let's revisit ownership once this is implemented. Bug: 1016655 Change-Id: I498758fa1b7875e0bd782c32d3cc60cee7a94d33 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915640Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Tony Yeoman <tby@chromium.org> Cr-Commit-Position: refs/heads/master@{#715458}
-
Matt Jones authored
This patch also removes some obsolete logic from the days of Chrome Home like whether the android version of the toolbar is showing. Bug: 986310, 1002277 Change-Id: Ib939d02cdf783cc0fde30dbf7089171ae52c92a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1909930Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#715457}
-
Donn Denman authored
Extends the expiration of Finch experiment flags for: - OverlayNewLayout to 81 - ContextualSearchDefinitions to 82 - ContextualSearchLongpress to 83 - Ephemeral Tab experiments to 82 - Simplified Server to 79 (since it's at 100%). Change-Id: Ia4a496183addae6a6e80d296415e138a1f3e02f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918141 Auto-Submit: Donn Denman <donnd@chromium.org> Reviewed-by:
Jinsuk Kim <jinsukkim@chromium.org> Commit-Queue: Donn Denman <donnd@chromium.org> Cr-Commit-Position: refs/heads/master@{#715456}
-
Sam Maier authored
Allow users to set the compression level while using CompressHelper(). There is a 2x to 4x compression speed difference between levels [1..9], where it is possible to trade speed for better compression ratios (and vice-versa). It is up the user to experiment with the data to find the sweet spot for its application (default compression level is 6). Bug: 833361 Change-Id: Ia710cc7322707ace242133b283610eaa48def31f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912791 Commit-Queue: Adenilson Cavalcanti <cavalcantii@chromium.org> Commit-Queue: Sam Maier <smaier@chromium.org> Auto-Submit: Sam Maier <smaier@chromium.org> Reviewed-by:
Adenilson Cavalcanti <cavalcantii@chromium.org> Cr-Commit-Position: refs/heads/master@{#715455}
-
tby authored
This creates a directory for the structured metrics XML file. The next CLs will add codegen and presubmits similar to UKM's here too. Some notes: Naming: In an attempt to use short names, I've called this 'smc' in code, for 'structured metrics collection'. I'm not wedded to this though, let me know if simply 'structured_metrics' is better. Ownership: For ease of development I've added myself as OWNER, but I don't think this is needed in perpetuity. It might be wise to hand off ownership at some point, let's talk more about that later on. My intention was for the whole Metrics team to inherit ownership of this directory. I'm not sure how the owners system works though, so let me know if that's not the case. Types: There was some discussion of allowing three kinds of metrics: unhashed, globally hashed, and per-user hashed. That will require some changes to the XML file format, but I don't think that's finalized yet, so let's tackle it in a follow-up CL. Bug: 1016655 Change-Id: I064bca4ca7bd7b818451f6b49eb400fab43496d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911067Reviewed-by:
-
Daniele Castagna authored
Some Chromebooks come with an ambient sensor light. Powerd will send signals containing an ambient temperature in Kelvin. This CL lands a few functions to convert the temperature coming from powerd to a RGB scale factors. Those factors will be used to adjust the screen color temperature. The table to convert the readings to overall temperature and the functions to map the overall temperature to RGB scale factors were provided by the Create team. Bug: b/138731765, 1021193 Test: ash_unittests --gtest_filter="*AmbientTemp*" Change-Id: Iffed207d0ceca89328c42a0e9bedbfc20a434ac3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902290Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Commit-Queue: Daniele Castagna <dcastagna@chromium.org> Cr-Commit-Position: refs/heads/master@{#715453}
-
Kevin Bailey authored
Since we now consider scores of 0 in tail culling, move DemoteOnDeviceSearchSuggestions() to before tail culling, in case a suggestion score gets cleared. If done after, this could remove most of the remaining suggestions. Note that this doesn't move deduplication. This is because, if done prior to tail culling, it could theoretically remove a suggestion that would have played a significant role in culling. Bug: 1023821 Change-Id: I50dbe38066209a900dc63e5ee0b95cb3540ffec6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913632 Commit-Queue: Kevin Bailey <krb@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#715452}
-
