FrameView can be null in addTextMatchMaker() after appendChild(iframe)

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

Index: third_party/WebKit/Source/core/testing/Internals.cpp
diff --git a/third_party/WebKit/Source/core/testing/Internals.cpp b/third_party/WebKit/Source/core/testing/Internals.cpp
index c13b7b919ecaf60e10933212f232a8481f64116a..9125fef91a77a15662bf5a97258eba0e7688824d 100644
--- a/third_party/WebKit/Source/core/testing/Internals.cpp
+++ b/third_party/WebKit/Source/core/testing/Internals.cpp
@@ -1047,7 +1047,8 @@ void Internals::addTextMatchMarker(const Range* range, bool isActive) {
 
   // This simulates what the production code does after
   // DocumentMarkerController::addTextMatchMarker().
-  range->ownerDocument().view()->invalidatePaintForTickmarks();
+  if (range->ownerDocument().view())

[yosin_UTC9, 2017/02/21 23:24:17]

Could you do early return at top of function, e.g. L1044?
Since we don't need to update layout and update markers.

void Internals::addTextMatchMarker(const Range* range, bool isActive) {
  DCHECK(range);
  if (!range->ownerDocument().view())
   return;
  range->ownerDocument().updateStyleAndLayoutIgnorePendingStylesheets();

[hiroshige, 2017/02/21 23:29:56]

Done.

+    range->ownerDocument().view()->invalidatePaintForTickmarks();
 }
 
 static bool parseColor(const String& value,