⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Apr 26, 2010:

11:28 PM Changeset in webkit [58294] by jam@chromium.org
  • 3 edits
    1 add in trunk/WebKit/chromium

2010-04-26 John Abd-El-Malek <​jam@chromium.org>

Reviewed by Darin Fisher.

[chromium] Prepare to making WebThemeEngine cross-platform
​https://bugs.webkit.org/show_bug.cgi?id=38077

  • WebKit.gyp:
  • public/WebThemeEngine.h: Added.
  • public/win/WebThemeEngine.h:
11:19 PM Changeset in webkit [58293] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-04-26 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Remove unused code from text_diff.py
​https://bugs.webkit.org/show_bug.cgi?id=38170

  • Scripts/webkitpy/layout_tests/test_types/text_diff.py:
    • Remove is_render_tree_dump which appears unused.
11:10 PM Changeset in webkit [58292] by darin@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-04-26 Darin Fisher <​darin@chromium.org>

Reviewed by Adam Barth.

[Chromium] Crash after calling window.print() on a window object that has been closed.
​https://bugs.webkit.org/show_bug.cgi?id=38148

It is unfortunately not possible to write a layout test for this as the
crash occurs in an API that is only called by Chrome while showing a
print dialog.

  • src/WebViewImpl.cpp: (WebKit::WebView::willEnterModalLoop): (WebKit::WebView::didExitModalLoop):
10:56 PM Changeset in webkit [58291] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2010-04-26 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

webkit-patch pretty-diff is broken
​https://bugs.webkit.org/show_bug.cgi?id=38172

We need to register for these options because they're used when we look
up the diff.

  • Scripts/webkitpy/tool/steps/confirmdiff.py:
10:30 PM Changeset in webkit [58290] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2010-04-26 Adam Barth <​abarth@webkit.org>

Unreviewed. Update comment to better reflect reality.

  • bindings/scripts/test/TestObj.idl:
10:01 PM Changeset in webkit [58289] by abarth@webkit.org
  • 3 edits
    8 adds in trunk

2010-04-26 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Add ObjC and GObject to run-bindings-test
​https://bugs.webkit.org/show_bug.cgi?id=38168

Added expected results.

  • bindings/scripts/test/GObject/WebKitDOMTestObj.cpp: Added. (WebKit::kit): (webkit_dom_test_obj_void_method): (webkit_dom_test_obj_void_method_with_args): (webkit_dom_test_obj_int_method): (webkit_dom_test_obj_int_method_with_args): (webkit_dom_test_obj_obj_method): (webkit_dom_test_obj_obj_method_with_args): (webkit_dom_test_obj_method_with_exception): (webkit_dom_test_obj_method_with_optional_arg): (webkit_dom_test_obj_method_with_non_optional_arg_and_optional_arg): (webkit_dom_test_obj_method_with_non_optional_arg_and_two_optional_args): (webkit_dom_test_obj_get_read_only_int_attr): (webkit_dom_test_obj_get_read_only_string_attr): (webkit_dom_test_obj_get_read_only_test_obj_attr): (webkit_dom_test_obj_get_int_attr): (webkit_dom_test_obj_set_int_attr): (webkit_dom_test_obj_get_string_attr): (webkit_dom_test_obj_set_string_attr): (webkit_dom_test_obj_get_test_obj_attr): (webkit_dom_test_obj_set_test_obj_attr): (webkit_dom_test_obj_get_attr_with_exception): (webkit_dom_test_obj_set_attr_with_exception): (webkit_dom_test_obj_get_attr_with_setter_exception): (webkit_dom_test_obj_set_attr_with_setter_exception): (webkit_dom_test_obj_get_attr_with_getter_exception): (webkit_dom_test_obj_set_attr_with_getter_exception): (WebKit::wrapTestObj): (WebKit::core): (webkit_dom_test_obj_finalize): (webkit_dom_test_obj_set_property): (webkit_dom_test_obj_get_property): (webkit_dom_test_obj_class_init): (webkit_dom_test_obj_init):
  • bindings/scripts/test/GObject/WebKitDOMTestObj.h: Added.
  • bindings/scripts/test/GObject/WebKitDOMTestObjPrivate.h: Added.
  • bindings/scripts/test/ObjC/DOMTestObj.h: Added.
  • bindings/scripts/test/ObjC/DOMTestObj.mm: Added. (-[DOMTestObj dealloc]): (-[DOMTestObj finalize]): (-[DOMTestObj readOnlyIntAttr]): (-[DOMTestObj readOnlyStringAttr]): (-[DOMTestObj readOnlyTestObjAttr]): (-[DOMTestObj intAttr]): (-[DOMTestObj setIntAttr:]): (-[DOMTestObj stringAttr]): (-[DOMTestObj setStringAttr:]): (-[DOMTestObj testObjAttr]): (-[DOMTestObj setTestObjAttr:]): (-[DOMTestObj attrWithException]): (-[DOMTestObj setAttrWithException:]): (-[DOMTestObj attrWithSetterException]): (-[DOMTestObj setAttrWithSetterException:]): (-[DOMTestObj attrWithGetterException]): (-[DOMTestObj setAttrWithGetterException:]): (-[DOMTestObj customAttr]): (-[DOMTestObj setCustomAttr:]): (-[DOMTestObj voidMethod]): (-[DOMTestObj voidMethodWithArgs:strArg:objArg:]): (-[DOMTestObj intMethod]): (-[DOMTestObj intMethodWithArgs:strArg:objArg:]): (-[DOMTestObj objMethod]): (-[DOMTestObj objMethodWithArgs:strArg:objArg:]): (-[DOMTestObj methodWithException]): (-[DOMTestObj customMethod]): (-[DOMTestObj customMethodWithArgs:strArg:objArg:]): (-[DOMTestObj methodWithOptionalArg:]): (-[DOMTestObj methodWithNonOptionalArgAndOptionalArg:opt:]): (-[DOMTestObj methodWithNonOptionalArgAndTwoOptionalArgs:opt1:opt2:]): (core): (kit):
  • bindings/scripts/test/ObjC/DOMTestObjInternal.h: Added.

2010-04-26 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Add ObjC and GObject to run-bindings-test
​https://bugs.webkit.org/show_bug.cgi?id=38168

  • Scripts/run-bindings-tests:
9:49 PM Changeset in webkit [58288] by abarth@webkit.org
  • 4 edits
    2 moves
    4 adds in trunk

2010-04-26 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Add testing infrastructure for JSC bindings generator
​https://bugs.webkit.org/show_bug.cgi?id=38167

This required moving the test files around a bit more.

  • bindings/scripts/test/JS/JSTestObj.cpp: Added. (WebCore::): (WebCore::JSTestObjConstructor::JSTestObjConstructor): (WebCore::JSTestObjConstructor::classInfo): (WebCore::JSTestObjConstructor::createStructure): (WebCore::JSTestObjConstructor::getOwnPropertySlot): (WebCore::JSTestObjConstructor::getOwnPropertyDescriptor): (WebCore::JSTestObjPrototype::self): (WebCore::JSTestObjPrototype::getOwnPropertySlot): (WebCore::JSTestObjPrototype::getOwnPropertyDescriptor): (WebCore::JSTestObj::JSTestObj): (WebCore::JSTestObj::~JSTestObj): (WebCore::JSTestObj::createPrototype): (WebCore::JSTestObj::getOwnPropertySlot): (WebCore::JSTestObj::getOwnPropertyDescriptor): (WebCore::jsTestObjReadOnlyIntAttr): (WebCore::jsTestObjReadOnlyStringAttr): (WebCore::jsTestObjReadOnlyTestObjAttr): (WebCore::jsTestObjIntAttr): (WebCore::jsTestObjStringAttr): (WebCore::jsTestObjTestObjAttr): (WebCore::jsTestObjAttrWithException): (WebCore::jsTestObjAttrWithSetterException): (WebCore::jsTestObjAttrWithGetterException): (WebCore::jsTestObjCustomAttr): (WebCore::jsTestObjConstructor): (WebCore::JSTestObj::put): (WebCore::setJSTestObjIntAttr): (WebCore::setJSTestObjStringAttr): (WebCore::setJSTestObjTestObjAttr): (WebCore::setJSTestObjAttrWithException): (WebCore::setJSTestObjAttrWithSetterException): (WebCore::setJSTestObjAttrWithGetterException): (WebCore::setJSTestObjCustomAttr): (WebCore::JSTestObj::getConstructor): (WebCore::jsTestObjPrototypeFunctionVoidMethod): (WebCore::jsTestObjPrototypeFunctionVoidMethodWithArgs): (WebCore::jsTestObjPrototypeFunctionIntMethod): (WebCore::jsTestObjPrototypeFunctionIntMethodWithArgs): (WebCore::jsTestObjPrototypeFunctionObjMethod): (WebCore::jsTestObjPrototypeFunctionObjMethodWithArgs): (WebCore::jsTestObjPrototypeFunctionMethodWithException): (WebCore::jsTestObjPrototypeFunctionCustomMethod): (WebCore::jsTestObjPrototypeFunctionCustomMethodWithArgs): (WebCore::jsTestObjPrototypeFunctionMethodWithOptionalArg): (WebCore::jsTestObjPrototypeFunctionMethodWithNonOptionalArgAndOptionalArg): (WebCore::jsTestObjPrototypeFunctionMethodWithNonOptionalArgAndTwoOptionalArgs): (WebCore::toJS): (WebCore::toTestObj):
  • bindings/scripts/test/JS/JSTestObj.h: Added. (WebCore::JSTestObj::classInfo): (WebCore::JSTestObj::createStructure): (WebCore::JSTestObj::impl): (WebCore::JSTestObjPrototype::classInfo): (WebCore::JSTestObjPrototype::createStructure): (WebCore::JSTestObjPrototype::JSTestObjPrototype):
  • bindings/scripts/test/TestObj.idl:
  • bindings/scripts/test/V8/V8TestObj.cpp: Renamed from WebCore/bindings/scripts/test/V8TestObj.cpp. (WebCore::): (WebCore::TestObjInternal::V8_USE): (WebCore::TestObjInternal::readOnlyIntAttrAttrGetter): (WebCore::TestObjInternal::readOnlyStringAttrAttrGetter): (WebCore::TestObjInternal::readOnlyTestObjAttrAttrGetter): (WebCore::TestObjInternal::intAttrAttrGetter): (WebCore::TestObjInternal::intAttrAttrSetter): (WebCore::TestObjInternal::stringAttrAttrGetter): (WebCore::TestObjInternal::stringAttrAttrSetter): (WebCore::TestObjInternal::testObjAttrAttrGetter): (WebCore::TestObjInternal::testObjAttrAttrSetter): (WebCore::TestObjInternal::attrWithExceptionAttrGetter): (WebCore::TestObjInternal::attrWithExceptionAttrSetter): (WebCore::TestObjInternal::attrWithSetterExceptionAttrGetter): (WebCore::TestObjInternal::attrWithSetterExceptionAttrSetter): (WebCore::TestObjInternal::attrWithGetterExceptionAttrGetter): (WebCore::TestObjInternal::attrWithGetterExceptionAttrSetter): (WebCore::TestObjInternal::voidMethodCallback): (WebCore::TestObjInternal::voidMethodWithArgsCallback): (WebCore::TestObjInternal::intMethodCallback): (WebCore::TestObjInternal::intMethodWithArgsCallback): (WebCore::TestObjInternal::objMethodCallback): (WebCore::TestObjInternal::objMethodWithArgsCallback): (WebCore::TestObjInternal::methodWithExceptionCallback): (WebCore::TestObjInternal::methodWithOptionalArgCallback): (WebCore::TestObjInternal::methodWithNonOptionalArgAndOptionalArgCallback): (WebCore::TestObjInternal::methodWithNonOptionalArgAndTwoOptionalArgsCallback): (WebCore::TestObjInternal::overloadedMethod1Callback): (WebCore::TestObjInternal::overloadedMethod2Callback): (WebCore::TestObjInternal::overloadedMethod3Callback): (WebCore::TestObjInternal::overloadedMethod4Callback): (WebCore::TestObjInternal::overloadedMethodCallback): (WebCore::ConfigureV8TestObjTemplate): (WebCore::V8TestObj::GetRawTemplate): (WebCore::V8TestObj::GetTemplate): (WebCore::V8TestObj::toNative): (WebCore::V8TestObj::HasInstance): (WebCore::V8TestObj::wrap): (WebCore::toV8): (WebCore::V8TestObj::derefObject):
  • bindings/scripts/test/V8/V8TestObj.h: Renamed from WebCore/bindings/scripts/test/V8TestObj.h.

2010-04-26 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Add testing infrastructure for JSC bindings generator
​https://bugs.webkit.org/show_bug.cgi?id=38167

Add support for testing more than one bindings. Also, converted the
script to PEP8 style.

  • Scripts/run-bindings-tests:
9:39 PM Changeset in webkit [58287] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Fix windows

9:22 PM Changeset in webkit [58286] by oliver@apple.com
  • 21 edits
    3 adds in trunk

2010-04-26 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

Need to support more efficient dispatch of lightweight builtins
​https://bugs.webkit.org/show_bug.cgi?id=38155

The problem with calling lightweight builtins is that marshalling
the call from the nitro calling convention to the system calling
convention is very expensive relative to the cost of the actually
evaluating the function. To work around this problem this patch
adds the ability to define a custom thunk for a builtin.

This allows us to use high performance custom implementations of
the common and sensible versions of simple builtins. This patch
includes a simple (use of which is currently hardcoded) thunk for
charCodeAt.

This adds a JSInterfaceJIT subclass called SpecializedThunkJIT
that has helper functions to remove (or at least reduce) the need
to have separate thunk implementations for each JSValue encoding.

  • create_hash_table: Add thunk generation callbacks to the hash tables, currently we hardcode the script to only support charCodeAt
  • jit/JITStubCall.h:
  • jit/JITStubs.cpp: (JSC::JITThunks::~JITThunks): (JSC::JITThunks::specializedThunk):
  • jit/JITStubs.h:
  • jit/SpecializedThunkJIT.h: Added. (JSC::SpecializedThunkJIT::SpecializedThunkJIT): (JSC::SpecializedThunkJIT::loadCellArgument): (JSC::SpecializedThunkJIT::loadJSStringArgument): (JSC::SpecializedThunkJIT::loadInt32Argument): (JSC::SpecializedThunkJIT::appendFailure): (JSC::SpecializedThunkJIT::returnInt32): (JSC::SpecializedThunkJIT::finalize): (JSC::SpecializedThunkJIT::argumentToVirtualRegister): (JSC::SpecializedThunkJIT::tagReturnAsInt32):
  • jit/ThunkGenerators.cpp: Added. (JSC::ThunkHelpers::stringImplDataOffset): (JSC::ThunkHelpers::jsStringLengthOffset): (JSC::ThunkHelpers::jsStringValueOffset): (JSC::charCodeAtThunkGenerator):
  • jit/ThunkGenerators.h: Added.
  • runtime/Executable.h: (JSC::NativeExecutable::NativeExecutable):
  • runtime/JSFunction.cpp: (JSC::JSFunction::JSFunction):
  • runtime/JSFunction.h:
  • runtime/JSGlobalData.h: (JSC::JSGlobalData::getThunk):
  • runtime/JSString.h: (JSC::):
  • runtime/JSValue.h: Protect some of the JSVALUE32 and JSVALUE32_64 only constants and function behind appropriate USE() guards to make it harder to use the wrong flags for the target build.
  • runtime/Lookup.cpp: (JSC::HashTable::createTable): (JSC::setUpStaticFunctionSlot):
  • runtime/Lookup.h: (JSC::HashEntry::initialize): (JSC::HashEntry::generator): (JSC::HashEntry::): Make the lookup tables use a specialized thunkGenerator if present
  • wtf/text/StringImpl.h:

2010-04-26 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

Need to support more efficient dispatch of lightweight builtins
​https://bugs.webkit.org/show_bug.cgi?id=38155

Update bindings generation to include new thunk generator field
in the property map hash tables.

  • bindings/scripts/CodeGeneratorJS.pm:
9:21 PM Changeset in webkit [58285] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2010-04-26 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Refactor results.html generation out into a new method and test it
​https://bugs.webkit.org/show_bug.cgi?id=38164

Hopefully this results in no change in functionality.

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
9:12 PM Changeset in webkit [58284] by oliver@apple.com
  • 3 edits in trunk/JavaScriptCore

Build fix

8:45 PM Changeset in webkit [58283] by oliver@apple.com
  • 4 edits
    1 add in trunk/JavaScriptCore

2010-04-26 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

Factor some basic JS interface logic out of JIT and into a superclass
​https://bugs.webkit.org/show_bug.cgi?id=38163

Create JSInterfaceJIT to hold some common low level functions for
interacting with the JS RegisterFile. Doing this separately from
​https://bugs.webkit.org/show_bug.cgi?id=38155 to ease reviewing.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • jit/JIT.h:
  • jit/JITInlineMethods.h: (JSC::JIT::emitJumpSlowCaseIfNotJSCell):
  • jit/JSInterfaceJIT.h: Added. (JSC::JSInterfaceJIT::emitLoadJSCell): (JSC::JSInterfaceJIT::emitJumpIfNotJSCell): (JSC::JSInterfaceJIT::emitLoadInt32): (JSC::JSInterfaceJIT::tagFor): (JSC::JSInterfaceJIT::payloadFor): (JSC::JSInterfaceJIT::addressFor):
8:36 PM Changeset in webkit [58282] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2010-04-26 Adam Barth <​abarth@webkit.org>

Unreviewed, rolling out r58265.
​http://trac.webkit.org/changeset/58265
​https://bugs.webkit.org/show_bug.cgi?id=38021

This change prevents me from uploading patches. It also breaks sheriff-bot.

  • Scripts/webkitpy/tool/steps/options.py:
8:35 PM Changeset in webkit [58281] by abarth@webkit.org
  • 3 edits
    3 moves
    1 add in trunk

2010-04-26 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Move bindings test directory into the scripts directory
​https://bugs.webkit.org/show_bug.cgi?id=38161

This test is really of the bindings scripts not of the bindings
themselves. By moving the files into the scripts directory, we'll be
able to test JSC as well.

  • bindings/scripts/test/TestObj.idl: Renamed from WebCore/bindings/v8/test/TestObj.idl.
  • bindings/scripts/test/V8TestObj.cpp: Renamed from WebCore/bindings/v8/test/V8TestObj.cpp. (WebCore::): (WebCore::TestObjInternal::V8_USE): (WebCore::TestObjInternal::readOnlyIntAttrAttrGetter): (WebCore::TestObjInternal::readOnlyStringAttrAttrGetter): (WebCore::TestObjInternal::readOnlyTestObjAttrAttrGetter): (WebCore::TestObjInternal::intAttrAttrGetter): (WebCore::TestObjInternal::intAttrAttrSetter): (WebCore::TestObjInternal::stringAttrAttrGetter): (WebCore::TestObjInternal::stringAttrAttrSetter): (WebCore::TestObjInternal::testObjAttrAttrGetter): (WebCore::TestObjInternal::testObjAttrAttrSetter): (WebCore::TestObjInternal::attrWithExceptionAttrGetter): (WebCore::TestObjInternal::attrWithExceptionAttrSetter): (WebCore::TestObjInternal::attrWithSetterExceptionAttrGetter): (WebCore::TestObjInternal::attrWithSetterExceptionAttrSetter): (WebCore::TestObjInternal::attrWithGetterExceptionAttrGetter): (WebCore::TestObjInternal::attrWithGetterExceptionAttrSetter): (WebCore::TestObjInternal::voidMethodCallback): (WebCore::TestObjInternal::voidMethodWithArgsCallback): (WebCore::TestObjInternal::intMethodCallback): (WebCore::TestObjInternal::intMethodWithArgsCallback): (WebCore::TestObjInternal::objMethodCallback): (WebCore::TestObjInternal::objMethodWithArgsCallback): (WebCore::TestObjInternal::methodWithExceptionCallback): (WebCore::TestObjInternal::methodWithOptionalArgCallback): (WebCore::TestObjInternal::methodWithNonOptionalArgAndOptionalArgCallback): (WebCore::TestObjInternal::methodWithNonOptionalArgAndTwoOptionalArgsCallback): (WebCore::TestObjInternal::overloadedMethod1Callback): (WebCore::TestObjInternal::overloadedMethod2Callback): (WebCore::TestObjInternal::overloadedMethod3Callback): (WebCore::TestObjInternal::overloadedMethod4Callback): (WebCore::TestObjInternal::overloadedMethodCallback): (WebCore::ConfigureV8TestObjTemplate): (WebCore::V8TestObj::GetRawTemplate): (WebCore::V8TestObj::GetTemplate): (WebCore::V8TestObj::toNative): (WebCore::V8TestObj::HasInstance): (WebCore::V8TestObj::wrap): (WebCore::toV8): (WebCore::V8TestObj::derefObject):
  • bindings/scripts/test/V8TestObj.h: Renamed from WebCore/bindings/v8/test/V8TestObj.h.

2010-04-26 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Move bindings test directory into the scripts directory
​https://bugs.webkit.org/show_bug.cgi?id=38161

Change script to point to the new location of these data files.

  • Scripts/run-bindings-tests:
8:07 PM Changeset in webkit [58280] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-04-26 Daniel Cheng <​dcheng@chromium.org>

Reviewed by Jian Li.

[chromium] Fix ChromiumDataObject::setURL to not populate its internal URL list with empty URLs.
​https://bugs.webkit.org/show_bug.cgi?id=38159

No new tests.

  • platform/chromium/ChromiumDataObject.h: (WebCore::ChromiumDataObject::setURL):
7:33 PM Changeset in webkit [58279] by eric@webkit.org
  • 10 edits
    2 adds in trunk/WebKitTools

2010-04-26 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

[chromium] new-run-webkit-tests hangs on Chromium Bots (OS X and Linux)
​https://bugs.webkit.org/show_bug.cgi?id=37987

Rolled out:
​http://trac.webkit.org/changeset/58062
​http://trac.webkit.org/changeset/58060
​http://trac.webkit.org/changeset/58059
​http://trac.webkit.org/changeset/58055
​http://trac.webkit.org/changeset/58054
and parts of:
​http://trac.webkit.org/changeset/58050

I also wrote some new comments and a tiny amount of new
code to help make ChromiumDriver.run_test easier to read.

In order to unit-test my new code, I had to change ChromiumDriver
to not automatically start itself when created. That ended up
being a lot of plumbing, but is hopefully easier to understand now.

There are no tests for the (restored) wdiff code. wdiff does not
exist on all systems, so for now we will assume it worked since
it is just old code being reverted.

  • Scripts/webkitpy/layout_tests/driver_test.py:
    • Use create_driver instead of start_driver, and be sure to call .stop()
  • Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
    • Use create_driver instead of start_driver
  • Scripts/webkitpy/layout_tests/port/base.py:
    • Added a comment to explain that diffs are binary files.
    • Various patch reverts relating to wdiff
    • Add Driver._command_wrapper to share code between WebKitDriver and ChromiumDriver.
    • Made _command_wrapper use shlex.split to get rid of the FIXME.
  • Scripts/webkitpy/layout_tests/port/base_unittest.py: Added.
    • test the new _command_wrapper
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Use _command_wrapper to get rid of a bunch of ugly code.
  • Make init stop auto-starting.
  • Rename create_driver to start_driver.
  • Added _write_command_and_read_line to make it possible to put a FIXME next to read_line() w/o having to put it in two places.
  • Moved test_shell command building into _test_shell_command and tested it.
  • Fix comments to say test_shell since ChromiumDriver is test_shell only.
  • Scripts/webkitpy/layout_tests/port/chromium_unittest.py: Added.
    • Test the new test_shell_command method.
  • Scripts/webkitpy/layout_tests/port/dryrun.py:
    • Rename create_driver to start_driver.
  • Scripts/webkitpy/layout_tests/port/test.py:
    • Rename create_driver to start_driver.
  • Scripts/webkitpy/layout_tests/port/webkit.py:
    • Rename create_driver to start_driver.
    • Treat output as binary arrays.
  • Scripts/webkitpy/layout_tests/test_types/test_type_base.py:
    • Treat diff files as binary.
  • Scripts/webkitpy/layout_tests/test_types/text_diff.py:
    • Treat diff files as binary.
7:22 PM Changeset in webkit [58278] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2010-04-26 Adam Barth <​abarth@webkit.org>

Unreviewed. Mark run-bindings-tests executable so we can execute it.

  • Scripts/run-bindings-tests:
7:12 PM Changeset in webkit [58277] by yuzo@google.com
  • 1 edit
    1 move
    1 add
    1 delete in trunk/LayoutTests

2010-04-26 Yuzo Fujishima <​yuzo@google.com>

Unreviewed, Chromium test expectation change.

To keep up with the change that turned on HTML5 DB AUTO_VACUUM.
​https://bugs.webkit.org/show_bug.cgi?id=36251

  • platform/chromium-mac/storage/quota-tracking-expected.txt: Removed.
  • platform/chromium-win/storage/quota-tracking-expected.txt: Removed.
  • platform/chromium/storage/quota-tracking-expected.txt: Added.
6:32 PM Changeset in webkit [58276] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-04-26 Daniel Cheng <​dcheng@chromium.org>

Reviewed by Jian Li.

[Chromium] Don't make file paths available in text/uri-list when dragging files.
​https://bugs.webkit.org/show_bug.cgi?id=25882

Test will be checked in with WebCore/platform/mac fix.

  • platform/chromium/ClipboardChromium.cpp: (WebCore::ClipboardChromium::getData): (WebCore::ClipboardChromium::types):
6:26 PM Changeset in webkit [58275] by tony@chromium.org
  • 4 edits in trunk/WebKitTools

2010-04-26 Tony Chang <​tony@chromium.org>

Reviewed by David Levin.

[chromium] build-webkit --chromium should build release by default
​https://bugs.webkit.org/show_bug.cgi?id=38028

  • Scripts/build-dumprendertree: Also should build release by default
  • Scripts/build-webkit: Make sure to pass command line args through
  • Scripts/webkitdirs.pm: Build the right configuration
5:37 PM Changeset in webkit [58274] by dumi@chromium.org
  • 3 edits in trunk/LayoutTests

Unreviewed, fixing the build breakage introduced by r58269.

Turning on AUTO_VACUUM has increased the amount of space used by databases.

  • storage/quota-tracking-expected.txt:
  • storage/quota-tracking.html:
5:09 PM Changeset in webkit [58273] by mjs@apple.com
  • 9 edits in trunk/WebCore

2010-04-26 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin Adler.

REGRESSION (r57292): 1.5% page load speed regression from visited link information leak fix
​https://bugs.webkit.org/show_bug.cgi?id=38131

I did a number of separate optimizations which speed up style
resolution enough to more than make up for the regression. This
measures as a total PLT speedup of somewhere between 1.5% and
3.7%.


Optimizations done:

  • Cache determineLinkState results, to avoid the need to repeatedly compute the visited link hash for the same element. This directly addresses much of the slowdown, since all elements get their style computed twice now.
  • Added a fast way to get the length of a CSSMutableStyleDeclaration, and use in CSSStyleSelector::matchRulesForList, since it was hot there.
  • Hoist some loop invariant code that's not detected by the compiler out of the main loop in matchRulesForList
  • inline CSSStyleSelector::initElement and locateSharedStyle, since there is only one call site in each case
  • Inline the common non-line fast case of determineLinkState, and split the rest into out-of-line determineLinkStateSlowCase.
  • Added inline versions of the functions called by visitedLinkHash (the version called by determineLinkState).
  • css/CSSMutableStyleDeclaration.cpp: (WebCore::CSSMutableStyleDeclaration::length): Implemented in terms of new inline nonvirtual mutableLength().
  • css/CSSMutableStyleDeclaration.h: (WebCore::CSSMutableStyleDeclaration::mutableLength): Added new nonvirtual inline way to get the length if you know you have a mutable style decl.
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::init): Clear cached link state. (WebCore::CSSStyleSelector::matchRulesForList): hoist some code out of the main loop and get style decl length more efficiently. (WebCore::CSSStyleSelector::initElement): inline (only one call site) (WebCore::CSSStyleSelector::SelectorChecker::determineLinkState): Inline fast case, call slow case. (WebCore::CSSStyleSelector::SelectorChecker::determineLinkStateSlowCase): Split most of the above function into this slow case helper. (WebCore::CSSStyleSelector::canShareStyleWithElement): Use the cache-enabled way to get the current link state. (WebCore::CSSStyleSelector::locateSharedStyle): inline (WebCore::CSSStyleSelector::styleForElement): Use the cache-enabled way to get the current link state.
  • css/CSSStyleSelector.h: (WebCore::CSSStyleSelector::currentElementLinkState): inline way to get link state for the current element; manages the cache
  • platform/LinkHash.cpp: (WebCore::visitedLinkHashInline): inline version of below function (WebCore::visitedLinkHash): call the inline version (WebCore::visitedURLInline): inline version of below function (WebCore::visitedURL): call the inline version (WebCore::visitedURL): call inline versions of above two functions
4:30 PM Changeset in webkit [58272] by weinig@apple.com
  • 2 edits in trunk/WebCore

Remove last use of WEB_THREAD.

Reviewed by Alexey Proskuryakov.

  • platform/mac/WebCoreObjCExtras.mm:

(WebCoreObjCScheduleDeallocateOnMainThread): This can always use
isMainThread().

4:10 PM Changeset in webkit [58271] by crogers@google.com
  • 2 adds in branches/audio/WebCore/audio

initial commit test

3:58 PM Changeset in webkit [58270] by dumi@chromium.org
  • 7 edits in trunk/WebCore

Turn on AUTO_VACUUM = INCREMENTAL for all HTML5 databases, and
vacuum all databases when the number of free pages is at least 10%
of the number of total pages. Also, adding a guard against a bug
that was fixed in SQLite only starting with version 3.6.16.

Reviewed by Dimitri Glazkov.

​https://bugs.webkit.org/show_bug.cgi?id=36251

  • platform/sql/SQLiteDatabase.cpp:

(WebCore::SQLiteDatabase::totalSize):
(WebCore::SQLiteDatabase::runIncrementalVacuumCommand):
(WebCore::SQLiteDatabase::turnOnIncrementalAutoVacuum):

  • platform/sql/SQLiteDatabase.h:
  • platform/sql/SQLiteStatement.cpp:

(WebCore::SQLiteStatement::prepare):

  • storage/Database.cpp:

(WebCore::Database::performOpenAndVerify):
(WebCore::Database::incrementalVacuumIfNeeded):

  • storage/Database.h:
  • storage/SQLTransaction.cpp:

(WebCore::SQLTransaction::postflightAndCommit):

3:45 PM Changeset in webkit [58269] by crogers@google.com
  • 1 copy in branches/audio

creating private audio branch to implement new audio features with javascript API

3:45 PM Changeset in webkit [58268] by weinig@apple.com
  • 2 edits in trunk/JavaScriptCore

Fix Tiger build.

  • wtf/mac/MainThreadMac.mm:

(WTF::isMainThread):

3:44 PM Changeset in webkit [58267] by ggaren@apple.com
  • 9 edits in trunk

<rdar://problem/7766413>

Reviewed by Sam Weinig.

Fixed a crash seen when using the JavaScriptCore API with WebKit.

No layout test because DumpRenderTree doesn't use the JavaScriptCore API
in this way.

JavaScriptCore:

  • interpreter/RegisterFile.cpp:

(JSC::RegisterFile::setGlobalObject):
(JSC::RegisterFile::clearGlobalObject):
(JSC::RegisterFile::globalObject):

  • interpreter/RegisterFile.h:

(JSC::RegisterFile::RegisterFile): Use WeakGCPtr for managing m_globalObject,
since it's a weak pointer. (We never noticed this error before because,
in WebKit, global objects always have a self-reference in a global variable,
so marking the register file's global variables would keep m_globalObject
alive. In the JavaScriptCore API, you can allocate a global object with
no self-reference.)

  • runtime/JSActivation.h: Removed unneeded #include.
  • runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::~JSGlobalObject): Don't use ==, since a weak
pointer is 0 when the object it points to runs its destructor.

  • runtime/WeakGCPtr.h:

(JSC::WeakGCPtr::clear): Changed to return a bool indicating whether the
clear actually happened.
(JSC::WeakGCPtr::assign): Changed to forbid assignment of 0 as a shorthand
for calling clear(). A client should never clear by assigning 0, since
clear() should be conditional on whether the object doing the clearing
is still pointed to by the weak pointer. (Otherwise, a zombie object might
clear a pointer to a new, valid object.)

WebCore:

  • bindings/js/JSEventListener.cpp:

(WebCore::JSEventListener::JSEventListener): Don't assign 0 to a WeakGCPtr.
JavaScriptCore no longer allows this.

  • bindings/js/ScriptWrappable.h:

(WebCore::ScriptWrappable::setWrapper): No need to initialize a WeakGCPtr
to 0, or ASSERT a non-0 value before assigning to a WeakGCPtr -- both are
default behaviors.

3:38 PM Changeset in webkit [58266] by weinig@apple.com
  • 56 edits in trunk

Fix for ​https://bugs.webkit.org/show_bug.cgi?id=38097
Disentangle initializing the main thread from initializing threading

Reviewed by Maciej Stachowiak.

JavaScriptCore:

This is the first step in getting rid of the WEB_THREAD #define and
paving the way to allowing mac both WebKit and WebKit2 to use the same
WebCore.

  • JavaScriptCore.exp: Export initializeMainThread and initializeMainThreadToProcessMainThread.
  • JavaScriptCore.xcodeproj/project.pbxproj: Sort files.
  • wtf/MainThread.cpp:

(WTF::mainThreadInitializationMutex):
(WTF::mainThreadFunctionQueueMutex):
(WTF::initializeMainThread):
(WTF::deprecatedInitializeMainThread):

  • wtf/MainThread.h:

Added Mac only initializeMainThreadToProcessMainThread which sets up callOnMainThread
and isMainThead calls to assume that WebCore's main thread is also the
process's main thread. Since it is possible that both initializeMainThread
and initializeMainThreadToProcessMainThread could be called, the first one called
wins (either will work).

  • wtf/Threading.h: Moved to top of file.
  • wtf/ThreadingPthreads.cpp:

(WTF::initializeThreading): Remove call to initializeMainThread.

  • wtf/ThreadingWin.cpp:

(WTF::initializeThreading): Ditto.

  • wtf/gtk/ThreadingGtk.cpp:

(WTF::initializeThreading): Ditto.

  • wtf/mac/MainThreadMac.mm:

(WTF::initializeMainThreadPlatform):
(WTF::initializeMainThreadToProcessMainThread):
(WTF::scheduleDispatchFunctionsOnMainThread):
(WTF::isMainThread):
Add runtime switch between the old behavior of using the system's main
thread and a stored pointer to the main thread. Tiger always uses the
system's main thread.

  • wtf/qt/ThreadingQt.cpp:

(WTF::initializeThreading): Remove call to initializeMainThread.

  • wtf/win/MainThreadWin.cpp:

(WTF::initializeMainThreadPlatform): Add call to initializeCurrentThreadInternal
removed from initializeThreading.

WebCore:

  • WebCore.xcodeproj/project.pbxproj: Sort files.
  • accessibility/mac/AccessibilityObjectWrapper.mm: Remove unnessary

+initialize method. The AccessibilityObjectWrapper has no data members
that need to be derefed, so it does not need to be finalized on the main
thread.

  • bindings/js/ScriptController.cpp:

(WebCore::ScriptController::initializeThreading): Add call to initializeMainThread.

  • bindings/objc/WebScriptObject.mm:

(+[WebScriptObject initialize]): Add call to initializeMainThreadToProcessMainThread.

  • bindings/v8/ScriptController.cpp:

(WebCore::ScriptController::initializeThreading): Add call to initializeMainThread.

  • platform/mac/SharedBufferMac.mm:

(+[WebCoreSharedBufferData initialize]): Add call to initializeMainThreadToProcessMainThread.

WebKit/chromium:

  • src/WebKit.cpp:

(WebKit::initialize): Add call to initializeMainThread.

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::WebViewImpl): Ditto.

WebKit/gtk:

  • webkit/webkitprivate.cpp:

(webkit_init): Add call to initializeMainThread.

WebKit/mac:

Calls initializeMainThreadToProcessMainThread since there is no way to ensure
that the initialize method will be called on the main thread.

  • Carbon/CarbonWindowAdapter.mm:

(+[CarbonWindowAdapter initialize]): Add call to initializeMainThreadToProcessMainThread.

  • History/WebBackForwardList.mm:

(+[WebBackForwardList initialize]): Ditto.

  • History/WebHistoryItem.mm:

(+[WebHistoryItem initialize]): Ditto.

  • Misc/WebElementDictionary.mm:

(+[WebElementDictionary initialize]): Ditto.

  • Misc/WebIconDatabase.mm:

(+[WebIconDatabase initialize]): Ditto.

  • Plugins/Hosted/WebHostedNetscapePluginView.mm:

(+[WebHostedNetscapePluginView initialize]): Ditto.

  • Plugins/WebBaseNetscapePluginView.mm:

(+[WebBaseNetscapePluginView initialize]): Ditto.

  • Plugins/WebBasePluginPackage.mm:

(+[WebBasePluginPackage initialize]): Ditto.

  • Plugins/WebNetscapePluginView.mm:

(+[WebNetscapePluginView initialize]): Ditto.

  • WebCoreSupport/WebEditorClient.mm:

(+[WebEditCommand initialize]): Ditto.

  • WebCoreSupport/WebFrameLoaderClient.mm:

(+[WebFramePolicyListener initialize]): Ditto.

  • WebView/WebArchive.mm:

(+[WebArchivePrivate initialize]): Ditto.

  • WebView/WebDataSource.mm:

(+[WebDataSourcePrivate initialize]): Ditto.

  • WebView/WebHTMLView.mm:

(+[WebHTMLViewPrivate initialize]): Ditto.
(+[WebHTMLView initialize]): Ditto.

  • WebView/WebResource.mm:

(+[WebResourcePrivate initialize]): Ditto.

  • WebView/WebTextIterator.mm:

(+[WebTextIteratorPrivate initialize]): Ditto.

  • WebView/WebView.mm:
  • WebView/WebViewData.mm: Ditto.

(+[WebViewPrivate initialize]): Ditto.

WebKit/qt:

  • Api/qwebpage.cpp:

(QWebPagePrivate::QWebPagePrivate): Add call to initializeMainThread.

WebKit/win:

  • WebKitClassFactory.cpp:

(WebKitClassFactory::WebKitClassFactory): Add call to initializeMainThread.

  • WebView.cpp:

(WebView::WebView): Ditto.

WebKit/wx:

  • WebView.cpp:

(wxWebView::Create): Add call to initializeMainThread.

WebKit2:

  • UIProcess/Launcher/mac/WebProcessLauncher.mm:

(WebKit::webThreadBody): Add call to initializeMainThread.

  • UIProcess/Launcher/win/WebProcessLauncher.cpp:

(WebKit::webThreadBody): Ditto.

  • WebProcess/Launching/mac/WebProcessMain.mm:

(main): Ditto.

  • WebProcess/win/WebProcessMain.cpp:

(WebKit::WebProcessMain): Ditto.

2:52 PM Changeset in webkit [58265] by ojan@chromium.org
  • 2 edits in trunk/WebKitTools

2010-04-26 Ojan Vafai <​ojan@chromium.org>

Reviewed by Eric Seidel.

complete rietveld bugzilla integration
​https://bugs.webkit.org/show_bug.cgi?id=38021

Makes --fancy-review the default. All this means is that the patch will
be uploaded to rietveld in addition to bugs.webkit.org.

  • Scripts/webkitpy/tool/steps/options.py:
2:43 PM Changeset in webkit [58264] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-04-26 Yongjun Zhang <​yongjun_zhang@apple.com>

Reviewed by Alexey Proskuryakov.

​https://bugs.webkit.org/show_bug.cgi?id=38128
Don't add empty credential to CredentialStorage.

An empty credential is currently regarded as a invalid login because it means a missing value in
protectionSpaceToCredentialMap. This change makes it consistent with current WebCore's behavior
by ignoring empty credentials.

No new tests added since this only affects credential entered by user from credential dialog,
which is not testable in DRT.

  • platform/network/mac/ResourceHandleMac.mm: (WebCore::ResourceHandle::receivedCredential):
2:01 PM Changeset in webkit [58263] by Chris Jerdonek
  • 3 edits
    1 move in trunk/WebKitTools

For check-webkit-style, renamed the style/processors/ directory
to style/checkers/.

Reviewed by Shinichiro Hamaji.

​https://bugs.webkit.org/show_bug.cgi?id=38122

  • Scripts/webkitpy/style/checker.py:
    • Updated import statements.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Updated import statements.
  • Scripts/webkitpy/style/checkers: Copied from WebKitTools/Scripts/webkitpy/style/processors.
  • Scripts/webkitpy/style/processors: Removed.
  • Scripts/webkitpy/style/processors/init.py: Removed.
  • Scripts/webkitpy/style/processors/common.py: Removed.
  • Scripts/webkitpy/style/processors/common_unittest.py: Removed.
  • Scripts/webkitpy/style/processors/cpp.py: Removed.
  • Scripts/webkitpy/style/processors/cpp_unittest.py: Removed.
  • Scripts/webkitpy/style/processors/python.py: Removed.
  • Scripts/webkitpy/style/processors/python_unittest.py: Removed.
  • Scripts/webkitpy/style/processors/python_unittest_input.py: Removed.
  • Scripts/webkitpy/style/processors/text.py: Removed.
  • Scripts/webkitpy/style/processors/text_unittest.py: Removed.
12:26 PM Changeset in webkit [58262] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-04-26 Laszlo Gombos <Laszlo Gombos>

Reviewed by Simon Hausmann.

[Qt] Clean-up WebCore.pro after now that JavaScriptCore is built separately
​https://bugs.webkit.org/show_bug.cgi?id=38123

HAVE_STDINT_H, and HAVE_PTHREAD_NP_H no longer needs to be defined
for QtWebKit as JavaScriptCore is built separately. These defines are only
used by JavaScriptCore and for JavaScriptCore these are already
properly defined in JavaScriptCore/config.h.

No new tests, no new functionality.

  • WebCore.pro:
10:48 AM Changeset in webkit [58261] by ojan@chromium.org
  • 26 edits in trunk/WebKitTools

2010-04-06 Ojan Vafai <​ojan@chromium.org>

Reviewed by Adam Barth.

Include git commits in the diff for webkit-patch upload/land.
​https://bugs.webkit.org/show_bug.cgi?id=36394

Adds --squash, --no-squash and --git-commit.

--git-commit will use a specific local commit for land/upload.
If a commit-range is specified, then that range is treated as
a single squashed commit.

--squash will squash all local changes including working copy changes
into a single patch.

--no-squash is the legacy behavior (upload only considers the working copy,
land commits the working copy and then each local commit separately to SVN)

If neither is specified, then an informative error is raised if there is
more than one local commit or when there are local commit(s) and working
copy changes.

If the webkit-patch.squash git config parameter is set, then
that will be respected instead of raising an error.

  • Scripts/check-webkit-style:
  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/api_unittest.py:
  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/common/checkout/scm_unittest.py:
  • Scripts/webkitpy/style/optparser.py: --git-since is removed and --git-commit no longer implies commit_id.. Instead, it treats that individual commit, but also supports commit ranges (e.g. commit_id..) as arguments.
  • Scripts/webkitpy/style/optparser_unittest.py:
  • Scripts/webkitpy/style_references.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/abstractstep.py:
  • Scripts/webkitpy/tool/steps/applypatchwithlocalcommit.py:
  • Scripts/webkitpy/tool/steps/checkstyle.py:
  • Scripts/webkitpy/tool/steps/commit.py:
  • Scripts/webkitpy/tool/steps/options.py:
  • Scripts/webkitpy/tool/steps/postdiff.py:
  • Scripts/webkitpy/tool/steps/preparechangelog.py:
  • Scripts/webkitpy/tool/steps/preparechangelogforrevert.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py:
  • Scripts/webkitpy/tool/steps/validatereviewer.py:
10:03 AM Changeset in webkit [58260] by evan@chromium.org
  • 1 edit
    1 add in trunk/WebCore

2010-04-26 Roman Gershman <​romange@google.com>

Reviewed by Adam Barth.

[Chromium] Font size in suggestions popup menu should be correlated with the font size of its text field.
The following manual test checks for desired behavior.

​https://bugs.webkit.org/show_bug.cgi?id=37977

  • manual-tests/chromium/suggestions-popup-font-change.html: Added.
9:56 AM Changeset in webkit [58259] by evan@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-04-26 Roman Gershman <​romange@google.com>

Reviewed by Adam Barth.

[Chromium] Font size in suggestions popup menu should be correlated with the font size of its text field.

​https://bugs.webkit.org/show_bug.cgi?id=37977

  • src/SuggestionsPopupMenuClient.cpp: (WebKit::SuggestionsPopupMenuClient::initialize):
9:21 AM Changeset in webkit [58258] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-04-26 Luiz Agostini <​luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

View modes names in CSSValueKeywords.in
​https://bugs.webkit.org/show_bug.cgi?id=38125

View mode names has changed in previous patch but CSSValueKeywords.in file was
not updated.

  • css/CSSValueKeywords.in:
8:16 AM QtWebKitBackportingFixes edited by Simon Hausmann
(diff)
8:10 AM Changeset in webkit [58257] by Simon Hausmann
  • 2 edits in trunk/WebCore

[Qt] Fix compilation with RVCT 4
​https://bugs.webkit.org/show_bug.cgi?id=37727

Reviewed by Kenneth Rohde Christiansen.

Swap extern and declspec to fix visibility of symbol imported from QtGui.

  • page/qt/EventHandlerQt.cpp:
7:58 AM Changeset in webkit [58256] by eric@webkit.org
  • 8 edits in trunk/WebCore

2010-04-26 Ruben Van Boxem <​vanboxem.ruben@gmail.com>

Reviewed by Adam Roben.

Fixes for Win64 compilation under gcc (mingw-w64)

  • WebCore/bridge/npapi.h: for win64 compatibility, mirroring mozilla-central, see Mozilla bug 560298
  • WebCore/platform/Arena.h: uword is used to cast from pointers here. unsigned long is 32-bit on Windows (but 64-bit on mac), and too small to hold a pointer. uintptr_t is 32-bit on 32-bit systems (mac, linux and windows) and 64-bit on all 64-bit systems
  • WebCore/platform/graphics/transforms/TransformationMatrix.h: let mingw-w64/w32 use MSVC codepath
  • WebCore/platform/text/TextStream.cpp: let mingw-w64 use MSVC codepath
  • WebCore/platform/text/TextStream.h: let mingw-w64 use MSVC codepath
  • WebCore/plugins/PluginView.cpp: fix pointer casts on WIN64 and let mingw-w64 use MSVC codepath
  • WebCore/plugins/win/PluginViewWin.cpp: fix pointer casts on WIN64
7:54 AM Changeset in webkit [58255] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

[Qt] Fix the include header <qstring.h> -> <QtCore/qstring.h>

Patch by Thiago Macieira <​thiago.macieira@nokia.com> on 2010-04-26
Reviewed by Simon Hausmann.

The module/header.h style inclusion removes the need to have -I$QTDIR/include/depending-module
in the include search path for the application.

  • Api/qwebkitversion.h:
7:37 AM QtWebKitBackportingFixes edited by Simon Hausmann
(diff)
7:33 AM Changeset in webkit [58254] by Simon Hausmann
  • 2 edits in trunk/WebCore

[Qt] HTTP pipelining efficiency increase
​https://bugs.webkit.org/show_bug.cgi?id=38062

Patch by Markus Goetz <​Markus.Goetz@nokia.com> on 2010-04-26
Reviewed by Simon Hausmann.

Increase number of network requests that are fed into
QNetworkAccessManager.

  • platform/network/qt/ResourceRequestQt.cpp:

(WebCore::initializeMaximumHTTPConnectionCountPerHost):

6:56 AM Changeset in webkit [58253] by pfeldman@chromium.org
  • 9 edits
    5 deletes in trunk/WebKit/chromium

2010-04-26 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium Dev Tools: brush up images.

​https://bugs.webkit.org/show_bug.cgi?id=38124

  • WebKit.gypi:
  • src/js/DevTools.js: (WebInspector.setToolbarColors):
  • src/js/Images/segmentChromium.png:
  • src/js/Images/segmentChromium2.png: Removed.
  • src/js/Images/segmentHoverChromium2.png: Removed.
  • src/js/Images/segmentSelectedChromium2.png: Removed.
  • src/js/Images/statusbarBackgroundChromium.png:
  • src/js/Images/statusbarBackgroundChromium2.png: Removed.
  • src/js/Images/statusbarBottomBackgroundChromium.png:
  • src/js/Images/statusbarButtonsChromium.png:
  • src/js/Images/statusbarMenuButtonChromium.png:
  • src/js/Images/statusbarMenuButtonChromium2.png: Removed.
  • src/js/devTools.css: (.section > .header): (.console-group-messages .section > .header): (#resources-filter): (.crumbs .crumb): (.crumbs .crumb.end): (.crumbs .crumb.selected): (.crumbs .crumb.selected:hover): (.crumbs .crumb.selected.end, .crumbs .crumb.selected.end:hover): (.crumbs .crumb:hover): (.crumbs .crumb.dimmed:hover): (.crumbs .crumb.end:hover): (body.drawer-visible #main-status-bar): (.status-bar): (button.status-bar-item): (select.status-bar-item:active): (#drawer): (select.status-bar-item):
6:37 AM Changeset in webkit [58252] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-04-26 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: undefined shows up on inspector.
​https://bugs.webkit.org/show_bug.cgi?id=38120

  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.hide):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype.set visible):
6:31 AM QtWebKitRelease20 edited by Simon Hausmann
(diff)
6:01 AM Changeset in webkit [58251] by Chris Jerdonek
  • 3 edits in trunk/WebKitTools

2010-04-26 Chris Jerdonek <Chris Jerdonek>

Reviewed by Shinichiro Hamaji.

Deleted the StyleChecker-related classes that are no longer
being used.

​https://bugs.webkit.org/show_bug.cgi?id=38118

  • Scripts/webkitpy/style/checker.py:
    • Deleted the DeprecatedStyleChecker class.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Deleted the StyleCheckerTest, StyleCheckerCheckFileBase, StyleCheckerCheckFileTest, and StyleCheckerCheckPathsTest classes.
5:47 AM Changeset in webkit [58250] by eric@webkit.org
  • 4 edits in trunk

2010-04-26 Bruno Schmidt <​bruno.schmidt@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Exposing an QVariantMap containing QObjectStar to Javascript
causes Segmentation Fault
​https://bugs.webkit.org/show_bug.cgi?id=34729

If an QVariantMap containing QObjectStar is added to the to QtWebkit
Javascript, it's use causes Segmentation Fault.
It happens because, in the case QMetaType::QVariantMap, the "root"
object that is inside of a PassRefPtr is passed recursively inside a
loop to recover the content of the map, but the PassRefPtr semantics
prohibit its use inside a loop, so the "root" object mus be passed
using the method "PassRefPtr::get" in order to keep the current
reference.

  • bridge/qt/qt_runtime.cpp: (JSC::Bindings::convertValueToQVariant): (JSC::Bindings::convertQVariantToValue): change to root.get()

2010-04-26 Bruno Schmidt <​bruno.schmidt@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Exposing an QVariantMap containing QObjectStar to Javascript
causes Segmentation Fault
​https://bugs.webkit.org/show_bug.cgi?id=34729

If an QVariantMap containing QObjectStar is added to the to QtWebkit
Javascript, it's use causes Segmentation Fault.
It happens because, in the case QMetaType::QVariantMap, the "root"
object that is inside of a PassRefPtr is passed recursively inside a
loop to recover the content of the map, but the PassRefPtr semantics
prohibit its use inside a loop, so the "root" object mus be passed
using the method "PassRefPtr::get" in order to keep the current
reference.

  • tests/qwebframe/tst_qwebframe.cpp: (MyQObject::MyQObject): new property variantMapProperty (MyQObject::variantMapProperty): read variantMapProperty (MyQObject::setVariantMapProperty): write variantMapProperty
5:28 AM Changeset in webkit [58249] by Chris Jerdonek
  • 6 edits in trunk/WebKitTools

2010-04-26 Chris Jerdonek <Chris Jerdonek>

Reviewed by Shinichiro Hamaji.

Changed the StyleChecker class to use the new TextFileReader class.

​https://bugs.webkit.org/show_bug.cgi?id=37066

This revision separates the code responsible for reading and iterating
over text files from the rest of check-webkit-style.

  • Scripts/check-webkit-style:
    • Changed the script to use the new StyleProcessor and TextFileReader classes instead of the StyleChecker class.
  • Scripts/webkitpy/style/checker.py:
    • Added a FIXME to rename many of uses of the word "processor" to "checker". We did this to clarify the difference between ProcessorBase instances passed to the TextFileReader and classes that process and check lines for style.
    • Added a FIXME to remove FileType.NONE as a possible return value of ProcessorDispatcher._file_type(). This will better consolidate the logic of which files should be skipped.
    • Added a FIXME to delete the StyleChecker class.
    • Added the StyleProcessor class which implements ProcessorBase. This class is responsible for processing lines to check style (but not for reading files). For each file, this class creates creates both a carriage-return checker and a format-specific style checker (e.g. one of C++, Python, etc).
    • Updated the PatchChecker class to use a TextFileReader instance instead of a StyleChecker.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Added the StyleProcessor_EndToEndTest class to do "end-to-end" tests of the StyleProcessor class.
    • Added the StyleProcessor_CodeCoverageTest to test the StyleProcessor class with more complete code coverage. Unlike the StyleProcessor_EndToEndTest class, this class makes heavy use of mock parameters.
    • Added FIXME's to delete the unit test classes that are no longer needed.
    • Updated the PatchCheckerTest class to use a MockTextFileReader instead of a MockStyleChecker.
  • Scripts/webkitpy/style/filereader.py:
  • Scripts/webkitpy/style/filereader_unittest.py:
    • In the TextFileReaderTest class:
      • Moved the test_process_fileshould_not_process() method.
      • Added a test_process_filefile_stdin() method to test the file path "-".
5:28 AM Changeset in webkit [58248] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-04-26 Jedrzej Nowacki <​jedrzej.nowacki@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Fix a memory leak in the QScriptString.

QScriptStringPrivate's constructor shouldn't call JSStringRetain as
QScriptConverter implicitly pass a JSStringRef ownership.

[Qt] QScriptStringPrivate c'tor problem
​https://bugs.webkit.org/show_bug.cgi?id=38110

  • qt/api/qscriptstring_p.h: (QScriptStringPrivate::QScriptStringPrivate):
4:36 AM Changeset in webkit [58247] by pfeldman@chromium.org
  • 3 edits in trunk

2010-04-26 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Render clip in summary bar using "arc", not "quadraticCurveTo".

​https://bugs.webkit.org/show_bug.cgi?id=38107

  • inspector/front-end/SummaryBar.js: (WebInspector.SummaryBar.prototype._drawSummaryGraph.drawPill): (WebInspector.SummaryBar.prototype._drawSummaryGraph):
4:32 AM Changeset in webkit [58246] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-04-26 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Fix compilation against Qt 4.7

Some QtMultimedia things have been moved into QtMediaServices
​https://bugs.webkit.org/show_bug.cgi?id=38111

  • WebCore.pro:
  • platform/graphics/qt/MediaPlayerPrivateQt.cpp: (WebCore::MediaPlayerPrivate::supportsType): (WebCore::MediaPlayerPrivate::totalBytes):
4:23 AM Changeset in webkit [58245] by eric@webkit.org
  • 2 edits
    3 adds in trunk/LayoutTests

2010-04-26 Yael Aharon <​yael.aharon@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Add expected results after r58228
​https://bugs.webkit.org/show_bug.cgi?id=38098

  • platform/qt/Skipped:
  • platform/qt/fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element-expected.checksum: Added.
  • platform/qt/fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element-expected.png: Added.
  • platform/qt/fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element-expected.txt: Added.
4:01 AM Changeset in webkit [58244] by jorlow@chromium.org
  • 2 edits in trunk/LayoutTests

c

3:25 AM QtWebKitBugs edited by jocelyn.turcotte@nokia.com
(diff)
3:23 AM QtWebKitBugs edited by jocelyn.turcotte@nokia.com
Triaging should now use the Triaged keyword instead of the assignee (diff)
3:07 AM QtWebKitJournal edited by Simon Hausmann
(diff)
3:05 AM Changeset in webkit [58243] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-04-26 Zoltan Herczeg <​zherczeg@webkit.org>

Reviewed by Dirk Schulze.

Call setStrokeStyle in applyStrokeStyleToContext
​https://bugs.webkit.org/show_bug.cgi?id=37849

Not all platforms set the stroke style to solid if they
get an empty line dash array. Some just ignore the operation.

  • rendering/SVGRenderSupport.cpp: (WebCore::applyStrokeStyleToContext):
2:35 AM Changeset in webkit [58242] by loislo@chromium.org
  • 5 edits in trunk/WebCore

2010-04-23 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInspector: Aggregated stats of time spent in children records should be visible
in the popup panel of parent record. Crash in pushGCEventRecord also was fixed.
​https://bugs.webkit.org/show_bug.cgi?id=37820

  • English.lproj/localizedStrings.js:
  • inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::pushGCEventRecords):
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelinePanel.prototype._createRootRecord): (WebInspector.TimelinePanel.prototype._showPopover): (WebInspector.TimelineCalculator.prototype.computeBarGraphPercentages): (WebInspector.TimelineCalculator.prototype.computeBarGraphWindowPosition): (WebInspector.TimelineRecordGraphRow): (WebInspector.TimelineRecordGraphRow.prototype.update): (WebInspector.TimelinePanel.FormattedRecord): (WebInspector.TimelinePanel.FormattedRecord.prototype._generateAggregatedInfo): (WebInspector.TimelinePanel.FormattedRecord.prototype._generatePopupContent): (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails): (WebInspector.TimelinePanel.FormattedRecord.prototype._calculateAggregatedStats):
  • inspector/front-end/inspector.css: (.timeline-graph-bar.with-children): (.timeline-graph-bar.cpu): (.timeline-aggregated-category): (.timeline-loading): (.timeline-scripting): (.timeline-rendering): (.popover .timeline-aggregated-category.timeline-loading): (.timeline-details-title):
1:23 AM Changeset in webkit [58241] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

2010-04-26 Tony Chang <​tony@chromium.org>

Not reviewed, removing duplicate expectations.

[chromium] Check and rebaseline SVG test expectations for Chromium
​https://bugs.webkit.org/show_bug.cgi?id=38106

  • platform/chromium/test_expectations.txt:
1:23 AM Changeset in webkit [58240] by tkent@chromium.org
  • 3 edits in trunk/WebKit/chromium

Unreviewed. Roll out r58237 and r58238 because of Chromium/Linux build break.

1:21 AM QtWebKitTableOfFeatures20 edited by Henry Haverinen
(diff)
1:21 AM QtWebKitTableOfFeatures20 edited by Henry Haverinen
(diff)
1:19 AM QtWebKitTableOfFeatures20 edited by Henry Haverinen
(diff)
1:11 AM QtWebKitTableOfFeatures20 edited by Henry Haverinen
(diff)
1:06 AM QtWebKitTableOfFeatures20 edited by Henry Haverinen
(diff)
1:02 AM Changeset in webkit [58239] by yuzo@google.com
  • 2 edits in trunk/LayoutTests

2010-04-26 Yuzo Fujishima <​yuzo@google.com>

Unreviewed, Chromium test expectation change.

Change SVG test expectations for Chromium.
​https://bugs.webkit.org/show_bug.cgi?id=38106

  • platform/chromium/test_expectations.txt:
1:01 AM QtWebKitTableOfFeatures20 edited by Henry Haverinen
(diff)
12:56 AM Changeset in webkit [58238] by tkent@chromium.org
  • 1 edit in trunk/WebKit/chromium/DEPS

Unreviewd. Chromium build fix for r58237.

  • DEPS: Add third_party/mesa
12:33 AM Changeset in webkit [58237] by tkent@chromium.org
  • 3 edits in trunk/WebKit/chromium

2010-04-26 Kent Tamura <​tkent@chromium.org>

Reviewed by Shinichiro Hamaji.

[DRT/Chromium] Enable Windows project generation
​https://bugs.webkit.org/show_bug.cgi?id=38103

  • Bump the Chromium revision to r45515 to have GYP r818, which fixes a Windows project generation bug.
  • Avoid platform.system(), which causes a crash with python.exe in depot_tools. Use sys.platform instead.
  • Process DumpRenderTree.gyp on all platforms.
  • DEPS:
  • gyp_webkit:
12:07 AM Changeset in webkit [58236] by tkent@chromium.org
  • 10 edits
    4 adds in trunk

2010-04-25 Kent Tamura <​tkent@chromium.org>

Reviewed by Darin Adler.

WebKit crashes with deeply nested divs.
​https://bugs.webkit.org/show_bug.cgi?id=18282

Add a test for deeply nested <span> elements like block-nesting-cap.html.

  • fast/parser/element-nesting-cap-expected.txt: Added.
  • fast/parser/element-nesting-cap.html: Added.
  • fast/parser/script-tests/element-nesting-cap.js: Added.

2010-04-25 Kent Tamura <​tkent@chromium.org>

Reviewed by Darin Adler.

WebKit crashes with deeply nested divs
​https://bugs.webkit.org/show_bug.cgi?id=18282

The HTML parser caps a tree depth by MAX_DOM_TREE_DEPTH defined in
TreeDepthLimit.h. This is performance-efficient, but does not work for
DOM operations such as Node.appendChild().

Test: fast/parser/element-nesting-cap.html

  • dom/Node.h:
  • dom/XMLTokenizer.cpp: (WebCore::XMLTokenizer::pushCurrentNode):
  • html/HTMLParser.cpp:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/TreeDepthLimit.h: Added. Define MAX_DOM_TREE_DEPTH. The default value is 5000.
  • dom/XMLTokenizer.cpp: Use maxDOMTreeDepth. (WebCore::XMLTokenizer::pushCurrentNode):
  • html/HTMLParser.cpp: Introduce m_treeDepth, which counts the depth of the tree. Unlike m_blocksInStack, it contains the number of non-block nodes. (WebCore::HTMLParser::HTMLParser): (WebCore::HTMLParser::reset): (WebCore::HTMLParser::limitDepth): Renamed from limitBlockDepth. Add check for m_nodeDepth. (WebCore::HTMLParser::insertNodeAfterLimitDepth): Renamed from insertNodeAfterLimitBlockDepth. (WebCore::HTMLParser::parseToken): (WebCore::HTMLParser::handleResidualStyleCloseTagAcrossBlocks): (WebCore::HTMLParser::pushBlock): (WebCore::HTMLParser::popOneBlockCommon): (WebCore::HTMLParser::freeBlock):
  • html/HTMLParser.h:

Apr 25, 2010:

11:49 PM Changeset in webkit [58235] by yuzo@google.com
  • 1 edit
    3 adds in trunk/LayoutTests

2010-04-25 Yuzo Fujishima <​yuzo@google.com>

Unreviewed, Chromium test expectation change.

Rebaseline test expectation for Chromium.
The following tests pass for Chromium.

​https://bugs.webkit.org/show_bug.cgi?id=38083

  • platform/chromium/fast/js/ToNumber-expected.txt: Added.
  • platform/chromium/fast/js/parseFloat-expected.txt: Added.
11:45 PM Changeset in webkit [58234] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

2010-04-25 Tony Chang <​tony@chromium.org>

Not reviewed, trying to get the chromium bots green.
Adding image diffs to pixel tests.

REGRESSION(r58191): Caret moved a pixel, pixel tests failing
​https://bugs.webkit.org/show_bug.cgi?id=38104

  • platform/chromium/test_expectations.txt:
11:12 PM Changeset in webkit [58233] by tony@chromium.org
  • 2 edits
    5 adds in trunk/LayoutTests

2010-04-25 Tony Chang <​tony@chromium.org>

Not reviewed, trying to get the chromium bots
green.

​https://bugs.webkit.org/show_bug.cgi?id=38099
Rebaseline a new test and add another new test to the
expected fail list on windows and linux.

  • platform/chromium-linux/fast/css-generated-content/after-duplicated-after-split-expected.checksum: Added.
  • platform/chromium-linux/fast/css-generated-content/after-duplicated-after-split-expected.png: Added.
  • platform/chromium-win/fast/css-generated-content/after-duplicated-after-split-expected.checksum: Added.
  • platform/chromium-win/fast/css-generated-content/after-duplicated-after-split-expected.png: Added.
  • platform/chromium-win/fast/css-generated-content/after-duplicated-after-split-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
10:56 PM Changeset in webkit [58232] by yuzo@google.com
  • 2 edits in trunk/LayoutTests

2010-04-25 Yuzo Fujishima <​yuzo@google.com>

Unreviewed, Chromium test expectation change.

editing/inserting/caret-position.html is expected to fail for Chromium.
This is because TextInputController::firstRectForCharacterRange is unimplemented for Chromium.

​https://bugs.webkit.org/show_bug.cgi?id=38100

  • platform/chromium/test_expectations.txt:
3:03 PM Changeset in webkit [58231] by kevino@webkit.org
  • 3 edits in trunk/WebKit/wx

Reviewed by Kevin Ollivier.

Update focus handling code to match current approaches used by other platforms,
and fix focus handling for corner cases such as when a mouse down pops up a dialog.

​https://bugs.webkit.org/show_bug.cgi?id=38086

9:16 AM Changeset in webkit [58230] by hamaji@chromium.org
  • 2 edits in trunk/WebKit/qt

2010-04-25 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by Simon Hausmann.

[Qt] layoutTestController.counterValueForElementById crashes for a nonexistent ID
​https://bugs.webkit.org/show_bug.cgi?id=34573

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp: (DumpRenderTreeSupportQt::counterValueForElementById):
8:46 AM Changeset in webkit [58229] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

2010-04-25 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Adam Barth.

Added tests for pages event handlers being called with
page context when an event is dispatched from an isolated
world.
​https://bugs.webkit.org/show_bug.cgi?id=37947

  • http/tests/security/isolatedWorld/dispatchEvent-expected.txt: Added.
  • http/tests/security/isolatedWorld/dispatchEvent.html: Added.

2010-04-25 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Adam Barth.

Get context properly when event handler was created on
the main world.
​https://bugs.webkit.org/show_bug.cgi?id=37947

Test: http/tests/security/isolatedWorld/dispatchEvent.html

  • bindings/v8/WorldContextHandle.cpp: (WebCore::WorldContextHandle::adjustedContext):
6:52 AM Changeset in webkit [58228] by eric@webkit.org
  • 15 edits
    5 adds in trunk

2010-04-25 yael aharon <​yael.aharon@nokia.com>

Reviewed by Adele Peterson.

Allow styling of HTMLProgressElement.
​https://bugs.webkit.org/show_bug.cgi?id=37901

Added a test for changing the -webkit-appearance statically and dynamically.

  • fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/fast/dom/HTMLProgressElement: Added.
  • platform/mac/fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element-expected.checksum: Added.
  • platform/mac/fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element-expected.png: Added.
  • platform/mac/fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element-expected.txt: Added.
  • platform/qt/Skipped:
  • platform/win/Skipped:

2010-04-25 yael aharon <​yael.aharon@nokia.com>

Reviewed by Adele Peterson.

Allow styling of HTMLProgressElement.
​https://bugs.webkit.org/show_bug.cgi?id=37901

Added a new pseudo element to represent the value portion of the progress element.
Web developers can style this pseudo element separate from the rest of the progress element.

Test: fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element.html

  • css/CSSPrimitiveValueMappings.h: (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
  • css/CSSSelector.cpp: (WebCore::CSSSelector::pseudoId): (WebCore::nameToPseudoTypeMap): (WebCore::CSSSelector::extractPseudoType):
  • css/CSSSelector.h:
  • css/CSSValueKeywords.in:
  • platform/ThemeTypes.h:
  • rendering/RenderProgress.cpp: (WebCore::ProgressValueElement::isShadowNode): (WebCore::ProgressValueElement::shadowParentNode): (WebCore::ProgressValueElement::ProgressValueElement): (WebCore::RenderProgress::RenderProgress): (WebCore::RenderProgress::~RenderProgress): (WebCore::RenderProgress::layout): (WebCore::RenderProgress::styleDidChange): (WebCore::RenderProgress::updateFromElement): (WebCore::RenderProgress::updateValuePartState): (WebCore::RenderProgress::createStyleForValuePart): (WebCore::RenderProgress::updateAnimationState):
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::isControlStyled):
  • rendering/RenderProgress.h:
  • rendering/style/RenderStyleConstants.h:
4:37 AM Changeset in webkit [58227] by eric@webkit.org
  • 3 edits in trunk/WebKit/win

2010-04-25 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: inspector client shouldn't check if it can be opened
docked if it is already in that state.

​https://bugs.webkit.org/show_bug.cgi?id=37946

  • WebCoreSupport/WebInspectorClient.cpp: (WebInspectorFrontendClient::WebInspectorFrontendClient): (WebInspectorFrontendClient::showWindowWithoutNotifications):
  • WebCoreSupport/WebInspectorClient.h:
Note: See TracTimeline for information about the timeline view.