<?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>58507</bug_id>
          
          <creation_ts>2011-04-13 19:31:09 -0700</creation_ts>
          <short_desc>Add GTest unit-testing framework to WebKit</short_desc>
          <delta_ts>2011-04-15 18:18:06 -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>OS X 10.5</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>
          
          <blocked>21010</blocked>
    
    <blocked>58509</blocked>
          <everconfirmed>0</everconfirmed>
          <reporter name="Dmitry Lomov">dslomov</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>dslomov</cc>
    
    <cc>levin</cc>
    
    <cc>mrowe</cc>
    
    <cc>timothy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>385463</commentid>
    <comment_count>0</comment_count>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-04-13 19:31:09 -0700</bug_when>
    <thetext>Add GTest to Source/ThirdParty, create examples of its use.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>385466</commentid>
    <comment_count>1</comment_count>
      <attachid>89519</attachid>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-04-13 20:03:07 -0700</bug_when>
    <thetext>Created attachment 89519
Adding gtest to Source/ThirdParty + sample test case</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>385468</commentid>
    <comment_count>2</comment_count>
      <attachid>89519</attachid>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-04-13 20:05:50 -0700</bug_when>
    <thetext>Comment on attachment 89519
Adding gtest to Source/ThirdParty + sample test case

Breaking up into two separate patches.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>385475</commentid>
    <comment_count>3</comment_count>
      <attachid>89521</attachid>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-04-13 20:28:30 -0700</bug_when>
    <thetext>Created attachment 89521
Adding GTest framework to Source/ThirdParty</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>385478</commentid>
    <comment_count>4</comment_count>
      <attachid>89522</attachid>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-04-13 20:36:58 -0700</bug_when>
    <thetext>Created attachment 89522
Adding GTest to Source/ThirdParty</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>385514</commentid>
    <comment_count>5</comment_count>
      <attachid>89522</attachid>
    <who name="David Levin">levin</who>
    <bug_when>2011-04-13 22:16:16 -0700</bug_when>
    <thetext>Comment on attachment 89522
Adding GTest to Source/ThirdParty

Based on the status of ews, I suspect this patch is too big for the commit queue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>385987</commentid>
    <comment_count>6</comment_count>
    <who name="David Levin">levin</who>
    <bug_when>2011-04-14 13:13:30 -0700</bug_when>
    <thetext>Committed as http://trac.webkit.org/changeset/83815</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>386792</commentid>
    <comment_count>7</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2011-04-15 11:55:29 -0700</bug_when>
    <thetext>The GTest targets have errors on systems that don&apos;t have $(DEVELOPER_SDK_DIR)/MacOSX10.4u.sdk. It should not have an SDK and use the current system.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>387101</commentid>
    <comment_count>8</comment_count>
    <who name="Mark Rowe (bdash)">mrowe</who>
    <bug_when>2011-04-15 17:57:15 -0700</bug_when>
    <thetext>Can someone expand on how we arrived at using gtest? There appears to have been zero discussion of this and the rather large and complicated library was simply checked in to the source tree.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>89519</attachid>
            <date>2011-04-13 20:03:07 -0700</date>
            <delta_ts>2011-04-13 20:36:58 -0700</delta_ts>
            <desc>Adding gtest to Source/ThirdParty + sample test case</desc>
            <filename>adding-gtest.patch</filename>
            <type>text/plain</type>
            <size>0</size>
            <attacher name="Dmitry Lomov">dslomov</attacher>
            
              <data encoding="base64"></data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>89521</attachid>
            <date>2011-04-13 20:28:30 -0700</date>
            <delta_ts>2011-04-13 20:36:58 -0700</delta_ts>
            <desc>Adding GTest framework to Source/ThirdParty</desc>
            <filename>adding-gtest.patch</filename>
            <type>text/plain</type>
            <size>0</size>
            <attacher name="Dmitry Lomov">dslomov</attacher>
            
              <data encoding="base64"></data>
<flag name="review"
          id="82212"
          type_id="1"
          status="-"
          setter="dslomov"
    />
    <flag name="commit-queue"
          id="82213"
          type_id="3"
          status="-"
          setter="dslomov"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>89522</attachid>
            <date>2011-04-13 20:36:58 -0700</date>
            <delta_ts>2011-04-13 22:16:16 -0700</delta_ts>
            <desc>Adding GTest to Source/ThirdParty</desc>
            <filename>adding-gtest.patch</filename>
            <type>text/plain</type>
            <size>0</size>
            <attacher name="Dmitry Lomov">dslomov</attacher>
            
              <data encoding="base64"></data>
<flag name="review"
          id="82214"
          type_id="1"
          status="+"
          setter="levin"
    />
    <flag name="commit-queue"
          id="82215"
          type_id="3"
          status="-"
          setter="levin"
    />
          </attachment>
      

    </bug>

</bugzilla>