Speculative fix for crasher.

third_party/WebKit/Source/core/frame/ResizeViewportAnchor.cpp

Index: third_party/WebKit/Source/core/frame/ResizeViewportAnchor.cpp
diff --git a/third_party/WebKit/Source/core/frame/ResizeViewportAnchor.cpp b/third_party/WebKit/Source/core/frame/ResizeViewportAnchor.cpp
index 445d705ce230c74c30720d588a71300f77c3a59b..dcacc1ac53ee149fcd0d27ab20e8814e8c0fef0b 100644
--- a/third_party/WebKit/Source/core/frame/ResizeViewportAnchor.cpp
+++ b/third_party/WebKit/Source/core/frame/ResizeViewportAnchor.cpp
@@ -15,7 +15,7 @@ namespace blink {
 
 void ResizeViewportAnchor::ResizeFrameView(IntSize size) {
   FrameView* frame_view = RootFrameView();
-  DCHECK(frame_view);
+  SECURITY_CHECK(frame_view);

[pdr., 2017/05/19 22:47:50]

Is this actually related to security? If not, what about just CHECK?

[szager, 2017/05/19 22:52:39]

Done.

[dcheng, 2017/05/19 22:55:08]

If this absolutely must be a CHECK(), it should be temporary and should come
with a TODO to remove:
https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/c++.md...

(This CHECK normally doesn't add any value; it's possible to figure out that
frame_view is null from the crash dumps)

[szager, 2017/05/19 23:04:40]

Done.

 
   ScrollableArea* root_viewport = frame_view->GetScrollableArea();
   ScrollOffset offset = root_viewport->GetScrollOffset();