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

Remove error message about favicon when the error is just not-found. #44

Closed
wants to merge 2 commits into from
Closed

Conversation

ianb
Copy link
Contributor

@ianb ianb commented Jan 5, 2016

An example site that emits this warning continuously is https://bugzilla.mozilla.org

@mozilla-autolander-deprecated
Copy link
Contributor

Autolander could not find a bug number in your pull request title. All pull requests should be in the format of: Bug [number] - [description].

@mozilla-autolander-deprecated
Copy link
Contributor

Autolander could not find a bug number in your pull request title. All pull requests should be in the format of: Bug [number] - [description].

@Mardak
Copy link
Member

Mardak commented Jan 6, 2016

Fyi, you pushed Fix the profile creation documentation 7ac0b81 to master (so it's included on this PR).

@Standard8
Copy link
Member

@ianb you could land this with r=Standard8 if you fix the branch the branch/patch to only land the MozLoopAPI.jsm changes and not the profile creation docs.

@ianb
Copy link
Contributor Author

ianb commented Jan 12, 2016

Fixed commit and merged in #70

@ianb ianb closed this Jan 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants