<?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>238556</bug_id>
          
          <creation_ts>2022-03-30 10:07:28 -0700</creation_ts>
          <short_desc>Turn DEVELOPER_MODE ON for all non-Apple ports in build-webkit</short_desc>
          <delta_ts>2022-03-30 19:38:17 -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>Tools / Tests</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Don Olmstead">don.olmstead</reporter>
          <assigned_to name="Don Olmstead">don.olmstead</assigned_to>
          <cc>achristensen</cc>
    
    <cc>annulen</cc>
    
    <cc>bfulgham</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>jbedard</cc>
    
    <cc>pvollan</cc>
    
    <cc>ryuan.choi</cc>
    
    <cc>sergio</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1856428</commentid>
    <comment_count>0</comment_count>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2022-03-30 10:07:28 -0700</bug_when>
    <thetext>PlayStation isn&apos;t building with it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856460</commentid>
    <comment_count>1</comment_count>
      <attachid>456146</attachid>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2022-03-30 10:49:44 -0700</bug_when>
    <thetext>Created attachment 456146
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856462</commentid>
    <comment_count>2</comment_count>
      <attachid>456146</attachid>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2022-03-30 10:51:06 -0700</bug_when>
    <thetext>Comment on attachment 456146
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=456146&amp;action=review

&gt; Tools/Scripts/webkitdirs.pm:2565
&gt;      # Some ports have production mode, but build-webkit should always use developer mode.
&gt; -    push @args, &quot;-DDEVELOPER_MODE=ON&quot; if isGtk() || isJSCOnly() || isWPE() || isWin64();
&gt; +    push @args, &quot;-DDEVELOPER_MODE=ON&quot; unless isAppleWebKit();

Originally this seemed to want to be turned ON for non-AppleWin ports. I&apos;m really not sure if this should still be the case. I&apos;m leaning towards it being on for all CMake ports but wanted to get an opinion from Apple before modifying it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856469</commentid>
    <comment_count>3</comment_count>
      <attachid>456146</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2022-03-30 10:55:04 -0700</bug_when>
    <thetext>Comment on attachment 456146
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=456146&amp;action=review

&gt;&gt; Tools/Scripts/webkitdirs.pm:2565
&gt;&gt; +    push @args, &quot;-DDEVELOPER_MODE=ON&quot; unless isAppleWebKit();
&gt; 
&gt; Originally this seemed to want to be turned ON for non-AppleWin ports. I&apos;m really not sure if this should still be the case. I&apos;m leaning towards it being on for all CMake ports but wanted to get an opinion from Apple before modifying it.

No strong opinion here.  This seems fine.  I wonder if we could work to remove DEVELOPER_MODE</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856479</commentid>
    <comment_count>4</comment_count>
      <attachid>456146</attachid>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2022-03-30 11:03:01 -0700</bug_when>
    <thetext>Comment on attachment 456146
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=456146&amp;action=review

&gt;&gt;&gt; Tools/Scripts/webkitdirs.pm:2565
&gt;&gt;&gt; +    push @args, &quot;-DDEVELOPER_MODE=ON&quot; unless isAppleWebKit();
&gt;&gt; 
&gt;&gt; Originally this seemed to want to be turned ON for non-AppleWin ports. I&apos;m really not sure if this should still be the case. I&apos;m leaning towards it being on for all CMake ports but wanted to get an opinion from Apple before modifying it.
&gt; 
&gt; No strong opinion here.  This seems fine.  I wonder if we could work to remove DEVELOPER_MODE

If there&apos;s no historical reason for not turning `DEVELOPER_MODE` `ON` for AppleWin then I think we should just turn it `ON` for all here so the above comment is actually true.

We could possibly roll `DEVELOPER_MODE` and `ENABLE_EXPERIMENTAL_FEATURES` together. For Igalia ports `DEVELOPER_MODE=OFF` seems to signal a production build for them. I&apos;m not sure if you&apos;d ever want to do `DEVELOPER_MODE=OFF` and `ENABLE_EXPERIMENTAL_FEATURES=ON`. I would guess there would need to be a transition period with that though.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856480</commentid>
    <comment_count>5</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2022-03-30 11:04:04 -0700</bug_when>
    <thetext>I think we should be working towards removing DEVELOPER_MODE and ENABLE_EXPERIMENTAL_FEATURES and having development be more like production.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856493</commentid>
    <comment_count>6</comment_count>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2022-03-30 11:28:12 -0700</bug_when>
    <thetext>(In reply to Alex Christensen from comment #5)
&gt; I think we should be working towards removing DEVELOPER_MODE and
&gt; ENABLE_EXPERIMENTAL_FEATURES and having development be more like production.

My understanding is the Igalia releases use DEVELOPER_MODE=OFF. In those cases it doesn&apos;t build any testing bits or the MiniBrowser. Then they make their browser off of that.

So I&apos;ll propose we add in the DEVELOPER_MODE=ON for all ports. Then get the PlayStation port on that, which is my main concern. Then we can talk to Igalians about their release and hopefully merge ENABLE_EXPERIMENTAL_FEATURES into DEVELOPER_MODE. Then we can figure out if there&apos;s a way to get us off DEVELOPER_MDOE completely.

I&apos;m not sure everyone&apos;s requirements for a production build so it might need to stick around completely. For our builds I&apos;d like to minimize the number of functions exported and do a full LTO.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856497</commentid>
    <comment_count>7</comment_count>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2022-03-30 11:36:59 -0700</bug_when>
    <thetext>Also from grepping the code I do see uses of ENABLE(DEVELOPER_MODE) and ENABLE(EXPERIMENTAL_FEATURES).

ENABLE(DEVELOPER_MODE) appears in a number of .h/.cpp files. ENABLE(EXPERIMENTAL_FEATURES) only appears in a single .cpp file BUT is used in two WebPrefences yaml files.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856501</commentid>
    <comment_count>8</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2022-03-30 11:39:22 -0700</bug_when>
    <thetext>EXPERIMENTAL_FEATURES is on in SafariTechnologyPreview and off everywhere else in Apple&apos;s ecosystem.  We discussed it a few months ago and decided it is better to have SafariTechnologyPreview more similar to everything else.

The more differences between DEVELOPER_MODE=ON and DEVELOPER_MODE=OFF there are, the more likely it is that a bug in some obscure configuration will go unfixed because no developers noticed it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856509</commentid>
    <comment_count>9</comment_count>
      <attachid>456159</attachid>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2022-03-30 11:54:55 -0700</bug_when>
    <thetext>Created attachment 456159
Patch enabling all</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856510</commentid>
    <comment_count>10</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2022-03-30 11:56:17 -0700</bug_when>
    <thetext>We&apos;ll never be able to get rid of DEVELOPER_MODE because some things just need to be different for developer builds vs. real builds. E.g. if you&apos;re a developer you want to load the injected bundle from your build directory instead of the install prefix, otherwise there&apos;s no way it will work. But in releases we need to load the installed injected bundle. Another example: in developer builds we need to mount the build directory in the sandbox. For a more recent example, in bug #237107 where clopez needs to run a suboptimal codepath for WPT tests that we don&apos;t want to run in production.

Some cases look suspicious, though. E.g. I see we have Xvfb-related code guarded by DEVELOPER_MODE. I doubt those guards are truly needed.

Removing EXPERIMENTAL_FEATURES is probably possible: that&apos;s just a matter of replacing build guards with runtime guards.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856512</commentid>
    <comment_count>11</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2022-03-30 12:04:09 -0700</bug_when>
    <thetext>Apple developers use DYLD_FRAMEWORK_PATH so habitually and it works so well that I think we don&apos;t want ENABLE(DEVELOPER_MODE) on Apple platforms.  I understand if it might be useful for linux developers, but we should keep it minimal.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856515</commentid>
    <comment_count>12</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2022-03-30 12:11:37 -0700</bug_when>
    <thetext>I&apos;m removing EXPERIMENTAL_FEATURES in https://bugs.webkit.org/show_bug.cgi?id=238565 and would appreciate some input from Linux devs because I&apos;m not familiar with its use on Linux.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856538</commentid>
    <comment_count>13</comment_count>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2022-03-30 12:57:58 -0700</bug_when>
    <thetext>(In reply to Alex Christensen from comment #12)
&gt; I&apos;m removing EXPERIMENTAL_FEATURES in
&gt; https://bugs.webkit.org/show_bug.cgi?id=238565 and would appreciate some
&gt; input from Linux devs because I&apos;m not familiar with its use on Linux.

Ok if that&apos;s the plan can we get a review on this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856645</commentid>
    <comment_count>14</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-03-30 19:37:37 -0700</bug_when>
    <thetext>Committed r292137 (249044@main): &lt;https://commits.webkit.org/249044@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 456146.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1856646</commentid>
    <comment_count>15</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-03-30 19:38:17 -0700</bug_when>
    <thetext>&lt;rdar://problem/91080551&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>456146</attachid>
            <date>2022-03-30 10:49:44 -0700</date>
            <delta_ts>2022-03-30 19:37:40 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-238556.diff</filename>
            <type>text/plain</type>
            <size>3529</size>
            <attacher name="Don Olmstead">don.olmstead</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0NoYW5nZUxvZyBiL0NoYW5nZUxvZwppbmRleCBjNjQ0MDg5MmNiNjcuLjQz
YjUyNTJjZDY2YyAxMDA2NDQKLS0tIGEvQ2hhbmdlTG9nCisrKyBiL0NoYW5nZUxvZwpAQCAtMSwz
ICsxLDE1IEBACisyMDIyLTAzLTMwICBEb24gT2xtc3RlYWQgIDxkb24ub2xtc3RlYWRAc29ueS5j
b20+CisKKyAgICAgICAgVHVybiBERVZFTE9QRVJfTU9ERSBPTiBmb3IgYWxsIG5vbi1BcHBsZSBw
b3J0cyBpbiBidWlsZC13ZWJraXQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTIzODU1NgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgIFVzZSBERVZFTE9QRVJfTU9ERSB0byBkZXRlcm1pbmUgd2hhdCB0aGUgZGVm
YXVsdHMgb2YgRU5BQkxFX0FQSV9URVNUUyBhbmQKKyAgICAgICAgRU5BQkxFX01JTklCUk9XU0VS
IHNob3VsZCBiZS4gTWFrZSBFTkFCTEVfV0VCS0lUIHRoZSBkZWZhdWx0IG9mIEVOQUJMRV9NSU5J
QlJPV1NFUi4KKworICAgICAgICAqIFNvdXJjZS9jbWFrZS9PcHRpb25zUGxheVN0YXRpb24uY21h
a2U6CisKIDIwMjItMDMtMjkgIFJ5b3N1a2UgTml3YSAgPHJuaXdhQHdlYmtpdC5vcmc+CiAKICAg
ICAgICAgQWRkIGRlc2NyaXB0aW9ucyBvbiBob3cgdG8gc3RhcnQgV1BUIGFuZCBodHRwIHRlc3Qg
c2VydmVycyB0byBJbnRyb2R1Y3Rpb24ubWQKZGlmZiAtLWdpdCBhL1NvdXJjZS9jbWFrZS9PcHRp
b25zUGxheVN0YXRpb24uY21ha2UgYi9Tb3VyY2UvY21ha2UvT3B0aW9uc1BsYXlTdGF0aW9uLmNt
YWtlCmluZGV4IGUxYzk5YzhhYThkOS4uNDIyMjQ4ZjkyOWRjIDEwMDY0NAotLS0gYS9Tb3VyY2Uv
Y21ha2UvT3B0aW9uc1BsYXlTdGF0aW9uLmNtYWtlCisrKyBiL1NvdXJjZS9jbWFrZS9PcHRpb25z
UGxheVN0YXRpb24uY21ha2UKQEAgLTE2LDEyICsxNiwxOCBAQCBhZGRfZGVmaW5pdGlvbnMoLURT
Q0VfTElCQ19ESVNBQkxFX0NQUDE0X0hFQURFUl9XQVJOSU5HPSAtRFNDRV9MSUJDX0RJU0FCTEVf
Q1BQMQogIyBidWctMjI0NDYyCiBXRUJLSVRfUFJFUEVORF9HTE9CQUxfQ09NUElMRVJfRkxBR1Mo
LVduby1kbGwtYXR0cmlidXRlLW9uLXJlZGVjbGFyYXRpb24pCiAKLXNldChFTkFCTEVfQVBJX1RF
U1RTIE9OIENBQ0hFIEJPT0wgIkJ1aWxkIEFQSSBUZXN0cyIpCiBzZXQoRU5BQkxFX1dFQktJVF9M
RUdBQ1kgT0ZGKQogc2V0KEVOQUJMRV9XRUJJTlNQRUNUT1JVSSBPRkYpCiAKIFdFQktJVF9PUFRJ
T05fQkVHSU4oKQogCisjIERldmVsb3BlciBtb2RlIG9wdGlvbnMKK1NFVF9BTkRfRVhQT1NFX1RP
X0JVSUxEKEVOQUJMRV9ERVZFTE9QRVJfTU9ERSAke0RFVkVMT1BFUl9NT0RFfSkKK2lmIChFTkFC
TEVfREVWRUxPUEVSX01PREUpCisgICAgV0VCS0lUX09QVElPTl9ERUZBVUxUX1BPUlRfVkFMVUUo
RU5BQkxFX0FQSV9URVNUUyBQUklWQVRFIE9OKQorICAgIFdFQktJVF9PUFRJT05fREVGQVVMVF9Q
T1JUX1ZBTFVFKEVOQUJMRV9NSU5JQlJPV1NFUiBQVUJMSUMgJHtFTkFCTEVfV0VCS0lUfSkKK2Vu
ZGlmICgpCisKICMgUGxheVN0YXRpb24gU3BlY2lmaWMgT3B0aW9ucwogV0VCS0lUX09QVElPTl9E
RUZJTkUoRU5BQkxFX1NUQVRJQ19KU0MgIkNvbnRyb2wgd2hldGhlciB0byBidWlsZCBhIG5vbi1z
aGFyZWQgSlNDIiBQVUJMSUMgT04pCiAKZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rv
b2xzL0NoYW5nZUxvZwppbmRleCA3MTc5ZTkzN2MzZDAuLjEwMjAwOWFlNWZiZCAxMDA2NDQKLS0t
IGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBA
CisyMDIyLTAzLTMwICBEb24gT2xtc3RlYWQgIDxkb24ub2xtc3RlYWRAc29ueS5jb20+CisKKyAg
ICAgICAgVHVybiBERVZFTE9QRVJfTU9ERSBPTiBmb3IgYWxsIG5vbi1BcHBsZSBwb3J0cyBpbiBi
dWlsZC13ZWJraXQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTIzODU1NgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg
ICAgIEFkZCBERVZFTE9QRVJfTU9ERT1PTiB0byB0aGUgQ01ha2UgYXJndW1lbnQgbGlzdCBmb3Ig
YWxsIG5vbi1BcHBsZSBwb3J0cyBpbiBidWlsZC13ZWJraXQuCisgICAgICAgIFVzZSBFTkFCTEVf
TUlOSUJST1dTRVIgdG8gZGV0ZXJtaW5lIHdoZXRoZXIgdG8gYnVpbGQgdGhlIGV4ZWN1dGFibGUg
b24gUGxheVN0YXRpb24uCisKKyAgICAgICAgKiBQbGF0Zm9ybVBsYXlTdGF0aW9uLmNtYWtlOgor
ICAgICAgICAqIFNjcmlwdHMvd2Via2l0ZGlycy5wbToKKyAgICAgICAgKGdlbmVyYXRlQnVpbGRT
eXN0ZW1Gcm9tQ01ha2VQcm9qZWN0KToKKwogMjAyMi0wMy0yOSAgQWxleCBDaHJpc3RlbnNlbiAg
PGFjaHJpc3RlbnNlbkB3ZWJraXQub3JnPgogCiAgICAgICAgIERvbid0IGNyZWF0ZSBkaXJlY3Rv
cmllcyBvbiBpT1MgaWYgd2UgYXJlIG9ubHkgdXNpbmcgZXBoZW1lcmFsIHN0b3JhZ2VzCmRpZmYg
LS1naXQgYS9Ub29scy9QbGF0Zm9ybVBsYXlTdGF0aW9uLmNtYWtlIGIvVG9vbHMvUGxhdGZvcm1Q
bGF5U3RhdGlvbi5jbWFrZQppbmRleCBhNTM1YjlhYmE4YTguLjcyNTQ3MmE5YjViOSAxMDA2NDQK
LS0tIGEvVG9vbHMvUGxhdGZvcm1QbGF5U3RhdGlvbi5jbWFrZQorKysgYi9Ub29scy9QbGF0Zm9y
bVBsYXlTdGF0aW9uLmNtYWtlCkBAIC0xLDMgKzEsMyBAQAotaWYgKEVOQUJMRV9XRUJLSVQpCitp
ZiAoRU5BQkxFX01JTklCUk9XU0VSKQogICAgIGFkZF9zdWJkaXJlY3RvcnkoTWluaUJyb3dzZXIv
cGxheXN0YXRpb24pCiBlbmRpZiAoKQpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy93ZWJraXRk
aXJzLnBtIGIvVG9vbHMvU2NyaXB0cy93ZWJraXRkaXJzLnBtCmluZGV4IDYyMzRiYTE4NDJkNi4u
OTNmOTVkMzk4MDQ5IDEwMDc1NQotLS0gYS9Ub29scy9TY3JpcHRzL3dlYmtpdGRpcnMucG0KKysr
IGIvVG9vbHMvU2NyaXB0cy93ZWJraXRkaXJzLnBtCkBAIC0yNTYyLDcgKzI1NjIsNyBAQCBzdWIg
Z2VuZXJhdGVCdWlsZFN5c3RlbUZyb21DTWFrZVByb2plY3QKICAgICBwdXNoIEBhcmdzLCAnLURT
SE9XX0JJTkRJTkdTX0dFTkVSQVRJT05fUFJPR1JFU1M9MScgdW5sZXNzICgkd2lsbFVzZU5pbmph
ICYmIC10IFNURE9VVCk7CiAKICAgICAjIFNvbWUgcG9ydHMgaGF2ZSBwcm9kdWN0aW9uIG1vZGUs
IGJ1dCBidWlsZC13ZWJraXQgc2hvdWxkIGFsd2F5cyB1c2UgZGV2ZWxvcGVyIG1vZGUuCi0gICAg
cHVzaCBAYXJncywgIi1EREVWRUxPUEVSX01PREU9T04iIGlmIGlzR3RrKCkgfHwgaXNKU0NPbmx5
KCkgfHwgaXNXUEUoKSB8fCBpc1dpbjY0KCk7CisgICAgcHVzaCBAYXJncywgIi1EREVWRUxPUEVS
X01PREU9T04iIHVubGVzcyBpc0FwcGxlV2ViS2l0KCk7CiAKICAgICBpZiAoYXJjaGl0ZWN0dXJl
KCkgZXEgIng4Nl82NCIgJiYgc2hvdWxkQnVpbGQzMkJpdCgpKSB7CiAgICAgICAgICMgQ01BS0Vf
TElCUkFSWV9BUkNISVRFQ1RVUkUgaXMgbmVlZGVkIHRvIGdldCB0aGUgcmlnaHQgLnBjCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>456159</attachid>
            <date>2022-03-30 11:54:55 -0700</date>
            <delta_ts>2022-03-30 13:12:19 -0700</delta_ts>
            <desc>Patch enabling all</desc>
            <filename>bug-238556.diff</filename>
            <type>text/plain</type>
            <size>5479</size>
            <attacher name="Don Olmstead">don.olmstead</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0NoYW5nZUxvZyBiL0NoYW5nZUxvZwppbmRleCBjNjQ0MDg5MmNiNjcuLjRm
NjliYjEwMmZlOCAxMDA2NDQKLS0tIGEvQ2hhbmdlTG9nCisrKyBiL0NoYW5nZUxvZwpAQCAtMSwz
ICsxLDIyIEBACisyMDIyLTAzLTMwICBEb24gT2xtc3RlYWQgIDxkb24ub2xtc3RlYWRAc29ueS5j
b20+CisKKyAgICAgICAgVHVybiBERVZFTE9QRVJfTU9ERSBPTiBmb3IgYWxsIENNYWtlIHBvcnRz
IGluIGJ1aWxkLXdlYmtpdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MjM4NTU2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgQWRkIERFVkVMT1BFUl9NT0RFIGFzIGFuIG9wdGlvbiBmb3IgYWxsIENNYWtlIHBv
cnRzLiBFeHBvc2UgRU5BQkxFX0RFVkVMT1BFUl9NT0RFIHRvIHRoZQorICAgICAgICBidWlsZCBi
YXNlZCBvbiB0aGlzIHZhbHVlLiBGb3IgdGhlIFBsYXlTdGF0aW9uIHBvcnQgdXNlIERFVkVMT1BF
Ul9NT0RFIHRvIGhlbHAgZGV0ZXJtaW5lCisgICAgICAgIHdoYXQgdGhlIGRlZmF1bHRzIG9mIEVO
QUJMRV9BUElfVEVTVFMgYW5kIEVOQUJMRV9NSU5JQlJPV1NFUiBzaG91bGQgYmUuCisKKyAgICAg
ICAgWENvZGUgYnVpbGRzIGhhdmUgYSBEZWJ1ZywgUmVsZWFzZSBhbmQgUHJvZHVjdGlvbiBidWls
ZCBzbyB1c2UgREVWRUxPUEVSX01PREU9T0ZGIHRvCisgICAgICAgIHNpZ25pZnkgYSBQcm9kdWN0
aW9uIGJ1aWxkIHZhcmlhbnQuCisKKyAgICAgICAgKiBTb3VyY2UvY21ha2UvT3B0aW9uc1BsYXlT
dGF0aW9uLmNtYWtlOgorICAgICAgICAqIFNvdXJjZS9jbWFrZS9PcHRpb25zV1BFLmNtYWtlOgor
ICAgICAgICAqIFNvdXJjZS9jbWFrZS9PcHRpb25zV2luQ2Fpcm8uY21ha2U6CisgICAgICAgICog
U291cmNlL2NtYWtlL1dlYktpdEZlYXR1cmVzLmNtYWtlOgorCiAyMDIyLTAzLTI5ICBSeW9zdWtl
IE5pd2EgIDxybml3YUB3ZWJraXQub3JnPgogCiAgICAgICAgIEFkZCBkZXNjcmlwdGlvbnMgb24g
aG93IHRvIHN0YXJ0IFdQVCBhbmQgaHR0cCB0ZXN0IHNlcnZlcnMgdG8gSW50cm9kdWN0aW9uLm1k
CmRpZmYgLS1naXQgYS9Tb3VyY2UvY21ha2UvT3B0aW9uc1BsYXlTdGF0aW9uLmNtYWtlIGIvU291
cmNlL2NtYWtlL09wdGlvbnNQbGF5U3RhdGlvbi5jbWFrZQppbmRleCBlMWM5OWM4YWE4ZDkuLjU2
OGMxOTRiZDhmZiAxMDA2NDQKLS0tIGEvU291cmNlL2NtYWtlL09wdGlvbnNQbGF5U3RhdGlvbi5j
bWFrZQorKysgYi9Tb3VyY2UvY21ha2UvT3B0aW9uc1BsYXlTdGF0aW9uLmNtYWtlCkBAIC0xNiwx
MiArMTYsMTcgQEAgYWRkX2RlZmluaXRpb25zKC1EU0NFX0xJQkNfRElTQUJMRV9DUFAxNF9IRUFE
RVJfV0FSTklORz0gLURTQ0VfTElCQ19ESVNBQkxFX0NQUDEKICMgYnVnLTIyNDQ2MgogV0VCS0lU
X1BSRVBFTkRfR0xPQkFMX0NPTVBJTEVSX0ZMQUdTKC1Xbm8tZGxsLWF0dHJpYnV0ZS1vbi1yZWRl
Y2xhcmF0aW9uKQogCi1zZXQoRU5BQkxFX0FQSV9URVNUUyBPTiBDQUNIRSBCT09MICJCdWlsZCBB
UEkgVGVzdHMiKQogc2V0KEVOQUJMRV9XRUJLSVRfTEVHQUNZIE9GRikKIHNldChFTkFCTEVfV0VC
SU5TUEVDVE9SVUkgT0ZGKQogCiBXRUJLSVRfT1BUSU9OX0JFR0lOKCkKIAorIyBEZXZlbG9wZXIg
bW9kZSBvcHRpb25zCitpZiAoRU5BQkxFX0RFVkVMT1BFUl9NT0RFKQorICAgIFdFQktJVF9PUFRJ
T05fREVGQVVMVF9QT1JUX1ZBTFVFKEVOQUJMRV9BUElfVEVTVFMgUFJJVkFURSBPTikKKyAgICBX
RUJLSVRfT1BUSU9OX0RFRkFVTFRfUE9SVF9WQUxVRShFTkFCTEVfTUlOSUJST1dTRVIgUFVCTElD
ICR7RU5BQkxFX1dFQktJVH0pCitlbmRpZiAoKQorCiAjIFBsYXlTdGF0aW9uIFNwZWNpZmljIE9w
dGlvbnMKIFdFQktJVF9PUFRJT05fREVGSU5FKEVOQUJMRV9TVEFUSUNfSlNDICJDb250cm9sIHdo
ZXRoZXIgdG8gYnVpbGQgYSBub24tc2hhcmVkIEpTQyIgUFVCTElDIE9OKQogCmRpZmYgLS1naXQg
YS9Tb3VyY2UvY21ha2UvT3B0aW9uc1dQRS5jbWFrZSBiL1NvdXJjZS9jbWFrZS9PcHRpb25zV1BF
LmNtYWtlCmluZGV4IGQxZmQyZjAxZThhNi4uNmUxNDg5MDViOGY3IDEwMDY0NAotLS0gYS9Tb3Vy
Y2UvY21ha2UvT3B0aW9uc1dQRS5jbWFrZQorKysgYi9Tb3VyY2UvY21ha2UvT3B0aW9uc1dQRS5j
bWFrZQpAQCAtMjYsOCArMjYsNiBAQCBmaW5kX3BhY2thZ2UoWkxJQiBSRVFVSVJFRCkKIAogV0VC
S0lUX09QVElPTl9CRUdJTigpCiAKLVNFVF9BTkRfRVhQT1NFX1RPX0JVSUxEKEVOQUJMRV9ERVZF
TE9QRVJfTU9ERSAke0RFVkVMT1BFUl9NT0RFfSkKLQogaW5jbHVkZShHU3RyZWFtZXJEZWZpbml0
aW9ucykKIAogIyBQdWJsaWMgb3B0aW9ucyBzaGFyZWQgd2l0aCBvdGhlciBXZWJLaXQgcG9ydHMu
IERvIG5vdCBhZGQgYW55IG9wdGlvbnMgaGVyZQpkaWZmIC0tZ2l0IGEvU291cmNlL2NtYWtlL09w
dGlvbnNXaW5DYWlyby5jbWFrZSBiL1NvdXJjZS9jbWFrZS9PcHRpb25zV2luQ2Fpcm8uY21ha2UK
aW5kZXggNjJiNGJjNmU5MWIxLi5jNTFiYmFiY2IyYjMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9jbWFr
ZS9PcHRpb25zV2luQ2Fpcm8uY21ha2UKKysrIGIvU291cmNlL2NtYWtlL09wdGlvbnNXaW5DYWly
by5jbWFrZQpAQCAtNjMsOCArNjMsNiBAQCBTRVRfQU5EX0VYUE9TRV9UT19CVUlMRChVU0VfVEVY
VFVSRV9NQVBQRVJfR0wgT04pCiBTRVRfQU5EX0VYUE9TRV9UT19CVUlMRChVU0VfTUVESUFfRk9V
TkRBVElPTiBPTikKIFNFVF9BTkRfRVhQT1NFX1RPX0JVSUxEKFVTRV9JTlNQRUNUT1JfU09DS0VU
X1NFUlZFUiAke0VOQUJMRV9SRU1PVEVfSU5TUEVDVE9SfSkKIAotU0VUX0FORF9FWFBPU0VfVE9f
QlVJTEQoRU5BQkxFX0RFVkVMT1BFUl9NT0RFICR7REVWRUxPUEVSX01PREV9KQotCiBTRVRfQU5E
X0VYUE9TRV9UT19CVUlMRChIQVZFX09TX0RBUktfTU9ERV9TVVBQT1JUIDEpCiAKICMgQ29yZUZv
dW5kYXRpb24gaXMgcmVxdWlyZWQgd2hlbiBidWlsZGluZyBXZWJLaXRMZWdhY3kKZGlmZiAtLWdp
dCBhL1NvdXJjZS9jbWFrZS9XZWJLaXRGZWF0dXJlcy5jbWFrZSBiL1NvdXJjZS9jbWFrZS9XZWJL
aXRGZWF0dXJlcy5jbWFrZQppbmRleCA5YTA4ZGE0ZjJlZDMuLmZmMzU0NmUwZGEwNiAxMDA2NDQK
LS0tIGEvU291cmNlL2NtYWtlL1dlYktpdEZlYXR1cmVzLmNtYWtlCisrKyBiL1NvdXJjZS9jbWFr
ZS9XZWJLaXRGZWF0dXJlcy5jbWFrZQpAQCAtNDI5LDUgKzQyOSw4IEBAIG1hY3JvKFdFQktJVF9D
SEVDS19TT1VSQ0VfQ09NUElMRVMgX3ZhcmlhYmxlIF9zb3VyY2UpCiAgICAgU0VUX0FORF9FWFBP
U0VfVE9fQlVJTEQoJHtfdmFyaWFibGV9ICR7JHtfdmFyaWFibGV9X3ZhbHVlfSkKIGVuZG1hY3Jv
KCkKIAorb3B0aW9uKERFVkVMT1BFUl9NT0RFICJFbmFibGUgZGV2ZWxvcGVyIG1vZGUiIE9GRikK
K1NFVF9BTkRfRVhQT1NFX1RPX0JVSUxEKEVOQUJMRV9ERVZFTE9QRVJfTU9ERSAke0RFVkVMT1BF
Ul9NT0RFfSkKKwogb3B0aW9uKEVOQUJMRV9FWFBFUklNRU5UQUxfRkVBVFVSRVMgIkVuYWJsZSBl
eHBlcmltZW50YWwgZmVhdHVyZXMiIE9GRikKIFNFVF9BTkRfRVhQT1NFX1RPX0JVSUxEKEVOQUJM
RV9FWFBFUklNRU5UQUxfRkVBVFVSRVMgJHtFTkFCTEVfRVhQRVJJTUVOVEFMX0ZFQVRVUkVTfSkK
ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA3MTc5
ZTkzN2MzZDAuLmYwYTgxN2Q1OTRiNSAxMDA2NDQKLS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBi
L1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDIyLTAzLTMwICBEb24gT2xtc3Rl
YWQgIDxkb24ub2xtc3RlYWRAc29ueS5jb20+CisKKyAgICAgICAgVHVybiBERVZFTE9QRVJfTU9E
RSBPTiBmb3IgYWxsIENNYWtlIHBvcnRzIGluIGJ1aWxkLXdlYmtpdAorICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjM4NTU2CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQWRkIERFVkVMT1BFUl9NT0RFPU9OIHRv
IHRoZSBDTWFrZSBhcmd1bWVudCBsaXN0IGZvciBhbGwgQ01ha2UgcG9ydHMgaW4gYnVpbGQtd2Vi
a2l0LgorICAgICAgICAKKyAgICAgICAgVXNlIEVOQUJMRV9NSU5JQlJPV1NFUiB0byBkZXRlcm1p
bmUgd2hldGhlciB0byBidWlsZCB0aGUgZXhlY3V0YWJsZSBvbiBQbGF5U3RhdGlvbi4KKworICAg
ICAgICAqIFBsYXRmb3JtUGxheVN0YXRpb24uY21ha2U6CisgICAgICAgICogU2NyaXB0cy93ZWJr
aXRkaXJzLnBtOgorICAgICAgICAoZ2VuZXJhdGVCdWlsZFN5c3RlbUZyb21DTWFrZVByb2plY3Qp
OgorCiAyMDIyLTAzLTI5ICBBbGV4IENocmlzdGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5v
cmc+CiAKICAgICAgICAgRG9uJ3QgY3JlYXRlIGRpcmVjdG9yaWVzIG9uIGlPUyBpZiB3ZSBhcmUg
b25seSB1c2luZyBlcGhlbWVyYWwgc3RvcmFnZXMKZGlmZiAtLWdpdCBhL1Rvb2xzL1BsYXRmb3Jt
UGxheVN0YXRpb24uY21ha2UgYi9Ub29scy9QbGF0Zm9ybVBsYXlTdGF0aW9uLmNtYWtlCmluZGV4
IGE1MzViOWFiYThhOC4uNzI1NDcyYTliNWI5IDEwMDY0NAotLS0gYS9Ub29scy9QbGF0Zm9ybVBs
YXlTdGF0aW9uLmNtYWtlCisrKyBiL1Rvb2xzL1BsYXRmb3JtUGxheVN0YXRpb24uY21ha2UKQEAg
LTEsMyArMSwzIEBACi1pZiAoRU5BQkxFX1dFQktJVCkKK2lmIChFTkFCTEVfTUlOSUJST1dTRVIp
CiAgICAgYWRkX3N1YmRpcmVjdG9yeShNaW5pQnJvd3Nlci9wbGF5c3RhdGlvbikKIGVuZGlmICgp
CmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3dlYmtpdGRpcnMucG0gYi9Ub29scy9TY3JpcHRz
L3dlYmtpdGRpcnMucG0KaW5kZXggNjIzNGJhMTg0MmQ2Li5mZjM1ODE2YWU4NzggMTAwNzU1Ci0t
LSBhL1Rvb2xzL1NjcmlwdHMvd2Via2l0ZGlycy5wbQorKysgYi9Ub29scy9TY3JpcHRzL3dlYmtp
dGRpcnMucG0KQEAgLTI1NjIsNyArMjU2Miw3IEBAIHN1YiBnZW5lcmF0ZUJ1aWxkU3lzdGVtRnJv
bUNNYWtlUHJvamVjdAogICAgIHB1c2ggQGFyZ3MsICctRFNIT1dfQklORElOR1NfR0VORVJBVElP
Tl9QUk9HUkVTUz0xJyB1bmxlc3MgKCR3aWxsVXNlTmluamEgJiYgLXQgU1RET1VUKTsKIAogICAg
ICMgU29tZSBwb3J0cyBoYXZlIHByb2R1Y3Rpb24gbW9kZSwgYnV0IGJ1aWxkLXdlYmtpdCBzaG91
bGQgYWx3YXlzIHVzZSBkZXZlbG9wZXIgbW9kZS4KLSAgICBwdXNoIEBhcmdzLCAiLURERVZFTE9Q
RVJfTU9ERT1PTiIgaWYgaXNHdGsoKSB8fCBpc0pTQ09ubHkoKSB8fCBpc1dQRSgpIHx8IGlzV2lu
NjQoKTsKKyAgICBwdXNoIEBhcmdzLCAiLURERVZFTE9QRVJfTU9ERT1PTiI7CiAKICAgICBpZiAo
YXJjaGl0ZWN0dXJlKCkgZXEgIng4Nl82NCIgJiYgc2hvdWxkQnVpbGQzMkJpdCgpKSB7CiAgICAg
ICAgICMgQ01BS0VfTElCUkFSWV9BUkNISVRFQ1RVUkUgaXMgbmVlZGVkIHRvIGdldCB0aGUgcmln
aHQgLnBjCg==
</data>
<flag name="review"
          id="484374"
          type_id="1"
          status="-"
          setter="achristensen"
    />
          </attachment>
      

    </bug>

</bugzilla>