<?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>32024</bug_id>
          
          <creation_ts>2009-12-01 08:50:46 -0800</creation_ts>
          <short_desc>[GTK] WebKit does not compile without JAVASCRIPT_DEBUGGER</short_desc>
          <delta_ts>2009-12-07 11:23:03 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</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>0</everconfirmed>
          <reporter name="Gyuyoung Kim">gyuyoung</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>eric</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>167647</commentid>
    <comment_count>0</comment_count>
      <attachid>44085</attachid>
    <who name="Gyuyoung Kim">gyuyoung</who>
    <bug_when>2009-12-01 08:50:46 -0800</bug_when>
    <thetext>Created attachment 44085
patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk

When I try to build webkit without ENABLE(JAVASCRIPT_DEBUGGER), there are some problems. 

First, even though I run &apos;./configure --enable-javascript-debugger=no&apos;, the ENABLE(JAVASCRIPT_DEBUGGER) is still enabled. I&apos;m going to report the reason.

Second, when I defined &apos;ENABLE_JAVASCRIPT_DEBUGGER 0&apos; as IPHONE, ANDROID in wtf/Platform.h, there are two build errors as below, 

-f &quot;WebKit/gtk/webkit/.deps/libwebkit_1_0_la-webkitwebinspector.Tpo&quot;; exit 1; fi
WebKit/gtk/webkit/webkitwebinspector.cpp: In function &apos;void webkit_web_inspector_set_property(GObject*, guint, const GValue*, GParamSpec*)&apos;:
WebKit/gtk/webkit/webkitwebinspector.cpp:353: error: &apos;class WebCore::InspectorController&apos; has no member named &apos;enableProfiler&apos;
WebKit/gtk/webkit/webkitwebinspector.cpp:355: error: &apos;class WebCore::InspectorController&apos; has no member named &apos;disableProfiler&apos;
WebKit/gtk/webkit/webkitwebinspector.cpp: In function &apos;void webkit_web_inspector_get_property(GObject*, guint, GValue*, GParamSpec*)&apos;:
WebKit/gtk/webkit/webkitwebinspector.cpp:388: error: &apos;class WebCore::InspectorController&apos; has no member named &apos;profilerEnabled&apos;
make[1]: *** [WebKit/gtk/webkit/libwebkit_1_0_la-webkitwebinspector.lo] Error 1
make[1]: Leaving directory `/root/workspace/WebKit&apos;
make: *** [all] Error 2

Tpo&quot; &quot;WebKit/gtk/webkit/.deps/libwebkit_1_0_la-webkitwebinspector.Plo&quot;; else rm -f &quot;WebKit/gtk/webkit/.deps/libwebkit_1_0_la-webkitwebinspector.Tpo&quot;; exit 1; fi
WebKit/gtk/webkit/webkitwebinspector.cpp: In function &apos;void webkit_web_inspector_get_property(GObject*, guint, GValue*, GParamSpec*)&apos;:
WebKit/gtk/webkit/webkitwebinspector.cpp:388: error: &apos;class WebCore::InspectorController&apos; has no member named &apos;profilerEnabled&apos;
make[1]: *** [WebKit/gtk/webkit/libwebkit_1_0_la-webkitwebinspector.lo] Error 1
make[1]: Leaving directory `/root/workspace/WebKit&apos;
make: *** [all] Error 2

The reason of upper build errors is that webkitwebinspector.cpp still invokes the enableProfiler(), disableProfiler() and profilerEnabled() defined in WebCore/inspector/InspectorController.h.
In my opinion, it seems embedded device based on gtk doesn&apos;t need to use JAVASCRIPT_DEBUGGER feature.

I make a try a patch for webkitwebinspector.cpp. If there are any nits or problems, please let me know.

Thanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>167789</commentid>
    <comment_count>1</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2009-12-01 17:04:13 -0800</bug_when>
    <thetext>style-queue ran check-webkit-style on attachment 44085 without any errors.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>167793</commentid>
    <comment_count>2</comment_count>
      <attachid>44085</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-12-01 17:08:31 -0800</bug_when>
    <thetext>Comment on attachment 44085
patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk

I&apos;m surprised that these case statements would be outisde the #if.  Does the UI still show JS profiling stuff, and does the console object still expose profiling JS commands?

This looks OK as a build fix, but is probably not the full fix.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>167794</commentid>
    <comment_count>3</comment_count>
      <attachid>44085</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-12-01 17:09:23 -0800</bug_when>
    <thetext>Comment on attachment 44085
patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk

Actually, nevermind.  This has no ChangeLog.  see http://webkit.org/coding/contributing.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>167843</commentid>
    <comment_count>4</comment_count>
      <attachid>44126</attachid>
    <who name="Gyuyoung Kim">gyuyoung</who>
    <bug_when>2009-12-01 19:09:22 -0800</bug_when>
    <thetext>Created attachment 44126
 patch with Changelog

Thank you for your comment. When I define &quot;ENABLE_JAVASCRIPT_DEBUGGER 0&quot; in Platform.h, there are no build errors on this patch. However, I also think that this patch is not full fix. I will report full patch to this bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>168055</commentid>
    <comment_count>5</comment_count>
      <attachid>44126</attachid>
    <who name="Gustavo Noronha (kov)">gustavo</who>
    <bug_when>2009-12-02 10:26:31 -0800</bug_when>
    <thetext>Comment on attachment 44126
 patch with Changelog

Hrm. Yeah, I think the case statements should indeed be outside the #if clauses, because it would be kind of an API breakage otherwise, but you should perhaps add a g_message to the #else case, and make sure get_property already sets false to the value, just to be sure.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>168207</commentid>
    <comment_count>6</comment_count>
      <attachid>44192</attachid>
    <who name="Gyuyoung Kim">gyuyoung</who>
    <bug_when>2009-12-02 16:18:04 -0800</bug_when>
    <thetext>Created attachment 44192
Patch with g_message

I add a g_message to #else case. However, I&apos;m not sure it is you want. If there are problems, please let me know.
Thanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>168847</commentid>
    <comment_count>7</comment_count>
      <attachid>44323</attachid>
    <who name="Gyuyoung Kim">gyuyoung</who>
    <bug_when>2009-12-04 11:05:33 -0800</bug_when>
    <thetext>Created attachment 44323
 patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk

I have tried to solve the build breaks. As I mentioned in description, there are two cases of error. First, there are build errors as below when the javascript-debugger is disabled in configure(./configure --enable-javascript-debugger=no).

WebCore/bindings/js/JSConsoleCustom.cpp:42: error: no ‘JSC::JSValue WebCore::JSConsole::profiles(JSC::ExecState*) const’ member function declared in class ‘WebCore::JSConsole’
WebCore/bindings/js/JSConsoleCustom.cpp:54: error: no ‘JSC::JSValue WebCore::JSConsole::profile(JSC::ExecState*, const JSC::ArgList&amp;)’ member function declared in class ‘WebCore::JSConsole’
WebCore/bindings/js/JSConsoleCustom.cpp:62: error: no ‘JSC::JSValue WebCore::JSConsole::profileEnd(JSC::ExecState*, const JSC::ArgList&amp;)’ member function declared in class ‘WebCore::JSConsole’
make[1]: *** [WebCore/bindings/js/libWebCore_la-JSConsoleCustom.lo] Error 1
make[1]: Leaving directory `/home/gyuyoung/workspace/WebKit&apos;
make: *** [all] Error 2

I think that JSConsole.h of DerivedSources doesn&apos;t make the JSConsole::profiles() because of the configure setting, but JSConsoleCustom.cpp still compiles the JSConsole::profiles(), which are wrapped by JAVASCRIPT_DEBUGGER as below,
 
#if ENABLE(JAVASCRIPT_DEBUGGER)

typedef Vector&lt;RefPtr&lt;JSC::Profile&gt; &gt; ProfilesArray;

JSValue JSConsole::profiles(ExecState* exec) const
{
    const ProfilesArray&amp; profiles = impl()-&gt;profiles();
    MarkedArgumentBuffer list;

    ProfilesArray::const_iterator end = profiles.end();
    for (ProfilesArray::const_iterator iter = profiles.begin(); iter != end; ++iter)
        list.append(toJS(exec, iter-&gt;get()));

    return constructArray(exec, list);
}
...
#endif

The reason is that JAVASCRIPT_DEBUGGER macro is enabled by Platform.h as below,

679 #if !defined(ENABLE_JAVASCRIPT_DEBUGGER)
680 #define ENABLE_JAVASCRIPT_DEBUGGER 1
681 #endif

Upper condition means that if ENABLE_JAVASCRIPT_DEBUGGER is not defined, the ENABLE_JAVASCRIPT_DEBUGGER should be enabled.
(It seems the ENABLE_JAVASCRIPT_DEBUGGER is enabled by WebCore/GNUmakefile.am when it is set as &quot;yes&quot; in configure) 
I think this make the build errors even though the &quot;javascript-debugger&quot; is disabled by configure.

IMO, why don&apos;t we make the ENABLE_JAVASCRIPT_DEBUGGER set as 0(disable) when it is not defined ? If user sets the javascript-debugger as &quot;no&quot;, it means that user doesn&apos;t want to use it. So, I think the feature can be disabled as default.

BTW, even if the macro is set as 0 basically, as mentioned in description, there are still build breaks as below,

-f &quot;WebKit/gtk/webkit/.deps/libwebkit_1_0_la-webkitwebinspector.Tpo&quot;; exit 1;
fi
WebKit/gtk/webkit/webkitwebinspector.cpp: In function &apos;void
webkit_web_inspector_set_property(GObject*, guint, const GValue*,
GParamSpec*)&apos;:
WebKit/gtk/webkit/webkitwebinspector.cpp:353: error: &apos;class
WebCore::InspectorController&apos; has no member named &apos;enableProfiler&apos;
WebKit/gtk/webkit/webkitwebinspector.cpp:355: error: &apos;class
WebCore::InspectorController&apos; has no member named &apos;disableProfiler&apos;
WebKit/gtk/webkit/webkitwebinspector.cpp: In function &apos;void
webkit_web_inspector_get_property(GObject*, guint, GValue*, GParamSpec*)&apos;:
WebKit/gtk/webkit/webkitwebinspector.cpp:388: error: &apos;class
WebCore::InspectorController&apos; has no member named &apos;profilerEnabled&apos;
make[1]: *** [WebKit/gtk/webkit/libwebkit_1_0_la-webkitwebinspector.lo] Error 1
make[1]: Leaving directory `/root/workspace/WebKit&apos;
make: *** [all] Error 2

Tpo&quot; &quot;WebKit/gtk/webkit/.deps/libwebkit_1_0_la-webkitwebinspector.Plo&quot;; else rm
-f &quot;WebKit/gtk/webkit/.deps/libwebkit_1_0_la-webkitwebinspector.Tpo&quot;; exit 1;
fi
WebKit/gtk/webkit/webkitwebinspector.cpp: In function &apos;void
webkit_web_inspector_get_property(GObject*, guint, GValue*, GParamSpec*)&apos;:
WebKit/gtk/webkit/webkitwebinspector.cpp:388: error: &apos;class
WebCore::InspectorController&apos; has no member named &apos;profilerEnabled&apos;
make[1]: *** [WebKit/gtk/webkit/libwebkit_1_0_la-webkitwebinspector.lo] Error 1
make[1]: Leaving directory `/root/workspace/WebKit&apos;
make: *** [all] Error 2

The webkitwebinspector.cpp invokes some functions related to JAVASCRIPT_DEBUGGER macro regardless of enabling/disabling it.

I make a patch for it. Please review it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>168849</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2009-12-04 11:10:06 -0800</bug_when>
    <thetext>style-queue ran check-webkit-style on attachment 44323 without any errors.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>168992</commentid>
    <comment_count>9</comment_count>
    <who name="Gyuyoung Kim">gyuyoung</who>
    <bug_when>2009-12-04 17:20:00 -0800</bug_when>
    <thetext>Dear Eric :
Could you review this patch ?

(In reply to comment #7)
&gt; Created an attachment (id=44323) [details]
&gt;  patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk
&gt; 
&gt; I have tried to solve the build breaks. As I mentioned in description, there
&gt; are two cases of error. First, there are build errors as below when the
&gt; javascript-debugger is disabled in configure(./configure
&gt; --enable-javascript-debugger=no).
&gt; 
&gt; WebCore/bindings/js/JSConsoleCustom.cpp:42: error: no ‘JSC::JSValue
&gt; WebCore::JSConsole::profiles(JSC::ExecState*) const’ member function declared
&gt; in class ‘WebCore::JSConsole’
&gt; WebCore/bindings/js/JSConsoleCustom.cpp:54: error: no ‘JSC::JSValue
&gt; WebCore::JSConsole::profile(JSC::ExecState*, const JSC::ArgList&amp;)’ member
&gt; function declared in class ‘WebCore::JSConsole’
&gt; WebCore/bindings/js/JSConsoleCustom.cpp:62: error: no ‘JSC::JSValue
&gt; WebCore::JSConsole::profileEnd(JSC::ExecState*, const JSC::ArgList&amp;)’ member
&gt; function declared in class ‘WebCore::JSConsole’
&gt; make[1]: *** [WebCore/bindings/js/libWebCore_la-JSConsoleCustom.lo] Error 1
&gt; make[1]: Leaving directory `/home/gyuyoung/workspace/WebKit&apos;
&gt; make: *** [all] Error 2
&gt; 
&gt; I think that JSConsole.h of DerivedSources doesn&apos;t make the
&gt; JSConsole::profiles() because of the configure setting, but JSConsoleCustom.cpp
&gt; still compiles the JSConsole::profiles(), which are wrapped by
&gt; JAVASCRIPT_DEBUGGER as below,
&gt; 
&gt; #if ENABLE(JAVASCRIPT_DEBUGGER)
&gt; 
&gt; typedef Vector&lt;RefPtr&lt;JSC::Profile&gt; &gt; ProfilesArray;
&gt; 
&gt; JSValue JSConsole::profiles(ExecState* exec) const
&gt; {
&gt;     const ProfilesArray&amp; profiles = impl()-&gt;profiles();
&gt;     MarkedArgumentBuffer list;
&gt; 
&gt;     ProfilesArray::const_iterator end = profiles.end();
&gt;     for (ProfilesArray::const_iterator iter = profiles.begin(); iter != end;
&gt; ++iter)
&gt;         list.append(toJS(exec, iter-&gt;get()));
&gt; 
&gt;     return constructArray(exec, list);
&gt; }
&gt; ...
&gt; #endif
&gt; 
&gt; The reason is that JAVASCRIPT_DEBUGGER macro is enabled by Platform.h as below,
&gt; 
&gt; 679 #if !defined(ENABLE_JAVASCRIPT_DEBUGGER)
&gt; 680 #define ENABLE_JAVASCRIPT_DEBUGGER 1
&gt; 681 #endif
&gt; 
&gt; Upper condition means that if ENABLE_JAVASCRIPT_DEBUGGER is not defined, the
&gt; ENABLE_JAVASCRIPT_DEBUGGER should be enabled.
&gt; (It seems the ENABLE_JAVASCRIPT_DEBUGGER is enabled by WebCore/GNUmakefile.am
&gt; when it is set as &quot;yes&quot; in configure) 
&gt; I think this make the build errors even though the &quot;javascript-debugger&quot; is
&gt; disabled by configure.
&gt; 
&gt; IMO, why don&apos;t we make the ENABLE_JAVASCRIPT_DEBUGGER set as 0(disable) when it
&gt; is not defined ? If user sets the javascript-debugger as &quot;no&quot;, it means that
&gt; user doesn&apos;t want to use it. So, I think the feature can be disabled as
&gt; default.
&gt; 
&gt; BTW, even if the macro is set as 0 basically, as mentioned in description,
&gt; there are still build breaks as below,
&gt; 
&gt; -f &quot;WebKit/gtk/webkit/.deps/libwebkit_1_0_la-webkitwebinspector.Tpo&quot;; exit 1;
&gt; fi
&gt; WebKit/gtk/webkit/webkitwebinspector.cpp: In function &apos;void
&gt; webkit_web_inspector_set_property(GObject*, guint, const GValue*,
&gt; GParamSpec*)&apos;:
&gt; WebKit/gtk/webkit/webkitwebinspector.cpp:353: error: &apos;class
&gt; WebCore::InspectorController&apos; has no member named &apos;enableProfiler&apos;
&gt; WebKit/gtk/webkit/webkitwebinspector.cpp:355: error: &apos;class
&gt; WebCore::InspectorController&apos; has no member named &apos;disableProfiler&apos;
&gt; WebKit/gtk/webkit/webkitwebinspector.cpp: In function &apos;void
&gt; webkit_web_inspector_get_property(GObject*, guint, GValue*, GParamSpec*)&apos;:
&gt; WebKit/gtk/webkit/webkitwebinspector.cpp:388: error: &apos;class
&gt; WebCore::InspectorController&apos; has no member named &apos;profilerEnabled&apos;
&gt; make[1]: *** [WebKit/gtk/webkit/libwebkit_1_0_la-webkitwebinspector.lo] Error 1
&gt; make[1]: Leaving directory `/root/workspace/WebKit&apos;
&gt; make: *** [all] Error 2
&gt; 
&gt; Tpo&quot; &quot;WebKit/gtk/webkit/.deps/libwebkit_1_0_la-webkitwebinspector.Plo&quot;; else rm
&gt; -f &quot;WebKit/gtk/webkit/.deps/libwebkit_1_0_la-webkitwebinspector.Tpo&quot;; exit 1;
&gt; fi
&gt; WebKit/gtk/webkit/webkitwebinspector.cpp: In function &apos;void
&gt; webkit_web_inspector_get_property(GObject*, guint, GValue*, GParamSpec*)&apos;:
&gt; WebKit/gtk/webkit/webkitwebinspector.cpp:388: error: &apos;class
&gt; WebCore::InspectorController&apos; has no member named &apos;profilerEnabled&apos;
&gt; make[1]: *** [WebKit/gtk/webkit/libwebkit_1_0_la-webkitwebinspector.lo] Error 1
&gt; make[1]: Leaving directory `/root/workspace/WebKit&apos;
&gt; make: *** [all] Error 2
&gt; 
&gt; The webkitwebinspector.cpp invokes some functions related to
&gt; JAVASCRIPT_DEBUGGER macro regardless of enabling/disabling it.
&gt; 
&gt; I make a patch for it. Please review it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>169000</commentid>
    <comment_count>10</comment_count>
      <attachid>44323</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-12-04 17:48:57 -0800</bug_when>
    <thetext>Comment on attachment 44323
 patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk

The platform change is wrong.  I do not think you should change the default value for ENABLE_JAVASCRIPT_DEBUGGER.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>169009</commentid>
    <comment_count>11</comment_count>
      <attachid>44345</attachid>
    <who name="Gyuyoung Kim">gyuyoung</who>
    <bug_when>2009-12-04 19:04:22 -0800</bug_when>
    <thetext>Created attachment 44345
patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk-2

I add &apos;-DENABLE_JAVASCRIPT_DEBUGGER 0&apos; to WebCore/GNUmakefile.am as below,  
===========================================================
if ENABLE_JAVASCRIPT_DEBUGGER
FEATURE_DEFINES_JAVASCRIPT += ENABLE_JAVASCRIPT_DEBUGGER=1

webcore_cppflags += \
        -DENABLE_JAVASCRIPT_DEBUGGER=1

else
webcore_cppflags += \
        -DENABLE_JAVASCRIPT_DEBUGGER=0

endif # END ENABLE_JAVASCRIPT_DEBUGGER
===========================================================

I find that some features ,for example ENABLE_DATABASE, ENABLE_ICONDATABASE, ENABLE_WEB_SOCKETS, already use this way.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>169025</commentid>
    <comment_count>12</comment_count>
      <attachid>44348</attachid>
    <who name="Gyuyoung Kim">gyuyoung</who>
    <bug_when>2009-12-04 23:06:35 -0800</bug_when>
    <thetext>Created attachment 44348
patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk-3

Oops, I added an unnecessary blank line to Changelog. I make a patch again.
Thanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>169026</commentid>
    <comment_count>13</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2009-12-04 23:08:38 -0800</bug_when>
    <thetext>style-queue ran check-webkit-style on attachment 44348 without any errors.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>169321</commentid>
    <comment_count>14</comment_count>
    <who name="Gyuyoung Kim">gyuyoung</who>
    <bug_when>2009-12-07 07:33:05 -0800</bug_when>
    <thetext>Dear Eric,
Sorry for frequent review request. Could you review my latest patch again ? I want to fix this defect.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>169370</commentid>
    <comment_count>15</comment_count>
      <attachid>44348</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-12-07 10:35:15 -0800</bug_when>
    <thetext>Comment on attachment 44348
patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk-3

I don&apos;t know really anything about the GNUmakefile.am, but this looks sane enough.

Seems strange that whatever build system is using that file doesn&apos;t have a cleaner/more consistent way to set feature defines.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>169394</commentid>
    <comment_count>16</comment_count>
      <attachid>44348</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2009-12-07 11:22:57 -0800</bug_when>
    <thetext>Comment on attachment 44348
patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk-3

Clearing flags on attachment: 44348

Committed r51783: &lt;http://trac.webkit.org/changeset/51783&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>169395</commentid>
    <comment_count>17</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2009-12-07 11:23:03 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>44085</attachid>
            <date>2009-12-01 08:50:46 -0800</date>
            <delta_ts>2009-12-01 19:09:22 -0800</delta_ts>
            <desc>patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk</desc>
            <filename>javascript_debugger_patch.patch</filename>
            <type>text/plain</type>
            <size>1126</size>
            <attacher name="Gyuyoung Kim">gyuyoung</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC9ndGsvd2Via2l0L3dlYmtpdHdlYmluc3BlY3Rvci5jcHAKPT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PQotLS0gV2ViS2l0L2d0ay93ZWJraXQvd2Via2l0d2ViaW5zcGVjdG9yLmNwcAkocmV2aXNpb24g
NTE1MzIpCisrKyBXZWJLaXQvZ3RrL3dlYmtpdC93ZWJraXR3ZWJpbnNwZWN0b3IuY3BwCSh3b3Jr
aW5nIGNvcHkpCkBAIC0zNDYsMTIgKzM0NiwxNCBAQAogCiAgICAgc3dpdGNoKHByb3BfaWQpIHsK
ICAgICBjYXNlIFBST1BfSkFWQVNDUklQVF9QUk9GSUxJTkdfRU5BQkxFRDogeworI2lmIEVOQUJM
RShKQVZBU0NSSVBUX0RFQlVHR0VSKQogICAgICAgICBib29sIGVuYWJsZWQgPSBnX3ZhbHVlX2dl
dF9ib29sZWFuKHZhbHVlKTsKICAgICAgICAgV2ViQ29yZTo6SW5zcGVjdG9yQ29udHJvbGxlciog
Y29udHJvbGxlciA9IHByaXYtPnBhZ2UtPmluc3BlY3RvckNvbnRyb2xsZXIoKTsKICAgICAgICAg
aWYgKGVuYWJsZWQpCiAgICAgICAgICAgICBjb250cm9sbGVyLT5lbmFibGVQcm9maWxlcigpOwog
ICAgICAgICBlbHNlCiAgICAgICAgICAgICBjb250cm9sbGVyLT5kaXNhYmxlUHJvZmlsZXIoKTsK
KyNlbmRpZgogICAgICAgICBicmVhazsKICAgICB9CiAgICAgY2FzZSBQUk9QX1RJTUVMSU5FX1BS
T0ZJTElOR19FTkFCTEVEOiB7CkBAIC0zODIsNyArMzg0LDkgQEAKICAgICAgICAgZ192YWx1ZV9z
ZXRfc3RyaW5nKHZhbHVlLCBwcml2LT5pbnNwZWN0ZWRfdXJpKTsKICAgICAgICAgYnJlYWs7CiAg
ICAgY2FzZSBQUk9QX0pBVkFTQ1JJUFRfUFJPRklMSU5HX0VOQUJMRUQ6CisjaWYgRU5BQkxFKEpB
VkFTQ1JJUFRfREVCVUdHRVIpCiAgICAgICAgIGdfdmFsdWVfc2V0X2Jvb2xlYW4odmFsdWUsIHBy
aXYtPnBhZ2UtPmluc3BlY3RvckNvbnRyb2xsZXIoKS0+cHJvZmlsZXJFbmFibGVkKCkpOworI2Vu
ZGlmCiAgICAgICAgIGJyZWFrOwogICAgIGNhc2UgUFJPUF9USU1FTElORV9QUk9GSUxJTkdfRU5B
QkxFRDoKICAgICAgICAgZ192YWx1ZV9zZXRfYm9vbGVhbih2YWx1ZSwgcHJpdi0+cGFnZS0+aW5z
cGVjdG9yQ29udHJvbGxlcigpLT50aW1lbGluZUFnZW50KCkgIT0gMCk7Cg==
</data>
<flag name="review"
          id="25889"
          type_id="1"
          status="-"
          setter="eric"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>44126</attachid>
            <date>2009-12-01 19:09:22 -0800</date>
            <delta_ts>2009-12-02 16:18:04 -0800</delta_ts>
            <desc> patch with Changelog</desc>
            <filename>javascript-debugger.patch</filename>
            <type>text/plain</type>
            <size>1765</size>
            <attacher name="Gyuyoung Kim">gyuyoung</attacher>
            
              <data encoding="base64">SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDUxNTY5
KQorKysgQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMDktMTIt
MDEgIEd5dXlvdW5nIEtpbSAgPGd5dXlvdW5nQGdtYWlsLmNvbT4KKworICAgICAgICBSZXZpZXdl
ZCBieSBFcmljIFNlaWRlbC4KKworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MzIwMjQKKyAgICAgICAgW0dUS10gV2ViS2l0IGRvZXMgbm90IGNvbXBpbGUg
d2l0aG91dCBKQVZBU0NSSVBUX0RFQlVHR0VSCisKKyAgICAgICAgRml4IGJ1aWxkIGVycm9ycyB3
aGVuIGphdmFzY3JpcHQtZGVidWdnZXIgaXMgZGlzYWJsZWQgb24gR1RLCisKKyAgICAgICAgKiB3
ZWJraXQvd2Via2l0d2ViaW5zcGVjdG9yLmNwcDoKKyAgICAgICAgKHdlYmtpdF93ZWJfaW5zcGVj
dG9yX3NldF9wcm9wZXJ0eSk6CisgICAgICAgICh3ZWJraXRfd2ViX2luc3BlY3Rvcl9nZXRfcHJv
cGVydHkpOgorCiAyMDA5LTEyLTAxICBOaWtvbGFzIFppbW1lcm1hbm4gIDxuemltbWVybWFubkBy
aW0uY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFNpbW9uIEZyYXNlci4KSW5kZXg6IHdlYmtp
dC93ZWJraXR3ZWJpbnNwZWN0b3IuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIHdlYmtpdC93ZWJraXR3ZWJp
bnNwZWN0b3IuY3BwCShyZXZpc2lvbiA1MTU2OSkKKysrIHdlYmtpdC93ZWJraXR3ZWJpbnNwZWN0
b3IuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0zNDYsMTIgKzM0NiwxNCBAQAogCiAgICAgc3dpdGNo
KHByb3BfaWQpIHsKICAgICBjYXNlIFBST1BfSkFWQVNDUklQVF9QUk9GSUxJTkdfRU5BQkxFRDog
eworI2lmIEVOQUJMRShKQVZBU0NSSVBUX0RFQlVHR0VSKQogICAgICAgICBib29sIGVuYWJsZWQg
PSBnX3ZhbHVlX2dldF9ib29sZWFuKHZhbHVlKTsKICAgICAgICAgV2ViQ29yZTo6SW5zcGVjdG9y
Q29udHJvbGxlciogY29udHJvbGxlciA9IHByaXYtPnBhZ2UtPmluc3BlY3RvckNvbnRyb2xsZXIo
KTsKICAgICAgICAgaWYgKGVuYWJsZWQpCiAgICAgICAgICAgICBjb250cm9sbGVyLT5lbmFibGVQ
cm9maWxlcigpOwogICAgICAgICBlbHNlCiAgICAgICAgICAgICBjb250cm9sbGVyLT5kaXNhYmxl
UHJvZmlsZXIoKTsKKyNlbmRpZgogICAgICAgICBicmVhazsKICAgICB9CiAgICAgY2FzZSBQUk9Q
X1RJTUVMSU5FX1BST0ZJTElOR19FTkFCTEVEOiB7CkBAIC0zODIsNyArMzg0LDkgQEAKICAgICAg
ICAgZ192YWx1ZV9zZXRfc3RyaW5nKHZhbHVlLCBwcml2LT5pbnNwZWN0ZWRfdXJpKTsKICAgICAg
ICAgYnJlYWs7CiAgICAgY2FzZSBQUk9QX0pBVkFTQ1JJUFRfUFJPRklMSU5HX0VOQUJMRUQ6Cisj
aWYgRU5BQkxFKEpBVkFTQ1JJUFRfREVCVUdHRVIpCiAgICAgICAgIGdfdmFsdWVfc2V0X2Jvb2xl
YW4odmFsdWUsIHByaXYtPnBhZ2UtPmluc3BlY3RvckNvbnRyb2xsZXIoKS0+cHJvZmlsZXJFbmFi
bGVkKCkpOworI2VuZGlmCiAgICAgICAgIGJyZWFrOwogICAgIGNhc2UgUFJPUF9USU1FTElORV9Q
Uk9GSUxJTkdfRU5BQkxFRDoKICAgICAgICAgZ192YWx1ZV9zZXRfYm9vbGVhbih2YWx1ZSwgcHJp
di0+cGFnZS0+aW5zcGVjdG9yQ29udHJvbGxlcigpLT50aW1lbGluZUFnZW50KCkgIT0gMCk7Cg==
</data>
<flag name="review"
          id="25905"
          type_id="1"
          status="-"
          setter="gustavo"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>44192</attachid>
            <date>2009-12-02 16:18:04 -0800</date>
            <delta_ts>2009-12-04 11:05:33 -0800</delta_ts>
            <desc>Patch with g_message</desc>
            <filename>javascript-debug-patch.patch</filename>
            <type>text/plain</type>
            <size>2016</size>
            <attacher name="Gyuyoung Kim">gyuyoung</attacher>
            
              <data encoding="base64">SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDUxNTg4
KQorKysgQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMDktMTIt
MDIgIEd5dXlvdW5nIEtpbSAgPGd5dXlvdW5nQGdtYWlsPgorCisgICAgICAgIFJldmlld2VkIGJ5
IEd1c3Rhdm8gTm9yb25oYS4KKworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MzIwMjQKKyAgICAgICAgW0dUS10gV2ViS2l0IGRvZXMgbm90IGNvbXBpbGUg
d2l0aG91dCBKQVZBU0NSSVBUX0RFQlVHR0VSCisKKyAgICAgICAgRml4IGJ1aWxkIGVycm9ycyB3
aGVuIGphdmFzY3JpcHQtZGVidWdnZXIgaXMgZGlzYWJsZWQgb24gR1RLCisKKyAgICAgICAgKiB3
ZWJraXQvd2Via2l0d2ViaW5zcGVjdG9yLmNwcDoKKyAgICAgICAgKHdlYmtpdF93ZWJfaW5zcGVj
dG9yX3NldF9wcm9wZXJ0eSk6CisgICAgICAgICh3ZWJraXRfd2ViX2luc3BlY3Rvcl9nZXRfcHJv
cGVydHkpOgorCiAyMDA5LTEyLTAxICBOaWtvbGFzIFppbW1lcm1hbm4gIDxuemltbWVybWFubkBy
aW0uY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFNpbW9uIEZyYXNlci4KSW5kZXg6IHdlYmtp
dC93ZWJraXR3ZWJpbnNwZWN0b3IuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIHdlYmtpdC93ZWJraXR3ZWJp
bnNwZWN0b3IuY3BwCShyZXZpc2lvbiA1MTU4OCkKKysrIHdlYmtpdC93ZWJraXR3ZWJpbnNwZWN0
b3IuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0zNDYsMTIgKzM0NiwxNiBAQAogCiAgICAgc3dpdGNo
KHByb3BfaWQpIHsKICAgICBjYXNlIFBST1BfSkFWQVNDUklQVF9QUk9GSUxJTkdfRU5BQkxFRDog
eworI2lmIEVOQUJMRShKQVZBU0NSSVBUX0RFQlVHR0VSKQogICAgICAgICBib29sIGVuYWJsZWQg
PSBnX3ZhbHVlX2dldF9ib29sZWFuKHZhbHVlKTsKICAgICAgICAgV2ViQ29yZTo6SW5zcGVjdG9y
Q29udHJvbGxlciogY29udHJvbGxlciA9IHByaXYtPnBhZ2UtPmluc3BlY3RvckNvbnRyb2xsZXIo
KTsKICAgICAgICAgaWYgKGVuYWJsZWQpCiAgICAgICAgICAgICBjb250cm9sbGVyLT5lbmFibGVQ
cm9maWxlcigpOwogICAgICAgICBlbHNlCiAgICAgICAgICAgICBjb250cm9sbGVyLT5kaXNhYmxl
UHJvZmlsZXIoKTsKKyNlbHNlCisgICAgICAgIGdfbWVzc2FnZSgiUFJPUF9KQVZBU0NSSVBUX1BS
T0ZJTElOR19FTkFCTEVEIGlzIG5vdCB3b3JrIGJlY2F1c2Ugb2YgdGhlIGphdmFzY3JpcHQgZGVi
dWdnZXIgaXMgZGlzYWJsZWRcbiIpOworI2VuZGlmCiAgICAgICAgIGJyZWFrOwogICAgIH0KICAg
ICBjYXNlIFBST1BfVElNRUxJTkVfUFJPRklMSU5HX0VOQUJMRUQ6IHsKQEAgLTM4Miw3ICszODYs
MTEgQEAKICAgICAgICAgZ192YWx1ZV9zZXRfc3RyaW5nKHZhbHVlLCBwcml2LT5pbnNwZWN0ZWRf
dXJpKTsKICAgICAgICAgYnJlYWs7CiAgICAgY2FzZSBQUk9QX0pBVkFTQ1JJUFRfUFJPRklMSU5H
X0VOQUJMRUQ6CisjaWYgRU5BQkxFKEpBVkFTQ1JJUFRfREVCVUdHRVIpCiAgICAgICAgIGdfdmFs
dWVfc2V0X2Jvb2xlYW4odmFsdWUsIHByaXYtPnBhZ2UtPmluc3BlY3RvckNvbnRyb2xsZXIoKS0+
cHJvZmlsZXJFbmFibGVkKCkpOworI2Vsc2UKKyAgICAgICAgZ19tZXNzYWdlKCJQUk9QX0pBVkFT
Q1JJUFRfUFJPRklMSU5HX0VOQUJMRUQgaXMgbm90IHdvcmsgYmVjYXVzZSBvZiB0aGUgamF2YXNj
cmlwdCBkZWJ1Z2dlciBpcyBkaXNhYmxlZFxuIik7CisjZW5kaWYKICAgICAgICAgYnJlYWs7CiAg
ICAgY2FzZSBQUk9QX1RJTUVMSU5FX1BST0ZJTElOR19FTkFCTEVEOgogICAgICAgICBnX3ZhbHVl
X3NldF9ib29sZWFuKHZhbHVlLCBwcml2LT5wYWdlLT5pbnNwZWN0b3JDb250cm9sbGVyKCktPnRp
bWVsaW5lQWdlbnQoKSAhPSAwKTsK
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>44323</attachid>
            <date>2009-12-04 11:05:33 -0800</date>
            <delta_ts>2009-12-04 19:04:22 -0800</delta_ts>
            <desc> patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk</desc>
            <filename>javascript-debugger-build-error.patch</filename>
            <type>text/plain</type>
            <size>3158</size>
            <attacher name="Gyuyoung Kim">gyuyoung</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZyBiL0phdmFTY3JpcHRDb3JlL0No
YW5nZUxvZwppbmRleCAxNDY3MDlkLi45YWZjNmU0IDEwMDY0NAotLS0gYS9KYXZhU2NyaXB0Q29y
ZS9DaGFuZ2VMb2cKKysrIGIvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQg
QEAKKzIwMDktMTItMDQgIEd5dXlvdW5nIEtpbSAgPGd5dXlvdW5nQGdtYWlsLmNvbT4KKworICAg
ICAgICBSZXZpZXdlZCBieSBFcmljIFNlaWRlbC4KKworICAgICAgICBodHRwczovL2J1Z3Mud2Vi
a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzIwMjQKKyAgICAgICAgW0dUS10gV2ViS2l0IGRvZXMg
bm90IGNvbXBpbGUgd2l0aG91dCBKQVZBU0NSSVBUX0RFQlVHR0VSCisKKyAgICAgICAgRml4IGJ1
aWxkIGVycm9ycyB3aGVuIGphdmFzY3JpcHQtZGVidWdnZXIgaXMgZGlzYWJsZWQgb24gR1RLCisK
KyAgICAgICAgKiB3dGYvUGxhdGZvcm0uaDoKKwogMjAwOS0xMi0wMyAgRHJldyBXaWxzb24gIDxh
dHdpbHNvbkBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUm9sbGVkIGJhY2sgcjUxNjMzIGJlY2F1
c2UgaXQgY2F1c2VzIGEgcGVyZiByZWdyZXNzaW9uIGluIENocm9taXVtLgpkaWZmIC0tZ2l0IGEv
SmF2YVNjcmlwdENvcmUvd3RmL1BsYXRmb3JtLmggYi9KYXZhU2NyaXB0Q29yZS93dGYvUGxhdGZv
cm0uaAppbmRleCA5ZTFkYmNiLi4yMDgzYzBmIDEwMDY0NAotLS0gYS9KYXZhU2NyaXB0Q29yZS93
dGYvUGxhdGZvcm0uaAorKysgYi9KYXZhU2NyaXB0Q29yZS93dGYvUGxhdGZvcm0uaApAQCAtNjc3
LDcgKzY3Nyw3IEBACiAjZW5kaWYKIAogI2lmICFkZWZpbmVkKEVOQUJMRV9KQVZBU0NSSVBUX0RF
QlVHR0VSKQotI2RlZmluZSBFTkFCTEVfSkFWQVNDUklQVF9ERUJVR0dFUiAxCisjZGVmaW5lIEVO
QUJMRV9KQVZBU0NSSVBUX0RFQlVHR0VSIDAKICNlbmRpZgogCiAjaWYgIWRlZmluZWQoRU5BQkxF
X0ZUUERJUikKZGlmZiAtLWdpdCBhL1dlYktpdC9ndGsvQ2hhbmdlTG9nIGIvV2ViS2l0L2d0ay9D
aGFuZ2VMb2cKaW5kZXggYzBjNDJkNC4uYWEzMzkwYSAxMDA2NDQKLS0tIGEvV2ViS2l0L2d0ay9D
aGFuZ2VMb2cKKysrIGIvV2ViS2l0L2d0ay9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNiBAQAorMjAw
OS0xMi0wNCAgR3l1eW91bmcgS2ltICA8Z3l1eW91bmdAZ21haWwuY29tPgorCisgICAgICAgIFJl
dmlld2VkIGJ5IEVyaWMgU2VpZGVsLgorCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD0zMjAyNAorICAgICAgICBbR1RLXSBXZWJLaXQgZG9lcyBub3QgY29t
cGlsZSB3aXRob3V0IEpBVkFTQ1JJUFRfREVCVUdHRVIKKworICAgICAgICBGaXggYnVpbGQgZXJy
b3JzIHdoZW4gamF2YXNjcmlwdC1kZWJ1Z2dlciBpcyBkaXNhYmxlZCBvbiBHVEsKKworICAgICAg
ICAqIHdlYmtpdC93ZWJraXR3ZWJpbnNwZWN0b3IuY3BwOgorICAgICAgICAod2Via2l0X3dlYl9p
bnNwZWN0b3Jfc2V0X3Byb3BlcnR5KToKKyAgICAgICAgKHdlYmtpdF93ZWJfaW5zcGVjdG9yX2dl
dF9wcm9wZXJ0eSk6CisKIDIwMDktMTItMDMgIEJyYWR5IEVpZHNvbiAgPGJlaWRzb25AYXBwbGUu
Y29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFNhbSBXZWluaWcuCmRpZmYgLS1naXQgYS9XZWJL
aXQvZ3RrL3dlYmtpdC93ZWJraXR3ZWJpbnNwZWN0b3IuY3BwIGIvV2ViS2l0L2d0ay93ZWJraXQv
d2Via2l0d2ViaW5zcGVjdG9yLmNwcAppbmRleCA2OGIwM2IyLi4yZmViMDY0IDEwMDY0NAotLS0g
YS9XZWJLaXQvZ3RrL3dlYmtpdC93ZWJraXR3ZWJpbnNwZWN0b3IuY3BwCisrKyBiL1dlYktpdC9n
dGsvd2Via2l0L3dlYmtpdHdlYmluc3BlY3Rvci5jcHAKQEAgLTM0NiwxMiArMzQ2LDE2IEBAIHN0
YXRpYyB2b2lkIHdlYmtpdF93ZWJfaW5zcGVjdG9yX3NldF9wcm9wZXJ0eShHT2JqZWN0KiBvYmpl
Y3QsIGd1aW50IHByb3BfaWQsIGNvCiAKICAgICBzd2l0Y2gocHJvcF9pZCkgewogICAgIGNhc2Ug
UFJPUF9KQVZBU0NSSVBUX1BST0ZJTElOR19FTkFCTEVEOiB7CisjaWYgRU5BQkxFKEpBVkFTQ1JJ
UFRfREVCVUdHRVIpCiAgICAgICAgIGJvb2wgZW5hYmxlZCA9IGdfdmFsdWVfZ2V0X2Jvb2xlYW4o
dmFsdWUpOwogICAgICAgICBXZWJDb3JlOjpJbnNwZWN0b3JDb250cm9sbGVyKiBjb250cm9sbGVy
ID0gcHJpdi0+cGFnZS0+aW5zcGVjdG9yQ29udHJvbGxlcigpOwogICAgICAgICBpZiAoZW5hYmxl
ZCkKICAgICAgICAgICAgIGNvbnRyb2xsZXItPmVuYWJsZVByb2ZpbGVyKCk7CiAgICAgICAgIGVs
c2UKICAgICAgICAgICAgIGNvbnRyb2xsZXItPmRpc2FibGVQcm9maWxlcigpOworI2Vsc2UKKyAg
ICAgICAgZ19tZXNzYWdlKCJQUk9QX0pBVkFTQ1JJUFRfUFJPRklMSU5HX0VOQUJMRUQgaXMgbm90
IHdvcmsgYmVjYXVzZSBvZiB0aGUgamF2YXNjcmlwdCBkZWJ1Z2dlciBpcyBkaXNhYmxlZFxuIik7
CisjZW5kaWYKICAgICAgICAgYnJlYWs7CiAgICAgfQogICAgIGNhc2UgUFJPUF9USU1FTElORV9Q
Uk9GSUxJTkdfRU5BQkxFRDogewpAQCAtMzgyLDcgKzM4NiwxMSBAQCBzdGF0aWMgdm9pZCB3ZWJr
aXRfd2ViX2luc3BlY3Rvcl9nZXRfcHJvcGVydHkoR09iamVjdCogb2JqZWN0LCBndWludCBwcm9w
X2lkLCBHVgogICAgICAgICBnX3ZhbHVlX3NldF9zdHJpbmcodmFsdWUsIHByaXYtPmluc3BlY3Rl
ZF91cmkpOwogICAgICAgICBicmVhazsKICAgICBjYXNlIFBST1BfSkFWQVNDUklQVF9QUk9GSUxJ
TkdfRU5BQkxFRDoKKyNpZiBFTkFCTEUoSkFWQVNDUklQVF9ERUJVR0dFUikKICAgICAgICAgZ192
YWx1ZV9zZXRfYm9vbGVhbih2YWx1ZSwgcHJpdi0+cGFnZS0+aW5zcGVjdG9yQ29udHJvbGxlcigp
LT5wcm9maWxlckVuYWJsZWQoKSk7CisjZWxzZQorICAgICAgICBnX21lc3NhZ2UoIlBST1BfSkFW
QVNDUklQVF9QUk9GSUxJTkdfRU5BQkxFRCBpcyBub3Qgd29yayBiZWNhdXNlIG9mIHRoZSBqYXZh
c2NyaXB0IGRlYnVnZ2VyIGlzIGRpc2FibGVkXG4iKTsKKyNlbmRpZgogICAgICAgICBicmVhazsK
ICAgICBjYXNlIFBST1BfVElNRUxJTkVfUFJPRklMSU5HX0VOQUJMRUQ6CiAgICAgICAgIGdfdmFs
dWVfc2V0X2Jvb2xlYW4odmFsdWUsIHByaXYtPnBhZ2UtPmluc3BlY3RvckNvbnRyb2xsZXIoKS0+
dGltZWxpbmVBZ2VudCgpICE9IDApOwo=
</data>
<flag name="review"
          id="26148"
          type_id="1"
          status="-"
          setter="eric"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>44345</attachid>
            <date>2009-12-04 19:04:22 -0800</date>
            <delta_ts>2009-12-04 23:06:35 -0800</delta_ts>
            <desc>patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk-2</desc>
            <filename>javascript-debugger.patch</filename>
            <type>text/plain</type>
            <size>3122</size>
            <attacher name="Gyuyoung Kim">gyuyoung</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
NGJmN2Y1OC4uZTc3YjI1OCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNSBAQAorMjAwOS0xMi0wNCAgR3l1eW91bmcgS2lt
ICA8Z3l1eW91bmdAZ21haWwuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IEVyaWMgU2VpZGVs
LgorCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zMjAy
NAorICAgICAgICBbR1RLXSBXZWJLaXQgZG9lcyBub3QgY29tcGlsZSB3aXRob3V0IEpBVkFTQ1JJ
UFRfREVCVUdHRVIKKworCisgICAgICAgIEZpeCBidWlsZCBlcnJvcnMgd2hlbiBqYXZhc2NyaXB0
LWRlYnVnZ2VyIGlzIGRpc2FibGVkIG9uIEdUSworCisgICAgICAgICogR05VbWFrZWZpbGUuYW06
CisKIDIwMDktMTItMDQgIEFkYW0gTGFuZ2xleSAgPGFnbEBnb29nbGUuY29tPgogCiAgICAgICAg
IFJldmlld2VkIGJ5IEVyaWMgU2VpZGVsLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9HTlVtYWtlZmls
ZS5hbSBiL1dlYkNvcmUvR05VbWFrZWZpbGUuYW0KaW5kZXggMmQ4ZWIyZi4uOGE2YTI5MCAxMDA2
NDQKLS0tIGEvV2ViQ29yZS9HTlVtYWtlZmlsZS5hbQorKysgYi9XZWJDb3JlL0dOVW1ha2VmaWxl
LmFtCkBAIC0yMTEyLDYgKzIxMTIsMTAgQEAgRkVBVFVSRV9ERUZJTkVTX0pBVkFTQ1JJUFQgKz0g
RU5BQkxFX0pBVkFTQ1JJUFRfREVCVUdHRVI9MQogd2ViY29yZV9jcHBmbGFncyArPSBcCiAJLURF
TkFCTEVfSkFWQVNDUklQVF9ERUJVR0dFUj0xCiAKK2Vsc2UKK3dlYmNvcmVfY3BwZmxhZ3MgKz0g
XAorCS1ERU5BQkxFX0pBVkFTQ1JJUFRfREVCVUdHRVI9MAorCiBlbmRpZiAjIEVORCBFTkFCTEVf
SkFWQVNDUklQVF9ERUJVR0dFUgogCiAjIC0tLS0KZGlmZiAtLWdpdCBhL1dlYktpdC9ndGsvQ2hh
bmdlTG9nIGIvV2ViS2l0L2d0ay9DaGFuZ2VMb2cKaW5kZXggYzBjNDJkNC4uYWEzMzkwYSAxMDA2
NDQKLS0tIGEvV2ViS2l0L2d0ay9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0L2d0ay9DaGFuZ2VMb2cK
QEAgLTEsMyArMSwxNiBAQAorMjAwOS0xMi0wNCAgR3l1eW91bmcgS2ltICA8Z3l1eW91bmdAZ21h
aWwuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IEVyaWMgU2VpZGVsLgorCisgICAgICAgIGh0
dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zMjAyNAorICAgICAgICBbR1RL
XSBXZWJLaXQgZG9lcyBub3QgY29tcGlsZSB3aXRob3V0IEpBVkFTQ1JJUFRfREVCVUdHRVIKKwor
ICAgICAgICBGaXggYnVpbGQgZXJyb3JzIHdoZW4gamF2YXNjcmlwdC1kZWJ1Z2dlciBpcyBkaXNh
YmxlZCBvbiBHVEsKKworICAgICAgICAqIHdlYmtpdC93ZWJraXR3ZWJpbnNwZWN0b3IuY3BwOgor
ICAgICAgICAod2Via2l0X3dlYl9pbnNwZWN0b3Jfc2V0X3Byb3BlcnR5KToKKyAgICAgICAgKHdl
YmtpdF93ZWJfaW5zcGVjdG9yX2dldF9wcm9wZXJ0eSk6CisKIDIwMDktMTItMDMgIEJyYWR5IEVp
ZHNvbiAgPGJlaWRzb25AYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFNhbSBXZWlu
aWcuCmRpZmYgLS1naXQgYS9XZWJLaXQvZ3RrL3dlYmtpdC93ZWJraXR3ZWJpbnNwZWN0b3IuY3Bw
IGIvV2ViS2l0L2d0ay93ZWJraXQvd2Via2l0d2ViaW5zcGVjdG9yLmNwcAppbmRleCA2OGIwM2Iy
Li4yZmViMDY0IDEwMDY0NAotLS0gYS9XZWJLaXQvZ3RrL3dlYmtpdC93ZWJraXR3ZWJpbnNwZWN0
b3IuY3BwCisrKyBiL1dlYktpdC9ndGsvd2Via2l0L3dlYmtpdHdlYmluc3BlY3Rvci5jcHAKQEAg
LTM0NiwxMiArMzQ2LDE2IEBAIHN0YXRpYyB2b2lkIHdlYmtpdF93ZWJfaW5zcGVjdG9yX3NldF9w
cm9wZXJ0eShHT2JqZWN0KiBvYmplY3QsIGd1aW50IHByb3BfaWQsIGNvCiAKICAgICBzd2l0Y2go
cHJvcF9pZCkgewogICAgIGNhc2UgUFJPUF9KQVZBU0NSSVBUX1BST0ZJTElOR19FTkFCTEVEOiB7
CisjaWYgRU5BQkxFKEpBVkFTQ1JJUFRfREVCVUdHRVIpCiAgICAgICAgIGJvb2wgZW5hYmxlZCA9
IGdfdmFsdWVfZ2V0X2Jvb2xlYW4odmFsdWUpOwogICAgICAgICBXZWJDb3JlOjpJbnNwZWN0b3JD
b250cm9sbGVyKiBjb250cm9sbGVyID0gcHJpdi0+cGFnZS0+aW5zcGVjdG9yQ29udHJvbGxlcigp
OwogICAgICAgICBpZiAoZW5hYmxlZCkKICAgICAgICAgICAgIGNvbnRyb2xsZXItPmVuYWJsZVBy
b2ZpbGVyKCk7CiAgICAgICAgIGVsc2UKICAgICAgICAgICAgIGNvbnRyb2xsZXItPmRpc2FibGVQ
cm9maWxlcigpOworI2Vsc2UKKyAgICAgICAgZ19tZXNzYWdlKCJQUk9QX0pBVkFTQ1JJUFRfUFJP
RklMSU5HX0VOQUJMRUQgaXMgbm90IHdvcmsgYmVjYXVzZSBvZiB0aGUgamF2YXNjcmlwdCBkZWJ1
Z2dlciBpcyBkaXNhYmxlZFxuIik7CisjZW5kaWYKICAgICAgICAgYnJlYWs7CiAgICAgfQogICAg
IGNhc2UgUFJPUF9USU1FTElORV9QUk9GSUxJTkdfRU5BQkxFRDogewpAQCAtMzgyLDcgKzM4Niwx
MSBAQCBzdGF0aWMgdm9pZCB3ZWJraXRfd2ViX2luc3BlY3Rvcl9nZXRfcHJvcGVydHkoR09iamVj
dCogb2JqZWN0LCBndWludCBwcm9wX2lkLCBHVgogICAgICAgICBnX3ZhbHVlX3NldF9zdHJpbmco
dmFsdWUsIHByaXYtPmluc3BlY3RlZF91cmkpOwogICAgICAgICBicmVhazsKICAgICBjYXNlIFBS
T1BfSkFWQVNDUklQVF9QUk9GSUxJTkdfRU5BQkxFRDoKKyNpZiBFTkFCTEUoSkFWQVNDUklQVF9E
RUJVR0dFUikKICAgICAgICAgZ192YWx1ZV9zZXRfYm9vbGVhbih2YWx1ZSwgcHJpdi0+cGFnZS0+
aW5zcGVjdG9yQ29udHJvbGxlcigpLT5wcm9maWxlckVuYWJsZWQoKSk7CisjZWxzZQorICAgICAg
ICBnX21lc3NhZ2UoIlBST1BfSkFWQVNDUklQVF9QUk9GSUxJTkdfRU5BQkxFRCBpcyBub3Qgd29y
ayBiZWNhdXNlIG9mIHRoZSBqYXZhc2NyaXB0IGRlYnVnZ2VyIGlzIGRpc2FibGVkXG4iKTsKKyNl
bmRpZgogICAgICAgICBicmVhazsKICAgICBjYXNlIFBST1BfVElNRUxJTkVfUFJPRklMSU5HX0VO
QUJMRUQ6CiAgICAgICAgIGdfdmFsdWVfc2V0X2Jvb2xlYW4odmFsdWUsIHByaXYtPnBhZ2UtPmlu
c3BlY3RvckNvbnRyb2xsZXIoKS0+dGltZWxpbmVBZ2VudCgpICE9IDApOwo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>44348</attachid>
            <date>2009-12-04 23:06:35 -0800</date>
            <delta_ts>2009-12-07 11:22:57 -0800</delta_ts>
            <desc>patch for the build error when JAVASCRIPT_DEBUGGER is disabled on gtk-3</desc>
            <filename>javascript-debugger-error.patch</filename>
            <type>text/plain</type>
            <size>3120</size>
            <attacher name="Gyuyoung Kim">gyuyoung</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
NGJmN2Y1OC4uZGZmNGExMSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNCBAQAorMjAwOS0xMi0wNCAgR3l1eW91bmcgS2lt
ICA8Z3l1eW91bmdAZ21haWwuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IEVyaWMgU2VpZGVs
LgorCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zMjAy
NAorICAgICAgICBbR1RLXSBXZWJLaXQgZG9lcyBub3QgY29tcGlsZSB3aXRob3V0IEpBVkFTQ1JJ
UFRfREVCVUdHRVIKKworICAgICAgICBGaXggYnVpbGQgZXJyb3JzIHdoZW4gamF2YXNjcmlwdC1k
ZWJ1Z2dlciBpcyBkaXNhYmxlZCBvbiBHVEsKKworICAgICAgICAqIEdOVW1ha2VmaWxlLmFtOgor
CiAyMDA5LTEyLTA0ICBBZGFtIExhbmdsZXkgIDxhZ2xAZ29vZ2xlLmNvbT4KIAogICAgICAgICBS
ZXZpZXdlZCBieSBFcmljIFNlaWRlbC4KZGlmZiAtLWdpdCBhL1dlYkNvcmUvR05VbWFrZWZpbGUu
YW0gYi9XZWJDb3JlL0dOVW1ha2VmaWxlLmFtCmluZGV4IDJkOGViMmYuLjhhNmEyOTAgMTAwNjQ0
Ci0tLSBhL1dlYkNvcmUvR05VbWFrZWZpbGUuYW0KKysrIGIvV2ViQ29yZS9HTlVtYWtlZmlsZS5h
bQpAQCAtMjExMiw2ICsyMTEyLDEwIEBAIEZFQVRVUkVfREVGSU5FU19KQVZBU0NSSVBUICs9IEVO
QUJMRV9KQVZBU0NSSVBUX0RFQlVHR0VSPTEKIHdlYmNvcmVfY3BwZmxhZ3MgKz0gXAogCS1ERU5B
QkxFX0pBVkFTQ1JJUFRfREVCVUdHRVI9MQogCitlbHNlCit3ZWJjb3JlX2NwcGZsYWdzICs9IFwK
KwktREVOQUJMRV9KQVZBU0NSSVBUX0RFQlVHR0VSPTAKKwogZW5kaWYgIyBFTkQgRU5BQkxFX0pB
VkFTQ1JJUFRfREVCVUdHRVIKIAogIyAtLS0tCmRpZmYgLS1naXQgYS9XZWJLaXQvZ3RrL0NoYW5n
ZUxvZyBiL1dlYktpdC9ndGsvQ2hhbmdlTG9nCmluZGV4IGMwYzQyZDQuLmFhMzM5MGEgMTAwNjQ0
Ci0tLSBhL1dlYktpdC9ndGsvQ2hhbmdlTG9nCisrKyBiL1dlYktpdC9ndGsvQ2hhbmdlTG9nCkBA
IC0xLDMgKzEsMTYgQEAKKzIwMDktMTItMDQgIEd5dXlvdW5nIEtpbSAgPGd5dXlvdW5nQGdtYWls
LmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBFcmljIFNlaWRlbC4KKworICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzIwMjQKKyAgICAgICAgW0dUS10g
V2ViS2l0IGRvZXMgbm90IGNvbXBpbGUgd2l0aG91dCBKQVZBU0NSSVBUX0RFQlVHR0VSCisKKyAg
ICAgICAgRml4IGJ1aWxkIGVycm9ycyB3aGVuIGphdmFzY3JpcHQtZGVidWdnZXIgaXMgZGlzYWJs
ZWQgb24gR1RLCisKKyAgICAgICAgKiB3ZWJraXQvd2Via2l0d2ViaW5zcGVjdG9yLmNwcDoKKyAg
ICAgICAgKHdlYmtpdF93ZWJfaW5zcGVjdG9yX3NldF9wcm9wZXJ0eSk6CisgICAgICAgICh3ZWJr
aXRfd2ViX2luc3BlY3Rvcl9nZXRfcHJvcGVydHkpOgorCiAyMDA5LTEyLTAzICBCcmFkeSBFaWRz
b24gIDxiZWlkc29uQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBTYW0gV2Vpbmln
LgpkaWZmIC0tZ2l0IGEvV2ViS2l0L2d0ay93ZWJraXQvd2Via2l0d2ViaW5zcGVjdG9yLmNwcCBi
L1dlYktpdC9ndGsvd2Via2l0L3dlYmtpdHdlYmluc3BlY3Rvci5jcHAKaW5kZXggNjhiMDNiMi4u
MmZlYjA2NCAxMDA2NDQKLS0tIGEvV2ViS2l0L2d0ay93ZWJraXQvd2Via2l0d2ViaW5zcGVjdG9y
LmNwcAorKysgYi9XZWJLaXQvZ3RrL3dlYmtpdC93ZWJraXR3ZWJpbnNwZWN0b3IuY3BwCkBAIC0z
NDYsMTIgKzM0NiwxNiBAQCBzdGF0aWMgdm9pZCB3ZWJraXRfd2ViX2luc3BlY3Rvcl9zZXRfcHJv
cGVydHkoR09iamVjdCogb2JqZWN0LCBndWludCBwcm9wX2lkLCBjbwogCiAgICAgc3dpdGNoKHBy
b3BfaWQpIHsKICAgICBjYXNlIFBST1BfSkFWQVNDUklQVF9QUk9GSUxJTkdfRU5BQkxFRDogewor
I2lmIEVOQUJMRShKQVZBU0NSSVBUX0RFQlVHR0VSKQogICAgICAgICBib29sIGVuYWJsZWQgPSBn
X3ZhbHVlX2dldF9ib29sZWFuKHZhbHVlKTsKICAgICAgICAgV2ViQ29yZTo6SW5zcGVjdG9yQ29u
dHJvbGxlciogY29udHJvbGxlciA9IHByaXYtPnBhZ2UtPmluc3BlY3RvckNvbnRyb2xsZXIoKTsK
ICAgICAgICAgaWYgKGVuYWJsZWQpCiAgICAgICAgICAgICBjb250cm9sbGVyLT5lbmFibGVQcm9m
aWxlcigpOwogICAgICAgICBlbHNlCiAgICAgICAgICAgICBjb250cm9sbGVyLT5kaXNhYmxlUHJv
ZmlsZXIoKTsKKyNlbHNlCisgICAgICAgIGdfbWVzc2FnZSgiUFJPUF9KQVZBU0NSSVBUX1BST0ZJ
TElOR19FTkFCTEVEIGlzIG5vdCB3b3JrIGJlY2F1c2Ugb2YgdGhlIGphdmFzY3JpcHQgZGVidWdn
ZXIgaXMgZGlzYWJsZWRcbiIpOworI2VuZGlmCiAgICAgICAgIGJyZWFrOwogICAgIH0KICAgICBj
YXNlIFBST1BfVElNRUxJTkVfUFJPRklMSU5HX0VOQUJMRUQ6IHsKQEAgLTM4Miw3ICszODYsMTEg
QEAgc3RhdGljIHZvaWQgd2Via2l0X3dlYl9pbnNwZWN0b3JfZ2V0X3Byb3BlcnR5KEdPYmplY3Qq
IG9iamVjdCwgZ3VpbnQgcHJvcF9pZCwgR1YKICAgICAgICAgZ192YWx1ZV9zZXRfc3RyaW5nKHZh
bHVlLCBwcml2LT5pbnNwZWN0ZWRfdXJpKTsKICAgICAgICAgYnJlYWs7CiAgICAgY2FzZSBQUk9Q
X0pBVkFTQ1JJUFRfUFJPRklMSU5HX0VOQUJMRUQ6CisjaWYgRU5BQkxFKEpBVkFTQ1JJUFRfREVC
VUdHRVIpCiAgICAgICAgIGdfdmFsdWVfc2V0X2Jvb2xlYW4odmFsdWUsIHByaXYtPnBhZ2UtPmlu
c3BlY3RvckNvbnRyb2xsZXIoKS0+cHJvZmlsZXJFbmFibGVkKCkpOworI2Vsc2UKKyAgICAgICAg
Z19tZXNzYWdlKCJQUk9QX0pBVkFTQ1JJUFRfUFJPRklMSU5HX0VOQUJMRUQgaXMgbm90IHdvcmsg
YmVjYXVzZSBvZiB0aGUgamF2YXNjcmlwdCBkZWJ1Z2dlciBpcyBkaXNhYmxlZFxuIik7CisjZW5k
aWYKICAgICAgICAgYnJlYWs7CiAgICAgY2FzZSBQUk9QX1RJTUVMSU5FX1BST0ZJTElOR19FTkFC
TEVEOgogICAgICAgICBnX3ZhbHVlX3NldF9ib29sZWFuKHZhbHVlLCBwcml2LT5wYWdlLT5pbnNw
ZWN0b3JDb250cm9sbGVyKCktPnRpbWVsaW5lQWdlbnQoKSAhPSAwKTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>