Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(351)

Issue 804163002: Revert 186849 "Bind Window focus ability to the ExecutionContext..." (Closed)

Created:
6 years ago by amineer_google
Modified:
6 years ago
CC:
blink-reviews, michaeln, jsbell+serviceworker_chromium.org, kenjibaheux+watch_chromium.org, sof, tzik, serviceworker-reviews, eae+blinkwatch, nhiroki, falken, blink-reviews-dom_chromium.org, dglazkov+blink, kinuko+serviceworker, horo+watch_chromium.org, rwlbuis
Project:
blink
Visibility:
Public.

Description

Revert 186849 "Bind Window focus ability to the ExecutionContext..." Reverting this on branch only in an attempt to ship a non-crashy dev. Still following up on trunk... BUG=441968 > Bind Window focus ability to the ExecutionContext instead of the process. > > WindowFocusAllowedIndicator is using a static variable that allows > the entire process to focus a window. In addition, it is entirely > stack based which prevents asynchronous methods to use it properly. > > By binding the window focus ability to the ExecutionContext, > we can allow asynchronous methods but also, reduces the chances > of unrelated scripts to focus windows when they should not be > able to. Those two concerns are slightly related because an > ExecutionContext will now be able to focus a window for a > long period of time, thus, we do not want other ExecutionContext > to be able to do that in the meantime. > > This is part of a three-sided CL: > Part 1: <this> > Part 2: https://codereview.chromium.org/786403003 > Part 3: https://codereview.chromium.org/722423004 > > BUG=440740 > > Review URL: https://codereview.chromium.org/777483004 TBR=mlamouri@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=187184

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -151 lines) Patch
M LayoutTests/http/tests/serviceworker/client-focus.html View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/core.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/dom/ExecutionContext.h View 2 chunks +0 lines, -10 lines 0 comments Download
M Source/core/dom/ExecutionContext.cpp View 3 chunks +0 lines, -22 lines 0 comments Download
D Source/core/dom/ScopedWindowFocusAllowedIndicator.h View 1 chunk +0 lines, -24 lines 0 comments Download
D Source/core/dom/ScopedWindowFocusAllowedIndicator.cpp View 1 chunk +0 lines, -26 lines 0 comments Download
M Source/core/frame/LocalDOMWindow.cpp View 2 chunks +8 lines, -7 lines 0 comments Download
M Source/modules/notifications/Notification.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/modules/serviceworkers/ServiceWorkerClient.cpp View 2 chunks +3 lines, -6 lines 0 comments Download
M Source/modules/serviceworkers/WaitUntilObserver.cpp View 2 chunks +0 lines, -11 lines 0 comments Download
M Source/web/WebScopedWindowFocusAllowedIndicator.cpp View 2 chunks +0 lines, -11 lines 0 comments Download
M Source/web/tests/WebScopedWindowFocusAllowedIndicatorTest.cpp View 2 chunks +1 line, -20 lines 0 comments Download
M public/web/WebScopedWindowFocusAllowedIndicator.h View 1 chunk +0 lines, -9 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
amineer_google
6 years ago (2014-12-15 22:26:44 UTC) #1
amineer_google
6 years ago (2014-12-15 22:27:15 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as r187184.

Powered by Google App Engine
This is Rietveld 408576698