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

WEBUI context error when autofill entry is saved #13421

Closed
srirambv opened this issue Mar 12, 2018 · 6 comments
Closed

WEBUI context error when autofill entry is saved #13421

srirambv opened this issue Mar 12, 2018 · 6 comments

Comments

@srirambv
Copy link
Collaborator

srirambv commented Mar 12, 2018

Description

WEBUI context error when autofill entry is saved

Steps to Reproduce

  1. Clean install 0.21.656
  2. Have an autofill entry created
  3. Open https://www.roboform.com/filling-test-all-fields, click on a filed and select autofill entry
  4. Click the auto-filled field and clear form
  5. Switch tab focus
    6 Console throws error after a few minutes

Actual result:

[7528:1280:0312/174543.509:ERROR:CONSOLE(0)] "(WEBUI context) chrome://brave/C:/Users/bellurs/AppData/Local/BraveBeta/app-0.21.656/resources/app.asar/app/extensions/brave/gen/app.entry.js:1: Uncaught Error: e=>{s.autofillSelectionClicked(t.get("tabId"),r,i,o)} could not be cloned.{Error: e=>{s.autofillSelectionClicked(t.get("tabId"),r,i,o)} could not be cloned.
    at EventEmitter.s.on (chrome://brave/C:/Users/bellurs/AppData/Local/BraveBeta/app-0.21.656/resources/app.asar/app/extensions/brave/gen/app.entry.js:1:961420)
    at emitTwo (extensions::event_emitter:105:13)
    at EventEmitter.emit (extensions::event_emitter:193:7)
    at EventEmitter.ipcRenderer.emit (extensions::ipc_utils:35:22)}", source: chrome://brave/C:/Users/bellurs/AppData/Local/BraveBeta/app-0.21.656/resources/app.asar/app/extensions/brave/index.html (0)

Expected result:
Shouldn't throw any error

Reproduces how often:
No idea

Brave Version

about:brave info:

Brave 0.21.656
Brave Sync v1.4.2
libchromiumcontent 65.0.3325.146
Muon 4.9.0
Node.js 7.9.0
OS Architecture x64
OS Platform Microsoft Windows
OS Release 10.0.16299
rev c8e9925
Update Channel Beta
V8 6.5.254.31

Reproducible on current live release:
No

Additional Information

cc: @btlechowski @LaurenWags @kjozwiak to verify on other platforms

@srirambv srirambv added bug OS/Windows needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. feature/autofill 0.21.x issue first seen in 0.21.x cr65 labels Mar 12, 2018
@srirambv srirambv added this to the 0.21.x w/ Chromium 65 (Beta Channel) milestone Mar 12, 2018
@srirambv
Copy link
Collaborator Author

cc: @darkdh @bridiver @bsclifton

@LaurenWags
Copy link
Member

Have not been able to reproduce on MacOS yet.

@kjozwiak
Copy link
Member

kjozwiak commented Mar 12, 2018

I managed to run into this as well on macOS but I'm not 100% sure if it's related to auto-fill. When I ran into the above issue, I was going through the following use case:

  • visited motorsport.com and opened a few tabs using command + click
  • visited reddit.com and bookmarked the website (Bookmark Toolbar)
  • opened several tabs from reddit using command + click
  • switched over to a tab that was opened from reddit and exited the browser
[24312:775:0312/120406.826495:ERROR:CONSOLE(0)] "(WEBUI context) chrome://brave/Applications/Brave-Beta.app/Contents/Resources/app.asar/app/extensions/brave/gen/app.entry.js:1: Uncaught Error: function (t,n){c({openerTabId:e})} could not be cloned.{Error: function (t,n){c({openerTabId:e})} could not be cloned.
    at EventEmitter.s.on (chrome://brave/Applications/Brave-Beta.app/Contents/Resources/app.asar/app/extensions/brave/gen/app.entry.js:1:961419)
    at emitTwo (extensions::event_emitter:105:13)
    at EventEmitter.emit (extensions::event_emitter:193:7)
    at EventEmitter.ipcRenderer.emit (extensions::ipc_utils:35:22)}", source: chrome://brave/Applications/Brave-Beta.app/Contents/Resources/app.asar/app/extensions/brave/index.html (0)

Once I relaunched the browser, I was missing all my tabs that were opened from reddit. There's definitely a bad bug somewhere as it resulted in losing session state. However, I can't seem to reproduce it again.

@darkdh
Copy link
Member

darkdh commented Mar 12, 2018

reduce the sessionSaveInterval

sessionSaveInterval: 1000 * 60 * 5,

would be much easier to reproduce the function is not serializable error causing by we put lots of functions into array template
screen shot 2018-03-12 at 4 10 09 pm

@darkdh darkdh removed the needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. label Mar 13, 2018
darkdh added a commit that referenced this issue Mar 13, 2018
fix #13421
fix #11576

Auditors: @bridiver

Test Plan:
1. Set `sessionSaveInterval` to `1` in js/constants/appConfig.js
2. Playing around with bookmark toolbar folder
3. Playing around with auotfill menu(fill with entry and clear)
4. Playing around with hamburger menu
5. Playing around with back/forward long pressed menu
6. There shouldn't be any exceptions about "function can be cloned"
@bsclifton
Copy link
Member

Fixed with #13432

@srirambv
Copy link
Collaborator Author

No console error based on steps from original issue description or from #13421 (comment)

bsclifton pushed a commit that referenced this issue Mar 16, 2018
fix #13421
fix #11576

Auditors: @bridiver

Test Plan:
1. Set `sessionSaveInterval` to `1` in js/constants/appConfig.js
2. Playing around with bookmark toolbar folder
3. Playing around with auotfill menu(fill with entry and clear)
4. Playing around with hamburger menu
5. Playing around with back/forward long pressed menu
6. There shouldn't be any exceptions about "function can be cloned"
bsclifton pushed a commit that referenced this issue Mar 16, 2018
fix #13421
fix #11576

Auditors: @bridiver

Test Plan:
1. Set `sessionSaveInterval` to `1` in js/constants/appConfig.js
2. Playing around with bookmark toolbar folder
3. Playing around with auotfill menu(fill with entry and clear)
4. Playing around with hamburger menu
5. Playing around with back/forward long pressed menu
6. There shouldn't be any exceptions about "function can be cloned"
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants