DevTools: [CSS] Add CSS.editRangeInStyleSheetText() to the protocol

Source/core/inspector/InspectorCSSAgent.cpp

 /*
  * Copyright (C) 2010, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *     notice, this list of conditions and the following disclaimer in the
@@ skipping 206 matching lines @@
 
         SetStyleSheetTextAction* other = static_cast<SetStyleSheetTextAction*>(action.get());
         m_text = other->m_text;
     }
 
 private:
     String m_text;
     String m_oldText;
 };
 
+class InspectorCSSAgent::EditRangeInStyleSheetTextAction FINAL : public InspectorCSSAgent::StyleSheetAction {
+    WTF_MAKE_NONCOPYABLE(EditRangeInStyleSheetTextAction);
+public:
+    EditRangeInStyleSheetTextAction(InspectorStyleSheet* styleSheet, const String& text, unsigned rangeStart, unsigned rangeEnd)
+        : InspectorCSSAgent::StyleSheetAction("EditRangeInStyleSheetTextAction", styleSheet)
+        , m_text(text)
+        , m_rangeStart(rangeStart)
+        , m_rangeEnd(rangeEnd)
+        , m_externalEditRangeResult(0)
+    {
+    }
+
+    virtual bool perform(ExceptionState& exceptionState) OVERRIDE
+    {
+        return redo(exceptionState);
+    }
+
+    virtual bool undo(ExceptionState& exceptionState) OVERRIDE
+    {
+        if (m_styleSheet->setText(m_oldText, exceptionState)) {
+            m_styleSheet->reparseStyleSheet(m_oldText);
+            return true;
+        }
+        return false;
+    }
+
+    virtual bool redo(ExceptionState& exceptionState) OVERRIDE
+    {
+        if (!m_styleSheet->getText(&m_oldText))
+            return false;
+        InspectorStyleSheet::EditRangeResult placeholder;
+        InspectorStyleSheet::EditRangeResult* editRangeResult = m_externalEditRangeResult ? m_externalEditRangeResult : &placeholder;
+        m_externalEditRangeResult = 0;
+        return m_styleSheet->editRange(m_text, m_rangeStart, m_rangeEnd, editRangeResult, exceptionState);
+    }
+
+    virtual String mergeId() OVERRIDE
+    {
+        return String::format("EditRangeInStyleSheetTextAction %s:%u:%u", m_styleSheet->id().utf8().data(), m_rangeStart, m_rangeEnd);
+    }
+
+    virtual void merge(PassOwnPtr<Action> action) OVERRIDE
+    {
+        ASSERT(action->mergeId() == mergeId());
+
+        EditRangeInStyleSheetTextAction* other = static_cast<EditRangeInStyleSheetTextAction*>(action.get());
+        m_text = other->m_text;
+    }
+
+    void setExternalEditRangeResult(InspectorStyleSheet::EditRangeResult* editRangeResult)
+    {
+        m_externalEditRangeResult = editRangeResult;
+    }
+
+private:
+    String m_text;
+    unsigned m_rangeStart;
+    unsigned m_rangeEnd;
+    String m_oldText;
+    InspectorStyleSheet::EditRangeResult* m_externalEditRangeResult;
+};
+
 class InspectorCSSAgent::SetPropertyTextAction FINAL : public InspectorCSSAgent::StyleSheetAction {
     WTF_MAKE_NONCOPYABLE(SetPropertyTextAction);
 public:
     SetPropertyTextAction(InspectorStyleSheet* styleSheet, const InspectorCSSId& cssId, unsigned propertyIndex, const String& text, bool overwrite)
         : InspectorCSSAgent::StyleSheetAction("SetPropertyText", styleSheet)
         , m_cssId(cssId)
         , m_propertyIndex(propertyIndex)
         , m_text(text)
         , m_overwrite(overwrite)
     {
@@ skipping 584 matching lines @@
     if (!inspectorStyleSheet)
         return;
 
     inspectorStyleSheet->getText(result);
 }
 
 void InspectorCSSAgent::setStyleSheetText(ErrorString* errorString, const String& styleSheetId, const String& text)
 {
     InspectorStyleSheet* inspectorStyleSheet = assertStyleSheetForId(errorString, styleSheetId);
     if (!inspectorStyleSheet) {
-        *errorString = "Style sheet with id " + styleSheetId + " not found.";
+        *errorString = "Style sheet with id " + styleSheetId + " not found";
         return;
     }
 
     TrackExceptionState exceptionState;
     m_domAgent->history()->perform(adoptPtr(new SetStyleSheetTextAction(inspectorStyleSheet, text)), exceptionState);
     *errorString = InspectorDOMAgent::toErrorString(exceptionState);
 }
 
+static bool extractRangeComponent(ErrorString* errorString, const RefPtr<JSONObject>& range, const String& component, unsigned& result)
+{
+    int parsedValue;
+    if (!range->getNumber(component, &parsedValue) || parsedValue < 0) {
+        *errorString = "range." + component + " must be a non-negative integer";
+        return false;
+    }
+    result = parsedValue;
+    return true;
+}
+
+void InspectorCSSAgent::editRangeInStyleSheetText(ErrorString* errorString, const String& styleSheetId, const RefPtr<JSONObject>& range, const String& text, RefPtr<TypeBuilder::CSS::CSSRule>& rule, RefPtr<TypeBuilder::CSS::CSSStyle>& style)
+{
+    InspectorStyleSheet* inspectorStyleSheet = assertStyleSheetForId(errorString, styleSheetId);
+    if (!inspectorStyleSheet) {
+        *errorString = "Stylesheet with id " + styleSheetId + " not found";
+        return;
+    }
+    unsigned startLineNumber;
+    unsigned startColumn;
+    unsigned endLineNumber;
+    unsigned endColumn;
+    if (!extractRangeComponent(errorString, range, "startLine", startLineNumber)
+        || !extractRangeComponent(errorString, range, "startColumn", startColumn)
+        || !extractRangeComponent(errorString, range, "endLine", endLineNumber)
+        || !extractRangeComponent(errorString, range, "endColumn", endColumn))
+        return;
+
+    unsigned startOffset;
+    unsigned endOffset;
+    bool success = inspectorStyleSheet->lineNumberAndColumnToOffset(startLineNumber, startColumn, &startOffset)
+        && inspectorStyleSheet->lineNumberAndColumnToOffset(endLineNumber, endColumn, &endOffset);
+    if (!success) {
+        *errorString = "Specified range is out of bounds";
+        return;
+    }
+
+    if (startOffset > endOffset) {
+        *errorString = "Range start must not succeed its end";
+        return;
+    }
+
+    TrackExceptionState exceptionState;
+    OwnPtr<EditRangeInStyleSheetTextAction> editAction = adoptPtr(new EditRangeInStyleSheetTextAction(inspectorStyleSheet, text, startOffset, endOffset));
+    InspectorStyleSheet::EditRangeResult editRangeResult;
+    editAction->setExternalEditRangeResult(&editRangeResult);

[vsevik, 2014/03/13 10:44:51]

Wouldn't it be simpler to add an actionResult getter on this action instead?

+    if (!m_domAgent->history()->perform(editAction.release(), exceptionState)) {
+        *errorString = InspectorDOMAgent::toErrorString(exceptionState);
+        return;
+    }
+    if (editRangeResult.rule) {
+        rule = inspectorStyleSheet->buildObjectForRule(editRangeResult.rule.get(), buildMediaListChain(editRangeResult.rule.get()));
+    } else if (editRangeResult.style) {
+        style = inspectorStyleSheet->buildObjectForStyle(editRangeResult.style.get());
+    }
+}
+
 void InspectorCSSAgent::setPropertyText(ErrorString* errorString, const RefPtr<JSONObject>& fullStyleId, int propertyIndex, const String& text, bool overwrite, RefPtr<TypeBuilder::CSS::CSSStyle>& result)
 {
     InspectorCSSId compoundId(fullStyleId);
     ASSERT(!compoundId.isEmpty());
 
     InspectorStyleSheet* inspectorStyleSheet = assertStyleSheetForId(errorString, compoundId.styleSheetId());
     if (!inspectorStyleSheet)
         return;
 
     TrackExceptionState exceptionState;
@@ skipping 487 matching lines @@
             documentsToChange.add(element->ownerDocument());
     }
 
     m_nodeIdToForcedPseudoState.clear();
     for (HashSet<Document*>::iterator it = documentsToChange.begin(), end = documentsToChange.end(); it != end; ++it)
         (*it)->setNeedsStyleRecalc(SubtreeStyleChange);
 }
 
 } // namespace WebCore