576330 - Minor code cleanup

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch

* Remove left over "resizeAndPaint" reference
* Remove WINCE from front-end code
* fix a TAB

This patch does not remove all WINCE references in mobile-browser, just the ones in the front-end code itself.

Comment 1

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 455491 [details] [diff] [review]
patch

>diff --git a/chrome/content/content.js b/chrome/content/content.js
>--- a/chrome/content/content.js
>+++ b/chrome/content/content.js
>@@ -484,17 +484,17 @@ Content.prototype = {
>         let element = elementFromPoint(x, y);
>         if (modifiers == Ci.nsIDOMNSEvent.CONTROL_MASK) {
>           let uri = Util.getHrefForElement(element);
>           if (uri)
>             sendAsyncMessage("Browser:OpenURI", { uri: uri });
>         } else if (!this._formAssistant.open(element)) {
>           this._sendMouseEvent("mouseup", element, x, y);
>         }
>-	break;
>+        break;
>       }

I thought to have seen some "tabs" too before the comments a few lines before.

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Found the other TABs

pushed:
http://hg.mozilla.org/mobile-browser/rev/1dea022304bb

Comment 3

[Dão Gottwald [:dao]]

bugspam