backgroundparser: Relax assert a bit
Concern Raised76765692538d

Authored by kfunk on Apr 19 2017, 9:54 AM.

Description

backgroundparser: Relax assert a bit

A file with just a space as file name is fine -- let's rather make sure
we don't pass a folder instead of a file as url

CCBUG: 378933

Details

Auditors
nalvarez
Committed
kfunkApr 19 2017, 9:56 AM
Parents
R33:080ed286fe10: Work-around issue in Path(QString) ctor
Branches
Unknown
Tags
Unknown
Reverted By
R33:4ddfdf733c96: Revert "backgroundparser: Relax assert a bit"
nalvarez raised a concern with this commit.Apr 21 2017, 4:32 AM
nalvarez added a subscriber: nalvarez.

This change broke test_qmljscontexts and test_qmljsdeclarations at least in master (I didn't check 5.1). The error is:

INVALID URL ENCOUNTERED: "/internal/1993918864-functionContext.js" QUrl("file:///internal/1993918864-functionContext.js")

This commit now has outstanding concerns.Apr 21 2017, 4:32 AM