Add DocumentMarker::MatchStatus enum for text match markers

third_party/WebKit/Source/core/testing/Internals.h

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2013 Apple 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.
@@ skipping 15 matching lines @@
 
 #ifndef Internals_h
 #define Internals_h
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/ScriptValue.h"
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "core/css/CSSComputedStyleDeclaration.h"
+#include "core/editing/markers/DocumentMarker.h"
 #include "core/page/scrolling/ScrollingCoordinator.h"
 #include "platform/heap/Handle.h"
 #include "wtf/Forward.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
 class Animation;
 class CallbackFunctionTest;
 class CanvasRenderingContext;
@@ skipping 136 matching lines @@
   String markerDescriptionForNode(Node*,
                                   const String& markerType,
                                   unsigned index,
                                   ExceptionState&);
   void addTextMatchMarker(const Range*, bool isActive);
   void addCompositionMarker(const Range*,
                             const String& underlineColorValue,
                             bool thick,
                             const String& backgroundColorValue,
                             ExceptionState&);
-  void setMarkersActive(Node*, unsigned startOffset, unsigned endOffset, bool);
+  void setMarkersActive(Node*,
+                        unsigned startOffset,
+                        unsigned endOffset,
+                        bool isActive);

[rlanday, 2017/04/04 20:48:23]

Note: I didn't convert this to use an enum because this is used by IDL code
(third_party/WebKit/Source/core/testing/Internals.idl) and I couldn't figure out
if it's possible to use a C++ enum there

[Xiaocheng, 2017/04/04 21:11:15]

We just use string in IDLs. There's a lot of examples in Internals.

   void setMarkedTextMatchesAreHighlighted(Document*, bool);
 
   void setFrameViewPosition(Document*, long x, long y, ExceptionState&);
   String viewportAsText(Document*,
                         float devicePixelRatio,
                         int availableWidth,
                         int availableHeight,
                         ExceptionState&);
 
   bool elementShouldAutoComplete(Element* inputElement, ExceptionState&);
@@ skipping 350 matching lines @@
                            const String& markerType,
                            unsigned index,
                            ExceptionState&);
   Member<InternalRuntimeFlags> m_runtimeFlags;
   Member<Document> m_document;
 };
 
 }  // namespace blink
 
 #endif  // Internals_h