<?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>93680</bug_id>
          
          <creation_ts>2012-08-09 20:23:17 -0700</creation_ts>
          <short_desc>Back/Forward list with error page</short_desc>
          <delta_ts>2022-03-14 16:27:35 -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>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>UNCONFIRMED</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Basavaraj Padmashali Sidda">basavarajaps</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>basavaraj.ps</cc>
    
    <cc>beidson</cc>
    
    <cc>cgarcia</cc>
    
    <cc>creis</cc>
    
    <cc>dglazkov</cc>
    
    <cc>fishd</cc>
    
    <cc>japhet</cc>
    
    <cc>jeffrey+webkit</cc>
    
    <cc>jjjaquanrice</cc>
    
    <cc>jvnforums10</cc>
    
    <cc>pravind.2k4</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>692146</commentid>
    <comment_count>0</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavarajaps</who>
    <bug_when>2012-08-09 20:23:17 -0700</bug_when>
    <thetext>Adding multiple Items in the history tree for a safari error page, i.e when calling loadAlternateHTMLString when back or forward key is pressed.

Steps to reproduce 1. Fetch any URL(eg, google.com) 2. &lt;unreachable URL&gt; which displays safari error page (eg, fdgdfgfdgf.com) 3. 3rd any URL (eg, apple.com) 
when you press back from 3rd page, loadAlternateHTMLString() is called with load type standard which adds one more history item of the error page in the History Tree.
same case with Forward aswell.

Reproducible in safari/WebKit2 nightly build

This issue is similar to &lt;https://bugs.webkit.org/show_bug.cgi?id=84041&gt; which i for the case Reload.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>692148</commentid>
    <comment_count>1</comment_count>
      <attachid>157623</attachid>
    <who name="Basavaraj Padmashali Sidda">basavarajaps</who>
    <bug_when>2012-08-09 20:26:11 -0700</bug_when>
    <thetext>Created attachment 157623
Patch for back/forward list with error page

Proposed patch for back/forward list with error page</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>692154</commentid>
    <comment_count>2</comment_count>
      <attachid>157626</attachid>
    <who name="Basavaraj Padmashali Sidda">basavarajaps</who>
    <bug_when>2012-08-09 20:36:52 -0700</bug_when>
    <thetext>Created attachment 157626
Updated proposed patch for back/forward list with error page

Updated proposed patch for back/forward list with error page</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>692695</commentid>
    <comment_count>3</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2012-08-10 11:13:50 -0700</bug_when>
    <thetext>Please mark the patch with review? flag if it&apos;s ready for review.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>693058</commentid>
    <comment_count>4</comment_count>
      <attachid>157626</attachid>
    <who name="Basavaraj Padmashali Sidda">basavarajaps</who>
    <bug_when>2012-08-10 23:42:38 -0700</bug_when>
    <thetext>Comment on attachment 157626
Updated proposed patch for back/forward list with error page

marked review flag to ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>695292</commentid>
    <comment_count>5</comment_count>
      <attachid>157626</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2012-08-14 12:56:52 -0700</bug_when>
    <thetext>Comment on attachment 157626
Updated proposed patch for back/forward list with error page

Patches need to come with change logs that explain why the code is changed, and test cases to show the bug and demonstrate the bug has been fixed. review- because this is missing those elements.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>778888</commentid>
    <comment_count>6</comment_count>
      <attachid>176694</attachid>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2012-11-29 03:14:23 -0800</bug_when>
    <thetext>Created attachment 176694
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>778892</commentid>
    <comment_count>7</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2012-11-29 03:23:44 -0800</bug_when>
    <thetext>(In reply to comment #5)
&gt; (From update of attachment 157626 [details])
&gt; Patches need to come with change logs that explain why the code is changed, and test cases to show the bug and demonstrate the bug has been fixed. review- because this is missing those elements.

Thanks for reviewing, i have uploaded the patch with change log. This issue is related to back/forward navigation required from the error page, so i am unable to figure out how to write layout test case.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>778978</commentid>
    <comment_count>8</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2012-11-29 07:01:37 -0800</bug_when>
    <thetext>(In reply to comment #0)
&gt; Adding multiple Items in the history tree for a safari error page, i.e when calling loadAlternateHTMLString when back or forward key is pressed.
&gt; 
&gt; Steps to reproduce 1. Fetch any URL(eg, google.com) 2. &lt;unreachable URL&gt; which displays safari error page (eg, fdgdfgfdgf.com) 3. 3rd any URL (eg, apple.com) 
&gt; when you press back from 3rd page, loadAlternateHTMLString() is called with load type standard which adds one more history item of the error page in the History Tree.
&gt; same case with Forward as well.
&gt; 
&gt; Reproducible in safari/WebKit2 nightly build
&gt; 
&gt; This issue is similar to &lt;https://bugs.webkit.org/show_bug.cgi?id=84041&gt; which i for the case Reload.

PN: This issue can be easily observed without Proxy in webkit2 safari browser,</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>779322</commentid>
    <comment_count>9</comment_count>
      <attachid>176694</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-11-29 12:29:39 -0800</bug_when>
    <thetext>Comment on attachment 176694
Patch

Attachment 176694 did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/15055045

New failing tests:
fast/dom/DeviceMotion/optional-event-properties.html
http/tests/navigation/redirect302-metaredirect.html
http/tests/navigation/back-to-slow-frame.html
http/tests/navigation/timerredirect-subframeload.html
http/tests/navigation/post-goback2.html
fast/events/pageshow-pagehide.html
fast/forms/button-style-color.html
http/tests/navigation/scrollstate-after-http-equiv-refresh-fragment-identifier-2.html
http/tests/navigation/success200-loadsame.html
http/tests/navigation/reload-subframe-frame.html
http/tests/navigation/go-back-to-error-page.html
http/tests/navigation/changing-frame-hierarchy-in-onload.html
http/tests/history/replacestate-post-to-get.html
fast/forms/range/ValidityState-stepMismatch-range.html
http/tests/navigation/postredirect-frames.html
http/tests/history/back-with-fragment-change.php
compositing/flat-with-transformed-child.html
http/tests/navigation/image-load-in-subframe-unload-handler.html
http/tests/navigation/javascriptlink-subframeload.html
http/tests/navigation/post-frames.html
http/tests/navigation/anchor-subframeload.html
http/tests/cache/subresource-fragment-identifier.html
http/tests/navigation/postredirect-goback2.html
fast/dom/Geolocation/not-enough-arguments.html
fast/dom/navigation-type-navigate.html
http/tests/navigation/postredirect-basic.html
fast/dom/DeviceOrientation/no-synchronous-events.html
fast/events/onunload-clears-onbeforeunload.html
http/tests/navigation/redirect-on-reload-updates-history-item.html
http/tests/navigation/metaredirect-subframeload.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>779824</commentid>
    <comment_count>10</comment_count>
      <attachid>176694</attachid>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2012-11-29 22:35:54 -0800</bug_when>
    <thetext>Comment on attachment 176694
Patch

This patch does not include a new test itself.  Requiring tests with behavior changes is a requirement of the project that Darin laid out in his comment.

Additionally it caused existing tests to fail, as the WebKit Review Bot mentioned.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>826398</commentid>
    <comment_count>11</comment_count>
      <attachid>186842</attachid>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-02-06 06:36:30 -0800</bug_when>
    <thetext>Created attachment 186842
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>826399</commentid>
    <comment_count>12</comment_count>
      <attachid>186842</attachid>
    <who name="Pravin D">pravind.2k4</who>
    <bug_when>2013-02-06 06:37:41 -0800</bug_when>
    <thetext>Comment on attachment 186842
Patch

Updated the patch to fix the testcase failures. 
Added a testcase for the same</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>826404</commentid>
    <comment_count>13</comment_count>
    <who name="Pravin D">pravind.2k4</who>
    <bug_when>2013-02-06 06:39:41 -0800</bug_when>
    <thetext>(In reply to comment #12)
&gt; (From update of attachment 186842 [details])
&gt; Updated the patch to fix the testcase failures. 
&gt; Added a testcase for the same

Sorry... updated the wrong bug... :( :(</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>826428</commentid>
    <comment_count>14</comment_count>
      <attachid>186842</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-02-06 07:25:23 -0800</bug_when>
    <thetext>Comment on attachment 186842
Patch

Attachment 186842 did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/16398200

New failing tests:
http/tests/navigation/back-navigation-to-error-page-substitute-data.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>826511</commentid>
    <comment_count>15</comment_count>
      <attachid>186842</attachid>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2013-02-06 09:42:16 -0800</bug_when>
    <thetext>Comment on attachment 186842
Patch

Still causes test failures.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>831741</commentid>
    <comment_count>16</comment_count>
      <attachid>188035</attachid>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-02-13 02:15:13 -0800</bug_when>
    <thetext>Created attachment 188035
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>831746</commentid>
    <comment_count>17</comment_count>
      <attachid>188035</attachid>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-02-13 02:25:16 -0800</bug_when>
    <thetext>Comment on attachment 188035
Patch

Have uploaded a new patch fixing the test case failure on cr-linux.
Chromium behavior towards handling failing URL is different from other ports. Chromium creates a new frame when back/forward navigation is performed. This set the load type to &quot;Standard&quot; by default, even during back/forward navigation.
Have added a chromium port spec expected file for the same.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>839517</commentid>
    <comment_count>18</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-02-22 05:47:31 -0800</bug_when>
    <thetext>Please review this patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>845238</commentid>
    <comment_count>19</comment_count>
      <attachid>188035</attachid>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2013-03-01 09:31:45 -0800</bug_when>
    <thetext>Comment on attachment 188035
Patch

It doesn&apos;t really make sense to me why chromium would be different here.  It seems like they still have the bug.  Is there a good enough explanation for this that we should accept it?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>846402</commentid>
    <comment_count>20</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-03-04 06:14:47 -0800</bug_when>
    <thetext>Hi Brady,

This issue is seen in webkit2 code based and not in webkit and chrome, and here is explanation, 

About the issue: 

When load with substitute data, LoadHTMLString() is called with m_loadType is Back/Forward Type for a error page(unreachable URL in our case) in FrameLoader::Load(), the loadType is set to “Reload” if shouldReloadToHandleUnreachableURL() returns true,  Standard Type otherwise. 

shouldReloadToHandleUnreachableURL() return value depends on m_delegateIsHandlingProvisionalLoadError. 

On Webkit(single process) or  when an error page is loaded, m_delegateIsHandlingProvisionalLoadError is set to true. Then a call is made to 

Client’s  dispatchDidFailProvisionalLoad(). This method does a load with substitute data (LoadHTMLString). On  LoadHTMLString , shouldReloadToHandleUnreachableURL () is called. It returns true. As a result the loadType is set to “Reload” and duplication of history item of the error page is not seen on Back/Forward navigation.  Then m_delegateIsHandlingProvisionalLoadError is reset. See the code below:

m_delegateIsHandlingProvisionalLoadError = true;
m_client-&gt;dispatchDidFailProvisionalLoad(error); -- sync call of LoadHTMLString() in webkit(single process)
m_delegateIsHandlingProvisionalLoadError = false;

However on webkit2 :
m_delegateIsHandlingProvisionalLoadError = true;
m_client-&gt;dispatchDidFailProvisionalLoad(error); -- async call of LoadHTMLString() in webkit(single process)
m_delegateIsHandlingProvisionalLoadError = false;

The call to client’s dispatchDidFailProvisionalLoad() is async. The m_delegateIsHandlingProvisionalLoadError is reset before LoadHTMLString is called from the client. Thus shouldReloadToHandleUnreachableURL() returns false instead of true.  As result a Back/Forward navigation is affected.


Have 2 approaches to fix the issue. 

Approach 1
Implementation of sendSync in client&apos;s dispatchDidFailProvisionalLoad(). However this may not be good approach, because this will stall the UI process.
Call flow for the same is WebProcess -&gt; UIProcess -&gt; App -&gt; UIProcess -&gt; Webprocess. Also overwriting loadType to “Reload” on history navigation does not seem to be proper.

Approach2
The proposed patch (https://bug-93680-attachments.webkit.org/attachment.cgi?id=188035). 
In this, we just restore the loadType if it is back/forward type when loading substitute data during Back/Forward Navigation, instead of overwriting loadType to &quot;reload&quot; and loading as reload.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>846484</commentid>
    <comment_count>21</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-03-04 07:44:29 -0800</bug_when>
    <thetext>(In reply to comment #20)
&gt; Hi Brady,
&gt; 
&gt; This issue is seen in webkit2 code based and not in webkit and chrome, and here is explanation, 
&gt; 
&gt; About the issue: 
&gt; 
&gt; When load with substitute data, LoadHTMLString() is called with m_loadType is Back/Forward Type for a error page(unreachable URL in our case) in FrameLoader::Load(), the loadType is set to “Reload” if shouldReloadToHandleUnreachableURL() returns true,  Standard Type otherwise. 
&gt; 
&gt; shouldReloadToHandleUnreachableURL() return value depends on m_delegateIsHandlingProvisionalLoadError. 
&gt; 
&gt; On Webkit(single process) or  when an error page is loaded, m_delegateIsHandlingProvisionalLoadError is set to true. Then a call is made to 
&gt; 
&gt; Client’s  dispatchDidFailProvisionalLoad(). This method does a load with substitute data (LoadHTMLString). On  LoadHTMLString , shouldReloadToHandleUnreachableURL () is called. It returns true. As a result the loadType is set to “Reload” and duplication of history item of the error page is not seen on Back/Forward navigation.  Then m_delegateIsHandlingProvisionalLoadError is reset. See the code below:
&gt; 
&gt; m_delegateIsHandlingProvisionalLoadError = true;
&gt; m_client-&gt;dispatchDidFailProvisionalLoad(error); -- sync call of LoadHTMLString() in webkit(single process)
&gt; m_delegateIsHandlingProvisionalLoadError = false;
&gt; 
&gt; However on webkit2 :
&gt; m_delegateIsHandlingProvisionalLoadError = true;
&gt; m_client-&gt;dispatchDidFailProvisionalLoad(error); -- async call of LoadHTMLString() in webkit(single process)
&gt; m_delegateIsHandlingProvisionalLoadError = false;
&gt; 
&gt; The call to client’s dispatchDidFailProvisionalLoad() is async. The m_delegateIsHandlingProvisionalLoadError is reset before LoadHTMLString is called from the client. Thus shouldReloadToHandleUnreachableURL() returns false instead of true.  As result a Back/Forward navigation is affected.
&gt; 
&gt; 
&gt; Have 2 approaches to fix the issue. 
&gt; 
&gt; Approach 1
&gt; Implementation of sendSync in client&apos;s dispatchDidFailProvisionalLoad(). However this may not be good approach, because this will stall the UI process.
&gt; Call flow for the same is WebProcess -&gt; UIProcess -&gt; App -&gt; UIProcess -&gt; Webprocess. Also overwriting loadType to “Reload” on history navigation does not seem to be proper.
&gt; 
&gt; Approach2
&gt; The proposed patch (https://bug-93680-attachments.webkit.org/attachment.cgi?id=188035). 
&gt; In this, we just restore the loadType if it is back/forward type when loading substitute data during Back/Forward Navigation, instead of overwriting loadType to &quot;reload&quot; and loading as reload.

working on approach 2: Reverting existing overwriting of loadType to reload, and on top of it adding proposed patch so that added test case to pass in chrome also and make sure that all existing old test cases to pass ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>846588</commentid>
    <comment_count>22</comment_count>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2013-03-04 09:49:24 -0800</bug_when>
    <thetext>(In reply to comment #20)
&gt; Hi Brady,
&gt; 
&gt; This issue is seen in webkit2 code based and not in webkit and chrome, and here is explanation, 

I was confused by the chromium specific test results.  They clearly still behave differently here.

But it&apos;s not important I guess.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>847600</commentid>
    <comment_count>23</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-03-05 05:59:24 -0800</bug_when>
    <thetext>Checked with the Aproach2, chrome test results still behaves same,
guess this result is due to some chrome specific change.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>850082</commentid>
    <comment_count>24</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-03-07 07:41:25 -0800</bug_when>
    <thetext>In case of Chromium, BackNavigation fails for unreachable url, i.e from 3rd URL to 2nd URL (didFailProvisionalLoad()), 
============== Back Forward List ==============
        http://127.0.0.1:8000/navigation/back-navigation-to-error-page-substitute-data.html  **nav target**
        http://127.0.0.1:2321/SomeUnreachableURL  **nav target**
curr-&gt;  http://127.0.0.1:8000/navigation/resources/page-to-go-back-from.html  **nav target**
===============================================
According to test case from patch (https://bug-93680-attachments.webkit.org/attachment.cgi?id=188035), when you call loadHTMLString() after Back Navigation, it overwrites the 3rd URL since m_loadType will be of &quot;Back/Forward&quot; type.
So test result will look different for chromium as shown below.
============== Back Forward List ==============
        http://127.0.0.1:8000/navigation/back-navigation-to-error-page-substitute-data.html  **nav target**
        http://127.0.0.1:2321/SomeUnreachableURL  **nav target**
curr-&gt;  http://127.0.0.1:2321/SomeUnreachableURL  **nav target**
===============================================</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>850167</commentid>
    <comment_count>25</comment_count>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2013-03-07 09:35:30 -0800</bug_when>
    <thetext>I know all of that, I don&apos;t know why we&apos;re still talking about this - I r+&apos;ed the patch days ago.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>850424</commentid>
    <comment_count>26</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2013-03-07 13:20:03 -0800</bug_when>
    <thetext>@fishd or @creis: Do you know why Chromium has a different result on this test?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>850594</commentid>
    <comment_count>27</comment_count>
    <who name="Charles Reis">creis</who>
    <bug_when>2013-03-07 16:18:56 -0800</bug_when>
    <thetext>(In reply to comment #26)
&gt; @fishd or @creis: Do you know why Chromium has a different result on this test?

I&apos;m not familiar with this code, but I pulled up the repro steps in a debugger.  In Chrome, we appear to hit the shouldReloadToHandleUnreachableURL(newDocumentLoader) check a few lines below this patch&apos;s change to FrameLoader::load, causing type to be set to FrameLoadTypeReload.  Maybe that differs with Safari?

I see a scary FIXME around that logic, though, so perhaps someone should revisit it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>853428</commentid>
    <comment_count>28</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-03-12 07:09:36 -0700</bug_when>
    <thetext>(In reply to comment #25)
&gt; I know all of that, I don&apos;t know why we&apos;re still talking about this - I r+&apos;ed the patch days ago.

&gt; 
Can you please give me a cq+ for the same...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>855207</commentid>
    <comment_count>29</comment_count>
    <who name="Basavaraj Padmashali Sidda">basavaraj.ps</who>
    <bug_when>2013-03-14 08:26:28 -0700</bug_when>
    <thetext>(In reply to comment #27)
&gt; (In reply to comment #26)
&gt; &gt; @fishd or @creis: Do you know why Chromium has a different result on this test?
&gt; 
&gt; I&apos;m not familiar with this code, but I pulled up the repro steps in a debugger.  In Chrome, we appear to hit the shouldReloadToHandleUnreachableURL(newDocumentLoader) check a few lines below this patch&apos;s change to FrameLoader::load, causing type to be set to FrameLoadTypeReload.  Maybe that differs with Safari?
&gt; 
This does not seem to be a causing the issue. Even on making  shouldReloadToHandleUnreachableURL() return always false(currently it return true in case of chromium) the test results did not change for chromium port.

&gt; I see a scary FIXME around that logic, though, so perhaps someone should revisit it.
&gt; 
Maybe the different results are due implementation difference in chromium DRT as compared to other ports (didFailProvisionalLoad() is not implemented in WebViewHost).</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>157623</attachid>
            <date>2012-08-09 20:26:11 -0700</date>
            <delta_ts>2012-08-09 20:36:52 -0700</delta_ts>
            <desc>Patch for back/forward list with error page</desc>
            <filename>FrameLoader.patch</filename>
            <type>text/plain</type>
            <size>499</size>
            <attacher name="Basavaraj Padmashali Sidda">basavarajaps</attacher>
            
              <data encoding="base64">SW5kZXg6IEZyYW1lTG9hZGVyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBGcmFtZUxvYWRlci5jcHAJKHJl
dmlzaW9uIDEyNTIzMSkKKysrIEZyYW1lTG9hZGVyLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTI4
MCw2ICsxMjgwLDggQEAKICAgICAgICAgdHlwZSA9IEZyYW1lTG9hZFR5cGVTYW1lOwogICAgIH0g
ZWxzZSBpZiAoc2hvdWxkVHJlYXRVUkxBc1NhbWVBc0N1cnJlbnQobmV3RG9jdW1lbnRMb2FkZXIt
PnVucmVhY2hhYmxlVVJMKCkpICYmIG1fbG9hZFR5cGUgPT0gRnJhbWVMb2FkVHlwZVJlbG9hZCkK
ICAgICAgICAgdHlwZSA9IEZyYW1lTG9hZFR5cGVSZWxvYWQ7CisgICAgZWxzZSBpZihpc0JhY2tG
b3J3YXJkTG9hZFR5cGUobV9sb2FkVHlwZSkpCisJdHlwZSA9IG1fbG9hZFR5cGU7CiAgICAgZWxz
ZQogICAgICAgICB0eXBlID0gRnJhbWVMb2FkVHlwZVN0YW5kYXJkOwogCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>157626</attachid>
            <date>2012-08-09 20:36:52 -0700</date>
            <delta_ts>2012-08-14 12:56:52 -0700</delta_ts>
            <desc>Updated proposed patch for back/forward list with error page</desc>
            <filename>FrameLoader.patch</filename>
            <type>text/plain</type>
            <size>506</size>
            <attacher name="Basavaraj Padmashali Sidda">basavarajaps</attacher>
            
              <data encoding="base64">SW5kZXg6IEZyYW1lTG9hZGVyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBGcmFtZUxvYWRlci5jcHAJKHJl
dmlzaW9uIDEyNTIzMSkKKysrIEZyYW1lTG9hZGVyLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTI4
MCw2ICsxMjgwLDggQEAKICAgICAgICAgdHlwZSA9IEZyYW1lTG9hZFR5cGVTYW1lOwogICAgIH0g
ZWxzZSBpZiAoc2hvdWxkVHJlYXRVUkxBc1NhbWVBc0N1cnJlbnQobmV3RG9jdW1lbnRMb2FkZXIt
PnVucmVhY2hhYmxlVVJMKCkpICYmIG1fbG9hZFR5cGUgPT0gRnJhbWVMb2FkVHlwZVJlbG9hZCkK
ICAgICAgICAgdHlwZSA9IEZyYW1lTG9hZFR5cGVSZWxvYWQ7CisgICAgZWxzZSBpZihpc0JhY2tG
b3J3YXJkTG9hZFR5cGUobV9sb2FkVHlwZSkpCisgICAgICAgIHR5cGUgPSBtX2xvYWRUeXBlOwog
ICAgIGVsc2UKICAgICAgICAgdHlwZSA9IEZyYW1lTG9hZFR5cGVTdGFuZGFyZDsKIAo=
</data>
<flag name="review"
          id="167836"
          type_id="1"
          status="-"
          setter="darin"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>176694</attachid>
            <date>2012-11-29 03:14:23 -0800</date>
            <delta_ts>2013-02-06 06:36:23 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-93680-20121129164656.patch</filename>
            <type>text/plain</type>
            <size>1547</size>
            <attacher name="Basavaraj Padmashali Sidda">basavaraj.ps</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTM2MTE1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYzhiMWI2ODhiOGViYjcy
NDdlZjE4ZmVjZjA1MWUyYjVmNDVlYWNlYS4uMmQxMmQ3MmIwOGFjZTlkMTRlYjdhZGI0YmM3M2Y3
MjFkZGQzMjJjYSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEyLTExLTI5ICBCYXNh
dmFyYWogUFMgIDxiYXNhdmFyYWoucHNAc2Ftc3VuZy5jb20+CisKKyAgICAgICAgQmFjay9Gb3J3
YXJkIGxpc3Qgd2l0aCBlcnJvciBwYWdlCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD05MzY4MAorCisgICAgICAgIEZpeCBmb3IgdGhlIG11bHRpcGxlIGhp
c3RvcnkgaXRlbXMgZm9yIGVycm9yIHBhZ2UsIHdoZW4gYmFjayBvciBmb3J3YXJkIGtleSBpcyBw
cmVzc2VkLiAKKyAgICAgICAgTXVsdGlwbGUgaGlzdG9yeSBpdGVtcyBhcmUgY3JlYXRlZCBkdWUg
dG8gbG9hZGluZyB0aGUgZXJyb3IgcGFnZSB3aXRoIGxvYWQgdHlwZSBhcyAKKyAgICAgICAgc3Rh
bmRhcmQgaW5zdGVhZCBvZiBiYWNrL2ZvcndhcmQgbG9hZCB0eXBlLgorCisgICAgICAgICogbG9h
ZGVyL0ZyYW1lTG9hZGVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkZyYW1lTG9hZGVyOjpsb2Fk
KToKKwogMjAxMi0xMS0yOSAgU2hlcmlmZiBCb3QgIDx3ZWJraXQucmV2aWV3LmJvdEBnbWFpbC5j
b20+CiAKICAgICAgICAgVW5yZXZpZXdlZCwgcm9sbGluZyBvdXQgcjEzNjExMS4KZGlmZiAtLWdp
dCBhL1NvdXJjZS9XZWJDb3JlL2xvYWRlci9GcmFtZUxvYWRlci5jcHAgYi9Tb3VyY2UvV2ViQ29y
ZS9sb2FkZXIvRnJhbWVMb2FkZXIuY3BwCm9sZCBtb2RlIDEwMDY0NApuZXcgbW9kZSAxMDA3NTUK
aW5kZXggMDM1MmIxZDhjYzI4OWNlMWU5NWI0YWI2MzFmZTU4ZTNhZDAyNmNkNC4uMWFlYjgzNTU1
ZjM5MzNkMzYzNjIxMzg1YWI3YWUwMDcxYjlkNDM4ZAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9sb2Fk
ZXIvRnJhbWVMb2FkZXIuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL2xvYWRlci9GcmFtZUxvYWRl
ci5jcHAKQEAgLTEzMDAsNiArMTMwMCw4IEBAIHZvaWQgRnJhbWVMb2FkZXI6OmxvYWQoRG9jdW1l
bnRMb2FkZXIqIG5ld0RvY3VtZW50TG9hZGVyKQogICAgICAgICB0eXBlID0gRnJhbWVMb2FkVHlw
ZVNhbWU7CiAgICAgfSBlbHNlIGlmIChzaG91bGRUcmVhdFVSTEFzU2FtZUFzQ3VycmVudChuZXdE
b2N1bWVudExvYWRlci0+dW5yZWFjaGFibGVVUkwoKSkgJiYgbV9sb2FkVHlwZSA9PSBGcmFtZUxv
YWRUeXBlUmVsb2FkKQogICAgICAgICB0eXBlID0gRnJhbWVMb2FkVHlwZVJlbG9hZDsKKyAgICBl
bHNlIGlmIChpc0JhY2tGb3J3YXJkTG9hZFR5cGUobV9sb2FkVHlwZSkpCisgICAgICAgIHR5cGUg
PSBtX2xvYWRUeXBlOwogICAgIGVsc2UKICAgICAgICAgdHlwZSA9IEZyYW1lTG9hZFR5cGVTdGFu
ZGFyZDsKIAo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>186842</attachid>
            <date>2013-02-06 06:36:30 -0800</date>
            <delta_ts>2013-02-13 02:15:07 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-93680-20130206201039.patch</filename>
            <type>text/plain</type>
            <size>5448</size>
            <attacher name="Basavaraj Padmashali Sidda">basavaraj.ps</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE0MTk5NSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIxIEBACisyMDEzLTAyLTA2ICBCYXNhdmFy
YWogUFMgIDxiYXNhdmFyYWoucHNAc2Ftc3VuZy5jb20+CisKKyAgICAgICAgQmFjay9Gb3J3YXJk
IGxpc3Qgd2l0aCBlcnJvciBwYWdlCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD05MzY4MAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgIEZpeCBmb3IgdGhlIG11bHRpcGxlIGhpc3RvcnkgaXRlbXMgZm9yIGVycm9y
IHBhZ2Ugd2hpY2ggaXMgbG9hZGVkIGFzIHN1YnN0aXR1dGUgZGF0YSwKKyAgICAgICAgd2hlbiBi
YWNrIG9yIGZvcndhcmQga2V5IGlzIHByZXNzZWQuIAorICAgICAgICBNdWx0aXBsZSBoaXN0b3J5
IGl0ZW1zIGFyZSBjcmVhdGVkIGR1ZSB0byBsb2FkaW5nIHRoZSBlcnJvciBwYWdlIHdpdGggbG9h
ZCB0eXBlIGFzIAorICAgICAgICBzdGFuZGFyZCBpbnN0ZWFkIG9mIGJhY2svZm9yd2FyZCBsb2Fk
IHR5cGUgZm9yIHVucmVhY2hhYmxlIFVSTHMuCisKKyAgICAgICAgVGVzdDogaHR0cC90ZXN0cy9u
YXZpZ2F0aW9uL2JhY2stbmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS5o
dG1sCisKKyAgICAgICAgKiBsb2FkZXIvRnJhbWVMb2FkZXIuY3BwOgorICAgICAgICAoV2ViQ29y
ZTo6RnJhbWVMb2FkZXI6OmxvYWQpOgorICAgICAgICAgIFJldHVybmluZyB0aGUgTG9hZCB0eXBl
IGluIGNhc2Ugb2YgQmFjay9Gb3J3YXJkIG5hdmlnYXRpb24gZm9yIHVucmVhY2hhYmxlIFVSTHMu
CisKIDIwMTMtMDItMDYgIEFuZHJlYXMgS2xpbmcgIDxha2xpbmdAYXBwbGUuY29tPgogCiAgICAg
ICAgIE9wdGltaXplIEdseXBoUGFnZSBmb3IgY2FzZSB3aGVyZSBhbGwgZ2x5cGhzIGFyZSBhdmFp
bGFibGUgaW4gdGhlIHNhbWUgZm9udC4KSW5kZXg6IFNvdXJjZS9XZWJDb3JlL2xvYWRlci9GcmFt
ZUxvYWRlci5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvbG9hZGVyL0ZyYW1lTG9h
ZGVyLmNwcAkocmV2aXNpb24gMTQxODYyKQorKysgU291cmNlL1dlYkNvcmUvbG9hZGVyL0ZyYW1l
TG9hZGVyLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTMxNSw2ICsxMzE1LDggQEAgdm9pZCBGcmFt
ZUxvYWRlcjo6bG9hZChEb2N1bWVudExvYWRlciogbgogICAgICAgICB0eXBlID0gRnJhbWVMb2Fk
VHlwZVNhbWU7CiAgICAgfSBlbHNlIGlmIChzaG91bGRUcmVhdFVSTEFzU2FtZUFzQ3VycmVudChu
ZXdEb2N1bWVudExvYWRlci0+dW5yZWFjaGFibGVVUkwoKSkgJiYgbV9sb2FkVHlwZSA9PSBGcmFt
ZUxvYWRUeXBlUmVsb2FkKQogICAgICAgICB0eXBlID0gRnJhbWVMb2FkVHlwZVJlbG9hZDsKKyAg
ICBlbHNlIGlmICghbmV3RG9jdW1lbnRMb2FkZXItPnVucmVhY2hhYmxlVVJMKCkuaXNFbXB0eSgp
ICYmIGlzQmFja0ZvcndhcmRMb2FkVHlwZShtX2xvYWRUeXBlKSkKKyAgICAgICAgdHlwZSA9IG1f
bG9hZFR5cGU7CiAgICAgZWxzZQogICAgICAgICB0eXBlID0gRnJhbWVMb2FkVHlwZVN0YW5kYXJk
OwogCkluZGV4OiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVz
dHMvQ2hhbmdlTG9nCShyZXZpc2lvbiAxNDE5OTUpCisrKyBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJ
KHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxMyBAQAorMjAxMy0wMi0wNiAgQmFzYXZhcmFqIFBT
ICA8YmFzYXZhcmFqLnBzQHNhbXN1bmcuY29tPgorCisgICAgICAgIEJhY2svRm9yd2FyZCBsaXN0
IHdpdGggZXJyb3IgcGFnZQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9OTM2ODAKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICAqIGh0dHAvdGVzdHMvbmF2aWdhdGlvbi9iYWNrLW5hdmlnYXRpb24tdG8tZXJyb3It
cGFnZS1zdWJzdGl0dXRlLWRhdGEtZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBodHRw
L3Rlc3RzL25hdmlnYXRpb24vYmFjay1uYXZpZ2F0aW9uLXRvLWVycm9yLXBhZ2Utc3Vic3RpdHV0
ZS1kYXRhLmh0bWw6IEFkZGVkLgorCiAyMDEzLTAyLTA2ICBNYXJqYSBIw7ZsdHTDpCAgPG1hcmph
QGNocm9taXVtLm9yZz4KIAogICAgICAgICBbV0syXSBOZXcgdGVzdHMgaW50cm9kdWNlZCBpbiBy
MTAwODk1IGZhaWwKSW5kZXg6IExheW91dFRlc3RzL2h0dHAvdGVzdHMvbmF2aWdhdGlvbi9iYWNr
LW5hdmlnYXRpb24tdG8tZXJyb3ItcGFnZS1zdWJzdGl0dXRlLWRhdGEtZXhwZWN0ZWQudHh0Cj09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT0KLS0tIExheW91dFRlc3RzL2h0dHAvdGVzdHMvbmF2aWdhdGlvbi9iYWNrLW5hdmln
YXRpb24tdG8tZXJyb3ItcGFnZS1zdWJzdGl0dXRlLWRhdGEtZXhwZWN0ZWQudHh0CShyZXZpc2lv
biAwKQorKysgTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9uYXZpZ2F0aW9uL2JhY2stbmF2aWdhdGlv
bi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS1leHBlY3RlZC50eHQJKHJldmlzaW9uIDAp
CkBAIC0wLDAgKzEsOCBAQAorVGVzdGNhc2UgZm9yIEJ1ZyBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9OTM2ODAuIAorVGhlIEJhY2sgRm9yd2FyZCBMaXN0IGJlbG93IHNo
b3VsZCBub3QgY29udGFpbiBtdWx0aXBsZSBlbnRyaWVzIG9mIGh0dHA6Ly8xMjcuMC4wLjE6MjMy
MS9Tb21lVW5yZWFjaGFibGVVUkwuCisKKz09PT09PT09PT09PT09IEJhY2sgRm9yd2FyZCBMaXN0
ID09PT09PT09PT09PT09CisgICAgICAgIGh0dHA6Ly8xMjcuMC4wLjE6ODAwMC9uYXZpZ2F0aW9u
L2JhY2stbmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS5odG1sICAqKm5h
diB0YXJnZXQqKgorY3Vyci0+ICBodHRwOi8vMTI3LjAuMC4xOjIzMjEvU29tZVVucmVhY2hhYmxl
VVJMICAqKm5hdiB0YXJnZXQqKgorICAgICAgICBodHRwOi8vMTI3LjAuMC4xOjgwMDAvbmF2aWdh
dGlvbi9yZXNvdXJjZXMvcGFnZS10by1nby1iYWNrLWZyb20uaHRtbCAgKipuYXYgdGFyZ2V0KioK
Kz09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09CkluZGV4OiBM
YXlvdXRUZXN0cy9odHRwL3Rlc3RzL25hdmlnYXRpb24vYmFjay1uYXZpZ2F0aW9uLXRvLWVycm9y
LXBhZ2Utc3Vic3RpdHV0ZS1kYXRhLmh0bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvaHR0
cC90ZXN0cy9uYXZpZ2F0aW9uL2JhY2stbmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1
dGUtZGF0YS5odG1sCShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9uYXZp
Z2F0aW9uL2JhY2stbmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS5odG1s
CShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDI2IEBACis8IURPQ1RZUEUgaHRtbD4KKzxodG1sPgor
PGhlYWQ+Cis8c2NyaXB0PgorZnVuY3Rpb24gcnVuVGVzdCgpCit7CisgICAgaWYgKCF3aW5kb3cu
dGVzdFJ1bm5lcikKKyAgICAgICAgcmV0dXJuOworCisgICAgdGVzdFJ1bm5lci5kdW1wQXNUZXh0
KCk7CisgICAgdGVzdFJ1bm5lci5kdW1wQmFja0ZvcndhcmRMaXN0KCk7CisKKyAgICB0ZXN0UnVu
bmVyLnF1ZXVlTG9hZEhUTUxTdHJpbmcoIjxzY3JpcHQ+ZG9jdW1lbnQud3JpdGUoJzxkaXYgaWQ9
cmVzdWx0PlN1YnN0aXR1dGUgRGF0YSBmb3IgdGhlIHVucmVhY2hhYmxlIFVSTDwvZGl2PicpOzxc
L3NjcmlwdD4iLCAicmVzb3VyY2VzIiwgImh0dHA6Ly8xMjcuMC4wLjE6MjMyMS9Tb21lVW5yZWFj
aGFibGVVUkwiKTsKKyAgICB0ZXN0UnVubmVyLnF1ZXVlTG9hZCgicmVzb3VyY2VzL3BhZ2UtdG8t
Z28tYmFjay1mcm9tLmh0bWwiKTsKKyAgICB0ZXN0UnVubmVyLnF1ZXVlQmFja05hdmlnYXRpb24o
MSk7CisgICAgdGVzdFJ1bm5lci5xdWV1ZUxvYWRIVE1MU3RyaW5nKCI8c2NyaXB0PmRvY3VtZW50
LndyaXRlKCc8ZGl2IGlkPXJlc3VsdD4gVGVzdGNhc2UgZm9yIEJ1ZyBodHRwczovL2J1Z3Mud2Vi
a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTM2ODAuIDxicj5UaGUgQmFjayBGb3J3YXJkIExpc3Qg
YmVsb3cgc2hvdWxkIG5vdCBjb250YWluIG11bHRpcGxlIGVudHJpZXMgb2YgaHR0cDovLzEyNy4w
LjAuMToyMzIxL1NvbWVVbnJlYWNoYWJsZVVSTC4gPC9kaXY+Jyk7PFwvc2NyaXB0PiIsICJyZXNv
dXJjZXMiLCAiaHR0cDovLzEyNy4wLjAuMToyMzIxL1NvbWVVbnJlYWNoYWJsZVVSTCIpOworfQor
PC9zY3JpcHQ+Cis8L2hlYWQ+Cis8Ym9keSBvbmxvYWQ9InJ1blRlc3QoKTsiPgorPGgzPiBUZXN0
Y2FzZSBmb3IgQnVnIDxhIGhyZWY9Imh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD05MzY4MCI+IDkzNjgwPC9hPjwvaDM+Cis8cD5UaGlzIHRlc3QgY2FzZSBpcyB0byBjaGVj
ayB0aGUgY29ycmVjdCBiZWhhdmlvdXIgd2hlbiBvZiBCYWNrL0ZvcndhcmQgbmF2aWdhdGlvbiB3
aGVuIGFuIHVucmVhY2hhYmxlIFVSTCBpcyBlbmNvdW50ZXJlZC4KK1doZW4gYSBsb2FkIHRvIGFu
IHVucmVhY2hhYmxlIHVybCBmYWlscyBhbmQgYSBIVE1MIHN0cmluZyBpcyBsb2FkZWQgYXMgc3Vi
c3RpdHV0ZSBkYXRhIGZvciB0aGUgc2FtZSwgYSBoaXN0b3J5IEl0ZW0gaXMgY3JlYXRlZC4gQWZ0
ZXIgYW5vdGhlciBmZXRjaCB0byBhIHZhbGlkIFVSTCwgd2hlbiB3ZSBnZXQgYmFjayB0byB0aGUg
dW5yZWFjaGFibGUgVVJMKFNvbWVVbnJlYWNoYWJsZVVSTCkgdXNpbmcgQmFjayBuYXZpZ2F0aW9u
LCB0aGUgdW5yZWFjaGFibGUgVVJMKFNvbWVVbnJlYWNoYWJsZVVSTCkgaXMgZmV0Y2hlZCBhZ2Fp
bihmYWlscykgYW5kIHdoZW4gYSBIVE1MIHN0cmluZyBpcyBsb2FkZWQgYXMgc3Vic3RpdHV0ZSBk
YXRhIGZvciBzYW1lIGEgbmV3IGhpc3RvcnkgSXRlbSBpcyBjcmVhdGVkLiBIb3dldmVyIGhpc3Rv
cnkgSXRlbSBjcmVhdGVkIHdoZW4gZmV0Y2hpbmcgdGhlIHVucmVhY2hhYmxlIFVSTChTb21lVW5y
ZWFjaGFibGVVUkwpIHNob3VsZCBiZSByZS11c2VkIGluIGNhc2Ugb2YgQmFjay9Gb3dhcmQgbmF2
aWdhdGlvbi4KKzwvcD4KKzwvYm9keT4KKzwvaHRtbD4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>188035</attachid>
            <date>2013-02-13 02:15:13 -0800</date>
            <delta_ts>2022-03-14 16:27:35 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-93680-20130213211139.patch</filename>
            <type>text/plain</type>
            <size>6528</size>
            <attacher name="Basavaraj Padmashali Sidda">basavaraj.ps</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE0MjczMCkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIxIEBACisyMDEzLTAyLTEzICBCYXNhdmFy
YWogUFMgIDxiYXNhdmFyYWoucHNAc2Ftc3VuZy5jb20+CisKKyAgICAgICAgQmFjay9Gb3J3YXJk
IGxpc3Qgd2l0aCBlcnJvciBwYWdlCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD05MzY4MAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgIEZpeCBmb3IgdGhlIG11bHRpcGxlIGhpc3RvcnkgaXRlbXMgZm9yIGVycm9y
IHBhZ2Ugd2hpY2ggaXMgbG9hZGVkIGFzIHN1YnN0aXR1dGUgZGF0YSwKKyAgICAgICAgd2hlbiBi
YWNrIG9yIGZvcndhcmQga2V5IGlzIHByZXNzZWQuCisgICAgICAgIE11bHRpcGxlIGhpc3Rvcnkg
aXRlbXMgYXJlIGNyZWF0ZWQgZHVlIHRvIGxvYWRpbmcgdGhlIGVycm9yIHBhZ2Ugd2l0aCBsb2Fk
IHR5cGUgYXMKKyAgICAgICAgc3RhbmRhcmQgaW5zdGVhZCBvZiBiYWNrL2ZvcndhcmQgbG9hZCB0
eXBlIGZvciB1bnJlYWNoYWJsZSBVUkxzLiAKKworICAgICAgICBUZXN0OiBodHRwL3Rlc3RzL25h
dmlnYXRpb24vYmFjay1uYXZpZ2F0aW9uLXRvLWVycm9yLXBhZ2Utc3Vic3RpdHV0ZS1kYXRhLmh0
bWwKKworICAgICAgICAqIGxvYWRlci9GcmFtZUxvYWRlci5jcHA6CisgICAgICAgIChXZWJDb3Jl
OjpGcmFtZUxvYWRlcjo6bG9hZCk6CisgICAgICAgICBSZXR1cm5pbmcgdGhlIExvYWQgdHlwZSBp
biBjYXNlIG9mIEJhY2svRm9yd2FyZCBuYXZpZ2F0aW9uIGZvciB1bnJlYWNoYWJsZSBVUkxzLgor
CiAyMDEzLTAyLTEyICBLZW50YXJvIEhhcmEgIDxoYXJha2VuQGNocm9taXVtLm9yZz4KIAogICAg
ICAgICBbVjhdIEdlbmVyYXRlIHdyYXBwZXIgbWV0aG9kcyBmb3IgY3VzdG9tIGdldHRlcnMvc2V0
dGVycwpJbmRleDogU291cmNlL1dlYkNvcmUvbG9hZGVyL0ZyYW1lTG9hZGVyLmNwcAo9PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9sb2FkZXIvRnJhbWVMb2FkZXIuY3BwCShyZXZpc2lvbiAx
NDIwMTEpCisrKyBTb3VyY2UvV2ViQ29yZS9sb2FkZXIvRnJhbWVMb2FkZXIuY3BwCSh3b3JraW5n
IGNvcHkpCkBAIC0xMzE1LDYgKzEzMTUsOCBAQCB2b2lkIEZyYW1lTG9hZGVyOjpsb2FkKERvY3Vt
ZW50TG9hZGVyKiBuCiAgICAgICAgIHR5cGUgPSBGcmFtZUxvYWRUeXBlU2FtZTsKICAgICB9IGVs
c2UgaWYgKHNob3VsZFRyZWF0VVJMQXNTYW1lQXNDdXJyZW50KG5ld0RvY3VtZW50TG9hZGVyLT51
bnJlYWNoYWJsZVVSTCgpKSAmJiBtX2xvYWRUeXBlID09IEZyYW1lTG9hZFR5cGVSZWxvYWQpCiAg
ICAgICAgIHR5cGUgPSBGcmFtZUxvYWRUeXBlUmVsb2FkOworICAgIGVsc2UgaWYgKCFuZXdEb2N1
bWVudExvYWRlci0+dW5yZWFjaGFibGVVUkwoKS5pc0VtcHR5KCkgJiYgaXNCYWNrRm9yd2FyZExv
YWRUeXBlKG1fbG9hZFR5cGUpKQorICAgICAgICB0eXBlID0gbV9sb2FkVHlwZTsKICAgICBlbHNl
CiAgICAgICAgIHR5cGUgPSBGcmFtZUxvYWRUeXBlU3RhbmRhcmQ7CiAKSW5kZXg6IExheW91dFRl
c3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlz
aW9uIDE0MjczMCkKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAt
MSwzICsxLDE0IEBACisyMDEzLTAyLTEzICBCYXNhdmFyYWogUFMgIDxiYXNhdmFyYWoucHNAc2Ft
c3VuZy5jb20+CisKKyAgICAgICAgQmFjay9Gb3J3YXJkIGxpc3Qgd2l0aCBlcnJvciBwYWdlCisg
ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD05MzY4MAorCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogaHR0cC90ZXN0
cy9uYXZpZ2F0aW9uL2JhY2stbmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0
YS1leHBlY3RlZC50eHQ6IEFkZGVkLgorICAgICAgICAqIGh0dHAvdGVzdHMvbmF2aWdhdGlvbi9i
YWNrLW5hdmlnYXRpb24tdG8tZXJyb3ItcGFnZS1zdWJzdGl0dXRlLWRhdGEuaHRtbDogQWRkZWQu
CisgICAgICAgICogcGxhdGZvcm0vY2hyb21pdW0vaHR0cC90ZXN0cy9uYXZpZ2F0aW9uL2JhY2st
bmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS1leHBlY3RlZC50eHQ6IEFk
ZGVkLgorCiAyMDEzLTAyLTEzICDDgWTDoW0gS2FsbGFpICA8a2FkYW1AaW5mLnUtc3plZ2VkLmh1
PgogCiAgICAgICAgIFtRdF0gVW5yZXZpZXdlZCBnYXJkZW5pbmcuIFNraXAgZmFpbGluZyB0ZXN0
cy4KSW5kZXg6IExheW91dFRlc3RzL2h0dHAvdGVzdHMvbmF2aWdhdGlvbi9iYWNrLW5hdmlnYXRp
b24tdG8tZXJyb3ItcGFnZS1zdWJzdGl0dXRlLWRhdGEtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
LS0tIExheW91dFRlc3RzL2h0dHAvdGVzdHMvbmF2aWdhdGlvbi9iYWNrLW5hdmlnYXRpb24tdG8t
ZXJyb3ItcGFnZS1zdWJzdGl0dXRlLWRhdGEtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysg
TGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9uYXZpZ2F0aW9uL2JhY2stbmF2aWdhdGlvbi10by1lcnJv
ci1wYWdlLXN1YnN0aXR1dGUtZGF0YS1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCkBAIC0wLDAg
KzEsOCBAQAorVGVzdGNhc2UgZm9yIEJ1ZyBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9OTM2ODAuIAorVGhlIEJhY2sgRm9yd2FyZCBMaXN0IGJlbG93IHNob3VsZCBub3Qg
Y29udGFpbiBtdWx0aXBsZSBlbnRyaWVzIG9mIGh0dHA6Ly8xMjcuMC4wLjE6MjMyMS9Tb21lVW5y
ZWFjaGFibGVVUkwuCisKKz09PT09PT09PT09PT09IEJhY2sgRm9yd2FyZCBMaXN0ID09PT09PT09
PT09PT09CisgICAgICAgIGh0dHA6Ly8xMjcuMC4wLjE6ODAwMC9uYXZpZ2F0aW9uL2JhY2stbmF2
aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS5odG1sICAqKm5hdiB0YXJnZXQq
KgorY3Vyci0+ICBodHRwOi8vMTI3LjAuMC4xOjIzMjEvU29tZVVucmVhY2hhYmxlVVJMICAqKm5h
diB0YXJnZXQqKgorICAgICAgICBodHRwOi8vMTI3LjAuMC4xOjgwMDAvbmF2aWdhdGlvbi9yZXNv
dXJjZXMvcGFnZS10by1nby1iYWNrLWZyb20uaHRtbCAgKipuYXYgdGFyZ2V0KioKKz09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09CkluZGV4OiBMYXlvdXRUZXN0
cy9odHRwL3Rlc3RzL25hdmlnYXRpb24vYmFjay1uYXZpZ2F0aW9uLXRvLWVycm9yLXBhZ2Utc3Vi
c3RpdHV0ZS1kYXRhLmh0bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9u
YXZpZ2F0aW9uL2JhY2stbmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS5o
dG1sCShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9uYXZpZ2F0aW9uL2Jh
Y2stbmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS5odG1sCShyZXZpc2lv
biAwKQpAQCAtMCwwICsxLDI2IEBACis8IURPQ1RZUEUgaHRtbD4KKzxodG1sPgorPGhlYWQ+Cis8
c2NyaXB0PgorZnVuY3Rpb24gcnVuVGVzdCgpCit7CisgICAgaWYgKCF3aW5kb3cudGVzdFJ1bm5l
cikKKyAgICAgICAgcmV0dXJuOworCisgICAgdGVzdFJ1bm5lci5kdW1wQXNUZXh0KCk7CisgICAg
dGVzdFJ1bm5lci5kdW1wQmFja0ZvcndhcmRMaXN0KCk7CisKKyAgICB0ZXN0UnVubmVyLnF1ZXVl
TG9hZEhUTUxTdHJpbmcoIjxzY3JpcHQ+ZG9jdW1lbnQud3JpdGUoJzxkaXYgaWQ9cmVzdWx0PlN1
YnN0aXR1dGUgRGF0YSBmb3IgdGhlIHVucmVhY2hhYmxlIFVSTDwvZGl2PicpOzxcL3NjcmlwdD4i
LCAicmVzb3VyY2VzIiwgImh0dHA6Ly8xMjcuMC4wLjE6MjMyMS9Tb21lVW5yZWFjaGFibGVVUkwi
KTsKKyAgICB0ZXN0UnVubmVyLnF1ZXVlTG9hZCgicmVzb3VyY2VzL3BhZ2UtdG8tZ28tYmFjay1m
cm9tLmh0bWwiKTsKKyAgICB0ZXN0UnVubmVyLnF1ZXVlQmFja05hdmlnYXRpb24oMSk7CisgICAg
dGVzdFJ1bm5lci5xdWV1ZUxvYWRIVE1MU3RyaW5nKCI8c2NyaXB0PmRvY3VtZW50LndyaXRlKCc8
ZGl2IGlkPXJlc3VsdD4gVGVzdGNhc2UgZm9yIEJ1ZyBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9OTM2ODAuIDxicj5UaGUgQmFjayBGb3J3YXJkIExpc3QgYmVsb3cgc2hv
dWxkIG5vdCBjb250YWluIG11bHRpcGxlIGVudHJpZXMgb2YgaHR0cDovLzEyNy4wLjAuMToyMzIx
L1NvbWVVbnJlYWNoYWJsZVVSTC4gPC9kaXY+Jyk7PFwvc2NyaXB0PiIsICJyZXNvdXJjZXMiLCAi
aHR0cDovLzEyNy4wLjAuMToyMzIxL1NvbWVVbnJlYWNoYWJsZVVSTCIpOworfQorPC9zY3JpcHQ+
Cis8L2hlYWQ+Cis8Ym9keSBvbmxvYWQ9InJ1blRlc3QoKTsiPgorPGgzPiBUZXN0Y2FzZSBmb3Ig
QnVnIDxhIGhyZWY9Imh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD05MzY4
MCI+IDkzNjgwPC9hPjwvaDM+Cis8cD5UaGlzIHRlc3QgY2FzZSBpcyB0byBjaGVjayB0aGUgY29y
cmVjdCBiZWhhdmlvdXIgd2hlbiBvZiBCYWNrL0ZvcndhcmQgbmF2aWdhdGlvbiB3aGVuIGFuIHVu
cmVhY2hhYmxlIFVSTCBpcyBlbmNvdW50ZXJlZC4KK1doZW4gYSBsb2FkIHRvIGFuIHVucmVhY2hh
YmxlIHVybCBmYWlscyBhbmQgYSBIVE1MIHN0cmluZyBpcyBsb2FkZWQgYXMgc3Vic3RpdHV0ZSBk
YXRhIGZvciB0aGUgc2FtZSwgYSBoaXN0b3J5IEl0ZW0gaXMgY3JlYXRlZC4gQWZ0ZXIgYW5vdGhl
ciBmZXRjaCB0byBhIHZhbGlkIFVSTCwgd2hlbiB3ZSBnZXQgYmFjayB0byB0aGUgdW5yZWFjaGFi
bGUgVVJMKFNvbWVVbnJlYWNoYWJsZVVSTCkgdXNpbmcgQmFjayBuYXZpZ2F0aW9uLCB0aGUgdW5y
ZWFjaGFibGUgVVJMKFNvbWVVbnJlYWNoYWJsZVVSTCkgaXMgZmV0Y2hlZCBhZ2FpbihmYWlscykg
YW5kIHdoZW4gYSBIVE1MIHN0cmluZyBpcyBsb2FkZWQgYXMgc3Vic3RpdHV0ZSBkYXRhIGZvciBz
YW1lIGEgbmV3IGhpc3RvcnkgSXRlbSBpcyBjcmVhdGVkLiBIb3dldmVyIGhpc3RvcnkgSXRlbSBj
cmVhdGVkIHdoZW4gZmV0Y2hpbmcgdGhlIHVucmVhY2hhYmxlIFVSTChTb21lVW5yZWFjaGFibGVV
UkwpIHNob3VsZCBiZSByZS11c2VkIGluIGNhc2Ugb2YgQmFjay9Gb3dhcmQgbmF2aWdhdGlvbi4K
KzwvcD4KKzwvYm9keT4KKzwvaHRtbD4KSW5kZXg6IExheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9t
aXVtL2h0dHAvdGVzdHMvbmF2aWdhdGlvbi9iYWNrLW5hdmlnYXRpb24tdG8tZXJyb3ItcGFnZS1z
dWJzdGl0dXRlLWRhdGEtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL3Bs
YXRmb3JtL2Nocm9taXVtL2h0dHAvdGVzdHMvbmF2aWdhdGlvbi9iYWNrLW5hdmlnYXRpb24tdG8t
ZXJyb3ItcGFnZS1zdWJzdGl0dXRlLWRhdGEtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysg
TGF5b3V0VGVzdHMvcGxhdGZvcm0vY2hyb21pdW0vaHR0cC90ZXN0cy9uYXZpZ2F0aW9uL2JhY2st
bmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS1leHBlY3RlZC50eHQJKHJl
dmlzaW9uIDApCkBAIC0wLDAgKzEsOCBAQAorVGVzdGNhc2UgZm9yIEJ1ZyBodHRwczovL2J1Z3Mu
d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTM2ODAuIAorVGhlIEJhY2sgRm9yd2FyZCBMaXN0
IGJlbG93IHNob3VsZCBub3QgY29udGFpbiBtdWx0aXBsZSBlbnRyaWVzIG9mIGh0dHA6Ly8xMjcu
MC4wLjE6MjMyMS9Tb21lVW5yZWFjaGFibGVVUkwuCisKKz09PT09PT09PT09PT09IEJhY2sgRm9y
d2FyZCBMaXN0ID09PT09PT09PT09PT09CisgICAgICAgIGh0dHA6Ly8xMjcuMC4wLjE6ODAwMC9u
YXZpZ2F0aW9uL2JhY2stbmF2aWdhdGlvbi10by1lcnJvci1wYWdlLXN1YnN0aXR1dGUtZGF0YS5o
dG1sICAqKm5hdiB0YXJnZXQqKgorICAgICAgICBodHRwOi8vMTI3LjAuMC4xOjIzMjEvU29tZVVu
cmVhY2hhYmxlVVJMICAqKm5hdiB0YXJnZXQqKgorY3Vyci0+ICBodHRwOi8vMTI3LjAuMC4xOjIz
MjEvU29tZVVucmVhY2hhYmxlVVJMICAqKm5hdiB0YXJnZXQqKgorPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
</data>
<flag name="review"
          id="208032"
          type_id="1"
          status="+"
          setter="beidson"
    />
          </attachment>
      

    </bug>

</bugzilla>