-
Evan Stade authored
The code in this component is unique to javascript dialogs, and does not cover all app modals. Further, it has some view code that isn't specifically for app modal javascript dialogs, but is re-used for tab modals. Some tab modal code from //chrome will be moved here soon, so rename from app_modal to javascript_dialogs. Class names reflect whether their use is specific to the app modal control flow. Also, rename some classes to better reflect their roles: . JavaScriptDialogManager (implements content::JavaScriptDialogManager) => AppModalDialogManager . JavaScriptAppModalDialog => AppModalDialogController . NativeAppModalDialog => AppModalDialogView etc. Bug: 1025256 Change-Id: I0a1865ec71897ae6549184bc317ede29cc56438a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2031936Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#737355}
7220e47a