Changeset 147731 in webkit


Ignore:
Timestamp:
Apr 5, 2013 2:25:25 AM (11 years ago)
Author:
commit-queue@webkit.org
Message:

REGRESSION(r145279): Build fails while linking when MEDIA_STREAM is enabled
​https://bugs.webkit.org/show_bug.cgi?id=113294

Patch by Carlos Garcia Campos <​cgarcia@igalia.com> on 2013-04-05
Reviewed by Xan Lopez.

In r145279 NamedGetter extended attribute is added to
RTCStatsResponse.idl which makes the code generator to add
prototypes for JSRTCStatsResponse::nameGetter and
JSRTCStatsResponse::canGetItemsForName to the generated
JSRTCStatsResponse header, but they are not implemented
anywhere. We need to provide a JSRTCStatsResponseCustom file with
the implementation of both functions.

  • GNUmakefile.list.am: Add JSRTCStatsResponseCustom.cpp to

compilation.

  • Target.pri: Ditto.
  • UseJSC.cmake: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.vcxproj/WebCore.vcxproj: Ditto.
  • WebCore.vcxproj/WebCore.vcxproj.filters: Ditto.
  • bindings/js/JSRTCStatsResponseCustom.cpp: Added.

(WebCore):
(WebCore::JSRTCStatsResponse::canGetItemsForName):
(WebCore::JSRTCStatsResponse::nameGetter):

Location:
trunk/Source/WebCore
Files:
1 added
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/ChangeLog

    r147730 r147731  
     12013-04-05  Carlos Garcia Campos  <cgarcia@igalia.com>
     2
     3        REGRESSION(r145279): Build fails while linking when MEDIA_STREAM is enabled
     4        https://bugs.webkit.org/show_bug.cgi?id=113294
     5
     6        Reviewed by Xan Lopez.
     7
     8        In r145279 NamedGetter extended attribute is added to
     9        RTCStatsResponse.idl which makes the code generator to add
     10        prototypes for JSRTCStatsResponse::nameGetter and
     11        JSRTCStatsResponse::canGetItemsForName to the generated
     12        JSRTCStatsResponse header, but they are not implemented
     13        anywhere. We need to provide a JSRTCStatsResponseCustom file with
     14        the implementation of both functions.
     15
     16        * GNUmakefile.list.am: Add JSRTCStatsResponseCustom.cpp to
     17        compilation.
     18        * Target.pri: Ditto.
     19        * UseJSC.cmake: Ditto.
     20        * WebCore.vcproj/WebCore.vcproj: Ditto.
     21        * WebCore.vcxproj/WebCore.vcxproj: Ditto.
     22        * WebCore.vcxproj/WebCore.vcxproj.filters: Ditto.
     23        * bindings/js/JSRTCStatsResponseCustom.cpp: Added.
     24        (WebCore):
     25        (WebCore::JSRTCStatsResponse::canGetItemsForName):
     26        (WebCore::JSRTCStatsResponse::nameGetter):
     27
    1282013-04-05  Ryosuke Niwa  <rniwa@webkit.org>
    229

  • trunk/Source/WebCore/GNUmakefile.list.am

    r147715 r147731  
    24582458        Source/WebCore/bindings/js/JSSQLTransactionCustom.cpp \
    24592459        Source/WebCore/bindings/js/JSSQLTransactionSyncCustom.cpp \
     2460        Source/WebCore/bindings/js/JSRTCStatsResponseCustom.cpp \
    24602461        Source/WebCore/bindings/js/JSSharedWorkerCustom.cpp \
    24612462        Source/WebCore/bindings/js/JSStorageCustom.cpp \

  • trunk/Source/WebCore/Target.pri

    r147545 r147731  
    175175     bindings/js/JSProcessingInstructionCustom.cpp \
    176176     bindings/js/JSRequestAnimationFrameCallbackCustom.cpp \
     177     bindings/js/JSRTCStatsResponseCustom.cpp \
    177178     bindings/js/JSStorageCustom.cpp \
    178179     bindings/js/JSStyleSheetCustom.cpp \

  • trunk/Source/WebCore/UseJSC.cmake

    r146585 r147731  
    264264endif ()
    265265
     266if (ENABLE_MEDIA_STREAM)
     267    list(APPEND WebCore_SOURCES
     268        bindings/js/JSRTCStatsResponseCustom.cpp
     269    )
     270endif ()
     271
    266272list(APPEND SCRIPTS_BINDINGS
    267273    ${WEBCORE_DIR}/bindings/scripts/CodeGenerator.pm

  • trunk/Source/WebCore/WebCore.vcproj/WebCore.vcproj

    r147643 r147731  
    7156371563                                </File>
    7156471564                                <File
     71565                                        RelativePath="..\bindings\js\JSRTCStatsResponseCustom.cpp"
     71566                                        >
     71567                                        <FileConfiguration
     71568                                                Name="Debug|Win32"
     71569                                                ExcludedFromBuild="true"
     71570                                                >
     71571                                                <Tool
     71572                                                        Name="VCCLCompilerTool"
     71573                                                />
     71574                                        </FileConfiguration>
     71575                                        <FileConfiguration
     71576                                                Name="Release|Win32"
     71577                                                ExcludedFromBuild="true"
     71578                                                >
     71579                                                <Tool
     71580                                                        Name="VCCLCompilerTool"
     71581                                                />
     71582                                        </FileConfiguration>
     71583                                        <FileConfiguration
     71584                                                Name="Debug_Cairo_CFLite|Win32"
     71585                                                ExcludedFromBuild="true"
     71586                                                >
     71587                                                <Tool
     71588                                                        Name="VCCLCompilerTool"
     71589                                                />
     71590                                        </FileConfiguration>
     71591                                        <FileConfiguration
     71592                                                Name="Release_Cairo_CFLite|Win32"
     71593                                                ExcludedFromBuild="true"
     71594                                                >
     71595                                                <Tool
     71596                                                        Name="VCCLCompilerTool"
     71597                                                />
     71598                                        </FileConfiguration>
     71599                                        <FileConfiguration
     71600                                                Name="Debug_All|Win32"
     71601                                                ExcludedFromBuild="true"
     71602                                                >
     71603                                                <Tool
     71604                                                        Name="VCCLCompilerTool"
     71605                                                />
     71606                                        </FileConfiguration>
     71607                                        <FileConfiguration
     71608                                                Name="Production|Win32"
     71609                                                ExcludedFromBuild="true"
     71610                                                >
     71611                                                <Tool
     71612                                                        Name="VCCLCompilerTool"
     71613                                                />
     71614                                        </FileConfiguration>
     71615                                </File>
     71616                                <File
     71617                                        RelativePath="..\bindings\js\JSRTCStatsResponseCustom.h"
     71618                                        >
     71619                                </File>
     71620                                <File
    7156571621                                        RelativePath="..\bindings\js\JSStorageCustom.cpp"
    7156671622                                        >

  • trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj

    r147643 r147731  
    99189918    </ClCompile>
    99199919    <ClCompile Include="..\bindings\js\JSSQLTransactionSyncCustom.cpp">
     9920      <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">true</ExcludedFromBuild>
     9921      <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Debug_WinCairo|Win32'">true</ExcludedFromBuild>
     9922      <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='DebugSuffix|Win32'">true</ExcludedFromBuild>
     9923      <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Release|Win32'">true</ExcludedFromBuild>
     9924      <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Release_WinCairo|Win32'">true</ExcludedFromBuild>
     9925      <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Production|Win32'">true</ExcludedFromBuild>
     9926    </ClCompile>
     9927    <ClCompile Include="..\bindings\js\JSRTCStatsResponseCustom.cpp">
    99209928      <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">true</ExcludedFromBuild>
    99219929      <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Debug_WinCairo|Win32'">true</ExcludedFromBuild>

  • trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters

    r147643 r147731  
    60826082      <Filter>bindings\js</Filter>
    60836083    </ClCompile>
     6084    <ClCompile Include="..\bindings\js\JSRTCStatsResponseCustom.cpp">
     6085      <Filter>bindings\js</Filter>
     6086    </ClCompile>
    60846087    <ClCompile Include="..\bindings\js\JSStorageCustom.cpp">
    60856088      <Filter>bindings\js</Filter>
Note: See TracChangeset for help on using the changeset viewer.