New issue
Advanced search Search tips

Issue 739542 link

Starred by 6 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug

Blocking:
issue 707375



Sign in to add a comment

V2 Sandbox: Forward elastic scrolling property from browser to renderer.

Reported by ale160...@gmail.com, Jul 5 2017

Issue description

on latest chrome canary version 61 enabling the new flag
mac-v2-sandbox (set to true )makes  impossible to disable Rubber-Band / Elastic Scroll in macOS using terminal command (defaults write -g NSScrollViewRubberbanding -bool false)

please solve the issue the main reason i switched to chrome years ago is that chrome(compared to safari) respects that termianl command to disable rubber  band scrooling( which i really hate)
 

Comment 1 by b...@chromium.org, Jul 6 2017

Components: Blink>Scroll
Cc: rsesek@chromium.org
Components: Internals>Sandbox
Owner: kerrnel@chromium.org
Status: Assigned (was: Unconfirmed)
We should continue to block access to cfprefs in sandbox v2, which is probably why this is failing. But perhaps we could ferry the value of NSScrollViewRubberbanding over IPC in theme_helper_mac.mm.
Blocking: 707375
Status: Started (was: Assigned)
Summary: V2 Sandbox: Forward elastic scrolling property from browser to renderer. (was: mac-v2-sandbox makes impossible to disable elastic scrooling on osx)
Project Member

Comment 5 by bugdroid1@chromium.org, Jul 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a7b943b9f3c63f03cec4dec83b351861d4c2d7a5

commit a7b943b9f3c63f03cec4dec83b351861d4c2d7a5
Author: Greg Kerr <kerrnel@chromium.org>
Date: Mon Jul 24 23:17:17 2017

Forward elastic scrolling information from browser to renderer.

Forward elastic scrolling preferences from the browser  to the renderer,
since the renderer can no longer access cfprefsd.

Bug:  739542 , 689306 
Change-Id: Id6e0a9bb66ddd51b4f61bd941a29d964c9376e90
Reviewed-on: https://chromium-review.googlesource.com/580672
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Reviewed-by: Nasko Oskov <nasko@chromium.org>
Reviewed-by: Mark Mentovai <mark@chromium.org>
Commit-Queue: Greg Kerr <kerrnel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#489131}
[modify] https://crrev.com/a7b943b9f3c63f03cec4dec83b351861d4c2d7a5/content/browser/theme_helper_mac.mm
[modify] https://crrev.com/a7b943b9f3c63f03cec4dec83b351861d4c2d7a5/content/common/renderer.mojom
[modify] https://crrev.com/a7b943b9f3c63f03cec4dec83b351861d4c2d7a5/content/renderer/render_thread_impl.cc

Status: Fixed (was: Started)

Sign in to add a comment