Skip to content

Commit

Permalink
feat: replace BrowserView with WebContentsView (#40759)
Browse files Browse the repository at this point in the history
* feat: replace BrowserView with WebContentsView

Co-authored-by: Jeremy Rose <jeremya@chromium.org>

* Update appveyor.yml

---------

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: Jeremy Rose <jeremya@chromium.org>
  • Loading branch information
trop[bot] and nornagon committed Jan 11, 2024
1 parent c2524d7 commit bd304f7
Show file tree
Hide file tree
Showing 75 changed files with 2,987 additions and 1,531 deletions.
7 changes: 0 additions & 7 deletions BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -693,13 +693,6 @@ source_set("electron_lib") {
]
}

if (enable_views_api) {
sources += [
"shell/browser/api/views/electron_api_image_view.cc",
"shell/browser/api/views/electron_api_image_view.h",
]
}

if (enable_printing) {
sources += [
"shell/browser/printing/print_view_manager_electron.cc",
Expand Down
6 changes: 0 additions & 6 deletions build/webpack/webpack.config.base.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,12 +53,6 @@ module.exports = ({

const ignoredModules = [];

if (defines.ENABLE_VIEWS_API === 'false') {
ignoredModules.push(
'@electron/internal/browser/api/views/image-view.js'
);
}

const plugins = [];

if (onlyPrintingGraph) {
Expand Down
1 change: 0 additions & 1 deletion buildflags/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ buildflag_header("buildflags") {
header = "buildflags.h"

flags = [
"ENABLE_VIEWS_API=$enable_views_api",
"ENABLE_PDF_VIEWER=$enable_pdf_viewer",
"ENABLE_ELECTRON_EXTENSIONS=$enable_electron_extensions",
"ENABLE_BUILTIN_SPELLCHECKER=$enable_builtin_spellchecker",
Expand Down
2 changes: 0 additions & 2 deletions buildflags/buildflags.gni
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@
# found in the LICENSE file.

declare_args() {
enable_views_api = true

enable_pdf_viewer = true

# Provide a fake location provider for mocking
Expand Down
4 changes: 3 additions & 1 deletion docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ These individual tutorials expand on topics discussed in the guide above.

* [app](api/app.md)
* [autoUpdater](api/auto-updater.md)
* [BrowserView](api/browser-view.md)
* [BaseWindow](api/base-window.md)
* [BrowserWindow](api/browser-window.md)
* [contentTracing](api/content-tracing.md)
* [desktopCapturer](api/desktop-capturer.md)
Expand Down Expand Up @@ -134,8 +134,10 @@ These individual tutorials expand on topics discussed in the guide above.
* [TouchBar](api/touch-bar.md)
* [Tray](api/tray.md)
* [utilityProcess](api/utility-process.md)
* [View](api/view.md)
* [webContents](api/web-contents.md)
* [webFrameMain](api/web-frame-main.md)
* [WebContentsView](api/web-contents-view.md)

### Modules for the Renderer Process (Web Page):

Expand Down

0 comments on commit bd304f7

Please sign in to comment.