<?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>30067</bug_id>
          
          <creation_ts>2009-10-05 03:43:38 -0700</creation_ts>
          <short_desc>Add a screenshot option to the menubar of QtLauncher</short_desc>
          <delta_ts>2009-10-05 16:03:31 -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>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>Qt</keywords>
          <priority>P3</priority>
          <bug_severity>Enhancement</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Girish Ramakrishnan">girish</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>commit-queue</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>152248</commentid>
    <comment_count>0</comment_count>
    <who name="Girish Ramakrishnan">girish</who>
    <bug_when>2009-10-05 03:43:38 -0700</bug_when>
    <thetext>Takes a screenshot of the current webview and optionally save it on disk.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>152249</commentid>
    <comment_count>1</comment_count>
      <attachid>40617</attachid>
    <who name="Girish Ramakrishnan">girish</who>
    <bug_when>2009-10-05 03:48:34 -0700</bug_when>
    <thetext>Created attachment 40617
Patch for adding snapshot option 

Patch is also available at http://gitorious.org/~girish/qtwebkit/girishs-clone/commits/qtlauncher_snapshot_30067.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>152266</commentid>
    <comment_count>2</comment_count>
    <who name="Tor Arne Vestbø">vestbo</who>
    <bug_when>2009-10-05 04:46:05 -0700</bug_when>
    <thetext>Thanks for your bug report.

In the future, please follow the QtWebKit bug reporting guidelines:

http://trac.webkit.org/wiki/QtWebKitContrib#ReportingBugs

In particular:

    - All bugs related to the Qt port of WebKit should have the keyword &apos;Qt&apos;
    - The &apos;WebKit Qt&apos; component should only be used for the QtWebKit API layer</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>152442</commentid>
    <comment_count>3</comment_count>
    <who name="Mark Rowe (bdash)">mrowe</who>
    <bug_when>2009-10-05 14:02:46 -0700</bug_when>
    <thetext>QtLauncher is intended only for testing of WebKit and not as a fully-featured browser.  How does this patch fit in to that?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>152480</commentid>
    <comment_count>4</comment_count>
      <attachid>40617</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2009-10-05 14:59:07 -0700</bug_when>
    <thetext>Comment on attachment 40617
Patch for adding snapshot option 

r=me. This helps towards making the launcher more useful for debugging. Wow, now it even has a shortcut ;)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>152506</commentid>
    <comment_count>5</comment_count>
      <attachid>40617</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2009-10-05 16:03:28 -0700</bug_when>
    <thetext>Comment on attachment 40617
Patch for adding snapshot option 

Clearing flags on attachment: 40617

Committed r49124: &lt;http://trac.webkit.org/changeset/49124&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>152507</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2009-10-05 16:03:31 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>40617</attachid>
            <date>2009-10-05 03:48:34 -0700</date>
            <delta_ts>2009-10-05 16:03:28 -0700</delta_ts>
            <desc>Patch for adding snapshot option </desc>
            <filename>qtlauncher_snapshot.patch</filename>
            <type>text/plain</type>
            <size>2772</size>
            <attacher name="Girish Ramakrishnan">girish</attacher>
            
              <data encoding="base64">Y29tbWl0IGFlNmFiZDc0YjVmNWUxMTFkMGQyYWJlOTE3NWJiMGZhZDRlZTNmM2IKQXV0aG9yOiBH
aXJpc2ggUmFtYWtyaXNobmFuIDxnaXJpc2hAZm9yd2FyZGJpYXMuaW4+CkRhdGU6ICAgV2VkIFNl
cCAzMCAwMDo0NDo0MiAyMDA5ICswNTMwCgogICAgQWRkIHNjcmVlbnNob3QgdG8gbWVudWJhcgoK
ZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9DaGFuZ2VMb2cgYi9XZWJLaXQvcXQvQ2hhbmdlTG9nCmlu
ZGV4IDcwMDU5MTYuLjdjNGRjOTAgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC9DaGFuZ2VMb2cKKysr
IGIvV2ViS2l0L3F0L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDA5LTEwLTA1ICBHaXJp
c2ggUmFtYWtyaXNobmFuICA8Z2lyaXNoQGZvcndhcmRiaWFzLmluPgorCisgICAgICAgIFJldmll
d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFkZCBzY3JlZW5zaG90IG9wdGlvbiB0
byBtZW51YmFyCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTMwMDY3CisKKyAgICAgICAgKiBRdExhdW5jaGVyL21haW4uY3BwOgorICAgICAgICAoTWFp
bldpbmRvdzo6c2NyZWVuc2hvdCk6CisgICAgICAgIChNYWluV2luZG93OjpzZXR1cFVJKToKKwog
MjAwOS0xMC0wMiAgTm9yYmVydCBMZXNlciAgPG5vcmJlcnQubGVzZXJAbm9raWEuY29tPgogCiAg
ICAgICAgIFJldmlld2VkIGJ5IFNpbW9uIEhhdXNtYW5uLgpkaWZmIC0tZ2l0IGEvV2ViS2l0L3F0
L1F0TGF1bmNoZXIvbWFpbi5jcHAgYi9XZWJLaXQvcXQvUXRMYXVuY2hlci9tYWluLmNwcAppbmRl
eCAwYjA2NzY5Li4wMWE2ODk3IDEwMDY0NAotLS0gYS9XZWJLaXQvcXQvUXRMYXVuY2hlci9tYWlu
LmNwcAorKysgYi9XZWJLaXQvcXQvUXRMYXVuY2hlci9tYWluLmNwcApAQCAtMSw1ICsxLDYgQEAK
IC8qCiAgKiBDb3B5cmlnaHQgKEMpIDIwMDkgTm9raWEgQ29ycG9yYXRpb24gYW5kL29yIGl0cyBz
dWJzaWRpYXJ5KC1pZXMpCisgKiBDb3B5cmlnaHQgKEMpIDIwMDkgR2lyaXNoIFJhbWFrcmlzaG5h
biA8Z2lyaXNoQGZvcndhcmRiaWFzLmluPgogICogQ29weXJpZ2h0IChDKSAyMDA2IEdlb3JnZSBT
dGFpa29zIDxzdGFpa29zQGtkZS5vcmc+CiAgKiBDb3B5cmlnaHQgKEMpIDIwMDYgRGlyayBNdWVs
bGVyIDxtdWVsbGVyQGtkZS5vcmc+CiAgKiBDb3B5cmlnaHQgKEMpIDIwMDYgWmFjayBSdXNpbiA8
emFja0BrZGUub3JnPgpAQCAtMjA4LDYgKzIwOSwyMiBAQCBwcm90ZWN0ZWQgc2xvdHM6CiAjZW5k
aWYKICAgICB9CiAKKyAgICB2b2lkIHNjcmVlbnNob3QoKQorICAgIHsKKyAgICAgICAgUVBpeG1h
cCBwaXhtYXAgPSBRUGl4bWFwOjpncmFiV2lkZ2V0KHZpZXcpOworICAgICAgICBRTGFiZWwqIGxh
YmVsID0gbmV3IFFMYWJlbDsKKyAgICAgICAgbGFiZWwtPnNldEF0dHJpYnV0ZShRdDo6V0FfRGVs
ZXRlT25DbG9zZSk7CisgICAgICAgIGxhYmVsLT5zZXRXaW5kb3dUaXRsZSgiU2NyZWVuc2hvdCAt
IFByZXZpZXciKTsKKyAgICAgICAgbGFiZWwtPnNldFBpeG1hcChwaXhtYXApOworICAgICAgICBs
YWJlbC0+c2hvdygpOworCisgICAgICAgIFFTdHJpbmcgZmlsZU5hbWUgPSBRRmlsZURpYWxvZzo6
Z2V0U2F2ZUZpbGVOYW1lKGxhYmVsLCAiU2NyZWVuc2hvdCIpOworICAgICAgICBpZiAoIWZpbGVO
YW1lLmlzRW1wdHkoKSkgeworICAgICAgICAgICAgcGl4bWFwLnNhdmUoZmlsZU5hbWUsICJwbmci
KTsKKyAgICAgICAgICAgIGxhYmVsLT5zZXRXaW5kb3dUaXRsZShRU3RyaW5nKCJTY3JlZW5zaG90
IC0gU2F2ZWQgYXQgJTEiKS5hcmcoZmlsZU5hbWUpKTsKKyAgICAgICAgfQorICAgIH0KKwogICAg
IHZvaWQgc2V0RWRpdGFibGUoYm9vbCBvbikgewogICAgICAgICB2aWV3LT5wYWdlKCktPnNldENv
bnRlbnRFZGl0YWJsZShvbik7CiAgICAgICAgIGZvcm1hdE1lbnVBY3Rpb24tPnNldFZpc2libGUo
b24pOwpAQCAtMjc1LDYgKzI5Miw3IEBAIHByaXZhdGU6CiAjaWYgUVRfVkVSU0lPTiA+PSAweDA0
MDQwMAogICAgICAgICBmaWxlTWVudS0+YWRkQWN0aW9uKHRyKCJQcmludCIpLCB0aGlzLCBTTE9U
KHByaW50KCkpKTsKICNlbmRpZgorICAgICAgICBRQWN0aW9uKiBzY3JlZW5zaG90ID0gZmlsZU1l
bnUtPmFkZEFjdGlvbigiU2NyZWVuc2hvdCIsIHRoaXMsIFNMT1Qoc2NyZWVuc2hvdCgpKSk7CiAg
ICAgICAgIGZpbGVNZW51LT5hZGRBY3Rpb24oIkNsb3NlIiwgdGhpcywgU0xPVChjbG9zZSgpKSk7
CiAKICAgICAgICAgUU1lbnUgKmVkaXRNZW51ID0gbWVudUJhcigpLT5hZGRNZW51KCImRWRpdCIp
OwpAQCAtMzEzLDYgKzMzMSw3IEBAIHByaXZhdGU6CiAgICAgICAgIHdyaXRpbmdNZW51LT5hZGRB
Y3Rpb24odmlldy0+cGFnZUFjdGlvbihRV2ViUGFnZTo6U2V0VGV4dERpcmVjdGlvblJpZ2h0VG9M
ZWZ0KSk7CiAKICAgICAgICAgbmV3V2luZG93LT5zZXRTaG9ydGN1dChRS2V5U2VxdWVuY2UoUXQ6
OkNUUkwgfCBRdDo6S2V5X04pKTsKKyAgICAgICAgc2NyZWVuc2hvdC0+c2V0U2hvcnRjdXQoUUtl
eVNlcXVlbmNlKFF0OjpDVFJMIHwgUXQ6OktleV9TKSk7CiAgICAgICAgIHZpZXctPnBhZ2VBY3Rp
b24oUVdlYlBhZ2U6OkJhY2spLT5zZXRTaG9ydGN1dChRS2V5U2VxdWVuY2U6OkJhY2spOwogICAg
ICAgICB2aWV3LT5wYWdlQWN0aW9uKFFXZWJQYWdlOjpTdG9wKS0+c2V0U2hvcnRjdXQoUXQ6Oktl
eV9Fc2NhcGUpOwogICAgICAgICB2aWV3LT5wYWdlQWN0aW9uKFFXZWJQYWdlOjpGb3J3YXJkKS0+
c2V0U2hvcnRjdXQoUUtleVNlcXVlbmNlOjpGb3J3YXJkKTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>