-
Notifications
You must be signed in to change notification settings - Fork 975
Regression - importing cookies not working #8849
Comments
It is not unresponsive of long running. The function on muon doesn't work due to brave/muon@d534ffd. |
Import runs in a service worker so shouldn't make the browser unresponsive. The code is mostly just moved to accommodate the removal of patches, can you look into this @darkdh?
… On May 13, 2017, at 10:13 PM, Anthony Tseng ***@***.***> wrote:
It is not unresponsive of long running. The function on muon doesn't work due to ***@***.***
Functions of importing cookies from both Firefox and Chrome is broken.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
along with process singleton this was a known possible regression with the patch cleanup
… On May 13, 2017, at 10:13 PM, Anthony Tseng ***@***.***> wrote:
It is not unresponsive of long running. The function on muon doesn't work due to ***@***.***
Functions of importing cookies from both Firefox and Chrome is broken.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I've already looked into it. almost done. |
fix brave/browser-laptop#8849 Auditors: @bridiver, @bbondy
fix brave/browser-laptop#8849 Auditors: @bridiver, @bbondy
I'm not sure if this was really fixed. STR:
Actual result: |
It is expected. We can only import unencrypted cookies from Chrome. Unfortunately, the cookies we test are encrypted. |
@darkdh importing cookie from Firefox still doesn't show up on Brave |
@srirambv, you did clear cookies before importing so there is nothing to import. you must have cookies set in firefox first |
fix brave/browser-laptop#8849 Auditors: @bridiver, @bbondy
…sion brave/browser-laptop#6366 brave/browser-laptop#6262 brave/browser-laptop#8389 Pinterest extension public key and string Fix broken import cookies fix brave/browser-laptop#8849 Auditors: @bridiver, @bbondy
…sion brave/browser-laptop#6366 brave/browser-laptop#6262 brave/browser-laptop#8389 Pinterest extension public key and string Fix broken import cookies fix brave/browser-laptop#8849 Auditors: @bridiver, @bbondy
…sion brave/browser-laptop#6366 brave/browser-laptop#6262 brave/browser-laptop#8389 Pinterest extension public key and string Fix broken import cookies fix brave/browser-laptop#8849 Auditors: @bridiver, @bbondy
STR:
The text was updated successfully, but these errors were encountered: