342226 - Fall back on localURI if a local microsummary generator has no sourceURI

Status: RESOLVED FIXED

(Firefox Graveyard :: Microsummaries, defect, P3)

Description

[Simon Bünzli]

... for all those developers who prefer not to have to make up a fake URL when testing generators locally. Of course, updating a generator where sourceURI.equals(localURI) won't make sense, but preventing that shouldn't be an issue...

Comment 1

[Simon Bünzli]

Attachment: fix

This fix is simpler than I thought. Am I missing something?

Comment 2

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 226431 [details] [diff] [review]
fix


This seems reasonable, the code looks good, and it works fine.  Note that we'll not want this functionality to trickle out into production generators, but we should be able to do that by making sure we continue to write the source URI into generators we install before we write them to disk.

Comment 3

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 226431 [details] [diff] [review]
fix

Per mconnor's blog post (http://steelgryphon.com/blog/?p=81), removing approval request until patch has landed on the trunk, baked, and has justification for landing on the branch for beta1.

Comment 4

[Myk Melez [:myk] [@mykmelez]]

Checked in to trunk.  Holding branch checkin per beta1 branch lockdown.

Comment 5

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 226431 [details] [diff] [review]
fix

This was checked into the trunk a week ago, and it doesn't seem to have caused any problems there.  It's not a critical fix, but it's a useful one for developers, who will be a disproportionately high percentage of initial microsummary consumers, so it'd be good to get it on the branch, which is where those developers will likely initially encounter the feature.  It presents a low risk of regression.

Comment 6

[Myk Melez [:myk] [@mykmelez]]

Fix checked in to branch.