Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(117)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/Source/core/editing/commands/EditorCommand.cpp

Issue 2578753002: [EditCommandSource] Rename and move |EditorCommandSource| to "CompositeEditCommand.h" (1/3) (Closed)
Patch Set: Xiaocheng and tkent's review: Rename to |EditCommandSource| and use kCamelCase Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/editing/commands/EditorCommand.cpp
diff --git a/third_party/WebKit/Source/core/editing/commands/EditorCommand.cpp b/third_party/WebKit/Source/core/editing/commands/EditorCommand.cpp
index 7f1aa4f15f695cc1b0c06a75771b711e95ea7e0b..07ae4740d2a9e786269f981caa1ec40b87dda88d 100644
--- a/third_party/WebKit/Source/core/editing/commands/EditorCommand.cpp
+++ b/third_party/WebKit/Source/core/editing/commands/EditorCommand.cpp
@@ -213,9 +213,9 @@ RangeVector* RangesFromCurrentSelectionOrExtendCaret(
class EditorInternalCommand {
public:
WebEditingCommandType commandType;
- bool (*execute)(LocalFrame&, Event*, EditorCommandSource, const String&);
+ bool (*execute)(LocalFrame&, Event*, EditCommandSource, const String&);
bool (*isSupportedFromDOM)(LocalFrame*);
- bool (*isEnabled)(LocalFrame&, Event*, EditorCommandSource);
+ bool (*isEnabled)(LocalFrame&, Event*, EditCommandSource);
TriState (*state)(LocalFrame&, Event*);
String (*value)(LocalFrame&, Event*);
bool isTextInsertion;
@@ -243,16 +243,16 @@ static LocalFrame* targetFrame(LocalFrame& frame, Event* event) {
}
static bool applyCommandToFrame(LocalFrame& frame,
- EditorCommandSource source,
+ EditCommandSource source,
InputEvent::InputType inputType,
StylePropertySet* style) {
// FIXME: We don't call shouldApplyStyle when the source is DOM; is there a
// good reason for that?
switch (source) {
- case CommandFromMenuOrKeyBinding:
+ case EditCommandSource::kMenuOrKeyBinding:
frame.editor().applyStyleToSelection(style, inputType);
return true;
- case CommandFromDOM:
+ case EditCommandSource::kDOM:
frame.editor().applyStyle(style, inputType);
return true;
}
@@ -261,7 +261,7 @@ static bool applyCommandToFrame(LocalFrame& frame,
}
static bool executeApplyStyle(LocalFrame& frame,
- EditorCommandSource source,
+ EditCommandSource source,
InputEvent::InputType inputType,
CSSPropertyID propertyID,
const String& propertyValue) {
@@ -272,7 +272,7 @@ static bool executeApplyStyle(LocalFrame& frame,
}
static bool executeApplyStyle(LocalFrame& frame,
- EditorCommandSource source,
+ EditCommandSource source,
InputEvent::InputType inputType,
CSSPropertyID propertyID,
CSSValueID propertyValue) {
@@ -287,7 +287,7 @@ static bool executeApplyStyle(LocalFrame& frame,
// Editor::selectionHasStyle to determine the current style but we cannot fix
// this until https://bugs.webkit.org/show_bug.cgi?id=27818 is resolved.
static bool executeToggleStyleInList(LocalFrame& frame,
- EditorCommandSource source,
+ EditCommandSource source,
InputEvent::InputType inputType,
CSSPropertyID propertyID,
CSSValue* value) {
@@ -320,7 +320,7 @@ static bool executeToggleStyleInList(LocalFrame& frame,
}
static bool executeToggleStyle(LocalFrame& frame,
- EditorCommandSource source,
+ EditCommandSource source,
InputEvent::InputType inputType,
CSSPropertyID propertyID,
const char* offValue,
@@ -342,7 +342,7 @@ static bool executeToggleStyle(LocalFrame& frame,
}
static bool executeApplyParagraphStyle(LocalFrame& frame,
- EditorCommandSource source,
+ EditCommandSource source,
InputEvent::InputType inputType,
CSSPropertyID propertyID,
const String& propertyValue) {
@@ -352,10 +352,10 @@ static bool executeApplyParagraphStyle(LocalFrame& frame,
// FIXME: We don't call shouldApplyStyle when the source is DOM; is there a
// good reason for that?
switch (source) {
- case CommandFromMenuOrKeyBinding:
+ case EditCommandSource::kMenuOrKeyBinding:
frame.editor().applyParagraphStyleToSelection(style, inputType);
return true;
- case CommandFromDOM:
+ case EditCommandSource::kDOM:
frame.editor().applyParagraphStyle(style, inputType);
return true;
}
@@ -506,14 +506,14 @@ static EphemeralRange unionEphemeralRanges(const EphemeralRange& range1,
static bool executeBackColor(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String& value) {
return executeApplyStyle(frame, source, InputEvent::InputType::None,
CSSPropertyBackgroundColor, value);
}
-static bool canWriteClipboard(LocalFrame& frame, EditorCommandSource source) {
- if (source == CommandFromMenuOrKeyBinding)
+static bool canWriteClipboard(LocalFrame& frame, EditCommandSource source) {
+ if (source == EditCommandSource::kMenuOrKeyBinding)
return true;
Settings* settings = frame.settings();
bool defaultValue = (settings && settings->javaScriptCanAccessClipboard()) ||
@@ -523,7 +523,7 @@ static bool canWriteClipboard(LocalFrame& frame, EditorCommandSource source) {
static bool executeCopy(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
// To support |allowExecutionWhenDisabled|, we need to check clipboard
// accessibility here rather than |Editor::Command::execute()|.
@@ -538,7 +538,7 @@ static bool executeCopy(LocalFrame& frame,
static bool executeCreateLink(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
if (value.isEmpty())
return false;
@@ -548,7 +548,7 @@ static bool executeCreateLink(LocalFrame& frame,
static bool executeCut(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
// To support |allowExecutionWhenDisabled|, we need to check clipboard
// accessibility here rather than |Editor::Command::execute()|.
@@ -563,7 +563,7 @@ static bool executeCut(LocalFrame& frame,
static bool executeDefaultParagraphSeparator(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
if (equalIgnoringCase(value, "div"))
frame.editor().setDefaultParagraphSeparator(EditorParagraphSeparatorIsDiv);
@@ -575,15 +575,15 @@ static bool executeDefaultParagraphSeparator(LocalFrame& frame,
static bool executeDelete(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
switch (source) {
- case CommandFromMenuOrKeyBinding: {
+ case EditCommandSource::kMenuOrKeyBinding: {
// Doesn't modify the text if the current selection isn't a range.
frame.editor().performDelete();
return true;
}
- case CommandFromDOM:
+ case EditCommandSource::kDOM:
// If the current selection is a caret, delete the preceding character. IE
// performs forwardDelete, but we currently side with Firefox. Doesn't
// scroll to make the selection visible, or modify the kill ring (this
@@ -601,7 +601,7 @@ static bool executeDelete(LocalFrame& frame,
static bool executeDeleteBackward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().deleteWithDirection(DeleteDirection::Backward,
CharacterGranularity, false, true);
@@ -611,7 +611,7 @@ static bool executeDeleteBackward(LocalFrame& frame,
static bool executeDeleteBackwardByDecomposingPreviousCharacter(
LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
DLOG(ERROR) << "DeleteBackwardByDecomposingPreviousCharacter is not "
"implemented, doing DeleteBackward instead";
@@ -622,7 +622,7 @@ static bool executeDeleteBackwardByDecomposingPreviousCharacter(
static bool executeDeleteForward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().deleteWithDirection(DeleteDirection::Forward,
CharacterGranularity, false, true);
@@ -631,7 +631,7 @@ static bool executeDeleteForward(LocalFrame& frame,
static bool executeDeleteToBeginningOfLine(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().deleteWithDirection(DeleteDirection::Backward, LineBoundary,
true, false);
@@ -640,7 +640,7 @@ static bool executeDeleteToBeginningOfLine(LocalFrame& frame,
static bool executeDeleteToBeginningOfParagraph(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().deleteWithDirection(DeleteDirection::Backward,
ParagraphBoundary, true, false);
@@ -649,7 +649,7 @@ static bool executeDeleteToBeginningOfParagraph(LocalFrame& frame,
static bool executeDeleteToEndOfLine(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
// Despite its name, this command should delete the newline at the end of a
// paragraph if you are at the end of a paragraph (like
@@ -661,7 +661,7 @@ static bool executeDeleteToEndOfLine(LocalFrame& frame,
static bool executeDeleteToEndOfParagraph(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
// Despite its name, this command should delete the newline at the end of
// a paragraph if you are at the end of a paragraph.
@@ -672,7 +672,7 @@ static bool executeDeleteToEndOfParagraph(LocalFrame& frame,
static bool executeDeleteToMark(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
const EphemeralRange mark =
frame.editor().mark().toNormalizedEphemeralRange();
@@ -692,7 +692,7 @@ static bool executeDeleteToMark(LocalFrame& frame,
static bool executeDeleteWordBackward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().deleteWithDirection(DeleteDirection::Backward, WordGranularity,
true, false);
@@ -701,7 +701,7 @@ static bool executeDeleteWordBackward(LocalFrame& frame,
static bool executeDeleteWordForward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().deleteWithDirection(DeleteDirection::Forward, WordGranularity,
true, false);
@@ -710,14 +710,14 @@ static bool executeDeleteWordForward(LocalFrame& frame,
static bool executeFindString(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
return frame.editor().findString(value, CaseInsensitive | WrapAround);
}
static bool executeFontName(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String& value) {
return executeApplyStyle(frame, source, InputEvent::InputType::None,
CSSPropertyFontFamily, value);
@@ -725,7 +725,7 @@ static bool executeFontName(LocalFrame& frame,
static bool executeFontSize(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String& value) {
CSSValueID size;
if (!HTMLFontElement::cssValueFromFontSizeNumber(value, size))
@@ -736,7 +736,7 @@ static bool executeFontSize(LocalFrame& frame,
static bool executeFontSizeDelta(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String& value) {
return executeApplyStyle(frame, source, InputEvent::InputType::None,
CSSPropertyWebkitFontSizeDelta, value);
@@ -744,7 +744,7 @@ static bool executeFontSizeDelta(LocalFrame& frame,
static bool executeForeColor(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String& value) {
return executeApplyStyle(frame, source, InputEvent::InputType::None,
CSSPropertyColor, value);
@@ -752,7 +752,7 @@ static bool executeForeColor(LocalFrame& frame,
static bool executeFormatBlock(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
String tagName = value.lower();
if (tagName[0] == '<' && tagName[tagName.length() - 1] == '>')
@@ -773,15 +773,15 @@ static bool executeFormatBlock(LocalFrame& frame,
static bool executeForwardDelete(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
EditingState editingState;
switch (source) {
- case CommandFromMenuOrKeyBinding:
+ case EditCommandSource::kMenuOrKeyBinding:
frame.editor().deleteWithDirection(DeleteDirection::Forward,
CharacterGranularity, false, true);
return true;
- case CommandFromDOM:
+ case EditCommandSource::kDOM:
// Doesn't scroll to make the selection visible, or modify the kill ring.
// ForwardDelete is not implemented in IE or Firefox, so this behavior is
// only needed for backward compatibility with ourselves, and for
@@ -798,7 +798,7 @@ static bool executeForwardDelete(LocalFrame& frame,
static bool executeIgnoreSpelling(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.spellChecker().ignoreSpelling();
return true;
@@ -806,7 +806,7 @@ static bool executeIgnoreSpelling(LocalFrame& frame,
static bool executeIndent(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
DCHECK(frame.document());
return IndentOutdentCommand::create(*frame.document(),
@@ -816,7 +816,7 @@ static bool executeIndent(LocalFrame& frame,
static bool executeInsertBacktab(LocalFrame& frame,
Event* event,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return targetFrame(frame, event)
->eventHandler()
@@ -825,7 +825,7 @@ static bool executeInsertBacktab(LocalFrame& frame,
static bool executeInsertHorizontalRule(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
DCHECK(frame.document());
HTMLHRElement* rule = HTMLHRElement::create(*frame.document());
@@ -836,7 +836,7 @@ static bool executeInsertHorizontalRule(LocalFrame& frame,
static bool executeInsertHTML(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
DCHECK(frame.document());
return executeInsertFragment(
@@ -845,7 +845,7 @@ static bool executeInsertHTML(LocalFrame& frame,
static bool executeInsertImage(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
DCHECK(frame.document());
HTMLImageElement* image = HTMLImageElement::create(*frame.document());
@@ -856,14 +856,14 @@ static bool executeInsertImage(LocalFrame& frame,
static bool executeInsertLineBreak(LocalFrame& frame,
Event* event,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
switch (source) {
- case CommandFromMenuOrKeyBinding:
+ case EditCommandSource::kMenuOrKeyBinding:
return targetFrame(frame, event)
->eventHandler()
.handleTextInputEvent("\n", event, TextEventInputLineBreak);
- case CommandFromDOM:
+ case EditCommandSource::kDOM:
// Doesn't scroll to make the selection visible, or modify the kill ring.
// InsertLineBreak is not implemented in IE or Firefox, so this behavior
// is only needed for backward compatibility with ourselves, and for
@@ -877,7 +877,7 @@ static bool executeInsertLineBreak(LocalFrame& frame,
static bool executeInsertNewline(LocalFrame& frame,
Event* event,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
LocalFrame* targetFrame = blink::targetFrame(frame, event);
return targetFrame->eventHandler().handleTextInputEvent(
@@ -888,7 +888,7 @@ static bool executeInsertNewline(LocalFrame& frame,
static bool executeInsertNewlineInQuotedContent(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
DCHECK(frame.document());
return TypingCommand::insertParagraphSeparatorInQuotedContent(
@@ -897,7 +897,7 @@ static bool executeInsertNewlineInQuotedContent(LocalFrame& frame,
static bool executeInsertOrderedList(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
DCHECK(frame.document());
return InsertListCommand::create(*frame.document(),
@@ -907,7 +907,7 @@ static bool executeInsertOrderedList(LocalFrame& frame,
static bool executeInsertParagraph(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
DCHECK(frame.document());
return TypingCommand::insertParagraphSeparator(*frame.document());
@@ -915,7 +915,7 @@ static bool executeInsertParagraph(LocalFrame& frame,
static bool executeInsertTab(LocalFrame& frame,
Event* event,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return targetFrame(frame, event)
->eventHandler()
@@ -924,7 +924,7 @@ static bool executeInsertTab(LocalFrame& frame,
static bool executeInsertText(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
DCHECK(frame.document());
TypingCommand::insertText(*frame.document(), value, 0);
@@ -933,7 +933,7 @@ static bool executeInsertText(LocalFrame& frame,
static bool executeInsertUnorderedList(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
DCHECK(frame.document());
return InsertListCommand::create(*frame.document(),
@@ -943,7 +943,7 @@ static bool executeInsertUnorderedList(LocalFrame& frame,
static bool executeJustifyCenter(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
return executeApplyParagraphStyle(frame, source,
InputEvent::InputType::FormatJustifyCenter,
@@ -952,7 +952,7 @@ static bool executeJustifyCenter(LocalFrame& frame,
static bool executeJustifyFull(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
return executeApplyParagraphStyle(frame, source,
InputEvent::InputType::FormatJustifyFull,
@@ -961,7 +961,7 @@ static bool executeJustifyFull(LocalFrame& frame,
static bool executeJustifyLeft(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
return executeApplyParagraphStyle(frame, source,
InputEvent::InputType::FormatJustifyLeft,
@@ -970,7 +970,7 @@ static bool executeJustifyLeft(LocalFrame& frame,
static bool executeJustifyRight(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
return executeApplyParagraphStyle(frame, source,
InputEvent::InputType::FormatJustifyRight,
@@ -979,7 +979,7 @@ static bool executeJustifyRight(LocalFrame& frame,
static bool executeMakeTextWritingDirectionLeftToRight(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
MutableStylePropertySet* style =
MutableStylePropertySet::create(HTMLQuirksMode);
@@ -992,7 +992,7 @@ static bool executeMakeTextWritingDirectionLeftToRight(LocalFrame& frame,
static bool executeMakeTextWritingDirectionNatural(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
MutableStylePropertySet* style =
MutableStylePropertySet::create(HTMLQuirksMode);
@@ -1004,7 +1004,7 @@ static bool executeMakeTextWritingDirectionNatural(LocalFrame& frame,
static bool executeMakeTextWritingDirectionRightToLeft(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
MutableStylePropertySet* style =
MutableStylePropertySet::create(HTMLQuirksMode);
@@ -1017,7 +1017,7 @@ static bool executeMakeTextWritingDirectionRightToLeft(LocalFrame& frame,
static bool executeMoveBackward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionBackward,
CharacterGranularity, UserTriggered);
@@ -1026,7 +1026,7 @@ static bool executeMoveBackward(LocalFrame& frame,
static bool executeMoveBackwardAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionBackward,
CharacterGranularity, UserTriggered);
@@ -1035,7 +1035,7 @@ static bool executeMoveBackwardAndModifySelection(LocalFrame& frame,
static bool executeMoveDown(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.selection().modify(FrameSelection::AlterationMove,
DirectionForward, LineGranularity,
@@ -1044,7 +1044,7 @@ static bool executeMoveDown(LocalFrame& frame,
static bool executeMoveDownAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionForward,
LineGranularity, UserTriggered);
@@ -1053,7 +1053,7 @@ static bool executeMoveDownAndModifySelection(LocalFrame& frame,
static bool executeMoveForward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionForward,
CharacterGranularity, UserTriggered);
@@ -1062,7 +1062,7 @@ static bool executeMoveForward(LocalFrame& frame,
static bool executeMoveForwardAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionForward,
CharacterGranularity, UserTriggered);
@@ -1071,7 +1071,7 @@ static bool executeMoveForwardAndModifySelection(LocalFrame& frame,
static bool executeMoveLeft(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.selection().modify(FrameSelection::AlterationMove, DirectionLeft,
CharacterGranularity, UserTriggered);
@@ -1079,7 +1079,7 @@ static bool executeMoveLeft(LocalFrame& frame,
static bool executeMoveLeftAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionLeft,
CharacterGranularity, UserTriggered);
@@ -1088,7 +1088,7 @@ static bool executeMoveLeftAndModifySelection(LocalFrame& frame,
static bool executeMovePageDown(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
unsigned distance = verticalScrollDistance(frame);
if (!distance)
@@ -1099,7 +1099,7 @@ static bool executeMovePageDown(LocalFrame& frame,
static bool executeMovePageDownAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
unsigned distance = verticalScrollDistance(frame);
if (!distance)
@@ -1110,7 +1110,7 @@ static bool executeMovePageDownAndModifySelection(LocalFrame& frame,
static bool executeMovePageUp(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
unsigned distance = verticalScrollDistance(frame);
if (!distance)
@@ -1121,7 +1121,7 @@ static bool executeMovePageUp(LocalFrame& frame,
static bool executeMovePageUpAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
unsigned distance = verticalScrollDistance(frame);
if (!distance)
@@ -1132,7 +1132,7 @@ static bool executeMovePageUpAndModifySelection(LocalFrame& frame,
static bool executeMoveRight(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.selection().modify(FrameSelection::AlterationMove,
DirectionRight, CharacterGranularity,
@@ -1141,7 +1141,7 @@ static bool executeMoveRight(LocalFrame& frame,
static bool executeMoveRightAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionRight,
CharacterGranularity, UserTriggered);
@@ -1150,7 +1150,7 @@ static bool executeMoveRightAndModifySelection(LocalFrame& frame,
static bool executeMoveToBeginningOfDocument(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionBackward,
DocumentBoundary, UserTriggered);
@@ -1160,7 +1160,7 @@ static bool executeMoveToBeginningOfDocument(LocalFrame& frame,
static bool executeMoveToBeginningOfDocumentAndModifySelection(
LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionBackward,
DocumentBoundary, UserTriggered);
@@ -1169,7 +1169,7 @@ static bool executeMoveToBeginningOfDocumentAndModifySelection(
static bool executeMoveToBeginningOfLine(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionBackward,
LineBoundary, UserTriggered);
@@ -1178,7 +1178,7 @@ static bool executeMoveToBeginningOfLine(LocalFrame& frame,
static bool executeMoveToBeginningOfLineAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionBackward,
LineBoundary, UserTriggered);
@@ -1187,7 +1187,7 @@ static bool executeMoveToBeginningOfLineAndModifySelection(LocalFrame& frame,
static bool executeMoveToBeginningOfParagraph(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionBackward,
ParagraphBoundary, UserTriggered);
@@ -1197,7 +1197,7 @@ static bool executeMoveToBeginningOfParagraph(LocalFrame& frame,
static bool executeMoveToBeginningOfParagraphAndModifySelection(
LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionBackward,
ParagraphBoundary, UserTriggered);
@@ -1206,7 +1206,7 @@ static bool executeMoveToBeginningOfParagraphAndModifySelection(
static bool executeMoveToBeginningOfSentence(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionBackward,
SentenceBoundary, UserTriggered);
@@ -1216,7 +1216,7 @@ static bool executeMoveToBeginningOfSentence(LocalFrame& frame,
static bool executeMoveToBeginningOfSentenceAndModifySelection(
LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionBackward,
SentenceBoundary, UserTriggered);
@@ -1225,7 +1225,7 @@ static bool executeMoveToBeginningOfSentenceAndModifySelection(
static bool executeMoveToEndOfDocument(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionForward,
DocumentBoundary, UserTriggered);
@@ -1234,7 +1234,7 @@ static bool executeMoveToEndOfDocument(LocalFrame& frame,
static bool executeMoveToEndOfDocumentAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionForward,
DocumentBoundary, UserTriggered);
@@ -1243,7 +1243,7 @@ static bool executeMoveToEndOfDocumentAndModifySelection(LocalFrame& frame,
static bool executeMoveToEndOfSentence(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionForward,
SentenceBoundary, UserTriggered);
@@ -1252,7 +1252,7 @@ static bool executeMoveToEndOfSentence(LocalFrame& frame,
static bool executeMoveToEndOfSentenceAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionForward,
SentenceBoundary, UserTriggered);
@@ -1261,7 +1261,7 @@ static bool executeMoveToEndOfSentenceAndModifySelection(LocalFrame& frame,
static bool executeMoveToEndOfLine(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionForward,
LineBoundary, UserTriggered);
@@ -1270,7 +1270,7 @@ static bool executeMoveToEndOfLine(LocalFrame& frame,
static bool executeMoveToEndOfLineAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionForward,
LineBoundary, UserTriggered);
@@ -1279,7 +1279,7 @@ static bool executeMoveToEndOfLineAndModifySelection(LocalFrame& frame,
static bool executeMoveToEndOfParagraph(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionForward,
ParagraphBoundary, UserTriggered);
@@ -1288,7 +1288,7 @@ static bool executeMoveToEndOfParagraph(LocalFrame& frame,
static bool executeMoveToEndOfParagraphAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionForward,
ParagraphBoundary, UserTriggered);
@@ -1297,7 +1297,7 @@ static bool executeMoveToEndOfParagraphAndModifySelection(LocalFrame& frame,
static bool executeMoveParagraphBackward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionBackward,
ParagraphGranularity, UserTriggered);
@@ -1306,7 +1306,7 @@ static bool executeMoveParagraphBackward(LocalFrame& frame,
static bool executeMoveParagraphBackwardAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionBackward,
ParagraphGranularity, UserTriggered);
@@ -1315,7 +1315,7 @@ static bool executeMoveParagraphBackwardAndModifySelection(LocalFrame& frame,
static bool executeMoveParagraphForward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionForward,
ParagraphGranularity, UserTriggered);
@@ -1324,7 +1324,7 @@ static bool executeMoveParagraphForward(LocalFrame& frame,
static bool executeMoveParagraphForwardAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionForward,
ParagraphGranularity, UserTriggered);
@@ -1333,7 +1333,7 @@ static bool executeMoveParagraphForwardAndModifySelection(LocalFrame& frame,
static bool executeMoveUp(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.selection().modify(FrameSelection::AlterationMove,
DirectionBackward, LineGranularity,
@@ -1342,7 +1342,7 @@ static bool executeMoveUp(LocalFrame& frame,
static bool executeMoveUpAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionBackward,
LineGranularity, UserTriggered);
@@ -1351,7 +1351,7 @@ static bool executeMoveUpAndModifySelection(LocalFrame& frame,
static bool executeMoveWordBackward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionBackward,
WordGranularity, UserTriggered);
@@ -1360,7 +1360,7 @@ static bool executeMoveWordBackward(LocalFrame& frame,
static bool executeMoveWordBackwardAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionBackward,
WordGranularity, UserTriggered);
@@ -1369,7 +1369,7 @@ static bool executeMoveWordBackwardAndModifySelection(LocalFrame& frame,
static bool executeMoveWordForward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionForward,
WordGranularity, UserTriggered);
@@ -1378,7 +1378,7 @@ static bool executeMoveWordForward(LocalFrame& frame,
static bool executeMoveWordForwardAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionForward,
WordGranularity, UserTriggered);
@@ -1387,7 +1387,7 @@ static bool executeMoveWordForwardAndModifySelection(LocalFrame& frame,
static bool executeMoveWordLeft(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionLeft,
WordGranularity, UserTriggered);
@@ -1396,7 +1396,7 @@ static bool executeMoveWordLeft(LocalFrame& frame,
static bool executeMoveWordLeftAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionLeft,
WordGranularity, UserTriggered);
@@ -1405,7 +1405,7 @@ static bool executeMoveWordLeftAndModifySelection(LocalFrame& frame,
static bool executeMoveWordRight(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionRight,
WordGranularity, UserTriggered);
@@ -1414,7 +1414,7 @@ static bool executeMoveWordRight(LocalFrame& frame,
static bool executeMoveWordRightAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionRight,
WordGranularity, UserTriggered);
@@ -1423,7 +1423,7 @@ static bool executeMoveWordRightAndModifySelection(LocalFrame& frame,
static bool executeMoveToLeftEndOfLine(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionLeft,
LineBoundary, UserTriggered);
@@ -1432,7 +1432,7 @@ static bool executeMoveToLeftEndOfLine(LocalFrame& frame,
static bool executeMoveToLeftEndOfLineAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionLeft,
LineBoundary, UserTriggered);
@@ -1441,7 +1441,7 @@ static bool executeMoveToLeftEndOfLineAndModifySelection(LocalFrame& frame,
static bool executeMoveToRightEndOfLine(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationMove, DirectionRight,
LineBoundary, UserTriggered);
@@ -1450,7 +1450,7 @@ static bool executeMoveToRightEndOfLine(LocalFrame& frame,
static bool executeMoveToRightEndOfLineAndModifySelection(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().modify(FrameSelection::AlterationExtend, DirectionRight,
LineBoundary, UserTriggered);
@@ -1459,7 +1459,7 @@ static bool executeMoveToRightEndOfLineAndModifySelection(LocalFrame& frame,
static bool executeOutdent(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
DCHECK(frame.document());
return IndentOutdentCommand::create(*frame.document(),
@@ -1469,14 +1469,14 @@ static bool executeOutdent(LocalFrame& frame,
static bool executeToggleOverwrite(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().toggleOverwriteModeEnabled();
return true;
}
-static bool canReadClipboard(LocalFrame& frame, EditorCommandSource source) {
- if (source == CommandFromMenuOrKeyBinding)
+static bool canReadClipboard(LocalFrame& frame, EditCommandSource source) {
+ if (source == EditCommandSource::kMenuOrKeyBinding)
return true;
Settings* settings = frame.settings();
bool defaultValue = settings && settings->javaScriptCanAccessClipboard() &&
@@ -1486,7 +1486,7 @@ static bool canReadClipboard(LocalFrame& frame, EditorCommandSource source) {
static bool executePaste(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
// To support |allowExecutionWhenDisabled|, we need to check clipboard
// accessibility here rather than |Editor::Command::execute()|.
@@ -1501,7 +1501,7 @@ static bool executePaste(LocalFrame& frame,
static bool executePasteGlobalSelection(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
// To support |allowExecutionWhenDisabled|, we need to check clipboard
// accessibility here rather than |Editor::Command::execute()|.
@@ -1512,7 +1512,7 @@ static bool executePasteGlobalSelection(LocalFrame& frame,
return false;
if (!frame.editor().behavior().supportsGlobalSelection())
return false;
- DCHECK_EQ(source, CommandFromMenuOrKeyBinding);
+ DCHECK_EQ(source, EditCommandSource::kMenuOrKeyBinding);
bool oldSelectionMode = Pasteboard::generalPasteboard()->isSelectionMode();
Pasteboard::generalPasteboard()->setSelectionMode(true);
@@ -1523,7 +1523,7 @@ static bool executePasteGlobalSelection(LocalFrame& frame,
static bool executePasteAndMatchStyle(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
frame.editor().pasteAsPlainText(source);
return true;
@@ -1531,7 +1531,7 @@ static bool executePasteAndMatchStyle(LocalFrame& frame,
static bool executePrint(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
FrameHost* host = frame.host();
if (!host)
@@ -1541,7 +1541,7 @@ static bool executePrint(LocalFrame& frame,
static bool executeRedo(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().redo();
return true;
@@ -1549,7 +1549,7 @@ static bool executeRedo(LocalFrame& frame,
static bool executeRemoveFormat(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().removeFormattingAndStyle();
return true;
@@ -1557,7 +1557,7 @@ static bool executeRemoveFormat(LocalFrame& frame,
static bool executeScrollPageBackward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.eventHandler().bubblingScroll(ScrollBlockDirectionBackward,
ScrollByPage);
@@ -1565,7 +1565,7 @@ static bool executeScrollPageBackward(LocalFrame& frame,
static bool executeScrollPageForward(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.eventHandler().bubblingScroll(ScrollBlockDirectionForward,
ScrollByPage);
@@ -1573,7 +1573,7 @@ static bool executeScrollPageForward(LocalFrame& frame,
static bool executeScrollLineUp(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.eventHandler().bubblingScroll(ScrollUpIgnoringWritingMode,
ScrollByLine);
@@ -1581,7 +1581,7 @@ static bool executeScrollLineUp(LocalFrame& frame,
static bool executeScrollLineDown(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.eventHandler().bubblingScroll(ScrollDownIgnoringWritingMode,
ScrollByLine);
@@ -1589,7 +1589,7 @@ static bool executeScrollLineDown(LocalFrame& frame,
static bool executeScrollToBeginningOfDocument(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.eventHandler().bubblingScroll(ScrollBlockDirectionBackward,
ScrollByDocument);
@@ -1597,7 +1597,7 @@ static bool executeScrollToBeginningOfDocument(LocalFrame& frame,
static bool executeScrollToEndOfDocument(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return frame.eventHandler().bubblingScroll(ScrollBlockDirectionForward,
ScrollByDocument);
@@ -1605,7 +1605,7 @@ static bool executeScrollToEndOfDocument(LocalFrame& frame,
static bool executeSelectAll(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().selectAll();
return true;
@@ -1613,28 +1613,28 @@ static bool executeSelectAll(LocalFrame& frame,
static bool executeSelectLine(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return expandSelectionToGranularity(frame, LineGranularity);
}
static bool executeSelectParagraph(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return expandSelectionToGranularity(frame, ParagraphGranularity);
}
static bool executeSelectSentence(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return expandSelectionToGranularity(frame, SentenceGranularity);
}
static bool executeSelectToMark(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
const EphemeralRange mark =
frame.editor().mark().toNormalizedEphemeralRange();
@@ -1649,14 +1649,14 @@ static bool executeSelectToMark(LocalFrame& frame,
static bool executeSelectWord(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
return expandSelectionToGranularity(frame, WordGranularity);
}
static bool executeSetMark(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().setMark(frame.selection().selection());
return true;
@@ -1664,7 +1664,7 @@ static bool executeSetMark(LocalFrame& frame,
static bool executeStrikethrough(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
CSSIdentifierValue* lineThrough =
CSSIdentifierValue::create(CSSValueLineThrough);
@@ -1675,7 +1675,7 @@ static bool executeStrikethrough(LocalFrame& frame,
static bool executeStyleWithCSS(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
frame.editor().setShouldStyleWithCSS(!equalIgnoringCase(value, "false"));
return true;
@@ -1683,7 +1683,7 @@ static bool executeStyleWithCSS(LocalFrame& frame,
static bool executeUseCSS(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String& value) {
frame.editor().setShouldStyleWithCSS(equalIgnoringCase(value, "false"));
return true;
@@ -1691,7 +1691,7 @@ static bool executeUseCSS(LocalFrame& frame,
static bool executeSubscript(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
return executeToggleStyle(frame, source,
InputEvent::InputType::FormatSubscript,
@@ -1700,7 +1700,7 @@ static bool executeSubscript(LocalFrame& frame,
static bool executeSuperscript(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
return executeToggleStyle(frame, source,
InputEvent::InputType::FormatSuperscript,
@@ -1709,7 +1709,7 @@ static bool executeSuperscript(LocalFrame& frame,
static bool executeSwapWithMark(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
const VisibleSelection& mark = frame.editor().mark();
const VisibleSelection& selection = frame.selection().selection();
@@ -1722,7 +1722,7 @@ static bool executeSwapWithMark(LocalFrame& frame,
static bool executeToggleBold(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
return executeToggleStyle(frame, source, InputEvent::InputType::FormatBold,
CSSPropertyFontWeight, "normal", "bold");
@@ -1730,7 +1730,7 @@ static bool executeToggleBold(LocalFrame& frame,
static bool executeToggleItalic(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
return executeToggleStyle(frame, source, InputEvent::InputType::FormatItalic,
CSSPropertyFontStyle, "normal", "italic");
@@ -1738,7 +1738,7 @@ static bool executeToggleItalic(LocalFrame& frame,
static bool executeTranspose(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().transpose();
return true;
@@ -1746,7 +1746,7 @@ static bool executeTranspose(LocalFrame& frame,
static bool executeUnderline(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
CSSIdentifierValue* underline = CSSIdentifierValue::create(CSSValueUnderline);
return executeToggleStyleInList(
@@ -1756,7 +1756,7 @@ static bool executeUnderline(LocalFrame& frame,
static bool executeUndo(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().undo();
return true;
@@ -1764,7 +1764,7 @@ static bool executeUndo(LocalFrame& frame,
static bool executeUnlink(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
DCHECK(frame.document());
return UnlinkCommand::create(*frame.document())->apply();
@@ -1772,7 +1772,7 @@ static bool executeUnlink(LocalFrame& frame,
static bool executeUnscript(LocalFrame& frame,
Event*,
- EditorCommandSource source,
+ EditCommandSource source,
const String&) {
return executeApplyStyle(frame, source, InputEvent::InputType::None,
CSSPropertyVerticalAlign, "baseline");
@@ -1780,7 +1780,7 @@ static bool executeUnscript(LocalFrame& frame,
static bool executeUnselect(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.selection().clear();
return true;
@@ -1788,7 +1788,7 @@ static bool executeUnselect(LocalFrame& frame,
static bool executeYank(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().insertTextWithoutSendingTextEvent(
frame.editor().killRing().yank(), false, 0);
@@ -1798,7 +1798,7 @@ static bool executeYank(LocalFrame& frame,
static bool executeYankAndSelect(LocalFrame& frame,
Event*,
- EditorCommandSource,
+ EditCommandSource,
const String&) {
frame.editor().insertTextWithoutSendingTextEvent(
frame.editor().killRing().yank(), true, 0);
@@ -1818,13 +1818,13 @@ static bool supportedFromMenuOrKeyBinding(LocalFrame*) {
// Enabled functions
-static bool enabled(LocalFrame&, Event*, EditorCommandSource) {
+static bool enabled(LocalFrame&, Event*, EditCommandSource) {
return true;
}
static bool enabledVisibleSelection(LocalFrame& frame,
Event* event,
- EditorCommandSource) {
+ EditCommandSource) {
frame.document()->updateStyleAndLayoutIgnorePendingStylesheets();
// The term "visible" here includes a caret in editable text or a range in any
@@ -1836,7 +1836,7 @@ static bool enabledVisibleSelection(LocalFrame& frame,
static bool enabledVisibleSelectionAndMark(LocalFrame& frame,
Event* event,
- EditorCommandSource) {
+ EditCommandSource) {
frame.document()->updateStyleAndLayoutIgnorePendingStylesheets();
const VisibleSelection& selection = frame.editor().selectionForCommand(event);
@@ -1847,20 +1847,20 @@ static bool enabledVisibleSelectionAndMark(LocalFrame& frame,
static bool enableCaretInEditableText(LocalFrame& frame,
Event* event,
- EditorCommandSource) {
+ EditCommandSource) {
frame.document()->updateStyleAndLayoutIgnorePendingStylesheets();
const VisibleSelection& selection = frame.editor().selectionForCommand(event);
return selection.isCaret() && selection.isContentEditable();
}
-static bool enabledCopy(LocalFrame& frame, Event*, EditorCommandSource source) {
+static bool enabledCopy(LocalFrame& frame, Event*, EditCommandSource source) {
if (!canWriteClipboard(frame, source))
return false;
return frame.editor().canDHTMLCopy() || frame.editor().canCopy();
}
-static bool enabledCut(LocalFrame& frame, Event*, EditorCommandSource source) {
+static bool enabledCut(LocalFrame& frame, Event*, EditCommandSource source) {
if (!canWriteClipboard(frame, source))
return false;
return frame.editor().canDHTMLCut() || frame.editor().canCut();
@@ -1868,7 +1868,7 @@ static bool enabledCut(LocalFrame& frame, Event*, EditorCommandSource source) {
static bool enabledInEditableText(LocalFrame& frame,
Event* event,
- EditorCommandSource) {
+ EditCommandSource) {
frame.document()->updateStyleAndLayoutIgnorePendingStylesheets();
// We should update selection to canonicalize with current layout and style,
@@ -1879,11 +1879,11 @@ static bool enabledInEditableText(LocalFrame& frame,
static bool enabledDelete(LocalFrame& frame,
Event* event,
- EditorCommandSource source) {
+ EditCommandSource source) {
switch (source) {
- case CommandFromMenuOrKeyBinding:
+ case EditCommandSource::kMenuOrKeyBinding:
return frame.editor().canDelete();
- case CommandFromDOM:
+ case EditCommandSource::kDOM:
// "Delete" from DOM is like delete/backspace keypress, affects selected
// range if non-empty, otherwise removes a character
return enabledInEditableText(frame, event, source);
@@ -1894,7 +1894,7 @@ static bool enabledDelete(LocalFrame& frame,
static bool enabledInRichlyEditableText(LocalFrame& frame,
Event*,
- EditorCommandSource) {
+ EditCommandSource) {
frame.document()->updateStyleAndLayoutIgnorePendingStylesheets();
// We should update selection to canonicalize with current layout and style,
@@ -1905,9 +1905,7 @@ static bool enabledInRichlyEditableText(LocalFrame& frame,
frame.selection().rootEditableElement();
}
-static bool enabledPaste(LocalFrame& frame,
- Event*,
- EditorCommandSource source) {
+static bool enabledPaste(LocalFrame& frame, Event*, EditCommandSource source) {
if (!canReadClipboard(frame, source))
return false;
return frame.editor().canPaste();
@@ -1915,7 +1913,7 @@ static bool enabledPaste(LocalFrame& frame,
static bool enabledRangeInEditableText(LocalFrame& frame,
Event*,
- EditorCommandSource) {
+ EditCommandSource) {
frame.document()->updateStyleAndLayoutIgnorePendingStylesheets();
// We should update selection to canonicalize with current layout and style,
@@ -1926,7 +1924,7 @@ static bool enabledRangeInEditableText(LocalFrame& frame,
static bool enabledRangeInRichlyEditableText(LocalFrame& frame,
Event*,
- EditorCommandSource) {
+ EditCommandSource) {
frame.document()->updateStyleAndLayoutIgnorePendingStylesheets();
// We should update selection to canonicalize with current layout and style,
@@ -1936,11 +1934,11 @@ static bool enabledRangeInRichlyEditableText(LocalFrame& frame,
frame.selection().isContentRichlyEditable();
}
-static bool enabledRedo(LocalFrame& frame, Event*, EditorCommandSource) {
+static bool enabledRedo(LocalFrame& frame, Event*, EditCommandSource) {
return frame.editor().canRedo();
}
-static bool enabledUndo(LocalFrame& frame, Event*, EditorCommandSource) {
+static bool enabledUndo(LocalFrame& frame, Event*, EditCommandSource) {
return frame.editor().canUndo();
}
@@ -2543,13 +2541,13 @@ static const EditorInternalCommand* internalCommand(const String& commandName) {
}
Editor::Command Editor::createCommand(const String& commandName) {
- return Command(internalCommand(commandName), CommandFromMenuOrKeyBinding,
- m_frame);
+ return Command(internalCommand(commandName),
+ EditCommandSource::kMenuOrKeyBinding, m_frame);
}
-Editor::Command Editor::createCommand(const String& commandName,
- EditorCommandSource source) {
- return Command(internalCommand(commandName), source, m_frame);
+Editor::Command Editor::createCommandFromDOM(const String& commandName) {
+ return Command(internalCommand(commandName), EditCommandSource::kDOM,
+ m_frame);
}
bool Editor::executeCommand(const String& commandName) {
@@ -2614,7 +2612,7 @@ bool Editor::executeCommand(const String& commandName, const String& value) {
Editor::Command::Command() : m_command(0) {}
Editor::Command::Command(const EditorInternalCommand* command,
- EditorCommandSource source,
+ EditCommandSource source,
LocalFrame* frame)
: m_command(command), m_source(source), m_frame(command ? frame : nullptr) {
// Use separate assertions so we can tell which bad thing happened.
@@ -2637,7 +2635,7 @@ bool Editor::Command::execute(const String& parameter,
return false;
}
- if (m_source == CommandFromMenuOrKeyBinding) {
+ if (m_source == EditCommandSource::kMenuOrKeyBinding) {
InputEvent::InputType inputType =
InputTypeFromCommandType(m_command->commandType, *m_frame);
if (inputType != InputEvent::InputType::None) {
@@ -2667,9 +2665,9 @@ bool Editor::Command::isSupported() const {
if (!m_command)
return false;
switch (m_source) {
- case CommandFromMenuOrKeyBinding:
+ case EditCommandSource::kMenuOrKeyBinding:
return true;
- case CommandFromDOM:
+ case EditCommandSource::kDOM:
return m_command->isSupportedFromDOM(m_frame.get());
}
NOTREACHED();
« no previous file with comments | « third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698