Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Android] sync crashes when node is removed and then same objectId comes from cloud - follow up to 8642 #8740

Closed
LaurenWags opened this issue Mar 18, 2020 · 4 comments
Assignees

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Mar 18, 2020

Description

Found while testing #8642

Issue 8642 is reproducible on x86 device. When resolved, should probably be retested with ARM to ensure no regressions.

Steps to reproduce

  1. Take a brave-browser desktop containing the changes from Restore other bookmarks brave-core#4404, I was using 1.8.19 on Nightly
  2. Create a chain on desktop
  3. Connect Android browser (1.5.111) to the chain
  4. On desktop, navigate to Bookmarks Manager and create folder OtherBookmark01 inside of Other bookmarks root folder
  5. On desktop create a bookmark OtherBookmark01.com inside of OtherBookmark01 folder from pt2
  6. Wait until OtherBookmark01 and OtherBookmark01.com appear on Android browser
  7. Verify the structure of bookmarks on Android is
Other bookmarks
 +OtherBookmark01
    +OtherBookmark01.com
  1. Delete folder Other bookmarks on Android
  2. Working quickly, on desktop create folder OtherBookmark02 inside of Other bookmarks root folder
  3. Again, working quickly, on desktop create a bookmark OtherBookmark02.com inside of OtherBookmark02 folder from pt9
  4. Wait for sync to complete

Actual result

  1. See crash on Android
    Screenshot_20200318-093440

Note - after the crash sync is basically unusable - any further additions/changes are not synced.

Expected result

  1. No crash on Android. Bookmarks structure is
Other bookmarks
 +OtherBookmark02
   +OtherBookmark02.com

Issue reproduces how often

easily, reproduced 3x

Issue happens on

  • Current Play Store version? n/a
  • Beta build? yes, 1.5.111

Device details

  • Install type (ARM, x86): x86
  • Device (Phone, Tablet, Phablet): all
  • Android version: all

Brave version

1.5.111

Website problems only

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Additional information

cc @srirambv @AlexeyBarabash

@AlexeyBarabash
Copy link
Contributor

Done two attempts on both versions 1.5.111 (current beta in GP) and 1.8.42 (current master build) on Android 9 x86 emulator - cannot reproduce this issue.

@LaurenWags which version of Android is installed on your x86 device?

@LaurenWags
Copy link
Member Author

@AlexeyBarabash - Android 6

@AlexeyBarabash
Copy link
Contributor

Thanks @LaurenWags
I could not reproduce it with Android 6 emulator, because emulator crashed once it tried to use the camera to scan QR code, but I could reproduce it with Android 5 emulator.

@AlexeyBarabash
Copy link
Contributor

I could reproduce this on arm Android 9 physical device.
The tricky step is
8.5 Don't wait until deleted folder would be on desktop, to to step 9 immediately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants