This for some reason only started being an issue now, but basically we
have to make sure that an empty line is left at the end of
`package.json`, otherwise we get presubmit failures.
Resolves https://github.com/brave/brave-browser/issues/46229
This change corrects all crate patches, changing them to point to
`brave/`. This eliminates the need of patching `gnrt vendor` to address
this patch mismatch.
Patches should be created now with:
```sh
git format-patch \
--start-number=101 \
--src-prefix=a/brave/ \
--dst-prefix=b/brave/ \
--output-directory \
third_party/rust/chromium_crates_io/patches/some-crate/ \
HEAD^
```
Resolves https://github.com/brave/brave-browser/issues/46201
fixbrave/brave-browser#46161
TEST=SplitViewLocationBarBrowserTest.*
launch browser with --enable-features=SideBySide
Select Add tab to split view from tab's context menu
Check mini urlbar is displayed on the inactive tab's contents
This change ensures that the deprecated `WKPreferences.javaScriptEnabled` is correctly reset between navigations and also changes the flow so that this legacy preference is only set in iOS 17 as iOS 18 solves the original issue without setting this value.
fix https://github.com/brave/brave-browser/issues/46088
MultiContentsView is container view that holds two contents view.
We'll migrate what we're doing with SplitView class into it.
This subclass will be used to apply our split view UX at contents area.
This PR adjusted Chromium's split view radius/border styles.
TEST=SideBySideEnabledBrowserTest.*
* add prettier to format, call format from PRESUBMIT
* make format.js
* update prettier
* Add unformatted dirs to .prettierignore
* Fix linter errors
* cleanup .prettierignore
* Convert format.js to a script
* review fixes
* add some prettier-ignore-*
* Address some review issues
* move commander declaration
* Add --presubmit and --dry-run for format
* Revert "add some prettier-ignore-*"
* Fix eslint, add printWidth exclusion
* Remove printWidth override
* Fix eslint and format warn
* use bracketSameLine
* Review fixes
* use a single git cl format call
* Use spawnSync to avoid log flooding
* More review fixes
* More review updates
* Update build/commands/scripts/format.js
Co-authored-by: Aleksei Khoroshilov <5928869+goodov@users.noreply.github.com>
* Renames some vars
* Use absolute path for format.js
---------
Co-authored-by: Aleksei Khoroshilov <5928869+goodov@users.noreply.github.com>
* wrap and expose vertical tab strip toggling as public function
* properly toggle independent vertical tab strip states
* Added test code
---------
Co-authored-by: Simon Hong <shong@brave.com>
Mikhail