<?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>36887</bug_id>
          
          <creation_ts>2010-03-31 08:52:18 -0700</creation_ts>
          <short_desc>[chromium] including WebKit.gyp:webkit should automatically add Skia and NPAPI include paths</short_desc>
          <delta_ts>2010-03-31 12:20:04 -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>All</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Darin Fisher (:fishd, Google)">fishd</reporter>
          <assigned_to name="Darin Fisher (:fishd, Google)">fishd</assigned_to>
          <cc>abarth</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>206531</commentid>
    <comment_count>0</comment_count>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2010-03-31 08:52:18 -0700</bug_when>
    <thetext>[chromium] including WebKit.gyp:webkit should automatically add Skia and NPAPI include paths

This is what WebCore.gyp does, and since I&apos;m working to remove direct usage of WebCore.gyp in the Chromium repository, it would be good to do the same for WebKit.gyp.  That way consumers of the WebKit API do not all have to manually include Skia and NPAPI settings.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>206532</commentid>
    <comment_count>1</comment_count>
      <attachid>52175</attachid>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2010-03-31 08:54:37 -0700</bug_when>
    <thetext>Created attachment 52175
v1 patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>206584</commentid>
    <comment_count>2</comment_count>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2010-03-31 11:01:12 -0700</bug_when>
    <thetext>Landed as http://trac.webkit.org/changeset/56853</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>206633</commentid>
    <comment_count>3</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-03-31 12:18:18 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/56853 appears to have broken Leopard Intel Debug (Tests)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>206637</commentid>
    <comment_count>4</comment_count>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2010-03-31 12:20:04 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; http://trac.webkit.org/changeset/56853 appears to have broken Leopard Intel
&gt; Debug (Tests)

That must be unrelated since this was purely a GYP change.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>52175</attachid>
            <date>2010-03-31 08:54:37 -0700</date>
            <delta_ts>2010-03-31 10:40:14 -0700</delta_ts>
            <desc>v1 patch</desc>
            <filename>gyp_1.txt</filename>
            <type>text/plain</type>
            <size>1366</size>
            <attacher name="Darin Fisher (:fishd, Google)">fishd</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC9jaHJvbWl1bS9DaGFuZ2VMb2cNCj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0NCi0tLSBXZWJLaXQv
Y2hyb21pdW0vQ2hhbmdlTG9nCShyZXZpc2lvbiA1Njg0MSkKKysrIFdlYktpdC9jaHJvbWl1bS9D
aGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxNCBAQAorMjAxMC0wMy0zMSAgRGFy
aW4gRmlzaGVyICA8ZGFyaW5AY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P
Qk9EWSAoT09QUyEpLgorCisgICAgICAgIFtjaHJvbWl1bV0gaW5jbHVkaW5nIFdlYktpdC5neXA6
d2Via2l0IHNob3VsZCBhdXRvbWF0aWNhbGx5IGFkZCBTa2lhCisgICAgICAgIGFuZCBOUEFQSSBp
bmNsdWRlIHBhdGhzCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu
Y2dpP2lkPTM2ODg3CisKKyAgICAgICAgKiBXZWJLaXQuZ3lwOgorCiAyMDEwLTAzLTMxICBZdXJ5
IFNlbWlraGF0c2t5ICA8eXVyeXNAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5
IFBhdmVsIEZlbGRtYW4uCkluZGV4OiBXZWJLaXQvY2hyb21pdW0vV2ViS2l0Lmd5cA0KPT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PQ0KLS0tIFdlYktpdC9jaHJvbWl1bS9XZWJLaXQuZ3lwCShyZXZpc2lvbiA1Njg0MSkKKysr
IFdlYktpdC9jaHJvbWl1bS9XZWJLaXQuZ3lwCSh3b3JraW5nIGNvcHkpCkBAIC01OSw2ICs1OSwx
MiBAQAogICAgICAgICAgICAgJ21zdnNfZ3VpZCc6ICc1RUNFQzlFNS04RjIzLTQ3QjYtOTNFMC1D
M0IzMjhCM0JFNjUnLAogICAgICAgICAgICAgJ2RlcGVuZGVuY2llcyc6IFsKICAgICAgICAgICAg
ICAgICAnLi4vLi4vV2ViQ29yZS9XZWJDb3JlLmd5cC9XZWJDb3JlLmd5cDp3ZWJjb3JlJywKKyAg
ICAgICAgICAgICAgICAnPChjaHJvbWl1bV9zcmNfZGlyKS9za2lhL3NraWEuZ3lwOnNraWEnLAor
ICAgICAgICAgICAgICAgICc8KGNocm9taXVtX3NyY19kaXIpL3RoaXJkX3BhcnR5L25wYXBpL25w
YXBpLmd5cDpucGFwaScsCisgICAgICAgICAgICBdLAorICAgICAgICAgICAgJ2V4cG9ydF9kZXBl
bmRlbnRfc2V0dGluZ3MnOiBbCisgICAgICAgICAgICAgICAgJzwoY2hyb21pdW1fc3JjX2Rpcikv
c2tpYS9za2lhLmd5cDpza2lhJywKKyAgICAgICAgICAgICAgICAnPChjaHJvbWl1bV9zcmNfZGly
KS90aGlyZF9wYXJ0eS9ucGFwaS9ucGFwaS5neXA6bnBhcGknLAogICAgICAgICAgICAgXSwKICAg
ICAgICAgICAgICdpbmNsdWRlX2RpcnMnOiBbCiAgICAgICAgICAgICAgICAgJ3B1YmxpYycsCg==
</data>
<flag name="review"
          id="35577"
          type_id="1"
          status="+"
          setter="levin"
    />
          </attachment>
      

    </bug>

</bugzilla>