1546577 - Make overloads of CanCut(), CanCopy(), CanDelete() and CanPaste() which return bool instead of nsresult

Status: RESOLVED FIXED

(Core :: DOM: Editor, task)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

CanCut(), CanCopy() and CanPaste() return error only when the editor has already been destroyed or not been initialized yet, or when failed to access clipboard when the document is not HTML/XHTML.

CanDelete() returns error only when the editor has already been destroyed or not been initialized yet.

So, these error result won't be exposed to the web in most cases and such exception shouldn't stop any content script because Chrome basically does not throw exception in such situation as far as I know.

So, there should be overloads of them to return bool result directly for making their callers simpler.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1546577 - Make overloads of CanCut(), CanCopy(), CanDelete() and CanPaste() which return bool instead of nsresult

CanCut(), CanCopy() and CanPaste() return error only when the editor has
already been destroyed or not been initialized yet, or when failed to access
clipboard when the document is not HTML/XHTML.

CanDelete() returns error only when the editor has already been destroyed or
not been initialized yet.

So, these error result won't be exposed to the web in most cases and such
exception shouldn't stop any content script because Chrome basically does not
throw exception in such situation as far as I know.

Therefore, there should be overloads of them to return bool result directly
for making their callers simpler.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/feb8846c6602
Make overloads of CanCut(), CanCopy(), CanDelete() and CanPaste() which return bool instead of nsresult r=m_kato

Comment 3

[Raul Gurzau (:RaulG)]

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