<?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>24345</bug_id>
          
          <creation_ts>2009-03-04 01:31:38 -0800</creation_ts>
          <short_desc>wxWebkit should make use of gcc(4 and greater) hidden visibility feature</short_desc>
          <delta_ts>2009-05-05 16:12:03 -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>WebKit wx</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Enhancement</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Malcolm MacLeod">mmacleod</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>112164</commentid>
    <comment_count>0</comment_count>
    <who name="Malcolm MacLeod">mmacleod</who>
    <bug_when>2009-03-04 01:31:38 -0800</bug_when>
    <thetext>wxWebKit does not currently make use of the gcc hidden visibility features that are available since gcc 4.
Changing it to make use of these features is trivial as everything is already in place to do so, so the gains that can be had are almost for free.

I have done so and it results in a +-35% binary size decrease (From 30Mib down to 19mib on a 64 bit system). Obviously this also speeds up linking time and other similar benefits.

I will attach the patch to enable this below.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>112165</commentid>
    <comment_count>1</comment_count>
      <attachid>28259</attachid>
    <who name="Malcolm MacLeod">mmacleod</who>
    <bug_when>2009-03-04 01:32:30 -0800</bug_when>
    <thetext>Created attachment 28259
Patch to enable hidden visibility for gcc &gt; 4</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>112166</commentid>
    <comment_count>2</comment_count>
    <who name="Malcolm MacLeod">mmacleod</who>
    <bug_when>2009-03-04 01:34:50 -0800</bug_when>
    <thetext>Forgot to mention that this has been tested successfully on both an x86 system with GCC 4.1.2 as well as an x86_64 system with GCC 4.3.3</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>120038</commentid>
    <comment_count>3</comment_count>
    <who name="Kevin Ollivier">kevino</who>
    <bug_when>2009-05-05 16:12:03 -0700</bug_when>
    <thetext>Landed in r43260, thanks!</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>28259</attachid>
            <date>2009-03-04 01:32:30 -0800</date>
            <delta_ts>2009-05-05 16:09:16 -0700</delta_ts>
            <desc>Patch to enable hidden visibility for gcc &gt; 4</desc>
            <filename>gcc_hidden_visibility.patch</filename>
            <type>text/plain</type>
            <size>1739</size>
            <attacher name="Malcolm MacLeod">mmacleod</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC93eC9XZWJGcmFtZS5oDQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09DQotLS0gV2ViS2l0L3d4L1dl
YkZyYW1lLmgJKHJldmlzaW9uIDQxNDE4KQ0KKysrIFdlYktpdC93eC9XZWJGcmFtZS5oCSh3b3Jr
aW5nIGNvcHkpDQpAQCAtNDYsNiArNDYsOSBAQA0KIAogI2lmbmRlZiBTV0lHCiAKKyNpZiAhd3hD
SEVDS19WRVJTSU9OKDIsOSwwKSAmJiB3eENIRUNLX0dDQ19WRVJTSU9OKDQsMCkKKyNkZWZpbmUg
V1hETExJTVBFWFBfV0VCS0lUIF9fYXR0cmlidXRlX18gKCh2aXNpYmlsaXR5KCJkZWZhdWx0Iikp
KQorI2Vsc2UKICNpZiBXWE1BS0lOR0RMTF9XRUJLSVQKICNkZWZpbmUgV1hETExJTVBFWFBfV0VC
S0lUIFdYRVhQT1JUCiAjZWxpZiBkZWZpbmVkKFdYVVNJTkdETExfV0VCS0lUKQpAQCAtNTQsOCAr
NTcsNyBAQA0KICNkZWZpbmUgV1hETExJTVBFWFBfV0VCS0lUCiAjZW5kaWYKIAotI2Vsc2UgCi0j
ZGVmaW5lIFdYRExMSU1QRVhQX1dFQktJVAorI2VuZGlmCiAjZW5kaWYgLy8gU1dJRwogCiBjbGFz
cyBXWERMTElNUEVYUF9XRUJLSVQgd3hXZWJWaWV3RE9NRWxlbWVudEluZm8KSW5kZXg6IFdlYktp
dC93eC9XZWJWaWV3LmgNCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT0NCi0tLSBXZWJLaXQvd3gvV2ViVmlldy5oCShyZXZp
c2lvbiA0MTQxOCkNCisrKyBXZWJLaXQvd3gvV2ViVmlldy5oCSh3b3JraW5nIGNvcHkpDQpAQCAt
NDYsNiArNDYsOSBAQA0KIAogI2lmbmRlZiBTV0lHCiAKKyNpZiAhd3hDSEVDS19WRVJTSU9OKDIs
OSwwKSAmJiB3eENIRUNLX0dDQ19WRVJTSU9OKDQsMCkKKyNkZWZpbmUgV1hETExJTVBFWFBfV0VC
S0lUIF9fYXR0cmlidXRlX18gKCh2aXNpYmlsaXR5KCJkZWZhdWx0IikpKQorI2Vsc2UKICNpZiBX
WE1BS0lOR0RMTF9XRUJLSVQKICNkZWZpbmUgV1hETExJTVBFWFBfV0VCS0lUIFdYRVhQT1JUCiAj
ZWxpZiBkZWZpbmVkKFdYVVNJTkdETExfV0VCS0lUKQpAQCAtNTQsOCArNTcsNyBAQA0KICNkZWZp
bmUgV1hETExJTVBFWFBfV0VCS0lUCiAjZW5kaWYKIAotI2Vsc2UgCi0jZGVmaW5lIFdYRExMSU1Q
RVhQX1dFQktJVAorI2VuZGlmCiAjZW5kaWYgLy8gU1dJRwogCiAjaWZuZGVmIFNXSUcKSW5kZXg6
IFdlYktpdFRvb2xzL3d4L2J1aWxkLXd4d2Via2l0DQo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09DQotLS0gV2ViS2l0VG9v
bHMvd3gvYnVpbGQtd3h3ZWJraXQJKHJldmlzaW9uIDQxNDE4KQ0KKysrIFdlYktpdFRvb2xzL3d4
L2J1aWxkLXd4d2Via2l0CSh3b3JraW5nIGNvcHkpDQpAQCAtMTc5LDcgKzE3OSw3IEBADQogICAg
ICAgICBjZCAkb2xkZGlyCiAgICAgZWxzZQogICAgICAgICBpZiBbIC1mICRkaXIvR05VbWFrZWZp
bGUgXTsgdGhlbgotICAgICAgICAgICAgbWFrZSAtQyAkZGlyIC1mIEdOVW1ha2VmaWxlICRNQUtF
X0FSR1MgQ1hYRkxBR1M9IiRjeHhmbGFncyIgJEAKKyAgICAgICAgICAgIG1ha2UgLUMgJGRpciAt
ZiBHTlVtYWtlZmlsZSAkTUFLRV9BUkdTIENYWEZMQUdTPSIkY3h4ZmxhZ3MgLWZ2aXNpYmlsaXR5
PWhpZGRlbiAtZnZpc2liaWxpdHktaW5saW5lcy1oaWRkZW4iICRACiAgICAgICAgIGZpCiAgICAg
ZmkKICAgICBpZiBbICQ/ICE9IDAgXTsgdGhlbgo=
</data>
<flag name="review"
          id="15075"
          type_id="1"
          status="+"
          setter="kevino"
    />
          </attachment>
      

    </bug>

</bugzilla>