summaryrefslogtreecommitdiffstats
path: root/editor
Commit message (Collapse)AuthorAgeLines
* Issue #1656 - Part 6: Clean up the build filesMoonchild2020-09-23-6/+0
|
* Issue #1656 - Part 3: Nuke more vim config lines in the tree.Moonchild2020-09-23-4/+0
| | | | Another S&R run with some smarter matching.
* Issue #1656 - Part 1: Nuke most vim config lines in the tree.Moonchild2020-09-23-1/+0
| | | | | | Since these are just interpreted comments, there's 0 impact on actual code. This removes all lines that match /* vim: set(.*)tw=80: */ with S&R -- there are a few others scattered around which will be removed manually in a second part.
* Issue #1621 - Part 4: Check whether node can be splited.Gaming4JC2020-07-31-7/+12
| | | | | | At first, HTMLEditor::GetActiveEditingHost might return null in this situation, we should check whether nullptr is returned. At second, SplitNodeDeep returns error since curent is design mode and selection node has no parent. So we should check error. Ref: Bug 1350772
* Issue #1621 - Part 3: Use nsIAtom to change attirbute if possible.Gaming4JC2020-07-31-64/+51
| | | | | | We can replace old nsIEditor API with nsIAtom version. Ref: Bug 1324996
* Issue #1621 - Part 2: Implement nsIAtom version of ↵Gaming4JC2020-07-31-100/+143
| | | | | | | | | | | | | SetAttribute/RemoveAttribute/CloneAttirubte. Add nsIAtom version of the following. - CloneAttribute - RemoveAttribute - RemoveAttributeOrEquivalent - SetAttribute - SetAttributeOrEquivalent Ref: Bug 1324996
* Issue #1621 - Part 1: CSSEditUtils should use atom for CSS property if possible.Gaming4JC2020-07-31-188/+224
| | | | | | There is a lot of string compare when using CSS property name. We should use nsGkAtoms instead. Ref: Bug 1323138
* Bug 1318570 - Clean up GetTextNode into TextEditRules.Gaming4JC2020-07-30-20/+14
| | | | | | EditorBase parameter of GetTextNode is unnecessary because it uses static method only. Also, we should return nsINode to reduce QI. Resolves #1617
* Bug 1316302 - Part 4: Refine HTMLEditRules::TryToJoinBlocks() and ↵Gaming4JC2020-06-14-53/+59
| | | | | | | | HTMLEditRules::MoveNodeSmart() with early return style for making scope of EditActionResult variable smaller For now, let's make the scope of EditActionResult variable in them smaller without big change. Tag #1563
* Bug 1316302 - Part 3: Create EditActionResult class for making the methods ↵Gaming4JC2020-06-14-149/+292
| | | | | | | | | | | | | which return nsresult, handled and canceled with out params In a lot of places, edit action handlers and their helper methods return nsresult and aHandled and aCanceled with out params. However, the out params cause the code complicated since: * it's not unclear if the method will overwrite aHandled and aCanceled value. * callers need to create temporary variable event if some of them are not necessary. This patch rewrites the helper methods of HTMLEditRules::WillDeleteSelection() with it. Tag #1563
* Bug 1316302 - Part 2: WillDeleteSelection() should retry to handle it when ↵Gaming4JC2020-06-14-3/+15
| | | | | | | | | | selection is collapsed and JoinBlocks() doesn't handle nor cancel the action When selection is collapsed and JoinBlocks() doesn't handle nor cancel the action, WillDeleteSelection() should move selection to the start/end of leftmost/rightmost editable leaf node and retry to handle the action again. For avoiding infinite loop, it checks if selected node is changed actually before calling itself again. Tag #1563
* Bug 1316302 - Part 1: Helper methods for ↵Gaming4JC2020-06-14-59/+149
| | | | | | | | | | | | HTMLEditRules::WillDeleteSelection() should have an out argument to indicates if it actually handles the action When HTMLEditRules::WillDeleteSelection() tries to remove something from the end/start of a block to its last/first text node but it's contained by block elements, it tries to join the container and the block. However, JoinBlocks() always fails to join them since it's impossible operation. In this case, HTMLEditRules::WillDeleteSelection() should retry to remove something in the leaf, however, it's impossible for now because JoinBlocks() and its helper methods don't return if it handles the action actually. This patch renames |JoinBlocks()| to |TryToJoinBlocks()| for representing what it is. And this patch adds |bool* aHandled| to the helper methods. Then, *aHandled and *aCancel are now always returns the result of each method. Therefore, for merging the result of multiple helper methods, callers need to receive the result with temporary variables and merge them by themselves. Note that when they modify DOM node actually or the action should do nothing (for example, selection is across tables), aHandled is set to true. Tag #1563
* Bug 1426494 - Share more code between nsIDocument and ShadowRootMatt A. Tobin2020-06-13-0/+1
| | | | Tag #1375
* Issue #1525 - Kill marquee elementMoonchild2020-06-01-1/+0
| | | | | | * Remove marquee code * Regenerate HTML Elements/parser code for Removal of Marquee. Co-authored-by: Gaming4JC <g4jc@hyperbola.info>
* Issue #1564 - Fix debug build dependencieswolfbeast2020-05-26-0/+10
|
* Issue #80 - De-unify editor/txmgr and editor/txtsvcMoonchild2020-05-16-2/+2
|
* Issue #80 - De-unify editor/libeditorMoonchild2020-05-16-1/+3
|
* Issue #80 - De-unify editor/composerMoonchild2020-05-16-1/+2
|
* Issue #1517 - Remove dom.event.highrestimestamp.enabled prefMoonchild2020-05-04-30/+10
| | | | This resolves #1517
* Bug 1360154 - nsIPlaintextEditor might have to have hasText property for ↵Matt A. Tobin2020-04-17-18/+11
| | | | | | | | | UpdateOverlayTextVisibility * DocumentIsBody should return bool, not nsresult * Add fast path to check whether valus is emtpy Tag #1375
* Bug 1418002 - Remove HTMLContentElementMatt A. Tobin2020-04-17-1/+0
| | | | Tag #1375
* Bug 1402941 - Add HTMLSlotElementMatt A. Tobin2020-04-17-34/+32
| | | | Tag #1375
* Bug 1396584 - Remove support for multiple ShadowRootsMatt A. Tobin2020-04-17-1/+0
| | | | Tag #1375
* Bug 1400777 - Slim down nsElementTable.hMatt A. Tobin2020-04-17-2/+2
| | | | | | | | | | | * Remove eHTMLTags * De-expose HTML group constants[] * De-expose gHTMLElements[] * Split nsHTMLElement * Clean up nsElementTable.{cpp,h} * Fixup for eHTMLTag removal in Parser Tag #1375
* Issue #1375 - Stop largely using the parser serviceMatt A. Tobin2020-04-17-73/+9
| | | | | | | | | This is based on Bug 1395828 * Add nsHTMLElement::IsBlock() * Rename nsHTMLTags methods * Remove AssertParserServiceIsCorrect() * Remove most uses of nsIParserService/nsParserService
* Bug 1377978 - Make nsRange use uint32_t to offsetMatt A. Tobin2020-04-17-72/+90
| | | | Tag #1375
* Bug 1372829 - Part 2: mozilla::EditorBase should cache raw pointer of ↵Matt A. Tobin2020-04-17-164/+281
| | | | | | nsISelectionController and nsIDocument with nsWeakPtr Tag #1375
* Bug 1332353 - Make it clearer when a stylesheet is really owned by its mDocumentMatt A. Tobin2020-04-17-2/+2
| | | | Tag #1375
* Bug 1372829 - Part 1: Make mozilla::PlaceholderTransaction inherit ↵Matt A. Tobin2020-04-17-43/+53
| | | | | | mozilla::SupportsWeakPtr instead of nsSupportsWeakReference Tag #1375
* Bug 1371170 - Add non-virtual EditorBase::GetSelectionControllerMatt A. Tobin2020-04-17-7/+14
| | | | Tag #1375
* Bug 1337698 - Use UniquePtr instead of nsAutoPtr in editorMatt A. Tobin2020-04-17-44/+32
| | | | | | | | * PlaceholderTransaction should use UniquePtr * HTMLEditor should use UniquePtr * TypeInState should use UniquePtr Tag #1375
* Bug 1367683 - Optimize initializing nsRangeMatt A. Tobin2020-04-17-70/+88
| | | | Tag #1375
* Bug 1348851 - Use new block when better selection isn't found.Matt A. Tobin2020-04-17-11/+35
| | | | Tag #1375
* Bug 1336011 - Fix Crash in InvalidArrayIndex_CRASH in ↵Matt A. Tobin2020-04-15-71/+124
| | | | | | | | | | mozilla::EditorBase::DeleteSelectionImpl * EditorBase shouldn't refer mActionListeners directly in loops because it might be removed during a loop * Create an alias of the type of mEditorObservers * Create an alias of the type of mDocStateListeners Tag #1375
* Issue #1514 - Perform validity check before trying to use the content.wolfbeast2020-04-13-1/+2
| | | | Resolves #1514
* Issue #1513 - Bail on orphan nodewolfbeast2020-04-13-1/+5
| | | | | | This was refactored by Mozilla into doing effectively the same thing so it aligns with them re: behavior. Resolves #1513
* Issue #1512 - Improve handling of multiple selections.wolfbeast2020-04-12-2/+8
| | | | | | | | | IsSelectionEditable should check whether the focus node and anchor node aren't null before trying to use them. This also changes the initialization of selections' aOutIndex to the last range in the selection as a fallback in case we don't add a range later (in AddItem) which could also end up with a null selection otherwise if the additional selection nodes are removed.
* Issue #1355 - Make nsTableCellFrame::GetColIndex/GetRowIndex fasterwin7-72020-02-16-2/+2
| | | | We can devirtualize it, remove some branches.
* Bug 1322938 - Basic implementation of HTMLDialogElement.Gaming4JC2020-01-07-3/+4
| | | | Tag #1343
* Issue #1219 - Align computed DOM styles with mainstream behvior.wolfbeast2019-12-20-4/+1
| | | | | | | | | This updates our behavior for computed DOM styling to no longer return null on elements that have no display, but return a 0-length (empty) style instead and don't throw. For this we stop looking at having a presentation for the style and just look at the document instead. This resolves #1219
* MoonchildProductions#1251 - Part 1: Restore initial Solaris support, fixed up.athenian2002019-10-21-1/+3
| | | | | | | | | | | | | | Compared with what Pale Moon had for Solaris originally, this is mostly the same zero point I started patching from, but I've made the following changes here after reviewing all this initial code I never looked at closely before. 1. In package-manifest.in for both Basilisk and Pale Moon, I've made the SPARC code for libfreebl not interefere with the x86 code, use the proper build flags, and also updated it to allow a SPARC64 build which is more likely to be used than the 32-bit SPARC code we had there. 2. See Mozilla bug #832272 and the old rules.mk patch from around Firefox 30 in oracle/solaris-userland. I believe they screwed up NSINSTALL on Solaris when they were trying to streamline the NSS buildsystem, because they started having unexplained issues with it around that time after Firefox 22 that they never properly resolved until Mozilla began building NSS with gyp files. I'm actually not even sure how relevant the thing they broke actually is to Solaris at this point, bug 665509 is so old it predates Firefox itself and goes back to the Mozilla suite days. I believe $(INSTALL) -t was wrong, and they meant $(NSINSTALL) -t because that makes more sense and is closer to what was there originally. It's what they have for WINNT, and it's possible a fix more like that could serve for Solaris as well. Alternatively, we could get rid of all these half-broken Makefiles and start building NSS with gyp files like Mozilla did. 3. I've completely cut out support for the Sun compiler and taken into account the reality that everyone builds Firefox (and therefore its forks) with GCC now on Solaris. This alone helped clean up a lot of the uglier parts of the code. 4. I've updated all remaining SOLARIS build flags to the newer XP_SOLARIS, because the SOLARIS flag is no longer set when building Solaris. 5. I've confirmed the workaround in gtxFontconfigFonts.cpp is no longer necessary. The Solaris people got impatient about implementing a half-baked patch for a fontconfig feature that wasn't ready yet back in 2009, and somehow convinced Mozilla to patch their software to work around it when really they should have just fixed or removed their broken fontconfig patch. The feature they wanted has since been implemented properly, and no version of Solaris still uses the broken patch that required this fix. If anyone had ever properly audited this code, it would have been removed a long time ago.
* Issue #187: Remove solaris 1st party code OS checks.wolfbeast2019-03-30-3/+1
|
* Fix Build Bustage - with "--enable-debug"janekptacijarabaci2018-07-12-1/+1
|
* Issue #12 Part 4: Don't access mEditorBase directly.wolfbeast2018-06-26-41/+54
| | | | | EditorEventListener should grab mEditorBase in a smaller scope wherever possible and shouldn't access it directly while handling an event. Each event listener method shouldn't access mEditorBase directly when calling its method since it might be changed to another instance.
* Issue #12 Part 3: EditorEventListener should check if it's removed during a ↵wolfbeast2018-06-26-99/+173
| | | | | | | | | call of editor's method. EditorEventListener doesn't check if mEditorBase is available even after it's removed from the editor. If it becomes nullptr, i.e., it's detached from editor, it shouldn't continue to handle event. This patch changes some methods' nsIDOM*Event argument to WidgetEvent since it's simpler.
* Issue #12 Part 2: Stop using nsIDOMEvent in IsAcceptableInputEvent.wolfbeast2018-06-26-41/+44
|
* Issue #12 Part 1: Stop using nsIDOMEvent in UpdateIMEComposition.wolfbeast2018-06-26-14/+22
|
* moebius#231: Consider blocking top level window data: URIs (tests)janekptacijarabaci2018-05-06-19/+19
| | | | https://github.com/MoonchildProductions/moebius/pull/231
* moebius#56: Fix: DataTransfer - Pasting image from clipboard fails in some casesjanekptacijarabaci2018-04-20-0/+64
| | | | https://github.com/MoonchildProductions/moebius/pull/56
* moebius#190: HTML - table - editor (contenteditable) - post process node ↵janekptacijarabaci2018-04-15-16/+82
| | | | | | array to remove all descendants of replacement node https://github.com/MoonchildProductions/moebius/issues/190
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-23 10:24:38 +0000