Skip to content
This repository has been archived by the owner on Jun 6, 2019. It is now read-only.

Add incognito to the extension manifest and set to split mode #67

Merged
merged 1 commit into from
Sep 27, 2018
Merged

Add incognito to the extension manifest and set to split mode #67

merged 1 commit into from
Sep 27, 2018

Conversation

bsclifton
Copy link
Member

Matches the behavior of browser-laptop which you can see here:
https://github.com/brave/browser-laptop/blob/41e46f9ce7e0d4f7a558646f377e40d29d0e101d/app/extensions.js#L36-L216

This should help in solving brave/brave-browser#1198

Auditors: @bridiver, @diracdeltas

@bsclifton bsclifton self-assigned this Sep 27, 2018
@bsclifton
Copy link
Member Author

More info about the manifest item here:
https://developer.chrome.com/extensions/manifest/incognito

yrliou
yrliou previously approved these changes Sep 27, 2018
Copy link
Member

@yrliou yrliou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yrliou
Copy link
Member

yrliou commented Sep 27, 2018

Please open an issue for this PR, thanks.
Also, will need to update DEPS in brave-core after this is merged.

@bsclifton
Copy link
Member Author

@yrliou issue brave/brave-browser#1323 created; I amended the commit to include the closing keywords 😄 Ready for re-review/re-approval

@yrliou yrliou merged commit 86e9553 into brave:master Sep 27, 2018
@bsclifton bsclifton deleted the manifest-add-incognito-split branch September 27, 2018 19:10
bsclifton added a commit to brave/brave-core that referenced this pull request Sep 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants