[webkit-reviews] review requested: [Bug 20620] Wrap uses of KJS in core classes with USE(JSC) : [Attachment 23152] Add #if USE(JSC) around KJS dependencies

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Sep 3 17:38:14 PDT 2008


Eric Seidel <eric at webkit.org> has asked  for review:
Bug 20620: Wrap uses of KJS in core classes with USE(JSC)
https://bugs.webkit.org/show_bug.cgi?id=20620

Attachment 23152: Add #if USE(JSC) around KJS dependencies
https://bugs.webkit.org/attachment.cgi?id=23152&action=edit

------- Additional Comments from Eric Seidel <eric at webkit.org>
 WebCore/ChangeLog				  |   13 ++++++++
 WebCore/bindings/js/JSPluginElementFunctions.cpp |   35 ----------------------
 WebCore/config.h				  |   22 +++++++------
 WebCore/html/HTMLAppletElement.cpp		  |    2 -
 WebCore/html/HTMLAppletElement.h		  |    2 -
 WebCore/html/HTMLEmbedElement.cpp		  |    6 ----
 WebCore/html/HTMLEmbedElement.h		  |    2 -
 WebCore/html/HTMLObjectElement.cpp		  |    5 ---
 WebCore/html/HTMLObjectElement.h		  |    2 -
 WebCore/html/HTMLPlugInElement.cpp		  |    6 ++--
 WebCore/html/HTMLPlugInElement.h		  |    8 ++--
 WebCore/platform/text/AtomicString.cpp 	  |    4 ++
 WebCore/platform/text/AtomicString.h		  |    6 ++++
 WebCore/platform/text/PlatformString.h 	  |    8 +++++
 WebCore/platform/text/String.cpp		  |    5 ++-
 WebCore/platform/text/StringImpl.cpp		  |    4 --
 WebCore/platform/text/StringImpl.h		  |    1 -
 17 files changed, 54 insertions(+), 77 deletions(-)


More information about the webkit-reviews mailing list