If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Make Mozmill-test testDisabledPermissions local

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: ashughes, Assigned: ashughes)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [litmus-data])

Attachments

(3 attachments, 6 obsolete attachments)

(Assignee)

Description

7 years ago
Module: testPrivateBrowsing/testDisabledPermissions.js
Test-page: Use any 1 page from test-files/layout/mozilla*
(Assignee)

Updated

7 years ago
Blocks: 579965
(Assignee)

Updated

7 years ago
Assignee: nobody → anthony.s.hughes
Status: NEW → ASSIGNED
(Assignee)

Comment 1

7 years ago
Created attachment 465017 [details] [diff] [review]
Patch v1 (default)
Attachment #465017 - Flags: review?(aaron.train)
Comment on attachment 465017 [details] [diff] [review]
Patch v1 (default)

>-const localTestFolder = collector.addHttpResource('../test-files/');
>+const LOCAL_TEST_FOLDER = collector.addHttpResource('../test-files/');
>+const LOCAL_TEST_PAGES = [
>+                          LOCAL_TEST_FOLDER + 'popups/popups_2.html',
>+                          LOCAL_TEST_FOLDER + 'cookies/cookie_single.html'
>+                         ];

Nit: Indentation, two spaces
Attachment #465017 - Flags: review?(aaron.train) → review-
(Assignee)

Comment 3

7 years ago
Created attachment 465462 [details] [diff] [review]
Patch v2 (default)
Attachment #465017 - Attachment is obsolete: true
Attachment #465462 - Flags: review?(aaron.train)
(Assignee)

Comment 4

7 years ago
(In reply to comment #3)
> Created attachment 465462 [details] [diff] [review]
> Patch v2 (default)

I forgot to add "r=aaronmt" to the qrefresh message.  I'll be sure to add that prior to check-in.
Comment on attachment 465462 [details] [diff] [review]
Patch v2 (default)

(In reply to comment #4)
> (In reply to comment #3)
> > Created attachment 465462 [details] [diff] [review] [details]
> > Patch v2 (default)
> 
> I forgot to add "r=aaronmt" to the qrefresh message.  I'll be sure to add that
> prior to check-in.

Thanks. r+, no issues I can see - on to Henrik
Attachment #465462 - Flags: review?(aaron.train) → review+
(Assignee)

Updated

7 years ago
Attachment #465462 - Flags: review?(hskupin)
Comment on attachment 465462 [details] [diff] [review]
Patch v2 (default)

> var setupModule = function(module)
> {

nit: Move up the opening brackets.

>   var allow = new elementslib.XPath(controller.window.document, 
>-                                    "/*[name()='window']/*[name()='popupset'][1]/*[name()='menupopup'][4]/*[name()='menuitem'][1]");
>+                                    '/*[name()="window"]' +
>+                                    '/*[name()="popupset"][1]' +
>+                                    '/*[name()="menupopup"][4]' +
>+                                    '/*[name()="menuitem"][1]');

Looking forward to our shared module refactoring which will eliminate all those kinds of wonky strings from the tests.

r=me for the little nit update.
Attachment #465462 - Flags: review?(hskupin) → review+
(Assignee)

Comment 7

7 years ago
> Comment on attachment 465462 [details] [diff] [review]
> Patch v2 (default)

Landed on default:
http://hg.mozilla.org/qa/mozmill-tests/rev/e4511cd65637

Does not cleanly apply to mozilla1.9.2 or mozilla1.9.1, patches forthcoming
(Assignee)

Comment 8

7 years ago
Created attachment 465780 [details] [diff] [review]
Patch v2 (mozilla1.9.2)
Attachment #465780 - Flags: review?(aaron.train)
(Assignee)

Comment 9

7 years ago
Created attachment 465786 [details] [diff] [review]
Patch v2 (mozilla1.9.1)
Attachment #465786 - Flags: review?(aaron.train)
(Assignee)

Comment 10

7 years ago
Created attachment 465788 [details] [diff] [review]
Patch v2 (mozilla1.9.2)
Attachment #465780 - Attachment is obsolete: true
Attachment #465788 - Flags: review?(aaron.train)
Attachment #465780 - Flags: review?(aaron.train)

Updated

7 years ago
Attachment #465786 - Flags: review?(aaron.train) → review+

Updated

7 years ago
Attachment #465788 - Flags: review?(aaron.train) → review+
(Assignee)

Updated

7 years ago
Attachment #465786 - Flags: review?(hskupin)
(Assignee)

Updated

7 years ago
Attachment #465788 - Flags: review?(hskupin)
Comment on attachment 465786 [details] [diff] [review]
Patch v2 (mozilla1.9.1)

>+var setupModule = function(module) {
>+var teardownModule = function(module) {

nit: please remove the module parameters from both functions

>-  var property = mozmill.isWindows ? "popupWarningButton.accesskey" : "popupWarningButtonUnix.accesskey";
>-  var accessKey = UtilsAPI.getProperty("chrome://browser/locale/browser.properties", property);
>+  var property = mozmill.isWindows ? "popupWarningButton.accesskey" 
>+                                   : "popupWarningButtonUnix.accesskey";
>+  var accessKey = UtilsAPI.getProperty(
>+    "chrome://browser/locale/browser.properties", 
>+    property
>+  );
>-  controller.keypress(null, accessKey, {ctrlKey: mozmill.isMac, altKey: !mozmill.isMac});
>+  controller.keypress(null, accessKey, 
>+                      {ctrlKey: mozmill.isMac, altKey: !mozmill.isMac});
[..]
>-  var blockImages = new elementslib.ID(controller.window.document, "context-blockimage");
>+  var blockImages = new elementslib.ID(controller.window.document, 
>+                                       "context-blockimage");

Please revert this change to be consistent with default. If you want to change that please file a follow-up bug (before we should get a style guide setup). But don't add new stuff to backported patches.

>-  var historyMode = new elementslib.ID(controller.window.document, "historyMode");
>-  controller.waitForElement(historyMode, gTimeout);
>+  var historyMode = new elementslib.ID(controller.window.document, 
>+                                       "historyMode");
>+  controller.waitForElement(historyMode, TIMEOUT);
>   controller.select(historyMode, null, null, "custom");
>-  controller.sleep(gDelay);
> 
>-  var acceptCookies = new elementslib.ID(controller.window.document, "acceptCookies");
>+  var acceptCookies = new elementslib.ID(controller.window.document, 
>+                                         "acceptCookies");
>   controller.assertChecked(acceptCookies);
> 
>   // Select "ask me every time"
>-  var keepCookies = new elementslib.ID(controller.window.document, "keepCookiesUntil");
>-  controller.waitForElement(keepCookies, gTimeout);
>+  var keepCookies = new elementslib.ID(controller.window.document, 
>+                                       "keepCookiesUntil");
>+  controller.waitForElement(keepCookies, TIMEOUT);
>   controller.select(keepCookies, null, null, 1);
>-  controller.sleep(gDelay);

Same here.
Attachment #465786 - Flags: review?(hskupin) → review-
Comment on attachment 465788 [details] [diff] [review]
Patch v2 (mozilla1.9.2)

Same comments as for the 1.9.1 patch.
Attachment #465788 - Flags: review?(hskupin) → review-
(Assignee)

Comment 13

7 years ago
Created attachment 478141 [details] [diff] [review]
Patch v3 (1.9.2)
Attachment #465786 - Attachment is obsolete: true
Attachment #465788 - Attachment is obsolete: true
Attachment #478141 - Flags: review?(hskupin)
(Assignee)

Comment 14

7 years ago
Comment on attachment 478141 [details] [diff] [review]
Patch v3 (1.9.2)

Noticed some cleanup for this patch...new one coming.
Attachment #478141 - Flags: review?(hskupin)
(Assignee)

Comment 15

7 years ago
Created attachment 478143 [details] [diff] [review]
Patch v3.1 (1.9.2)
Attachment #478141 - Attachment is obsolete: true
Attachment #478143 - Flags: review?(hskupin)
Attachment #478143 - Flags: review?(hskupin) → review+
Keywords: checkin-needed
(Assignee)

Comment 16

7 years ago
(In reply to comment #15)
> Created attachment 478143 [details] [diff] [review]
> Patch v3.1 (1.9.2)

Landed:
http://hg.mozilla.org/qa/mozmill-tests/rev/b07ed23483e6 [mozilla1.9.2]
(Assignee)

Comment 17

7 years ago
Created attachment 478386 [details] [diff] [review]
Patch v3.1 (1.9.1)

1.9.2 didn't apply cleanly.  Here is the 1.9.1 backport.
Attachment #478386 - Flags: review?(hskupin)
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
Attachment #478386 - Flags: review?(hskupin) → review+
(Assignee)

Comment 18

7 years ago
Created attachment 478442 [details] [diff] [review]
Patch v3.2 (default)

Final suggested revisions made.
(Assignee)

Comment 19

7 years ago
(In reply to comment #18)
> Created attachment 478442 [details] [diff] [review]
> Patch v3.2 (default)
> 
> Final suggested revisions made.

Wrong bug...sorry...
(Assignee)

Updated

7 years ago
Attachment #478442 - Attachment is obsolete: true
(Assignee)

Comment 20

7 years ago
(In reply to comment #17)
> Created attachment 478386 [details] [diff] [review]
> Patch v3.1 (1.9.1)
> 
> 1.9.2 didn't apply cleanly.  Here is the 1.9.1 backport.

Landed:
http://hg.mozilla.org/qa/mozmill-tests/rev/bdf38b387ce5 [mozilla1.9.1]
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Move of Mozmill Test related project bugs to newly created components. You can
filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.
Component: Mozmill Tests → Mozmill Tests
Product: Testing → Mozilla QA
You need to log in before you can comment on or make changes to this bug.