<?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>60991</bug_id>
          
          <creation_ts>2011-05-17 14:47:25 -0700</creation_ts>
          <short_desc>WebKit2: Needs API to set a custom drop target</short_desc>
          <delta_ts>2011-05-17 16:17: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>WebKit2</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, PlatformOnly</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Brian Weinstein">bweinstein</reporter>
          <assigned_to name="Brian Weinstein">bweinstein</assigned_to>
          <cc>aroben</cc>
    
    <cc>jeffm</cc>
    
    <cc>sfalken</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>405297</commentid>
    <comment_count>0</comment_count>
    <who name="Brian Weinstein">bweinstein</who>
    <bug_when>2011-05-17 14:47:25 -0700</bug_when>
    <thetext>WebKit2 needs an API to set a custom drop target for a view. It exists in WebKit1.

&lt;rdar://problem/9090868&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>405305</commentid>
    <comment_count>1</comment_count>
      <attachid>93821</attachid>
    <who name="Brian Weinstein">bweinstein</who>
    <bug_when>2011-05-17 14:54:41 -0700</bug_when>
    <thetext>Created attachment 93821
[PATCH] Fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>405307</commentid>
    <comment_count>2</comment_count>
      <attachid>93821</attachid>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-05-17 14:56:07 -0700</bug_when>
    <thetext>Comment on attachment 93821
[PATCH] Fix

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

&gt; Source/WebKit2/UIProcess/win/WebView.cpp:1128
&gt; +void WebView::setCustomDropTarget(IDropTarget* dropTarget)
&gt; +{
&gt; +    ::RevokeDragDrop(m_window);
&gt; +    ::RegisterDragDrop(m_window, dropTarget);
&gt; +}

If someone passes in 0 here, we should set the drop target back to our standard one. Otherwise clients will have no way to clear their custom drop target.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>405311</commentid>
    <comment_count>3</comment_count>
    <who name="Brian Weinstein">bweinstein</who>
    <bug_when>2011-05-17 14:58:49 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 93821 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=93821&amp;action=review
&gt; 
&gt; &gt; Source/WebKit2/UIProcess/win/WebView.cpp:1128
&gt; &gt; +void WebView::setCustomDropTarget(IDropTarget* dropTarget)
&gt; &gt; +{
&gt; &gt; +    ::RevokeDragDrop(m_window);
&gt; &gt; +    ::RegisterDragDrop(m_window, dropTarget);
&gt; &gt; +}
&gt; 
&gt; If someone passes in 0 here, we should set the drop target back to our standard one. Otherwise clients will have no way to clear their custom drop target.

Fixed. New patch coming soon.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>405333</commentid>
    <comment_count>4</comment_count>
      <attachid>93832</attachid>
    <who name="Brian Weinstein">bweinstein</who>
    <bug_when>2011-05-17 15:21:44 -0700</bug_when>
    <thetext>Created attachment 93832
[PATCH] Fix v2</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>405334</commentid>
    <comment_count>5</comment_count>
      <attachid>93832</attachid>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-05-17 15:24:31 -0700</bug_when>
    <thetext>Comment on attachment 93832
[PATCH] Fix v2

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

&gt; Source/WebKit2/UIProcess/win/WebView.cpp:1132
&gt; +void WebView::setCustomDropTarget(IDropTarget* dropTarget)
&gt; +{
&gt; +    ::RevokeDragDrop(m_window);
&gt; +
&gt; +    if (dropTarget)
&gt; +        ::RegisterDragDrop(m_window, dropTarget);
&gt; +    else
&gt; +        ::RegisterDragDrop(m_window, this);
&gt; +}

This won&apos;t work correctly if m_window has been destroyed or the page has already been closed. We shouldn&apos;t do anything in those cases.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>405338</commentid>
    <comment_count>6</comment_count>
      <attachid>93834</attachid>
    <who name="Brian Weinstein">bweinstein</who>
    <bug_when>2011-05-17 15:30:32 -0700</bug_when>
    <thetext>Created attachment 93834
[PATCH] Fix v3</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>405373</commentid>
    <comment_count>7</comment_count>
    <who name="Brian Weinstein">bweinstein</who>
    <bug_when>2011-05-17 16:17:35 -0700</bug_when>
    <thetext>Landed in r86717.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>93821</attachid>
            <date>2011-05-17 14:54:41 -0700</date>
            <delta_ts>2011-05-17 15:21:44 -0700</delta_ts>
            <desc>[PATCH] Fix</desc>
            <filename>set_custom_drop_target.patch</filename>
            <type>text/plain</type>
            <size>3225</size>
            <attacher name="Brian Weinstein">bweinstein</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDg2NzA4KQorKysgU291cmNlL1dlYktpdDIvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMjEgQEAKKzIwMTEtMDUtMTcgIEJyaWFuIFdl
aW5zdGVpbiAgPGJ3ZWluc3RlaW5AYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P
Qk9EWSAoT09QUyEpLgorCisgICAgICAgIFdlYktpdDI6IE5lZWRzIEFQSSB0byBzZXQgYSBjdXN0
b20gZHJvcCB0YXJnZXQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu
Y2dpP2lkPTYwOTkxCisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS85MDkwODY4PgorICAgICAgICAK
KyAgICAgICAgQWRkIEFQSSB0byBzZXQgYSBjdXN0b20gZHJvcCB0YXJnZXQgb24gYSBXS1ZpZXcu
CisKKyAgICAgICAgKiBVSVByb2Nlc3MvQVBJL0Mvd2luL1dLVmlldy5jcHA6CisgICAgICAgIChX
S1ZpZXdTZXRDdXN0b21Ecm9wVGFyZ2V0KTogQ2FsbCB0aHJvdWdoIHRvIFdlYlZpZXc6OnNldEN1
c3RvbURyb3BUYXJnZXQuCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9DL3dpbi9XS1ZpZXcuaDoK
KyAgICAgICAgKiBVSVByb2Nlc3Mvd2luL1dlYlZpZXcuY3BwOgorICAgICAgICAoV2ViS2l0OjpX
ZWJWaWV3OjpzZXRDdXN0b21Ecm9wVGFyZ2V0KTogUmV2b2tlIHRoZSBjdXJyZW50IGRyb3AgdGFy
Z2V0LCBhbmQgcmVnaXN0ZXIgdGhlCisgICAgICAgICAgICBjdXN0b20gb25lLgorICAgICAgICAq
IFVJUHJvY2Vzcy93aW4vV2ViVmlldy5oOgorCiAyMDExLTA1LTE3ICBBbmRyZWFzIEtsaW5nICA8
a2xpbmdAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBLZW5uZXRoIFJvaGRlIENo
cmlzdGlhbnNlbi4KSW5kZXg6IFNvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvQy93aW4vV0tW
aWV3LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL0Mvd2lu
L1dLVmlldy5jcHAJKHJldmlzaW9uIDg2NTgxKQorKysgU291cmNlL1dlYktpdDIvVUlQcm9jZXNz
L0FQSS9DL3dpbi9XS1ZpZXcuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xMTksMyArMTE5LDggQEAg
Ym9vbCBXS1ZpZXdEcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZChXSwogICAgIHJldHVybiB0b0lt
cGwodmlld1JlZiktPnBhZ2UoKS0+ZHJhd3NUcmFuc3BhcmVudEJhY2tncm91bmQoKTsKIH0KIAor
dm9pZCBXS1ZpZXdTZXRDdXN0b21Ecm9wVGFyZ2V0KFdLVmlld1JlZiB2aWV3UmVmLCBJRHJvcFRh
cmdldCogZHJvcFRhcmdldCkKK3sKKyAgICByZXR1cm4gdG9JbXBsKHZpZXdSZWYpLT5zZXRDdXN0
b21Ecm9wVGFyZ2V0KGRyb3BUYXJnZXQpOworfQorCkluZGV4OiBTb3VyY2UvV2ViS2l0Mi9VSVBy
b2Nlc3MvQVBJL0Mvd2luL1dLVmlldy5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQyL1VJ
UHJvY2Vzcy9BUEkvQy93aW4vV0tWaWV3LmgJKHJldmlzaW9uIDg2NTgxKQorKysgU291cmNlL1dl
YktpdDIvVUlQcm9jZXNzL0FQSS9DL3dpbi9XS1ZpZXcuaAkod29ya2luZyBjb3B5KQpAQCAtODMs
NiArODMsOCBAQCBXS19FWFBPUlQgYm9vbCBXS1ZpZXdJc1BhZ2VPdmVybGF5SW5zdGFsCiBXS19F
WFBPUlQgdm9pZCBXS1ZpZXdTZXREcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZChXS1ZpZXdSZWYg
dmlldywgYm9vbCBkcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZCk7CiBXS19FWFBPUlQgYm9vbCBX
S1ZpZXdEcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZChXS1ZpZXdSZWYgdmlldyk7CiAKK1dLX0VY
UE9SVCB2b2lkIFdLVmlld1NldEN1c3RvbURyb3BUYXJnZXQoV0tWaWV3UmVmIHZpZXcsIElEcm9w
VGFyZ2V0Kik7CisKICNpZmRlZiBfX2NwbHVzcGx1cwogfQogI2VuZGlmCkluZGV4OiBTb3VyY2Uv
V2ViS2l0Mi9VSVByb2Nlc3Mvd2luL1dlYlZpZXcuY3BwCj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9X
ZWJLaXQyL1VJUHJvY2Vzcy93aW4vV2ViVmlldy5jcHAJKHJldmlzaW9uIDg2NTgxKQorKysgU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL3dpbi9XZWJWaWV3LmNwcAkod29ya2luZyBjb3B5KQpAQCAt
MTEyMSw2ICsxMTIxLDEyIEBAIHZvaWQgV2ViVmlldzo6dW5hcHBseUVkaXRDb21tYW5kKFdlYkVk
aXQKICAgICBjb21tYW5kLT51bmFwcGx5KCk7CiB9CiAKK3ZvaWQgV2ViVmlldzo6c2V0Q3VzdG9t
RHJvcFRhcmdldChJRHJvcFRhcmdldCogZHJvcFRhcmdldCkKK3sKKyAgICA6OlJldm9rZURyYWdE
cm9wKG1fd2luZG93KTsKKyAgICA6OlJlZ2lzdGVyRHJhZ0Ryb3AobV93aW5kb3csIGRyb3BUYXJn
ZXQpOworfQorCiBGbG9hdFJlY3QgV2ViVmlldzo6Y29udmVydFRvRGV2aWNlU3BhY2UoY29uc3Qg
RmxvYXRSZWN0JiByZWN0KQogewogICAgIHJldHVybiByZWN0OwpJbmRleDogU291cmNlL1dlYktp
dDIvVUlQcm9jZXNzL3dpbi9XZWJWaWV3LmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYktpdDIv
VUlQcm9jZXNzL3dpbi9XZWJWaWV3LmgJKHJldmlzaW9uIDg2NTgxKQorKysgU291cmNlL1dlYktp
dDIvVUlQcm9jZXNzL3dpbi9XZWJWaWV3LmgJKHdvcmtpbmcgY29weSkKQEAgLTg5LDYgKzg5LDgg
QEAgcHVibGljOgogICAgIHZvaWQgcmVhcHBseUVkaXRDb21tYW5kKFdlYkVkaXRDb21tYW5kUHJv
eHkqKTsKICAgICB2b2lkIHVuYXBwbHlFZGl0Q29tbWFuZChXZWJFZGl0Q29tbWFuZFByb3h5Kik7
CiAKKyAgICB2b2lkIHNldEN1c3RvbURyb3BUYXJnZXQoSURyb3BUYXJnZXQqKTsKKwogICAgIC8v
IElVbmtub3duCiAgICAgdmlydHVhbCBIUkVTVUxUIFNURE1FVEhPRENBTExUWVBFIFF1ZXJ5SW50
ZXJmYWNlKFJFRklJRCByaWlkLCB2b2lkKiogcHB2T2JqZWN0KTsKICAgICB2aXJ0dWFsIFVMT05H
IFNURE1FVEhPRENBTExUWVBFIEFkZFJlZih2b2lkKTsK
</data>
<flag name="review"
          id="87048"
          type_id="1"
          status="-"
          setter="aroben"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>93832</attachid>
            <date>2011-05-17 15:21:44 -0700</date>
            <delta_ts>2011-05-17 15:30:32 -0700</delta_ts>
            <desc>[PATCH] Fix v2</desc>
            <filename>set_custom_drop_target.patch</filename>
            <type>text/plain</type>
            <size>3307</size>
            <attacher name="Brian Weinstein">bweinstein</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDg2NzA4KQorKysgU291cmNlL1dlYktpdDIvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMjEgQEAKKzIwMTEtMDUtMTcgIEJyaWFuIFdl
aW5zdGVpbiAgPGJ3ZWluc3RlaW5AYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P
Qk9EWSAoT09QUyEpLgorCisgICAgICAgIFdlYktpdDI6IE5lZWRzIEFQSSB0byBzZXQgYSBjdXN0
b20gZHJvcCB0YXJnZXQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu
Y2dpP2lkPTYwOTkxCisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS85MDkwODY4PgorICAgICAgICAK
KyAgICAgICAgQWRkIEFQSSB0byBzZXQgYSBjdXN0b20gZHJvcCB0YXJnZXQgb24gYSBXS1ZpZXcu
CisKKyAgICAgICAgKiBVSVByb2Nlc3MvQVBJL0Mvd2luL1dLVmlldy5jcHA6CisgICAgICAgIChX
S1ZpZXdTZXRDdXN0b21Ecm9wVGFyZ2V0KTogQ2FsbCB0aHJvdWdoIHRvIFdlYlZpZXc6OnNldEN1
c3RvbURyb3BUYXJnZXQuCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9DL3dpbi9XS1ZpZXcuaDoK
KyAgICAgICAgKiBVSVByb2Nlc3Mvd2luL1dlYlZpZXcuY3BwOgorICAgICAgICAoV2ViS2l0OjpX
ZWJWaWV3OjpzZXRDdXN0b21Ecm9wVGFyZ2V0KTogUmV2b2tlIHRoZSBjdXJyZW50IGRyb3AgdGFy
Z2V0LCBhbmQgcmVnaXN0ZXIgdGhlCisgICAgICAgICAgICBjdXN0b20gb25lLgorICAgICAgICAq
IFVJUHJvY2Vzcy93aW4vV2ViVmlldy5oOgorCiAyMDExLTA1LTE3ICBBbmRyZWFzIEtsaW5nICA8
a2xpbmdAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBLZW5uZXRoIFJvaGRlIENo
cmlzdGlhbnNlbi4KSW5kZXg6IFNvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvQy93aW4vV0tW
aWV3LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL0Mvd2lu
L1dLVmlldy5jcHAJKHJldmlzaW9uIDg2NTgxKQorKysgU291cmNlL1dlYktpdDIvVUlQcm9jZXNz
L0FQSS9DL3dpbi9XS1ZpZXcuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xMTksMyArMTE5LDggQEAg
Ym9vbCBXS1ZpZXdEcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZChXSwogICAgIHJldHVybiB0b0lt
cGwodmlld1JlZiktPnBhZ2UoKS0+ZHJhd3NUcmFuc3BhcmVudEJhY2tncm91bmQoKTsKIH0KIAor
dm9pZCBXS1ZpZXdTZXRDdXN0b21Ecm9wVGFyZ2V0KFdLVmlld1JlZiB2aWV3UmVmLCBJRHJvcFRh
cmdldCogZHJvcFRhcmdldCkKK3sKKyAgICByZXR1cm4gdG9JbXBsKHZpZXdSZWYpLT5zZXRDdXN0
b21Ecm9wVGFyZ2V0KGRyb3BUYXJnZXQpOworfQorCkluZGV4OiBTb3VyY2UvV2ViS2l0Mi9VSVBy
b2Nlc3MvQVBJL0Mvd2luL1dLVmlldy5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQyL1VJ
UHJvY2Vzcy9BUEkvQy93aW4vV0tWaWV3LmgJKHJldmlzaW9uIDg2NTgxKQorKysgU291cmNlL1dl
YktpdDIvVUlQcm9jZXNzL0FQSS9DL3dpbi9XS1ZpZXcuaAkod29ya2luZyBjb3B5KQpAQCAtODMs
NiArODMsOCBAQCBXS19FWFBPUlQgYm9vbCBXS1ZpZXdJc1BhZ2VPdmVybGF5SW5zdGFsCiBXS19F
WFBPUlQgdm9pZCBXS1ZpZXdTZXREcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZChXS1ZpZXdSZWYg
dmlldywgYm9vbCBkcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZCk7CiBXS19FWFBPUlQgYm9vbCBX
S1ZpZXdEcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZChXS1ZpZXdSZWYgdmlldyk7CiAKK1dLX0VY
UE9SVCB2b2lkIFdLVmlld1NldEN1c3RvbURyb3BUYXJnZXQoV0tWaWV3UmVmIHZpZXcsIElEcm9w
VGFyZ2V0Kik7CisKICNpZmRlZiBfX2NwbHVzcGx1cwogfQogI2VuZGlmCkluZGV4OiBTb3VyY2Uv
V2ViS2l0Mi9VSVByb2Nlc3Mvd2luL1dlYlZpZXcuY3BwCj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9X
ZWJLaXQyL1VJUHJvY2Vzcy93aW4vV2ViVmlldy5jcHAJKHJldmlzaW9uIDg2NTgxKQorKysgU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL3dpbi9XZWJWaWV3LmNwcAkod29ya2luZyBjb3B5KQpAQCAt
MTEyMSw2ICsxMTIxLDE2IEBAIHZvaWQgV2ViVmlldzo6dW5hcHBseUVkaXRDb21tYW5kKFdlYkVk
aXQKICAgICBjb21tYW5kLT51bmFwcGx5KCk7CiB9CiAKK3ZvaWQgV2ViVmlldzo6c2V0Q3VzdG9t
RHJvcFRhcmdldChJRHJvcFRhcmdldCogZHJvcFRhcmdldCkKK3sKKyAgICA6OlJldm9rZURyYWdE
cm9wKG1fd2luZG93KTsKKworICAgIGlmIChkcm9wVGFyZ2V0KQorICAgICAgICA6OlJlZ2lzdGVy
RHJhZ0Ryb3AobV93aW5kb3csIGRyb3BUYXJnZXQpOworICAgIGVsc2UKKyAgICAgICAgOjpSZWdp
c3RlckRyYWdEcm9wKG1fd2luZG93LCB0aGlzKTsKK30KKwogRmxvYXRSZWN0IFdlYlZpZXc6OmNv
bnZlcnRUb0RldmljZVNwYWNlKGNvbnN0IEZsb2F0UmVjdCYgcmVjdCkKIHsKICAgICByZXR1cm4g
cmVjdDsKSW5kZXg6IFNvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy93aW4vV2ViVmlldy5oCj09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy93aW4vV2ViVmlldy5oCShyZXZpc2lv
biA4NjU4MSkKKysrIFNvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy93aW4vV2ViVmlldy5oCSh3b3Jr
aW5nIGNvcHkpCkBAIC04OSw2ICs4OSw4IEBAIHB1YmxpYzoKICAgICB2b2lkIHJlYXBwbHlFZGl0
Q29tbWFuZChXZWJFZGl0Q29tbWFuZFByb3h5Kik7CiAgICAgdm9pZCB1bmFwcGx5RWRpdENvbW1h
bmQoV2ViRWRpdENvbW1hbmRQcm94eSopOwogCisgICAgdm9pZCBzZXRDdXN0b21Ecm9wVGFyZ2V0
KElEcm9wVGFyZ2V0Kik7CisKICAgICAvLyBJVW5rbm93bgogICAgIHZpcnR1YWwgSFJFU1VMVCBT
VERNRVRIT0RDQUxMVFlQRSBRdWVyeUludGVyZmFjZShSRUZJSUQgcmlpZCwgdm9pZCoqIHBwdk9i
amVjdCk7CiAgICAgdmlydHVhbCBVTE9ORyBTVERNRVRIT0RDQUxMVFlQRSBBZGRSZWYodm9pZCk7
Cg==
</data>
<flag name="review"
          id="87058"
          type_id="1"
          status="-"
          setter="aroben"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>93834</attachid>
            <date>2011-05-17 15:30:32 -0700</date>
            <delta_ts>2011-05-17 15:32:46 -0700</delta_ts>
            <desc>[PATCH] Fix v3</desc>
            <filename>set_custom_drop_target.patch</filename>
            <type>text/plain</type>
            <size>3368</size>
            <attacher name="Brian Weinstein">bweinstein</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDg2NzA4KQorKysgU291cmNlL1dlYktpdDIvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMjEgQEAKKzIwMTEtMDUtMTcgIEJyaWFuIFdl
aW5zdGVpbiAgPGJ3ZWluc3RlaW5AYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P
Qk9EWSAoT09QUyEpLgorCisgICAgICAgIFdlYktpdDI6IE5lZWRzIEFQSSB0byBzZXQgYSBjdXN0
b20gZHJvcCB0YXJnZXQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu
Y2dpP2lkPTYwOTkxCisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS85MDkwODY4PgorICAgICAgICAK
KyAgICAgICAgQWRkIEFQSSB0byBzZXQgYSBjdXN0b20gZHJvcCB0YXJnZXQgb24gYSBXS1ZpZXcu
CisKKyAgICAgICAgKiBVSVByb2Nlc3MvQVBJL0Mvd2luL1dLVmlldy5jcHA6CisgICAgICAgIChX
S1ZpZXdTZXRDdXN0b21Ecm9wVGFyZ2V0KTogQ2FsbCB0aHJvdWdoIHRvIFdlYlZpZXc6OnNldEN1
c3RvbURyb3BUYXJnZXQuCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9DL3dpbi9XS1ZpZXcuaDoK
KyAgICAgICAgKiBVSVByb2Nlc3Mvd2luL1dlYlZpZXcuY3BwOgorICAgICAgICAoV2ViS2l0OjpX
ZWJWaWV3OjpzZXRDdXN0b21Ecm9wVGFyZ2V0KTogUmV2b2tlIHRoZSBjdXJyZW50IGRyb3AgdGFy
Z2V0LCBhbmQgcmVnaXN0ZXIgdGhlCisgICAgICAgICAgICBjdXN0b20gb25lLgorICAgICAgICAq
IFVJUHJvY2Vzcy93aW4vV2ViVmlldy5oOgorCiAyMDExLTA1LTE3ICBBbmRyZWFzIEtsaW5nICA8
a2xpbmdAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBLZW5uZXRoIFJvaGRlIENo
cmlzdGlhbnNlbi4KSW5kZXg6IFNvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvQy93aW4vV0tW
aWV3LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL0Mvd2lu
L1dLVmlldy5jcHAJKHJldmlzaW9uIDg2NTgxKQorKysgU291cmNlL1dlYktpdDIvVUlQcm9jZXNz
L0FQSS9DL3dpbi9XS1ZpZXcuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xMTksMyArMTE5LDggQEAg
Ym9vbCBXS1ZpZXdEcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZChXSwogICAgIHJldHVybiB0b0lt
cGwodmlld1JlZiktPnBhZ2UoKS0+ZHJhd3NUcmFuc3BhcmVudEJhY2tncm91bmQoKTsKIH0KIAor
dm9pZCBXS1ZpZXdTZXRDdXN0b21Ecm9wVGFyZ2V0KFdLVmlld1JlZiB2aWV3UmVmLCBJRHJvcFRh
cmdldCogZHJvcFRhcmdldCkKK3sKKyAgICByZXR1cm4gdG9JbXBsKHZpZXdSZWYpLT5zZXRDdXN0
b21Ecm9wVGFyZ2V0KGRyb3BUYXJnZXQpOworfQorCkluZGV4OiBTb3VyY2UvV2ViS2l0Mi9VSVBy
b2Nlc3MvQVBJL0Mvd2luL1dLVmlldy5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQyL1VJ
UHJvY2Vzcy9BUEkvQy93aW4vV0tWaWV3LmgJKHJldmlzaW9uIDg2NTgxKQorKysgU291cmNlL1dl
YktpdDIvVUlQcm9jZXNzL0FQSS9DL3dpbi9XS1ZpZXcuaAkod29ya2luZyBjb3B5KQpAQCAtODMs
NiArODMsOCBAQCBXS19FWFBPUlQgYm9vbCBXS1ZpZXdJc1BhZ2VPdmVybGF5SW5zdGFsCiBXS19F
WFBPUlQgdm9pZCBXS1ZpZXdTZXREcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZChXS1ZpZXdSZWYg
dmlldywgYm9vbCBkcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZCk7CiBXS19FWFBPUlQgYm9vbCBX
S1ZpZXdEcmF3c1RyYW5zcGFyZW50QmFja2dyb3VuZChXS1ZpZXdSZWYgdmlldyk7CiAKK1dLX0VY
UE9SVCB2b2lkIFdLVmlld1NldEN1c3RvbURyb3BUYXJnZXQoV0tWaWV3UmVmIHZpZXcsIElEcm9w
VGFyZ2V0Kik7CisKICNpZmRlZiBfX2NwbHVzcGx1cwogfQogI2VuZGlmCkluZGV4OiBTb3VyY2Uv
V2ViS2l0Mi9VSVByb2Nlc3Mvd2luL1dlYlZpZXcuY3BwCj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9X
ZWJLaXQyL1VJUHJvY2Vzcy93aW4vV2ViVmlldy5jcHAJKHJldmlzaW9uIDg2NTgxKQorKysgU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL3dpbi9XZWJWaWV3LmNwcAkod29ya2luZyBjb3B5KQpAQCAt
MTEyMSw2ICsxMTIxLDE5IEBAIHZvaWQgV2ViVmlldzo6dW5hcHBseUVkaXRDb21tYW5kKFdlYkVk
aXQKICAgICBjb21tYW5kLT51bmFwcGx5KCk7CiB9CiAKK3ZvaWQgV2ViVmlldzo6c2V0Q3VzdG9t
RHJvcFRhcmdldChJRHJvcFRhcmdldCogZHJvcFRhcmdldCkKK3sKKyAgICBpZiAoIW1fcGFnZS0+
aXNWYWxpZCgpIHx8ICFtX3dpbmRvdykKKyAgICAgICAgcmV0dXJuOworCisgICAgOjpSZXZva2VE
cmFnRHJvcChtX3dpbmRvdyk7CisKKyAgICBpZiAoZHJvcFRhcmdldCkKKyAgICAgICAgOjpSZWdp
c3RlckRyYWdEcm9wKG1fd2luZG93LCBkcm9wVGFyZ2V0KTsKKyAgICBlbHNlCisgICAgICAgIDo6
UmVnaXN0ZXJEcmFnRHJvcChtX3dpbmRvdywgdGhpcyk7Cit9CisKIEZsb2F0UmVjdCBXZWJWaWV3
Ojpjb252ZXJ0VG9EZXZpY2VTcGFjZShjb25zdCBGbG9hdFJlY3QmIHJlY3QpCiB7CiAgICAgcmV0
dXJuIHJlY3Q7CkluZGV4OiBTb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3Mvd2luL1dlYlZpZXcuaAo9
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3Mvd2luL1dlYlZpZXcuaAkocmV2
aXNpb24gODY1ODEpCisrKyBTb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3Mvd2luL1dlYlZpZXcuaAko
d29ya2luZyBjb3B5KQpAQCAtODksNiArODksOCBAQCBwdWJsaWM6CiAgICAgdm9pZCByZWFwcGx5
RWRpdENvbW1hbmQoV2ViRWRpdENvbW1hbmRQcm94eSopOwogICAgIHZvaWQgdW5hcHBseUVkaXRD
b21tYW5kKFdlYkVkaXRDb21tYW5kUHJveHkqKTsKIAorICAgIHZvaWQgc2V0Q3VzdG9tRHJvcFRh
cmdldChJRHJvcFRhcmdldCopOworCiAgICAgLy8gSVVua25vd24KICAgICB2aXJ0dWFsIEhSRVNV
TFQgU1RETUVUSE9EQ0FMTFRZUEUgUXVlcnlJbnRlcmZhY2UoUkVGSUlEIHJpaWQsIHZvaWQqKiBw
cHZPYmplY3QpOwogICAgIHZpcnR1YWwgVUxPTkcgU1RETUVUSE9EQ0FMTFRZUEUgQWRkUmVmKHZv
aWQpOwo=
</data>
<flag name="review"
          id="87060"
          type_id="1"
          status="+"
          setter="aroben"
    />
          </attachment>
      

    </bug>

</bugzilla>