Skip to content
This repository was archived by the owner on Feb 26, 2022. It is now read-only.

Bug 1118614 - Remove redundant test/context-menu/utils openWindow and closeWindow functions#1812

Merged
erikvold merged 1 commit intomozilla:masterfrom
erikvold:1118614
Jan 7, 2015
Merged

Bug 1118614 - Remove redundant test/context-menu/utils openWindow and closeWindow functions#1812
erikvold merged 1 commit intomozilla:masterfrom
erikvold:1118614

Conversation

@erikvold
Copy link
Contributor

@erikvold erikvold commented Jan 7, 2015

No description provided.

@erikvold
Copy link
Contributor Author

erikvold commented Jan 7, 2015

@Gozala r?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this for ? It does not seems to be used

@Gozala
Copy link
Contributor

Gozala commented Jan 7, 2015

@erikvold please do not change test/test-context-menu@2.js it does not seems to be necessary. With changes to only test/context-menu/util it's r+. If changes to other file are necessary please ping me to explain why.

erikvold added a commit that referenced this pull request Jan 7, 2015
Bug 1118614 - Remove redundant test/context-menu/utils openWindow and closeWindow functions r=@Gozala
@erikvold erikvold merged commit 2e521be into mozilla:master Jan 7, 2015
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.

2 participants