Remove NOTREACHED in layout_frame_set.h

This was introduced in: https://chromium-review.googlesource.com/c/chromium/src/+/2099322 It turns out this was reachable! Bug: 1064432 Change-Id: I742f706434cd45ea8c30505e565b193921c3f6ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2136880 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#756566}

Remove NOTREACHED in layout_frame_set.h
This was introduced in: https://chromium-review.googlesource.com/c/chromium/src/+/2099322 It turns out this was reachable! Bug: 1064432 Change-Id: I742f706434cd45ea8c30505e565b193921c3f6ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2136880 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#756566}
fb635da2 · Ian Kilpatrick · Commit Bot · efccf473 · fb635da2 · fb635da2
Commit fb635da2 authored Apr 06, 2020 by Ian Kilpatrick Committed by Commit Bot Apr 06, 2020
2 changed files
--- a/third_party/blink/renderer/core/layout/layout_frame_set.h
+++ b/third_party/blink/renderer/core/layout/layout_frame_set.h
@@ -135,7 +135,6 @@ class LayoutFrameSet final : public LayoutBox {

  MinMaxSizes PreferredLogicalWidths() const override { return MinMaxSizes(); }
  MinMaxSizes ComputeIntrinsicLogicalWidths() const final {
-    NOTREACHED();
    return MinMaxSizes();
  }


--- a/third_party/blink/web_tests/external/wpt/css/css-sizing/frameset-intrinsic-crash.html
+++ b/third_party/blink/web_tests/external/wpt/css/css-sizing/frameset-intrinsic-crash.html
+<!DOCTYPE html>
+<link rel="help" href="https://crbug.com/1064432">
+<div id=target style="float: left;"></div>
+<script>
+document.getElementById('target').appendChild(document.createElement('frameset'));
+</script>