Diff
Modified: trunk/Source/WebCore/ChangeLog (102471 => 102472)
--- trunk/Source/WebCore/ChangeLog 2011-12-09 21:32:02 UTC (rev 102471)
+++ trunk/Source/WebCore/ChangeLog 2011-12-09 21:41:58 UTC (rev 102472)
@@ -1,3 +1,19 @@
+2011-12-09 Tony Chang <t...@chromium.org>
+
+ Unreviewed, rolling out r102416.
+ http://trac.webkit.org/changeset/102416
+ https://bugs.webkit.org/show_bug.cgi?id=73394
+
+ Chromium Win clobber builds are failing.
+
+ * WebCore.gyp/WebCore.gyp:
+ * WebCore.gyp/scripts/action_derivedsourcesallinone.py:
+ (main):
+ * WebCore.gypi:
+ * bindings/scripts/generate-bindings.pl:
+ * page/DOMWindow.idl:
+ * webaudio/DOMWindowWebAudio.idl: Removed.
+
2011-12-09 Eric Carlson <eric.carl...@apple.com>
JSC wrappers for TextTrack and TextTrackCue should not be collected during event dispatch or when owner is reachable
Modified: trunk/Source/WebCore/WebCore.gyp/WebCore.gyp (102471 => 102472)
--- trunk/Source/WebCore/WebCore.gyp/WebCore.gyp 2011-12-09 21:32:02 UTC (rev 102471)
+++ trunk/Source/WebCore/WebCore.gyp/WebCore.gyp 2011-12-09 21:41:58 UTC (rev 102472)
@@ -446,48 +446,9 @@
]
},
{
- 'target_name': 'generate_supplemental_dependency',
- 'type': 'none',
- 'actions': [
- {
- 'action_name': 'generateSupplementalDependency',
- 'variables': {
- # Write sources into a file, so that the action command line won't
- # exceed OS limits.
- 'idl_files_list': '<|(idl_files_list.tmp <@(bindings_idl_files))',
- },
- 'inputs': [
- '../bindings/scripts/resolve-supplemental.pl',
- '../bindings/scripts/IDLParser.pm',
- '<(idl_files_list)',
- '<!@(cat <(idl_files_list))',
- ],
- 'outputs': [
- '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
- ],
- 'action': [
- 'perl',
- '-w',
- '-I../bindings/scripts',
- '../bindings/scripts/resolve-supplemental.pl',
- '--defines',
- '<(feature_defines) LANGUAGE_JAVASCRIPT V8_BINDING',
- '--idlFilesList',
- '<(idl_files_list)',
- '--supplementalDependencyFile',
- '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
- ],
- 'message': 'Resolving [Supplemental=XXX] dependencies in all IDL files',
- }
- ]
- },
- {
'target_name': 'webcore_bindings_sources',
'type': 'none',
'hard_dependency': 1,
- 'dependencies': [
- 'generate_supplemental_dependency',
- ],
'sources': [
# bison rule
'../css/CSSGrammar.y',
@@ -900,9 +861,15 @@
},
{
'action_name': 'derived_sources_all_in_one',
+ 'variables': {
+ # Write sources into a file, so that the action command line won't
+ # exceed OS limites.
+ 'idls_list_temp_file': '<|(idls_list_temp_file.tmp <@(bindings_idl_files))',
+ },
'inputs': [
'scripts/action_derivedsourcesallinone.py',
- '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
+ '<(idls_list_temp_file)',
+ '<!@(cat <(idls_list_temp_file))',
],
'outputs': [
'<@(derived_sources_aggregate_files)',
@@ -910,7 +877,7 @@
'action': [
'python',
'scripts/action_derivedsourcesallinone.py',
- '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
+ '<(idls_list_temp_file)',
'--',
'<@(derived_sources_aggregate_files)',
],
@@ -963,7 +930,6 @@
'../bindings/scripts/IDLParser.pm',
'../bindings/scripts/IDLStructure.pm',
'../bindings/scripts/preprocessor.pm',
- '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
],
'outputs': [
# FIXME: The .cpp file should be in webkit/bindings once
@@ -1009,8 +975,6 @@
'--generator',
'V8',
'<@(generator_include_dirs)',
- '--supplementalDependencyFile',
- '<(SHARED_INTERMEDIATE_DIR)/supplemental_dependency.tmp',
'<(RULE_INPUT_PATH)',
],
'message': 'Generating binding from <(RULE_INPUT_PATH)',
Modified: trunk/Source/WebCore/WebCore.gyp/scripts/action_derivedsourcesallinone.py (102471 => 102472)
--- trunk/Source/WebCore/WebCore.gyp/scripts/action_derivedsourcesallinone.py 2011-12-09 21:32:02 UTC (rev 102471)
+++ trunk/Source/WebCore/WebCore.gyp/scripts/action_derivedsourcesallinone.py 2011-12-09 21:41:58 UTC (rev 102472)
@@ -32,12 +32,12 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
-# action_derivedsourcesallinone.py generates a single cpp file that includes
+# action_derivedsourceslist.py generates a single cpp file that includes
# all v8 bindings cpp files generated from idls. Files can be assigned into
# multiple output files, to reduce maximum compilation unit size and allow
# parallel compilation.
#
-# usage: action_derivedsourcesallinone.py IDL_FILES_LIST -- OUTPUT_FILE1 OUTPUT_FILE2 ...
+# usage: action_derivedsourceslist.py IDL_FILES_LIST -- OUTPUT_FILE1 OUTPUT_FILE2 ...
#
# Note that IDL_FILES_LIST is a text file containing the IDL file paths.
@@ -48,9 +48,6 @@
import subprocess
import sys
-sys.path.append("../../../Tools/Scripts/")
-from webkitpy.common.system import path
-
# A regexp for finding Conditional attributes in interface definitions.
conditionalPattern = re.compile('interface[\s]*\[[^\]]*Conditional=([\_0-9a-zA-Z&|]*)')
@@ -189,12 +186,7 @@
outputFileNames = args[inOutBreakIndex+1:]
inputFile = open(inputFileName, 'r')
- idlFileNames = []
- for line in inputFile:
- idlFileName = line[:-1].split(' ')[0]
- if idlFileName.find("/cygdrive") == 0:
- idlFileName = path.cygpath(idlFileName)
- idlFileNames.append(idlFileName)
+ idlFileNames = inputFile.read().split('\n')
inputFile.close()
filesMetaData = extractMetaData(idlFileNames)
Modified: trunk/Source/WebCore/WebCore.gypi (102471 => 102472)
--- trunk/Source/WebCore/WebCore.gypi 2011-12-09 21:32:02 UTC (rev 102471)
+++ trunk/Source/WebCore/WebCore.gypi 2011-12-09 21:41:58 UTC (rev 102472)
@@ -1495,7 +1495,6 @@
'webaudio/BiquadFilterNode.idl',
'webaudio/ConvolverNode.idl',
'webaudio/DelayNode.idl',
- 'webaudio/DOMWindowWebAudio.idl',
'webaudio/DynamicsCompressorNode.idl',
'webaudio/HighPass2FilterNode.idl',
'webaudio/_javascript_AudioNode.idl',
Modified: trunk/Source/WebCore/bindings/scripts/generate-bindings.pl (102471 => 102472)
--- trunk/Source/WebCore/bindings/scripts/generate-bindings.pl 2011-12-09 21:32:02 UTC (rev 102471)
+++ trunk/Source/WebCore/bindings/scripts/generate-bindings.pl 2011-12-09 21:41:58 UTC (rev 102472)
@@ -112,79 +112,6 @@
my $targetParser = IDLParser->new(!$verbose);
my $targetDocument = $targetParser->Parse($targetIdlFile, $defines, $preprocessor);
-# FIXME(haraken): Remove this if-else statement.
-# This if-else statement is temporary and will be removed
-# after build scripts for all platforms support [Supplemental] IDL.
-# The motivation for the [Supplemented] IDL is as follows:
-#
-# In order to support the [Supplemental] IDL, we need to
-# (1) run resolve-supplemental.pl and generate supplemental_dependency.tmp
-# (2) and run generate-bindings.pl with the supplemental_dependency.tmp.
-#
-# This build flow requires a change on the following build scripts,
-# but changing all the build scripts all at once without any regression is too difficult:
-#
-# - DerivedSources.make
-# - DerivedSources.pri
-# - GNUmakefile.am
-# - PlatformBlackBerry.cmake
-# - UseJSC.cmake
-# - UseV8.cmake
-# - WebCore.vcproj/MigrateScripts
-# - WebCore.vcproj/WebCore.vcproj
-# - bindings/gobject/GNUmakefile.am
-# - WebCore.gyp/WebCore.gyp
-#
-# Thus, we are planning to change the build scripts one by one, which implies that
-# we need to allow the temporary state in which some build scripts support [Supplemental] IDL
-# but others do not. To accomplish this, we introduce a temporal IDL, [Supplemented].
-# The [Supplemented] IDL on an attribute means that the attribute is marked with [Supplemental]
-# in another IDL file somewhere, like this:
-#
-# DOMWindowWebAudio.idl:
-# interface [
-# Supplemental=DOMWindow
-# ] DOMWindowWebAudio {
-# attribute attr1;
-# attribute attr2;
-# };
-#
-# DOMWindow.idl:
-# interface [
-# ] DOMWindow {
-# attribute [Supplemented] attr1; // This line will be removed after all build scripts support the [SupplementalL.
-# attribute [Supplemented] attr2; // This line will be removed after all build scripts support the [SupplementalL.
-# attribute attr3;
-# attribute attr4;
-# };
-#
-# Assuming these IDL files, the below code is doing the following logic:
-#
-# - If a given build script supports the [Supplemental] IDL (i.e. --supplementalDependencyFile is specified),
-# we ignore all attributes with the [Supplemented] IDL.
-# - Otherwise (i.e. --supplementalDependencyFile is not specified),
-# we treat all attributes with the [Supplemented] IDL as normal attributes
-# and instead ignore all attributes with the [Supplemental] IDL
-# (i.e. we generate nothing from the idl file with the [Supplemental] IDL).
-if ($supplementalDependencyFile) {
- foreach my $dataNode (@{$targetDocument->classes}) {
- my @nonSupplementedAttributes;
- foreach my $attribute (@{$dataNode->attributes}) {
- if (!$attribute->signature->extendedAttributes->{"Supplemented"}) {
- push(@nonSupplementedAttributes, $attribute);
- }
- }
- $dataNode->attributes(\@nonSupplementedAttributes);
- }
-} else {
- foreach my $dataNode (@{$targetDocument->classes}) {
- if ($dataNode->extendedAttributes->{"Supplemental"}) {
- exit 0;
- }
- }
-}
-# Temporary if-else statement until here.
-
foreach my $idlFile (@supplementedIdlFiles) {
next if $idlFile eq $targetIdlFile;
Modified: trunk/Source/WebCore/page/DOMWindow.idl (102471 => 102472)
--- trunk/Source/WebCore/page/DOMWindow.idl 2011-12-09 21:32:02 UTC (rev 102471)
+++ trunk/Source/WebCore/page/DOMWindow.idl 2011-12-09 21:41:58 UTC (rev 102472)
@@ -522,8 +522,8 @@
attribute [JSCCustomGetter] Float64ArrayConstructor Float64Array; // Usable with new operator
attribute [JSCCustomGetter] DataViewConstructor DataView; // Usable with new operator
- attribute [Supplemented, JSCCustomGetter, Conditional=WEB_AUDIO, EnabledAtRuntime] AudioContextConstructor webkitAudioContext; // Usable with new operator
- attribute [Supplemented, Conditional=WEB_AUDIO] AudioPannerNodeConstructor webkitAudioPannerNode; // Needed for panning model constants
+ attribute [JSCCustomGetter,Conditional=WEB_AUDIO,EnabledAtRuntime] AudioContextConstructor webkitAudioContext; // Usable with new operator
+ attribute [Conditional=WEB_AUDIO] AudioPannerNodeConstructor webkitAudioPannerNode; // Needed for panning model constants
// Event Constructors
attribute EventConstructor Event;
@@ -552,8 +552,8 @@
attribute [Conditional=TOUCH_EVENTS] TouchEventConstructor TouchEvent;
attribute [Conditional=WEB_SOCKETS] CloseEventConstructor CloseEvent;
attribute StorageEventConstructor StorageEvent;
- attribute [Supplemented, Conditional=WEB_AUDIO] AudioProcessingEventConstructor AudioProcessingEvent;
- attribute [Supplemented, Conditional=WEB_AUDIO] OfflineAudioCompletionEventConstructor OfflineAudioCompletionEvent;
+ attribute [Conditional=WEB_AUDIO] AudioProcessingEventConstructor AudioProcessingEvent;
+ attribute [Conditional=WEB_AUDIO] OfflineAudioCompletionEventConstructor OfflineAudioCompletionEvent;
attribute [Conditional=INPUT_SPEECH] SpeechInputEventConstructor SpeechInputEvent;
attribute [Conditional=MEDIA_STREAM] MediaStreamEventConstructor MediaStreamEvent;
attribute [Conditional=WEBGL] WebGLContextEventConstructor WebGLContextEvent;
Deleted: trunk/Source/WebCore/webaudio/DOMWindowWebAudio.idl (102471 => 102472)
--- trunk/Source/WebCore/webaudio/DOMWindowWebAudio.idl 2011-12-09 21:32:02 UTC (rev 102471)
+++ trunk/Source/WebCore/webaudio/DOMWindowWebAudio.idl 2011-12-09 21:41:58 UTC (rev 102472)
@@ -1,32 +0,0 @@
-/*
- * Copyright (C) 2011 Google Inc. All rights reserved.
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Library General Public
- * License as published by the Free Software Foundation; either
- * version 2 of the License, or (at your option) any later version.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * Library General Public License for more details.
- *
- * You should have received a copy of the GNU Library General Public License
- * along with this library; see the file COPYING.LIB. If not, write to
- * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
- * Boston, MA 02110-1301, USA.
- */
-
-module window {
-
- interface [
- Conditional=WEB_AUDIO,
- Supplemental=DOMWindow
- ] DOMWindowWebAudio {
- attribute [JSCCustomGetter, EnabledAtRuntime] AudioContextConstructor webkitAudioContext;
- attribute AudioPannerNodeConstructor webkitAudioPannerNode;
- attribute AudioProcessingEventConstructor AudioProcessingEvent;
- attribute OfflineAudioCompletionEventConstructor OfflineAudioCompletionEvent;
- };
-
-}
\ No newline at end of file