<?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>70345</bug_id>
          
          <creation_ts>2011-10-18 11:47:41 -0700</creation_ts>
          <short_desc>Add WebKit API to hide the definition window</short_desc>
          <delta_ts>2011-10-18 16:08:19 -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 API</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Mac</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>70363</dependson>
    
    <dependson>70366</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Damian Kaleta">dkaleta</reporter>
          <assigned_to name="Damian Kaleta">dkaleta</assigned_to>
          <cc>darin</cc>
    
    <cc>kdecker</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>486201</commentid>
    <comment_count>0</comment_count>
    <who name="Damian Kaleta">dkaleta</who>
    <bug_when>2011-10-18 11:47:41 -0700</bug_when>
    <thetext>Add WebKit API to hide the definition window.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486217</commentid>
    <comment_count>1</comment_count>
      <attachid>111478</attachid>
    <who name="Damian Kaleta">dkaleta</who>
    <bug_when>2011-10-18 12:18:06 -0700</bug_when>
    <thetext>Created attachment 111478
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486287</commentid>
    <comment_count>2</comment_count>
    <who name="Kevin Decker">kdecker</who>
    <bug_when>2011-10-18 13:50:14 -0700</bug_when>
    <thetext>Committed revision 97788.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486375</commentid>
    <comment_count>3</comment_count>
    <who name="Damian Kaleta">dkaleta</who>
    <bug_when>2011-10-18 15:54:53 -0700</bug_when>
    <thetext>Need to fix SL build.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486376</commentid>
    <comment_count>4</comment_count>
      <attachid>111515</attachid>
    <who name="Damian Kaleta">dkaleta</who>
    <bug_when>2011-10-18 15:55:57 -0700</bug_when>
    <thetext>Created attachment 111515
patch that doesn&apos;t break SL build</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486379</commentid>
    <comment_count>5</comment_count>
      <attachid>111515</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2011-10-18 15:57:16 -0700</bug_when>
    <thetext>Comment on attachment 111515
patch that doesn&apos;t break SL build

I might put the #ifndef inside the method and make it harmless but pointless to call it on SnowLeopard, but this is OK.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486381</commentid>
    <comment_count>6</comment_count>
      <attachid>111515</attachid>
    <who name="Sam Weinig">sam</who>
    <bug_when>2011-10-18 15:58:56 -0700</bug_when>
    <thetext>Comment on attachment 111515
patch that doesn&apos;t break SL build

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

&gt; Source/WebKit2/UIProcess/API/mac/WKViewPrivate.h:38
&gt; +#if !defined(BUILDING_ON_SNOW_LEOPARD)

I don&apos;t think you can do #if !defined(BUILDING_ON_SNOW_LEOPARD) in a header.  Reversing drain&apos;s r+.  You can just put the #if !defined(BUILDING_ON_SNOW_LEOPARD) in the implementation.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486383</commentid>
    <comment_count>7</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2011-10-18 15:59:19 -0700</bug_when>
    <thetext>Going down the reversing drain!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486389</commentid>
    <comment_count>8</comment_count>
      <attachid>111517</attachid>
    <who name="Damian Kaleta">dkaleta</who>
    <bug_when>2011-10-18 16:03:36 -0700</bug_when>
    <thetext>Created attachment 111517
yet another patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486393</commentid>
    <comment_count>9</comment_count>
      <attachid>111517</attachid>
    <who name="Sam Weinig">sam</who>
    <bug_when>2011-10-18 16:04:21 -0700</bug_when>
    <thetext>Comment on attachment 111517
yet another patch

Better, I will test and land.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>486399</commentid>
    <comment_count>10</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2011-10-18 16:08:19 -0700</bug_when>
    <thetext>Landed in 97806.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>111478</attachid>
            <date>2011-10-18 12:18:06 -0700</date>
            <delta_ts>2011-10-18 15:59:27 -0700</delta_ts>
            <desc>patch</desc>
            <filename>textmate stdin iWy8SM.txt</filename>
            <type>text/plain</type>
            <size>1757</size>
            <attacher name="Damian Kaleta">dkaleta</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDk3NzgwKQorKysgU291cmNlL1dlYktpdDIvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTEtMTAtMTggIERhbWlhbiBL
YWxldGEgIDxka2FsZXRhQGFwcGxlLmNvbT4KKworICAgICAgICBBZGQgV2ViS2l0IEFQSSB0byBo
aWRlIHRoZSBkZWZpbml0aW9uIHdpbmRvdworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9NzAzNDUKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P
UFMhKS4KKworICAgICAgICAqIFVJUHJvY2Vzcy9BUEkvbWFjL1dLVmlldy5tbToKKyAgICAgICAg
KC1bV0tWaWV3IGhpZGVXb3JkRGVmaW5pdGlvbldpbmRvd10pOiBIaWRlcyB0aGUgZGVmaW5pdGlv
biB3aW5kb3cuCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9tYWMvV0tWaWV3UHJpdmF0ZS5oOgor
CiAyMDExLTEwLTE4ICBBZGFtIEJhcnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CiAKICAgICAgICAg
QWx3YXlzIGVuYWJsZSBFTkFCTEUoWFBBVEgpCkluZGV4OiBTb3VyY2UvV2ViS2l0Mi9VSVByb2Nl
c3MvQVBJL21hYy9XS1ZpZXcubW0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYktpdDIvVUlQcm9j
ZXNzL0FQSS9tYWMvV0tWaWV3Lm1tCShyZXZpc2lvbiA5Nzc2MCkKKysrIFNvdXJjZS9XZWJLaXQy
L1VJUHJvY2Vzcy9BUEkvbWFjL1dLVmlldy5tbQkod29ya2luZyBjb3B5KQpAQCAtMjY1Myw2ICsy
NjUzLDExIEBAIC0gKHZvaWQpcGVyZm9ybURpY3Rpb25hcnlMb29rdXBBdEN1cnJlbnQKICAgICBf
ZGF0YS0+X3BhZ2UtPnBlcmZvcm1EaWN0aW9uYXJ5TG9va3VwQXRMb2NhdGlvbihGbG9hdFBvaW50
KHRoZVBvaW50LngsIHRoZVBvaW50LnkpKTsKIH0KIAorLSAodm9pZCloaWRlV29yZERlZmluaXRp
b25XaW5kb3cKK3sKKyAgICBXS0hpZGVXb3JkRGVmaW5pdGlvbldpbmRvdygpOworfQorCiAtIChO
U0ludGVnZXIpc3BlbGxDaGVja2VyRG9jdW1lbnRUYWcKIHsKICAgICBpZiAoIV9kYXRhLT5faGFz
U3BlbGxDaGVja2VyRG9jdW1lbnRUYWcpIHsKSW5kZXg6IFNvdXJjZS9XZWJLaXQyL1VJUHJvY2Vz
cy9BUEkvbWFjL1dLVmlld1ByaXZhdGUuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0Mi9V
SVByb2Nlc3MvQVBJL21hYy9XS1ZpZXdQcml2YXRlLmgJKHJldmlzaW9uIDk3NzYwKQorKysgU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9tYWMvV0tWaWV3UHJpdmF0ZS5oCSh3b3JraW5nIGNv
cHkpCkBAIC0zNCw2ICszNCw3IEBAIEBpbnRlcmZhY2UgV0tWaWV3IChQcml2YXRlKQogLSAoQk9P
TClmcmFtZVNpemVVcGRhdGVzRGlzYWJsZWQ7CiAKIC0gKHZvaWQpcGVyZm9ybURpY3Rpb25hcnlM
b29rdXBBdEN1cnJlbnRNb3VzZUxvY2F0aW9uOworLSAodm9pZCloaWRlV29yZERlZmluaXRpb25X
aW5kb3c7CiAKIC0gKE5TSW50ZWdlcilzcGVsbENoZWNrZXJEb2N1bWVudFRhZzsKIC0gKHZvaWQp
aGFuZGxlQ29ycmVjdGlvblBhbmVsUmVzdWx0OihOU1N0cmluZyopcmVzdWx0Owo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>111515</attachid>
            <date>2011-10-18 15:55:57 -0700</date>
            <delta_ts>2011-10-18 16:03:36 -0700</delta_ts>
            <desc>patch that doesn&apos;t break SL build</desc>
            <filename>textmate stdin ag3UXf.txt</filename>
            <type>text/plain</type>
            <size>1838</size>
            <attacher name="Damian Kaleta">dkaleta</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDk3Nzk5KQorKysgU291cmNlL1dlYktpdDIvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTEtMTAtMTggIERhbWlhbiBL
YWxldGEgIDxka2FsZXRhQGFwcGxlLmNvbT4KKworICAgICAgICBUaGlzIHJldmlzZWQgZml4IGRv
ZXNuJ3QgYnJlYWsgU0wgYnVpbGQuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD03MDM0NQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9tYWMvV0tWaWV3Lm1tOgorICAgICAgICAoK1tX
S1ZpZXcgaGlkZVdvcmREZWZpbml0aW9uV2luZG93XSk6IEhpZGVzIHRoZSBkZWZpbml0aW9uIHdp
bmRvdy4KKyAgICAgICAgKiBVSVByb2Nlc3MvQVBJL21hYy9XS1ZpZXdQcml2YXRlLmg6CisKIDIw
MTEtMTAtMTggIFNoZXJpZmYgQm90ICA8d2Via2l0LnJldmlldy5ib3RAZ21haWwuY29tPgogCiAg
ICAgICAgIFVucmV2aWV3ZWQsIHJvbGxpbmcgb3V0IHI5Nzc4OC4KSW5kZXg6IFNvdXJjZS9XZWJL
aXQyL1VJUHJvY2Vzcy9BUEkvbWFjL1dLVmlldy5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
S2l0Mi9VSVByb2Nlc3MvQVBJL21hYy9XS1ZpZXcubW0JKHJldmlzaW9uIDk3Nzk2KQorKysgU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9tYWMvV0tWaWV3Lm1tCSh3b3JraW5nIGNvcHkpCkBA
IC0yNjUzLDYgKzI2NTMsMTMgQEAgLSAodm9pZClwZXJmb3JtRGljdGlvbmFyeUxvb2t1cEF0Q3Vy
cmVudAogICAgIF9kYXRhLT5fcGFnZS0+cGVyZm9ybURpY3Rpb25hcnlMb29rdXBBdExvY2F0aW9u
KEZsb2F0UG9pbnQodGhlUG9pbnQueCwgdGhlUG9pbnQueSkpOwogfQogCisjaWYgIWRlZmluZWQo
QlVJTERJTkdfT05fU05PV19MRU9QQVJEKQorKyAodm9pZCloaWRlV29yZERlZmluaXRpb25XaW5k
b3cKK3sKKyAgICBXS0hpZGVXb3JkRGVmaW5pdGlvbldpbmRvdygpOworfQorI2VuZGlmCisKIC0g
KE5TSW50ZWdlcilzcGVsbENoZWNrZXJEb2N1bWVudFRhZwogewogICAgIGlmICghX2RhdGEtPl9o
YXNTcGVsbENoZWNrZXJEb2N1bWVudFRhZykgewpJbmRleDogU291cmNlL1dlYktpdDIvVUlQcm9j
ZXNzL0FQSS9tYWMvV0tWaWV3UHJpdmF0ZS5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQy
L1VJUHJvY2Vzcy9BUEkvbWFjL1dLVmlld1ByaXZhdGUuaAkocmV2aXNpb24gOTc3OTYpCisrKyBT
b3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL21hYy9XS1ZpZXdQcml2YXRlLmgJKHdvcmtpbmcg
Y29weSkKQEAgLTM1LDYgKzM1LDEwIEBAIEBpbnRlcmZhY2UgV0tWaWV3IChQcml2YXRlKQogCiAt
ICh2b2lkKXBlcmZvcm1EaWN0aW9uYXJ5TG9va3VwQXRDdXJyZW50TW91c2VMb2NhdGlvbjsKIAor
I2lmICFkZWZpbmVkKEJVSUxESU5HX09OX1NOT1dfTEVPUEFSRCkKKysgKHZvaWQpaGlkZVdvcmRE
ZWZpbml0aW9uV2luZG93OworI2VuZGlmCisKIC0gKE5TSW50ZWdlcilzcGVsbENoZWNrZXJEb2N1
bWVudFRhZzsKIC0gKHZvaWQpaGFuZGxlQ29ycmVjdGlvblBhbmVsUmVzdWx0OihOU1N0cmluZyop
cmVzdWx0OwogQGVuZAo=
</data>
<flag name="review"
          id="109365"
          type_id="1"
          status="-"
          setter="sam"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>111517</attachid>
            <date>2011-10-18 16:03:36 -0700</date>
            <delta_ts>2011-10-18 16:04:21 -0700</delta_ts>
            <desc>yet another patch</desc>
            <filename>textmate stdin aVtcgi.txt</filename>
            <type>text/plain</type>
            <size>1816</size>
            <attacher name="Damian Kaleta">dkaleta</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDk3Nzk5KQorKysgU291cmNlL1dlYktpdDIvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTEtMTAtMTggIERhbWlhbiBL
YWxldGEgIDxka2FsZXRhQGFwcGxlLmNvbT4KKworICAgICAgICBUaGlzIHJldmlzZWQgZml4IGRv
ZXNuJ3QgYnJlYWsgU0wgYnVpbGQuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD03MDM0NQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9tYWMvV0tWaWV3Lm1tOgorICAgICAgICAoK1tX
S1ZpZXcgaGlkZVdvcmREZWZpbml0aW9uV2luZG93XSk6IEhpZGVzIHRoZSBkZWZpbml0aW9uIHdp
bmRvdy4KKyAgICAgICAgKiBVSVByb2Nlc3MvQVBJL21hYy9XS1ZpZXdQcml2YXRlLmg6CisKIDIw
MTEtMTAtMTggIFNoZXJpZmYgQm90ICA8d2Via2l0LnJldmlldy5ib3RAZ21haWwuY29tPgogCiAg
ICAgICAgIFVucmV2aWV3ZWQsIHJvbGxpbmcgb3V0IHI5Nzc4OC4KSW5kZXg6IFNvdXJjZS9XZWJL
aXQyL1VJUHJvY2Vzcy9BUEkvbWFjL1dLVmlldy5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
S2l0Mi9VSVByb2Nlc3MvQVBJL21hYy9XS1ZpZXcubW0JKHJldmlzaW9uIDk3Nzk2KQorKysgU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9tYWMvV0tWaWV3Lm1tCSh3b3JraW5nIGNvcHkpCkBA
IC0yNjUzLDYgKzI2NTMsMTMgQEAgLSAodm9pZClwZXJmb3JtRGljdGlvbmFyeUxvb2t1cEF0Q3Vy
cmVudAogICAgIF9kYXRhLT5fcGFnZS0+cGVyZm9ybURpY3Rpb25hcnlMb29rdXBBdExvY2F0aW9u
KEZsb2F0UG9pbnQodGhlUG9pbnQueCwgdGhlUG9pbnQueSkpOwogfQogCisrICh2b2lkKWhpZGVX
b3JkRGVmaW5pdGlvbldpbmRvdworeworI2lmICFkZWZpbmVkKEJVSUxESU5HX09OX1NOT1dfTEVP
UEFSRCkKKyAgICBXS0hpZGVXb3JkRGVmaW5pdGlvbldpbmRvdygpOworI2VuZGlmCit9CisKIC0g
KE5TSW50ZWdlcilzcGVsbENoZWNrZXJEb2N1bWVudFRhZwogewogICAgIGlmICghX2RhdGEtPl9o
YXNTcGVsbENoZWNrZXJEb2N1bWVudFRhZykgewpJbmRleDogU291cmNlL1dlYktpdDIvVUlQcm9j
ZXNzL0FQSS9tYWMvV0tWaWV3UHJpdmF0ZS5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQy
L1VJUHJvY2Vzcy9BUEkvbWFjL1dLVmlld1ByaXZhdGUuaAkocmV2aXNpb24gOTc3OTYpCisrKyBT
b3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL21hYy9XS1ZpZXdQcml2YXRlLmgJKHdvcmtpbmcg
Y29weSkKQEAgLTM0LDYgKzM0LDcgQEAgQGludGVyZmFjZSBXS1ZpZXcgKFByaXZhdGUpCiAtIChC
T09MKWZyYW1lU2l6ZVVwZGF0ZXNEaXNhYmxlZDsKIAogLSAodm9pZClwZXJmb3JtRGljdGlvbmFy
eUxvb2t1cEF0Q3VycmVudE1vdXNlTG9jYXRpb247CisrICh2b2lkKWhpZGVXb3JkRGVmaW5pdGlv
bldpbmRvdzsKIAogLSAoTlNJbnRlZ2VyKXNwZWxsQ2hlY2tlckRvY3VtZW50VGFnOwogLSAodm9p
ZCloYW5kbGVDb3JyZWN0aW9uUGFuZWxSZXN1bHQ6KE5TU3RyaW5nKilyZXN1bHQ7Cg==
</data>
<flag name="review"
          id="109368"
          type_id="1"
          status="+"
          setter="sam"
    />
          </attachment>
      

    </bug>

</bugzilla>