Changes

Many (poorly behaved) web sites attempt to move and/or resize your browser window. It is possible to stop this behaviour (cf. dom.disable_window_move_resize) but it would be nice to have an infobar that informed the user that a web page attempted to move/resize the window, and allow it or ignore it (default). This behaviour is similar to the current Pop-up Blocker already present in Firefox. NOTE: it is not clear whether such a feature would be accepted in the tree or if this would need to be done as an extension.

'''09/04/2009''' - * Removed "NOTE: it is not clear whether such a feature would be accepted in the tree or if this would need to be done as an extension." from project description* Awaiting for review'''07/04/2009''' -* Apologies for the long break:* Made a fix to the patch: Removed text strings and placed in a properties file.* Update can be found on bug'''04/25/2009''' - * After some many many rebuilds and weird Assertion errors fixed...* Release 1.0! Its here!! Its here!!!'''04/24/2009''' - * Fixed a huge array of problems migrating to the latest version of firefox** Merged all code with kdiff3 (several conflicts)** Event number changed from 4000 back to 39 (instead of 36)** Reordered nsClassInfo.cpp (was throwing constructor errors at me)** Fixed a bug where the event type was mismatching causing no actual data to be passed'''04/23/2009''' -* Changed Event number to 4000 (cause 36 is used in the latest version of FireFox)* This number will be changed when this patch is official (to a more stable number)'''04/17/2009''' -* Video Tutorial for 0.9: http://www.youtube.com/watch?v=pOtZs4k54jg'''04/11/2009''' -* Not entirely related to the project, yesterday and today was spent on poster.* Releasing 0.9 as well.* The details can be found above.'''04/09/2009''' -* Added one new test to Mochitest testing resizeWindow.show_infobar* Added try/catch block to addEntry URI (So it doesn't throw an exception)* Changed some variable names to better reflect their usage** warningLabel changed to warningLabelKey seeing how this is a key** isByCommand changed to isAbsolute in nsGlobalWindow.cpp** fixed a bug where moveTo and moveBy was flipped around when using allowOnce* Deleted "showEditDialog()" from browser.js gResizedWindowObserver* Added a confirmation dialog to "allow once", with the x and y / command type info** That way users can choose to see if they want the window resized/moved.** And they can see if the numbers are okay first before they click accept.'''04/08/2009''' -* Added options to "Content.xul" to open exceptions, and to display the resize/move infobar.* Changed preferences.properties to reflect other exceptions types* Removed resizedWindow preference from firefox.js, and added to all.js'''03/31/2009''' -* Released 0.8 release for "Front End"* This patch contains a Mochitest specifically for replacing manual testing.'''03/29/2009''' -* Bug caused by an unsafe check for an existing infobar, and removing a null infobar.* Further testing continues* Replaced all removeNotificationByValue with removeAllNotifications(immediate) in test* removeNotificationByValue might have been conflicting with the code in browser.js which gets by value'''03/28/2009''' - * Discovered a bug with Mochitest test_bug413792.html which caused the infobar to become invisible'''03/27/2009''' - * Wrote the beginning of a mochitest* Changed access key of edit options, it was conflicting with Options'''03/12/2009''' - * Removed fillResizeList, removed "onpopupshowing"* Removed all text related to the above* Since Allow / Disable is gone, a and d access keys are freed. Reapplied to Allow/Deny (on options)* Fixed most lines greater than 80 columns in the frontend patch* Removed unwanted white space changes in frontend patch (previously only backend was done)* Changed label texts for better descriptions.* Removed resizeWindow.dtd from jar.mn in locales (Previous bug unfound!)* Allow Once is now the top menu choice (rearranged)'''03/11/2009''' - * The event is thrown now regardless of permissions.* However, the permissions still work in effect on nsGlobalWindow.cpp* So now even if the permissions states that a site can resize/can't resize, a user can still capture the event.* Removed "Disable Infobar option" from the menu* Removed Add/Disable disable_window_move_resize options* fillResizeList is currently empty'''02/19/2009''' -* Removed some code in browser.js which is no longer used (code which displayed even if disable_window_move_resize is off)* Fixed a bug where no matter what the command to trigger the infobar was, the infobar would say the page tried to "resize" the browser (now it says move, if a move command was used)* Removed all end white spaces after lines* Fixed all lines that were longer than 80 characters long in the backend* Released 0.6. Read release notes above!'''02/18/2009''' -* New file called resizedWindow.js created in browser/content/preferences/* Linked resizedWindow.xul with resizedWindow.js* To do: Code resizedWindow.js to insert permissions / load permissions from nsIPermissions.idl* Permissions will be loaded into a tree similar to the popup blocker* Added browser/content/preferences/resizedWindow.js to the folder's jar.mn* Changed resizeWindow.xul to follow the same size / look as other exceptions screens* Accesskeys have been changed to follow the regular keys in other option windows (similar feel throughout the entire app)* Temporarily removed the checkbox in resizedWindow.xul. I plan to add it to advanced javascript settings instead* resizedWindow.xul should strictly be for allow/denied sites only* '''And I realized xul is not even required, I can use permissions.xul entirely''' Good learning experience though!* Discarded resizedwindow.xul :(* discarded resizedWindow.dtd and js :(* browser.js calls permissions.xul with a new permission type (moveResize)* added new strings to permissions.properties: moveresizepermissiontext and moveresizepermissiontitle* Added a new function to nsGlobalWindow.cpp isSiteAllowedOrBlocked() which returns the permission/capacity of a URI** if its 0, it means it doesn't exist in the preference* Changed the moveTo, moveBy, resizeTo, resizeBy codes to use isSiteAllowedOrBlocked()* The code now completely ignores any blocks unless its a frame object (which can't be resized no matter what) if it is on the Allow list* and all denies are automatically denied regardless of situation'''02/16/2009''' -* Lots of research to do: Regarding preferences and xul objects'''02/15/2009''' -* Long break from OSD due to other courses* Read jst's review. Fixed some naming conventions and naming licenses* Changed isBy parameter to isAbsolute* Changed paramX and paramY to plain x and y* Removed case NS_WINDOWMOVERESIZE_EVENT from DuplicateData()* Removed some old comments on Window Move and Resize events, replaced with comments on one event* Changed code in nsGlobalWindow.cpp so that infobar does not appear when isFrame() is true* Added FireWindowMoveResize() function to nsGlobalWindow class, and declared it as a non static member function'''02/06/2009''' -* Patches released: InfobarBackend0.5.txt | InfobarFrontend0.5.txt** Refer to patch release details for further information* Tested splitting patch between backend and frontend.* Patches can be done from their root directories. (A LOT shorter!)** Backend: hg diff -p -U 8 ./dom ./content/events ./widget/public > InfobarBackend0.5.txt** Frontend: hg diff -p -U 8 ./browser ./toolkit/content/widgets > InfobarFrontend0.5.txt'''02/01/2009''' -* Fixed several white space changes in code / patch* Used JST Review to do changes prior to releasing an actual patch** In progress: Changing problems: Ignoring lines longer than 80 characters for now* There is apparently no way around the "if (nsnull == it)" statement when creating a new event. We have to compare whether the pointer to the new event is null or not, but we can't use NULL. Thus, ns_null is our only way.'''01/30/2009''' -* Fixed the tabbing and indentations of browser.js* Shortened Name of event thrown from nsGlobal.cpp so code isn't over 80 columns long* Converted several new files created to Unix format (hope this eliminates the windows endline characters)'''01/29/2009''' -* Event works. New event nsWindowMoveResizeEvent is now the only event thrown during a resize/move event'''01/28/2009''' -* browser.xml changed to accept just one event* browser.js updated to changes on browser.xml: uses the new parameter to check if command is resize or move* Mostly fixing compile errors'''01/27/2009''' - * Started work on nsWindowMoveEvent. This event has been renamed nsWindowMoveResizeEvent.** New Parameter: isMove: To specify whether this is a move or a resize command** Changed moveX and moveY to paramX and paramY** Changed Init* In progress: Changing nsGUIEvent.h, changing all files with WindowMove to WindowMoveResize, changing idl, h, cpp files* Removed FireWindowResizedEvent() in nsGlobalWindow.cpp and renamed FireWindowMovedEvent to FireWindowMoveResizeEvent()* Next: Removed nsIDOMWindowResizeEvent from the code, including its implementations* Deleted nsWindowResizeEvent from nsGUIEvent.h* Currently fixing nsGlobalWindow.cpp's calls to reflect the new FireWindowMoveResizeEvent();'''01/26/2009''' -* Received some feedback from jst, thanks to David* 0.5 release will focus mostly on combining the two events into one event (Yay!)* and... to fix all the indentations.* This will take me the rest of the week, then I'm going to try and see about splitting the patch between UI and Backend.'''01/22/2009''' -* Blog page about the entire project can be found [http://twlai1.blogspot.com/2009/01/lets-have-recap-my-project-and-how-to.html here]'''01/21/2009''' -* '''Patch 0.4 released!'''* Read above to see what has changed'''01/21/2009''' -* Added new menu option on browser.xul (menuitem) called "Allow Once"** Calls gWindowObserver's allowOnce() function when selected** Added new dtd localization text in browser.dtd for the new menuitem* Fixed a bug where nsWindowResizeEvent would constantly return 0,0 as the x,y (signature of getResizeX/Y())'''01/20/20082009''' -

* Updated browser.xml with a new report type MoveReport

'''01/19/20082009''' -

'''01/18/20082009''' -