758874 - Test "browser_sanitize-download-history.js" contains typo in window name

Status: RESOLVED FIXED

(Toolkit :: Downloads API, defect)

Description

[Benedikt Pfeifer]

The test at http://mxr.mozilla.org/mozilla-central/source/browser/base/content/test/browser_sanitize-download-history.js contains a check [1] that should close the UI if it's already open. The window's name is given with "Sanatize" and that seems to be a typo that breaks this part (it's the only reference to "Sanatize" that MXR can find!) and should read "Sanitize" instead.

[1] http://mxr.mozilla.org/mozilla-central/source/browser/base/content/test/browser_sanitize-download-history.js#109 (below "// Close the UI if necessary" if the line numbers should get messed up).

Comment 1

[Benedikt Pfeifer]

I hope setting "Blocks 431729" is correct, since it was introduced with this bug as it seems?

Comment 2

[Benjamin Kerensa [:bkerensa]]

Attachment: Fix the file

This should fix it.

Comment 3

[Marco Bonardo [:mak]]

Comment on attachment 641555 [details] [diff] [review]
Fix the file

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

while the change is correct, the patch format is not, please check https://developer.mozilla.org/en/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F

Comment 4

[Marco Bonardo [:mak]]

Comment on attachment 641555 [details] [diff] [review]
Fix the file

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

ehr, for some reason bugzilla showed me something else, sorry :)

Btw, if you could add user and commit message it would be welcome, then just add checkin-needed keyword

Comment 5

[Benjamin Kerensa [:bkerensa]]

I already nuked that branch what specific strings do you need added to the patch for it to be applied?

Comment 6

[Benedikt Pfeifer]

Attachment: Patch with header (commit message, user,..)

> Btw, if you could add user and commit message it would be welcome, then just
> add checkin-needed keyword

I created a new patch, including a username and commit message. I couldn't find any previous patchs and checkins from you though, Benjamin Kerensa. If you'd like to have your name and email adress of your choice there, then replace mine and add the checkin-needed keyword to the whiteboard.

The patch is against mozilla-central and I had updated the repository right before exporting it. If something is wrong, please let me know. It's the first patch I created for m-c.

Comment 7

[Benjamin Kerensa [:bkerensa]]

Comment on attachment 723230 [details] [diff] [review]
Patch with header (commit message, user,..)

># HG changeset patch
># User Benjamin Kerensa <[e-mail]>
># Date 1362941807 -3600
># Node ID 04cebce82099b05a46fc824adf91cbc91c2056a4
># Parent  b1a08130fae67ee3c02c63349628f5b645030cc1
>Bug 758874 - Test browser_sanitize-download-history.js contains typo in window name; r=mak77
>
>diff --git a/browser/base/content/test/browser_sanitize-download-history.js b/browser/base/content/test/browser_sanitize-download-history.js
>--- a/browser/base/content/test/browser_sanitize-download-history.js
>+++ b/browser/base/content/test/browser_sanitize-download-history.js
>@@ -103,17 +103,17 @@ function test()
>   let dm = Cc["@mozilla.org/download-manager;1"].
>            getService(Ci.nsIDownloadManager);
>   let db = dm.DBConnection;
> 
>   // Empty any old downloads
>   db.executeSimpleSQL("DELETE FROM moz_downloads");
> 
>   // Close the UI if necessary
>-  let win = Services.ww.getWindowByName("Sanatize", null);
>+  let win = Services.ww.getWindowByName("Sanitize", null);
>   if (win && (win instanceof Ci.nsIDOMWindow))
>     win.close();
> 
>   // Start the test when the sanitize window loads
>   Services.ww.registerNotification(function (aSubject, aTopic, aData) {
>     Services.ww.unregisterNotification(arguments.callee);
>     aSubject.QueryInterface(Ci.nsIDOMEventTarget)
>             .addEventListener("DOMContentLoaded", doTest, false);

Comment 8

[Benedikt Pfeifer]

Attachment: Patch with header (commit message, user,..)

This patch is attachment 641555 [details] [diff] [review] with headers and Benjamin Kerensa's user information now. I assume this carries mak77's r+ forward? I would have obsoleted the old patch but I haven't got sufficient rights to do so.


I'm not super-familiar with the checkin-process here but pasting an edited patch in a comment didn't look useful to me. When I wrote "replace" I had expected that you would download, edit and re-upload the diff I created for you.

Thanks for providing your user information!

Comment 9

[Ryan VanderMeulen [:RyanVM]]

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

Comment 10

[Ryan VanderMeulen [:RyanVM]]

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