Title: [128234] trunk/Source
Revision
128234
Author
mli...@apple.com
Date
2012-09-11 16:06:00 -0700 (Tue, 11 Sep 2012)

Log Message

OS X port should compile with newer versions of clang
https://bugs.webkit.org/show_bug.cgi?id=96434

Source/_javascript_Core: 

m_identIsVarDecl is unused - remove it.

Reviewed by Anders Carlsson.

* parser/NodeConstructors.h:
(JSC::ForInNode::ForInNode):
* parser/Nodes.h:
(ForInNode):

Source/WebCore: 

Reviewed by Anders Carlsson.

Guard m_hasTouchEventHandler behind ENABLE(TOUCH_EVENTS).
* html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::HTMLInputElement):
* html/HTMLInputElement.h:
(HTMLInputElement):

Fix uninitialized variable.
* platform/graphics/cg/GraphicsContextCG.cpp:
(WebCore::createLinearSRGBColorSpace):

Source/WebKit/mac: 

m_isTerminated is unused in the Hosted flavor of NetscapePluginStream.

Reviewed by Anders Carlsson.

* Plugins/Hosted/HostedNetscapePluginStream.h:
(HostedNetscapePluginStream):
* Plugins/Hosted/HostedNetscapePluginStream.mm:
(WebKit::HostedNetscapePluginStream::HostedNetscapePluginStream):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (128233 => 128234)


--- trunk/Source/_javascript_Core/ChangeLog	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/_javascript_Core/ChangeLog	2012-09-11 23:06:00 UTC (rev 128234)
@@ -1,3 +1,17 @@
+2012-09-11  Matt Lilek  <m...@apple.com>
+
+        OS X port should compile with newer versions of clang
+        https://bugs.webkit.org/show_bug.cgi?id=96434
+
+        m_identIsVarDecl is unused - remove it.
+
+        Reviewed by Anders Carlsson.
+
+        * parser/NodeConstructors.h:
+        (JSC::ForInNode::ForInNode):
+        * parser/Nodes.h:
+        (ForInNode):
+
 2012-09-11  Filip Pizlo  <fpi...@apple.com>
 
         LLInt should optimize and profile array length accesses

Modified: trunk/Source/_javascript_Core/parser/NodeConstructors.h (128233 => 128234)


--- trunk/Source/_javascript_Core/parser/NodeConstructors.h	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/_javascript_Core/parser/NodeConstructors.h	2012-09-11 23:06:00 UTC (rev 128234)
@@ -812,7 +812,6 @@
         , m_lexpr(l)
         , m_expr(expr)
         , m_statement(statement)
-        , m_identIsVarDecl(false)
     {
     }
 
@@ -822,7 +821,6 @@
         , m_lexpr(new (globalData) ResolveNode(location, ident, divot - startOffset))
         , m_expr(expr)
         , m_statement(statement)
-        , m_identIsVarDecl(true)
     {
         if (in) {
             AssignResolveNode* node = new (globalData) AssignResolveNode(location, ident, in);

Modified: trunk/Source/_javascript_Core/parser/Nodes.h (128233 => 128234)


--- trunk/Source/_javascript_Core/parser/Nodes.h	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/_javascript_Core/parser/Nodes.h	2012-09-11 23:06:00 UTC (rev 128234)
@@ -1204,7 +1204,6 @@
         ExpressionNode* m_lexpr;
         ExpressionNode* m_expr;
         StatementNode* m_statement;
-        bool m_identIsVarDecl;
     };
 
     class ContinueNode : public StatementNode, public ThrowableExpressionData {

Modified: trunk/Source/WebCore/ChangeLog (128233 => 128234)


--- trunk/Source/WebCore/ChangeLog	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/WebCore/ChangeLog	2012-09-11 23:06:00 UTC (rev 128234)
@@ -1,3 +1,20 @@
+2012-09-11  Matt Lilek  <m...@apple.com>
+
+        OS X port should compile with newer versions of clang
+        https://bugs.webkit.org/show_bug.cgi?id=96434
+
+        Reviewed by Anders Carlsson.
+
+        Guard m_hasTouchEventHandler behind ENABLE(TOUCH_EVENTS).
+        * html/HTMLInputElement.cpp:
+        (WebCore::HTMLInputElement::HTMLInputElement):
+        * html/HTMLInputElement.h:
+        (HTMLInputElement):
+
+        Fix uninitialized variable.
+        * platform/graphics/cg/GraphicsContextCG.cpp:
+        (WebCore::createLinearSRGBColorSpace):
+
 2012-09-11  Arnaud Renevier  <a.renev...@sisa.samsung.com>
 
         [Gtk] allow building with css-shaders

Modified: trunk/Source/WebCore/html/HTMLInputElement.cpp (128233 => 128234)


--- trunk/Source/WebCore/html/HTMLInputElement.cpp	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/WebCore/html/HTMLInputElement.cpp	2012-09-11 23:06:00 UTC (rev 128234)
@@ -119,7 +119,9 @@
     , m_valueAttributeWasUpdatedAfterParsing(false)
     , m_wasModifiedByUser(false)
     , m_canReceiveDroppedFiles(false)
+#if ENABLE(TOUCH_EVENTS)
     , m_hasTouchEventHandler(false)
+#endif
     , m_inputType(InputType::createText(this))
 {
     ASSERT(hasTagName(inputTag) || hasTagName(isindexTag));

Modified: trunk/Source/WebCore/html/HTMLInputElement.h (128233 => 128234)


--- trunk/Source/WebCore/html/HTMLInputElement.h	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/WebCore/html/HTMLInputElement.h	2012-09-11 23:06:00 UTC (rev 128234)
@@ -412,7 +412,9 @@
     bool m_valueAttributeWasUpdatedAfterParsing : 1;
     bool m_wasModifiedByUser : 1;
     bool m_canReceiveDroppedFiles : 1;
-    bool m_hasTouchEventHandler: 1;
+#if ENABLE(TOUCH_EVENTS)
+    bool m_hasTouchEventHandler : 1;
+#endif
     OwnPtr<InputType> m_inputType;
 #if ENABLE(DATALIST_ELEMENT)
     OwnPtr<ListAttributeTargetObserver> m_listAttributeTargetObserver;

Modified: trunk/Source/WebCore/platform/graphics/cg/GraphicsContextCG.cpp (128233 => 128234)


--- trunk/Source/WebCore/platform/graphics/cg/GraphicsContextCG.cpp	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/WebCore/platform/graphics/cg/GraphicsContextCG.cpp	2012-09-11 23:06:00 UTC (rev 128234)
@@ -197,7 +197,7 @@
     
     CFBundleRef webCoreBundle = CFBundleGetBundleWithIdentifier(CFSTR("com.apple.WebCore"));
     RetainPtr<CFURLRef> iccProfileURL(CFBundleCopyResourceURL(webCoreBundle, CFSTR("linearSRGB"), CFSTR("icc"), 0));
-    CFDataRef iccProfileData;
+    CFDataRef iccProfileData = 0;
     
     if (iccProfileURL && CFURLCreateDataAndPropertiesFromResource(0, iccProfileURL.get(), &iccProfileData, 0, 0, 0))
         linearSRGBSpace = CGColorSpaceCreateWithICCProfile(iccProfileData);

Modified: trunk/Source/WebKit/mac/ChangeLog (128233 => 128234)


--- trunk/Source/WebKit/mac/ChangeLog	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/WebKit/mac/ChangeLog	2012-09-11 23:06:00 UTC (rev 128234)
@@ -1,3 +1,17 @@
+2012-09-11  Matt Lilek  <m...@apple.com>
+
+        OS X port should compile with newer versions of clang
+        https://bugs.webkit.org/show_bug.cgi?id=96434
+
+        m_isTerminated is unused in the Hosted flavor of NetscapePluginStream.
+
+        Reviewed by Anders Carlsson.
+
+        * Plugins/Hosted/HostedNetscapePluginStream.h:
+        (HostedNetscapePluginStream):
+        * Plugins/Hosted/HostedNetscapePluginStream.mm:
+        (WebKit::HostedNetscapePluginStream::HostedNetscapePluginStream):
+
 2012-09-10  Jer Noble  <jer.no...@apple.com>
 
         Unreviewed; rolling out r128081.

Modified: trunk/Source/WebKit/mac/Plugins/Hosted/HostedNetscapePluginStream.h (128233 => 128234)


--- trunk/Source/WebKit/mac/Plugins/Hosted/HostedNetscapePluginStream.h	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/WebKit/mac/Plugins/Hosted/HostedNetscapePluginStream.h	2012-09-11 23:06:00 UTC (rev 128234)
@@ -89,7 +89,6 @@
     
     RefPtr<NetscapePluginInstanceProxy> m_instance;
     uint32_t m_streamID;
-    bool m_isTerminated;
     RetainPtr<NSMutableURLRequest> m_request;
 
     RetainPtr<NSURL> m_requestURL;

Modified: trunk/Source/WebKit/mac/Plugins/Hosted/HostedNetscapePluginStream.mm (128233 => 128234)


--- trunk/Source/WebKit/mac/Plugins/Hosted/HostedNetscapePluginStream.mm	2012-09-11 23:04:11 UTC (rev 128233)
+++ trunk/Source/WebKit/mac/Plugins/Hosted/HostedNetscapePluginStream.mm	2012-09-11 23:06:00 UTC (rev 128234)
@@ -54,7 +54,6 @@
 HostedNetscapePluginStream::HostedNetscapePluginStream(NetscapePluginInstanceProxy* instance, uint32_t streamID, NSURLRequest *request)
     : m_instance(instance)
     , m_streamID(streamID)
-    , m_isTerminated(false)
     , m_request(AdoptNS, [request mutableCopy])
     , m_requestURL([request URL])
     , m_frameLoader(0)
@@ -73,7 +72,6 @@
 HostedNetscapePluginStream::HostedNetscapePluginStream(NetscapePluginInstanceProxy* instance, WebCore::FrameLoader* frameLoader)
     : m_instance(instance)
     , m_streamID(1)
-    , m_isTerminated(false)
     , m_frameLoader(frameLoader)
 {
 #ifndef NDEBUG
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to