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

Sync failed DCHECK crash on initialization after Chromium update 78 #6435

Closed
AlexeyBarabash opened this issue Oct 11, 2019 · 1 comment · Fixed by brave/brave-core#3681
Closed

Comments

@AlexeyBarabash
Copy link
Contributor

AlexeyBarabash commented Oct 11, 2019

Description

Starting with version Version 0.72.89 Chromium: 78.0.3904.50 (Developer Build) (64-bit) browser crashes right after enabling sync if there is any created bookmark .

Steps to Reproduce

  1. Get a clean profile
  2. Create bookmark A1.com
  3. Go to brave://sync
  4. Create sync chain and wait

Actual result:

Failed DCHECK crash with stack of

[25896:25896:1011/171732.684433:INFO:syncer_helper.cc(105)] [BraveSync] AddBraveMetaInfo 000 node=A1.com
[25896:25896:1011/171732.684490:INFO:syncer_helper.cc(106)] [BraveSync] AddBraveMetaInfo stack=#0 0x7f1eb69dad8f base::debug::CollectStackTrace()
#1 0x7f1eb671662d base::debug::StackTrace::StackTrace()
#2 0x7f1eb67165e8 base::debug::StackTrace::StackTrace()
#3 0x55c65f7a500e brave_sync::AddBraveMetaInfo()
#4 0x55c66510f1b2 sync_bookmarks::BookmarkChangeProcessor::UpdateSyncNodeProperties()
#5 0x55c665110b21 sync_bookmarks::BookmarkChangeProcessor::CreateSyncNode()
#6 0x55c665122742 sync_bookmarks::BookmarkModelAssociator::BuildAssociations()
#7 0x55c665120ad2 sync_bookmarks::BookmarkModelAssociator::BuildAssociations()
#8 0x55c665120017 sync_bookmarks::BookmarkModelAssociator::AssociateModels()
#9 0x55c665117f7f syncer::FrontendDataTypeController::Associate()

Expected result:

No crash and sync works

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 0.72.89 Chromium: 78.0.3904.50 (Developer Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS Linux

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the dev channel?
  • Can you reproduce this issue with the nightly channel?
    Only on Nightly

Other Additional Information:

  • 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?
    NA

Miscellaneous Information:

NA

@AlexeyBarabash AlexeyBarabash changed the title Sync crash on initialization after Chromium update 78 Sync failed DCHECK crash on initialization after Chromium update 78 Oct 11, 2019
@bsclifton bsclifton added this to the 0.72.x - Nightly milestone Oct 11, 2019
@btlechowski
Copy link

btlechowski commented Oct 17, 2019

Verification passed on

Brave 0.70.118 Chromium: 78.0.3904.50 (Official Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS Ubuntu 18.04 LTS

Verified test plan from brave/brave-core#3681
No crash.

Verification passed on

Brave 0.70.118 Chromium: 78.0.3904.50 (Official Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified test plan from Disable Sync USS Bookmarks brave-core#3681
  • Verified that there is no crash when bookmarks are added before sync
  • Verified that there is no crash when multiple devices are added to sync chain

image

Verified passed with

Brave 0.70.119 Chromium: 78.0.3904.50 (Official Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS macOS Version 10.13.6 (Build 17G5019)
  • Verified test plan from Disable Sync USS Bookmarks brave-core#3681
  • Verified that there is no crash when bookmarks are added before sync
  • Verified that there is no crash when multiple devices are added to sync chain

Screen Shot 2019-10-21 at 4 54 28 PM

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