Junior [inactive] Reporter
	Description • 9 years ago

After Bug 1149868 lands, we need not to use |popPermissions| as a workaround in the test dom/apps/tests/test_third_party_homescreen.html. Just simply remove this two-lines chunk.

https://dxr.mozilla.org/mozilla-central/source/dom/apps/tests/test_third_party_homescreen.html#178

	mootoh Assignee
	Comment 1 • 9 years ago

Here is a quick patch for this, which looks too obvious though.

	Junior [inactive] Reporter
	Comment 2 • 9 years ago

Since Bug 1149868 has landed, here's something you could do next:
1. Take this bug
2. Rebase and test again to make sure your patch work (locally test and/or push to try server)
3. Request a review to a module owner

Enjoy!

	mootoh Assignee
	Comment 3 • 9 years ago

Thanks :junior and :glob, I asked a commit access Level 1 in Bug 1168288 to test it on try server.

	mootoh Assignee
	Comment 4 • 9 years ago

Tested the change on try server: https://treeherder.mozilla.org/#/jobs?repo=try&revision=5002ae21d4bc

It looks like it failed in browser/base/content/test/general/browser_popup_blocker.js, though I guess it is not related to this change.

	[:fabrice] Fabrice Desré
	Comment 5 • 9 years ago

Comment on attachment 8610094 [details] [diff] [review]
1167503.diff

Review of attachment 8610094 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, I re-triggered bc1 and it's green.

	mootoh Assignee
	Comment 6 • 9 years ago

Thanks Fabrice for the review.
Now I think the steps :junior mentioned have been completed. What else I can do to land this?

	[:fabrice] Fabrice Desré
	Comment 7 • 9 years ago

(In reply to mootoh from comment #6)
> Thanks Fabrice for the review.
> Now I think the steps :junior mentioned have been completed. What else I can
> do to land this?

You need to update your patch commit message to add r=fabrice, then you can set `checkin-needed` in the keyword field of the bug. Thanks for contributing!

	Junior [inactive] Reporter
	Comment 8 • 9 years ago

Hello mootoh, 
If you need some information from somebody, you could use needinfo flag.
Happy hacking!

	mootoh Assignee
	Comment 9 • 9 years ago

Updated the commit message of the patch to include the reviewer.

	mootoh Assignee
	Comment 10 • 9 years ago

Thanks Fabrice, the patch has been updated and I've just set the keyword as `checkin-needed`.
Alright Junior, good to know that and I'll do it next.

	Pulsebot
	Comment 11 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/cf6ccc2b40fd

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 12 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/cf6ccc2b40fd

	Junior [inactive] Reporter
	Comment 13 • 9 years ago

congrat!

	mootoh Assignee
	Comment 14 • 9 years ago

Thanks everyone helping me! Now I'm going to search another ones.