<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>169894</bug_id>
          
          <creation_ts>2017-03-20 15:20:40 -0700</creation_ts>
          <short_desc>Assertion failure in WebPage::reload() (!m_pendingNavigationID) when reloading a subframe history navigation</short_desc>
          <delta_ts>2017-03-23 09:45:45 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>History</component>
          <version>WebKit Local Build</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>macOS 10.12</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=135129</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Daniel Bates">dbates</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>andersca</cc>
    
    <cc>beidson</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>mitz</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1289594</commentid>
    <comment_count>0</comment_count>
      <attachid>304956</attachid>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2017-03-20 15:20:40 -0700</bug_when>
    <thetext>Created attachment 304956
Test

Using a debug build of WebKit, open the attached the test case in Safari and follow the instructions. Then the assert ASSERT(!m_mainFrame-&gt;coreFrame()-&gt;loader().frameHasLoaded() || !m_pendingNavigationID) will fail in WebPage::reload() because m_pendingNavigationID is nonzero.

For completeness, I used a debug build of Mac WebKit at r214187.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1289597</commentid>
    <comment_count>1</comment_count>
      <attachid>304958</attachid>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2017-03-20 15:28:28 -0700</bug_when>
    <thetext>Created attachment 304958
Back/Forward test case

Notice that there is ASSERT(!m_pendingNavigationID) in WebPage::go{Back, Forward}(). Attached archive with a test case that can be used to hit these assertions. Expand the archive and open back-or-forward-after-navigate-iframe.html in Safari with a debug build of WebKit and follow the instructions.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>304956</attachid>
            <date>2017-03-20 15:20:40 -0700</date>
            <delta_ts>2017-03-20 15:20:40 -0700</delta_ts>
            <desc>Test</desc>
            <filename>test.html</filename>
            <type>text/html</type>
            <size>370</size>
            <attacher name="Daniel Bates">dbates</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+DQo8aHRtbD4NCjxib2R5Pg0KPGlmcmFtZSBzcmNkb2M9JzwhRE9DVFlQ
RSBodG1sPg0KPGh0bWw+DQo8Ym9keT4NCjxwPlBlcmZvcm0gdGhlIGZvbGxvd2luZzo8L3A+DQo8
b2w+DQogICAgPGxpPkNsaWNrIDxhIGhyZWY9ImRhdGE6dGV4dC9odG1sLDxwPlByZXNzIHRoZSBi
YWNrIGJ1dHRvbiBpbiB0aGUgYnJvd3Nlci48L3A+Ij5oZXJlPC9hPi4NCiAgICA8bGk+UmVsb2Fk
IHRoZSBtYWluIHBhZ2UuPC9saT4NCjwvb2w+DQo8cD5UaGlzIHRlc3QgUEFTU0VEIGlmIHdlIGRv
IG5vdCBjYXVzZSBhbiBhc3NlcnRpb24gZmFpbHVyZS48L3A+DQo8L2JvZHk+DQo8L2h0bWw+Jz48
L2lmcmFtZT4NCjwvYm9keT4NCjwvaHRtbD4NCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>304958</attachid>
            <date>2017-03-20 15:28:28 -0700</date>
            <delta_ts>2017-03-20 15:28:28 -0700</delta_ts>
            <desc>Back/Forward test case</desc>
            <filename>back-or-forward-after-navigate-iframe.zip</filename>
            <type>application/zip</type>
            <size>1757</size>
            <attacher name="Daniel Bates">dbates</attacher>
            
              <data encoding="base64">UEsDBBQACAAIAPx6dEoAAAAAAAAAAAAAAAAwABAAYmFjay1vci1mb3J3YXJkLWFmdGVyLW5hdmln
YXRlLWlmcmFtZS1wYWdlMi5odG1sVVgMAKVW0Fh7VtBY9QEUAIWQwWrDMAyG74W+g9ZLLyO+F9cw
2p4XaC87KrG8mDlWkR2yvf3sdGUwNqaLQNL/6Zf0w/H5cHlpTzDkMZj1St9zx/ajZu8ER4IkveV+
v9X/Cq6mJXEsI+SBwHEIPPv4utPqWttch6GEDt4cgu/fQCMMQm6/sZhxl+k9qwp9rCihlBZQh2Wy
m3LmCD7eSsJzImkqeWMGEtIKTfON/6FmKXZkRrF/goqoeFSLybL+Mviip5ShfTqfT0fwDmYCyxA5
Q49TIsAImIo6+wJ06MMk1Hwdq+5PWQ4yW6PV7Z+/NNerT1BLBwisCrw64QAAAI0BAABQSwMECgAA
AAAAgnt0SgAAAAAAAAAAAAAAAAkAEABfX01BQ09TWC9VWAwAdFfQWHRX0Fj1ARQAUEsDBBQACAAI
APx6dEoAAAAAAAAAAAAAAAA7ABAAX19NQUNPU1gvLl9iYWNrLW9yLWZvcndhcmQtYWZ0ZXItbmF2
aWdhdGUtaWZyYW1lLXBhZ2UyLmh0bWxVWAwApVbQWHtW0Fj1ARQAY2AVY2dgYmDwTUxW8A9WiFCA
ApAYAycQGwHxIyAG8hlFGIgCjiEhQRAWWMcOII5BU8IEFbdhYBBNzs/VSywoyEnVKyxNLErMK8nM
SwWp+QLEClD53MTkovzcxJLUYr3kxKLUEoZCfQMDC2NrU4vklCRjA3Pr4MS0xKJMawsDJxdnAxNj
XTcTF3NdE1dLY11LI2MzXUsLU0tzAwMDNzcnR4aKOaufBTE0Pfkz59IdG9Zo1uBsA3t1Y33RQ6tZ
GXrYOdQBUEsHCAoMdLHDAAAAFAEAAFBLAwQUAAgACAD3enRKAAAAAAAAAAAAAAAAKgAQAGJhY2st
b3ItZm9yd2FyZC1hZnRlci1uYXZpZ2F0ZS1pZnJhbWUuaHRtbFVYDAClVtBYclbQWPUBFAA1jbEO
wjAMBXck/iF0dy0xmyyFGQYWRrd1mqhpg6wIxN+TIPqW01vu6HC+dvfH7WJ8XqLd72hjn8ZPZarP
lFEMtothmA2x8Sru1PQ8zJAUXNI36wjssiis/AoTZ4HglBeBJ09ybKu3sV5UCNm2hEVX9PjzE245
/Pe/UEsHCNoY1ZV5AAAAmQAAAFBLAwQUAAgACAD3enRKAAAAAAAAAAAAAAAANQAQAF9fTUFDT1NY
Ly5fYmFjay1vci1mb3J3YXJkLWFmdGVyLW5hdmlnYXRlLWlmcmFtZS5odG1sVVgMAKVW0FhyVtBY
9QEUAGNgFWNnYGJg8E1MVvAPVohQgAKQGAMnEBsB8RMgBvIZxRiIAo4hIUEQFljHDiCOQ1PCBBW3
YWAQTc7P1UssKMhJ1SssTSxKzCvJzEsFqfkCxEpQ+dzE5KL83MSS1GK95MSi1BKGQn0DAwtja1OL
5JQkYwNz6+DEtMSiTGsLAycXZwMTY103ExdzXRNXS2NdSyNjM11LC1NLcwMDAzc3J0eGai6QD5Pz
c0pz8xRsFUxMrMECOUB7QVxrrloAUEsHCIUIUXS8AAAAFgEAAFBLAQIVAxQACAAIAPx6dEqsCrw6
4QAAAI0BAAAwAAwAAAAAAAAAAEDtgQAAAABiYWNrLW9yLWZvcndhcmQtYWZ0ZXItbmF2aWdhdGUt
aWZyYW1lLXBhZ2UyLmh0bWxVWAgApVbQWHtW0FhQSwECFQMKAAAAAACCe3RKAAAAAAAAAAAAAAAA
CQAMAAAAAAAAAABA/UFPAQAAX19NQUNPU1gvVVgIAHRX0Fh0V9BYUEsBAhUDFAAIAAgA/Hp0SgoM
dLHDAAAAFAEAADsADAAAAAAAAAAAQKSBhgEAAF9fTUFDT1NYLy5fYmFjay1vci1mb3J3YXJkLWFm
dGVyLW5hdmlnYXRlLWlmcmFtZS1wYWdlMi5odG1sVVgIAKVW0Fh7VtBYUEsBAhUDFAAIAAgA93p0
StoY1ZV5AAAAmQAAACoADAAAAAAAAAAAQO2BwgIAAGJhY2stb3ItZm9yd2FyZC1hZnRlci1uYXZp
Z2F0ZS1pZnJhbWUuaHRtbFVYCAClVtBYclbQWFBLAQIVAxQACAAIAPd6dEqFCFF0vAAAABYBAAA1
AAwAAAAAAAAAAECkgaMDAABfX01BQ09TWC8uX2JhY2stb3ItZm9yd2FyZC1hZnRlci1uYXZpZ2F0
ZS1pZnJhbWUuaHRtbFVYCAClVtBYclbQWFBLBQYAAAAABQAFAPUBAADSBAAAAAA=
</data>

          </attachment>
      

    </bug>

</bugzilla>