Ignore:
Timestamp:
May 30, 2021, 9:11:40 AM (4 years ago)
Author:
Darin Adler
Message:

Remove WTF::Optional synonym for std::optional, using that class template directly instead
https://bugs.webkit.org/show_bug.cgi?id=226433

Reviewed by Chris Dumez.

Source/JavaScriptCore:

  • <many files>: Let the do-webcore-rename script rename Optional<> to std::optional<>.
  • inspector/scripts/codegen/generate_objc_protocol_types_implementation.py:

(ObjCProtocolTypesImplementationGenerator._generate_init_method_for_payload): Use auto instead
of Optional<>. Also use * instead of value() and nest the definition of the local inside an if
statement in the case where it's an optional.

  • inspector/scripts/tests/expected/*: Regenerated these results.

Source/WebCore:

  • <many files>: Let the do-webcore-rename script rename Optional<> to std::optional<>.

Source/WebCore/PAL:

  • <many files>: Let the do-webcore-rename script rename Optional<> to std::optional<>.

Source/WebDriver:

  • <many files>: Let the do-webcore-rename script rename Optional<> to std::optional<>.

Source/WebKit:

  • <many files>: Let the do-webcore-rename script rename Optional<> to std::optional<>.
  • Scripts/webkit/tests: Regenerated expected results, by running the command "python

Scripts/webkit/messages_unittest.py -r". (How am I supposed to know to do that?)

Source/WebKitLegacy/ios:

  • WebCoreSupport/WebChromeClientIOS.h: Let the do-webcore-rename script rename

Optional<> to std::optional<>.

Source/WebKitLegacy/mac:

  • <many files>: Let the do-webcore-rename script rename Optional<> to std::optional<>.

Source/WebKitLegacy/win:

  • <many files>: Let the do-webcore-rename script rename Optional<> to std::optional<>.

Source/WTF:

  • <many files>: Let the do-webcore-rename script rename Optional<> to std::optional<>.
  • wtf/Optional.h: Remove WTF::Optional.

Tools:

  • <many files>: Let the do-webcore-rename script rename Optional<> to std::optional<>.
File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/css/CSSTimingFunctionValue.h

    r261046 r278253  
    6565class CSSStepsTimingFunctionValue final : public CSSValue {
    6666public:
    67     static Ref<CSSStepsTimingFunctionValue> create(int steps, Optional<StepsTimingFunction::StepPosition> stepPosition)
     67    static Ref<CSSStepsTimingFunctionValue> create(int steps, std::optional<StepsTimingFunction::StepPosition> stepPosition)
    6868    {
    6969        return adoptRef(*new CSSStepsTimingFunctionValue(steps, stepPosition));
     
    7171
    7272    int numberOfSteps() const { return m_steps; }
    73     Optional<StepsTimingFunction::StepPosition> stepPosition() const { return m_stepPosition; }
     73    std::optional<StepsTimingFunction::StepPosition> stepPosition() const { return m_stepPosition; }
    7474
    7575    String customCSSText() const;
     
    7878
    7979private:
    80     CSSStepsTimingFunctionValue(int steps, Optional<StepsTimingFunction::StepPosition> stepPosition)
     80    CSSStepsTimingFunctionValue(int steps, std::optional<StepsTimingFunction::StepPosition> stepPosition)
    8181        : CSSValue(StepsTimingFunctionClass)
    8282        , m_steps(steps)
     
    8686
    8787    int m_steps;
    88     Optional<StepsTimingFunction::StepPosition> m_stepPosition;
     88    std::optional<StepsTimingFunction::StepPosition> m_stepPosition;
    8989};
    9090
Note: See TracChangeset for help on using the changeset viewer.