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

Timeline



Jul 5, 2011:

11:59 PM Changeset in webkit [90440] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-07-05 Eric Seidel <eric@webkit.org>

Re-enable NRWT on Leopard now that bug 63973 is fixed.

Reviewed by Adam Barth.

I also made the should-enable expression a black-list instead
of a white-list since every bot on build.webkit.org is
now using NRWT except windows and WebKit2 bots.

  • Scripts/run-webkit-tests: (useNewRunWebKitTests):
11:55 PM Changeset in webkit [90439] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-07-05 Eric Seidel <eric@webkit.org>

Perl string concat failure on Leopard (causes NRWT to fail)
https://bugs.webkit.org/show_bug.cgi?id=63973

Reviewed by Daniel Bates.

Leopard's "xcodebuild -version" has a different format from Snow Leopard/Lion
thus the version parsing logic in webkitdirs.pm seems to fail.

Default to "3.0" instead of undef when we can't parse the version number.
This should send us down the correct path for Leopard.

  • Scripts/webkitdirs.pm: (determineBaseProductDir):
11:36 PM Changeset in webkit [90438] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

2011-07-05 Kentaro Hara <haraken@google.com>

Trigger a 'change' event whenever selected files are changed
https://bugs.webkit.org/show_bug.cgi?id=63929

Reviewed by Kent Tamura.

Record previously selected files and then compare them with newly selected files.
If the previously selected files and their order are not equal to those of
the newly selected files, we trigger the 'change' event.

  • fast/forms/file-input-change-event-expected.txt: Added.
  • fast/forms/file-input-change-event.html: Added.

2011-07-05 Kentaro Hara <haraken@google.com>

Trigger a 'change' event whenever selected files are changed
https://bugs.webkit.org/show_bug.cgi?id=63929

Reviewed by Kent Tamura.

Record previously selected files and then compare them with newly selected files.
If the previously selected files and their order are not equal to those of
the newly selected files, we trigger the 'change' event.

Test: fast/forms/file-input-change-event.html

  • html/FileInputType.cpp: (WebCore::FileInputType::filesChosen): Calls dispathChangeEvent() only when the previously selected files are different from the newly selected files
11:35 PM Changeset in webkit [90437] by commit-queue@webkit.org
  • 4 edits in trunk/Source/JavaScriptCore

2011-07-05 Filip Pizlo <fpizlo@apple.com>

DFG JIT virtual call implementation is inefficient.
https://bugs.webkit.org/show_bug.cgi?id=63974

Reviewed by Gavin Barraclough.

  • dfg/DFGOperations.cpp:
  • runtime/Executable.h: (JSC::ExecutableBase::generatedJITCodeForCallWithArityCheck): (JSC::ExecutableBase::generatedJITCodeForConstructWithArityCheck): (JSC::ExecutableBase::generatedJITCodeWithArityCheckFor): (JSC::ExecutableBase::hasJITCodeForCall): (JSC::ExecutableBase::hasJITCodeForConstruct): (JSC::ExecutableBase::hasJITCodeFor):
  • runtime/JSFunction.h: (JSC::JSFunction::scopeUnchecked):
10:29 PM Changeset in webkit [90436] by tkent@chromium.org
  • 6 edits in trunk

2011-06-29 Kent Tamura <tkent@chromium.org>

Reviewed by Alexey Proskuryakov.

[Mac] REGRESSION (r88415): Range slider doesn't have focus appearance.
https://bugs.webkit.org/show_bug.cgi?id=63573

  • fast/forms/input-appearance-range.html: Give a focus on a slider to check focus appearance.
  • platform/chromium/test_expectations.txt:
  • platform/mac/fast/forms/input-appearance-range-expected.png:

2011-06-29 Kent Tamura <tkent@chromium.org>

Reviewed by Alexey Proskuryakov.

[Mac] REGRESSION (r88415): Range slider doesn't have focus appearance.
https://bugs.webkit.org/show_bug.cgi?id=63573

  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::paintSliderThumb): Need to pass the RenderObject of the owner <input> because updateFocusedState() checks the outline style of the specified renderer, and we'd like to make a slider thumb reflect the owner focus style.
9:36 PM Changeset in webkit [90435] by kevino@webkit.org
  • 2 edits in trunk/Tools

[wx] Unreviewed build fix after addition of Source/WebCore/xml/parser dir.

9:24 PM EFLWebKit edited by gyuyoung.kim@samsung.com
(diff)
9:11 PM EFLWebKit edited by gyuyoung.kim@samsung.com
(diff)
9:00 PM Changeset in webkit [90434] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-07-05 Eric Seidel <eric@webkit.org>

Disable NRWT for Leopard until https://bugs.webkit.org/show_bug.cgi?id=63973 can be fixed.

Reviewed by Adam Barth.

  • Scripts/run-webkit-tests: (useNewRunWebKitTests):
8:50 PM Changeset in webkit [90433] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-07-05 Eric Seidel <eric@webkit.org>

Make run-webkit-tests pass --verbose to new-run-webkit-tests on qt/gtk bots.

Reviewed by Adam Barth.

  • Scripts/run-webkit-tests: (runningOnBuildBot):
8:44 PM Changeset in webkit [90432] by abarth@webkit.org
  • 4 edits
    1 add in trunk/Tools

2011-07-05 Adam Barth <abarth@webkit.org>

Add results_unittests.js to garden-o-matic
https://bugs.webkit.org/show_bug.cgi?id=63971

Reviewed by Eric Seidel.

Adding these unit tests forced me to commit to an API for the results
module. I'm not sure if this is the API we'll end up with, but it's
something not entirely unreasonable.

Also, I've changed results.js over to WebKit style, as discussed previously.

  • Scripts/webkitpy/tool/servers/data/gardeningserver/index.html:
  • Scripts/webkitpy/tool/servers/data/gardeningserver/results.js:
  • Scripts/webkitpy/tool/servers/data/gardeningserver/results_unittests.js: Added.
  • Scripts/webkitpy/tool/servers/data/gardeningserver/run-unittests.html:
8:34 PM Changeset in webkit [90431] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-07-05 Eric Seidel <eric@webkit.org>

Teach run-webkit-tests how to translate --qt and --gtk for new-run-webkit-tests.

Reviewed by Adam Barth.

  • Scripts/run-webkit-tests:
8:15 PM Changeset in webkit [90430] by abarth@webkit.org
  • 2 edits
    2 adds in trunk/Tools

2011-07-05 Adam Barth <abarth@webkit.org>

Add first unit tests for garden-o-matic
https://bugs.webkit.org/show_bug.cgi?id=63969

Reviewed by Eric Seidel.

I've also reformated base.js to match WebKit style, as discussed in the
previous bug.

  • Scripts/webkitpy/tool/servers/data/gardeningserver/base.js:
  • Scripts/webkitpy/tool/servers/data/gardeningserver/base_unittests.js: Added.
  • Scripts/webkitpy/tool/servers/data/gardeningserver/run-unittests.html: Added.
8:11 PM Changeset in webkit [90429] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-07-05 Eric Seidel <eric@webkit.org>

Move Qt, Gtk and Leopard to NRWT.

We'll roll out any of these if we see problems.

Reviewed by Adam Barth.

  • Scripts/run-webkit-tests: (useNewRunWebKitTests):
7:32 PM Changeset in webkit [90428] by abarth@webkit.org
  • 4 edits
    1 add in trunk/Tools

2011-07-05 Adam Barth <abarth@webkit.org>

garden-o-matic should understand which tests have unexpected failures
https://bugs.webkit.org/show_bug.cgi?id=63965

Reviewed by Eric Seidel.

Currently, this code just logs the list of unexpected failures to the
console. A future patch will do something useful with this
information. A testing harness will also come in the next patch.

  • Scripts/webkitpy/tool/servers/data/gardeningserver/base.js: Added.
  • Scripts/webkitpy/tool/servers/data/gardeningserver/index.html:
  • Scripts/webkitpy/tool/servers/data/gardeningserver/results.js:
  • Scripts/webkitpy/tool/servers/gardeningserver.py:
7:30 PM Changeset in webkit [90427] by abarth@webkit.org
  • 1 edit
    15 adds in trunk/Source/ThirdParty

2011-07-05 Adam Barth <abarth@webkit.org>

Import qunit JavaScript unit testing framework
https://bugs.webkit.org/show_bug.cgi?id=63967

Reviewed by Eric Seidel.

We're going to use this unit testing framework to test garden-o-matic.

  • qunit: Added.
6:30 PM Changeset in webkit [90426] by oliver@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

2011-07-05 Oliver Hunt <oliver@apple.com>

Force inlining of simple functions that show up as not being inlined
https://bugs.webkit.org/show_bug.cgi?id=63964

Reviewed by Gavin Barraclough.

Looking at profile data indicates the gcc is failing to inline a
number of trivial functions. This patch hits the ones that show
up in profiles with the ALWAYS_INLINE hammer.

We also replace the memcpy() call in linking with a manual loop.
Apparently memcpy() is almost never faster than an inlined loop.

  • assembler/ARMv7Assembler.h: (JSC::ARMv7Assembler::add): (JSC::ARMv7Assembler::add_S): (JSC::ARMv7Assembler::ARM_and): (JSC::ARMv7Assembler::asr): (JSC::ARMv7Assembler::b): (JSC::ARMv7Assembler::blx): (JSC::ARMv7Assembler::bx): (JSC::ARMv7Assembler::clz): (JSC::ARMv7Assembler::cmn): (JSC::ARMv7Assembler::cmp): (JSC::ARMv7Assembler::eor): (JSC::ARMv7Assembler::it): (JSC::ARMv7Assembler::ldr): (JSC::ARMv7Assembler::ldrCompact): (JSC::ARMv7Assembler::ldrh): (JSC::ARMv7Assembler::ldrb): (JSC::ARMv7Assembler::lsl): (JSC::ARMv7Assembler::lsr): (JSC::ARMv7Assembler::movT3): (JSC::ARMv7Assembler::mov): (JSC::ARMv7Assembler::movt): (JSC::ARMv7Assembler::mvn): (JSC::ARMv7Assembler::neg): (JSC::ARMv7Assembler::orr): (JSC::ARMv7Assembler::orr_S): (JSC::ARMv7Assembler::ror): (JSC::ARMv7Assembler::smull): (JSC::ARMv7Assembler::str): (JSC::ARMv7Assembler::sub): (JSC::ARMv7Assembler::sub_S): (JSC::ARMv7Assembler::tst): (JSC::ARMv7Assembler::linkRecordSourceComparator): (JSC::ARMv7Assembler::link): (JSC::ARMv7Assembler::ARMInstructionFormatter::oneWordOp5Reg3Imm8): (JSC::ARMv7Assembler::ARMInstructionFormatter::oneWordOp5Imm5Reg3Reg3): (JSC::ARMv7Assembler::ARMInstructionFormatter::oneWordOp7Reg3Reg3Reg3): (JSC::ARMv7Assembler::ARMInstructionFormatter::oneWordOp8Imm8): (JSC::ARMv7Assembler::ARMInstructionFormatter::oneWordOp8RegReg143): (JSC::ARMv7Assembler::ARMInstructionFormatter::oneWordOp9Imm7): (JSC::ARMv7Assembler::ARMInstructionFormatter::oneWordOp10Reg3Reg3): (JSC::ARMv7Assembler::ARMInstructionFormatter::twoWordOp12Reg4FourFours): (JSC::ARMv7Assembler::ARMInstructionFormatter::twoWordOp16FourFours): (JSC::ARMv7Assembler::ARMInstructionFormatter::twoWordOp16Op16): (JSC::ARMv7Assembler::ARMInstructionFormatter::twoWordOp5i6Imm4Reg4EncodedImm): (JSC::ARMv7Assembler::ARMInstructionFormatter::twoWordOp12Reg4Reg4Imm12): (JSC::ARMv7Assembler::ARMInstructionFormatter::vfpOp): (JSC::ARMv7Assembler::ARMInstructionFormatter::vfpMemOp):
  • assembler/LinkBuffer.h: (JSC::LinkBuffer::linkCode):
  • assembler/MacroAssemblerARMv7.h: (JSC::MacroAssemblerARMv7::nearCall): (JSC::MacroAssemblerARMv7::call): (JSC::MacroAssemblerARMv7::ret): (JSC::MacroAssemblerARMv7::moveWithPatch): (JSC::MacroAssemblerARMv7::branchPtrWithPatch): (JSC::MacroAssemblerARMv7::storePtrWithPatch): (JSC::MacroAssemblerARMv7::tailRecursiveCall): (JSC::MacroAssemblerARMv7::makeTailRecursiveCall): (JSC::MacroAssemblerARMv7::jump): (JSC::MacroAssemblerARMv7::makeBranch):
6:20 PM Changeset in webkit [90425] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-07-05 Zoltan Herczeg <zherczeg@inf.u-szeged.hu>

Make "Add optimised paths for a few maths functions" work on Qt
https://bugs.webkit.org/show_bug.cgi?id=63893

Reviewed by Oliver Hunt.

Move the generated code to the .text section instead of .data section.
Fix alignment for the 32 bit thunk code.

  • jit/ThunkGenerators.cpp:
6:18 PM Changeset in webkit [90424] by abarth@webkit.org
  • 3 edits
    1 add in trunk/Tools

2011-07-05 Adam Barth <abarth@webkit.org>

garden-o-matic should know how to fetch test results from the (NRWT) bots
https://bugs.webkit.org/show_bug.cgi?id=63959

Reviewed by Eric Seidel.

Currently, this code just fetches the results and expectations JSON
blob for Linux from the server and logs it to the console.

  • Scripts/webkitpy/tool/servers/data/gardeningserver/index.html:
  • Scripts/webkitpy/tool/servers/data/gardeningserver/results.js: Added.
  • Scripts/webkitpy/tool/servers/gardeningserver.py:
5:56 PM Changeset in webkit [90423] by commit-queue@webkit.org
  • 26 edits in trunk/Source/JavaScriptCore

2011-07-05 Filip Pizlo <fpizlo@apple.com>

DFG JIT does not implement op_call.
https://bugs.webkit.org/show_bug.cgi?id=63858

Reviewed by Gavin Barraclough.

  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::unlinkCalls):
  • bytecode/CodeBlock.h: (JSC::CodeBlock::setNumberOfCallLinkInfos): (JSC::CodeBlock::numberOfCallLinkInfos):
  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitCall): (JSC::BytecodeGenerator::emitConstruct):
  • dfg/DFGAliasTracker.h: (JSC::DFG::AliasTracker::lookupGetByVal): (JSC::DFG::AliasTracker::recordCall): (JSC::DFG::AliasTracker::equalIgnoringLaterNumericConversion):
  • dfg/DFGByteCodeParser.cpp: (JSC::DFG::ByteCodeParser::ByteCodeParser): (JSC::DFG::ByteCodeParser::getLocal): (JSC::DFG::ByteCodeParser::getArgument): (JSC::DFG::ByteCodeParser::toInt32): (JSC::DFG::ByteCodeParser::addToGraph): (JSC::DFG::ByteCodeParser::addVarArgChild): (JSC::DFG::ByteCodeParser::predictInt32): (JSC::DFG::ByteCodeParser::parseBlock): (JSC::DFG::ByteCodeParser::processPhiStack): (JSC::DFG::ByteCodeParser::allocateVirtualRegisters):
  • dfg/DFGGraph.cpp: (JSC::DFG::Graph::opName): (JSC::DFG::Graph::dump): (JSC::DFG::Graph::refChildren):
  • dfg/DFGGraph.h:
  • dfg/DFGJITCodeGenerator.cpp: (JSC::DFG::JITCodeGenerator::useChildren): (JSC::DFG::JITCodeGenerator::emitCall):
  • dfg/DFGJITCodeGenerator.h: (JSC::DFG::JITCodeGenerator::addressOfCallData):
  • dfg/DFGJITCompiler.cpp: (JSC::DFG::JITCompiler::compileFunction):
  • dfg/DFGJITCompiler.h: (JSC::DFG::CallRecord::CallRecord): (JSC::DFG::JITCompiler::notifyCall): (JSC::DFG::JITCompiler::appendCallWithFastExceptionCheck): (JSC::DFG::JITCompiler::addJSCall): (JSC::DFG::JITCompiler::PropertyAccessRecord::PropertyAccessRecord): (JSC::DFG::JITCompiler::JSCallRecord::JSCallRecord):
  • dfg/DFGNode.h: (JSC::DFG::Node::Node): (JSC::DFG::Node::child1): (JSC::DFG::Node::child2): (JSC::DFG::Node::child3): (JSC::DFG::Node::firstChild): (JSC::DFG::Node::numChildren):
  • dfg/DFGNonSpeculativeJIT.cpp: (JSC::DFG::NonSpeculativeJIT::basicArithOp): (JSC::DFG::NonSpeculativeJIT::compare): (JSC::DFG::NonSpeculativeJIT::compile):
  • dfg/DFGOperations.cpp:
  • dfg/DFGOperations.h:
  • dfg/DFGRepatch.cpp: (JSC::DFG::dfgLinkCall):
  • dfg/DFGRepatch.h:
  • dfg/DFGSpeculativeJIT.cpp: (JSC::DFG::SpeculativeJIT::compilePeepHoleIntegerBranch): (JSC::DFG::SpeculativeJIT::compilePeepHoleCall): (JSC::DFG::SpeculativeJIT::compile):
  • dfg/DFGSpeculativeJIT.h: (JSC::DFG::SpeculativeJIT::detectPeepHoleBranch):
  • interpreter/CallFrame.h: (JSC::ExecState::calleeAsValue):
  • jit/JIT.cpp: (JSC::JIT::JIT): (JSC::JIT::privateCompileMainPass): (JSC::JIT::privateCompileSlowCases): (JSC::JIT::privateCompile): (JSC::JIT::linkCall): (JSC::JIT::linkConstruct):
  • jit/JITCall.cpp: (JSC::JIT::compileOpCall):
  • jit/JITCode.h: (JSC::JITCode::JITCode): (JSC::JITCode::jitType): (JSC::JITCode::HostFunction):
  • runtime/JSFunction.h:
  • runtime/JSGlobalData.h:
5:33 PM Changeset in webkit [90422] by dpranke@chromium.org
  • 4 edits in trunk/Tools

2011-07-05 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

Python tests are failing on leopard
https://bugs.webkit.org/show_bug.cgi?id=63842

Fix the missing flush() call that appears to be needed by the
logging package in python 2.5. Also fix the regressions
introduced in the run_webkit_test tests when we switched from
thread to processes (since processes aren't available on 2.5).

Also fix a missing "from future import with_statement' in
testfilehandler (unreviewed).

  • Scripts/webkitpy/layout_tests/layout_package/metered_stream.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:
  • TestResultServer/handlers/testfilehandler.py:
5:31 PM Changeset in webkit [90421] by oliver@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Initialize new MarkStack member

5:16 PM Changeset in webkit [90420] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-07-05 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

re-enabling new-run-webkit-tests on the Snow Leopard WebKit1 bots
now that we believe the stray http-server issue to be solved.

  • Scripts/run-webkit-tests: (useNewRunWebKitTests):
5:12 PM Changeset in webkit [90419] by dpranke@chromium.org
  • 3 edits in trunk/Tools

2011-07-05 Dirk Pranke <dpranke@chromium.org>

nrwt: allow for multiple http shards
https://bugs.webkit.org/show_bug.cgi?id=63116

Reviewed by Tony Chang.

This modifies the sharding logic to support multiple http
shards, but for now we clamp to one shard until we can test
perf impact and flakiness impact.

  • Scripts/webkitpy/layout_tests/layout_package/manager.py:
  • Scripts/webkitpy/layout_tests/layout_package/manager_unittest.py:
5:08 PM Changeset in webkit [90418] by abarth@webkit.org
  • 2 edits in trunk/Tools

2011-07-05 Adam Barth <abarth@webkit.org>

Turns out we need to support numerals too because that's what jQuery
uses by default.

  • TestResultServer/handlers/testfilehandler.py:
5:07 PM Changeset in webkit [90417] by dpranke@chromium.org
  • 3 edits in trunk/Tools

2011-07-05 Dirk Pranke <dpranke@chromium.org>

Re-land nrwt: make sharding tests needing locks less hard-coded
https://bugs.webkit.org/show_bug.cgi?id=63112

Reviewed by Ojan Vafai.

  • Scripts/webkitpy/layout_tests/layout_package/manager.py:
  • Scripts/webkitpy/layout_tests/layout_package/manager_unittest.py:
4:56 PM Changeset in webkit [90416] by abarth@webkit.org
  • 2 edits in trunk/Tools

2011-07-05 Adam Barth <abarth@webkit.org>

TestResultServer should support callback parameter for JSON
https://bugs.webkit.org/show_bug.cgi?id=63961

Reviewed by Ojan Vafai.

This makes using jQuery.ajax so much more pleasant.

  • TestResultServer/handlers/testfilehandler.py:
4:55 PM Changeset in webkit [90415] by oliver@apple.com
  • 8 edits in trunk/Source/JavaScriptCore

2011-07-05 Oliver Hunt <oliver@apple.com>

Don't throw out compiled code repeatedly
https://bugs.webkit.org/show_bug.cgi?id=63960

Reviewed by Gavin Barraclough.

Stop throwing away all compiled code every time
we're told to do a full GC. Instead unlink all
callsites during such GC passes to maximise the
number of collectable functions, but otherwise
leave compiled functions alone.

  • API/JSBase.cpp: (JSGarbageCollect):
  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::visitAggregate):
  • heap/Heap.cpp: (JSC::Heap::collectAllGarbage):
  • heap/MarkStack.h: (JSC::MarkStack::shouldUnlinkCalls): (JSC::MarkStack::setShouldUnlinkCalls):
  • runtime/JSGlobalData.cpp: (JSC::JSGlobalData::recompileAllJSFunctions): (JSC::JSGlobalData::releaseExecutableMemory):
  • runtime/RegExp.cpp: (JSC::RegExp::compile): (JSC::RegExp::invalidateCode):
  • runtime/RegExp.h:
4:18 PM Changeset in webkit [90414] by commit-queue@webkit.org
  • 5 edits in trunk/Source/JavaScriptCore

2011-07-05 Filip Pizlo <fpizlo@apple.com>

JSC JIT has code duplication for the handling of call and construct
https://bugs.webkit.org/show_bug.cgi?id=63957

Reviewed by Gavin Barraclough.

  • jit/JIT.cpp: (JSC::JIT::linkFor):
  • jit/JIT.h:
  • jit/JITStubs.cpp: (JSC::jitCompileFor): (JSC::DEFINE_STUB_FUNCTION): (JSC::arityCheckFor): (JSC::lazyLinkFor):
  • runtime/Executable.h: (JSC::ExecutableBase::generatedJITCodeFor): (JSC::FunctionExecutable::compileFor): (JSC::FunctionExecutable::isGeneratedFor): (JSC::FunctionExecutable::generatedBytecodeFor): (JSC::FunctionExecutable::generatedJITCodeWithArityCheckFor):
4:16 PM Changeset in webkit [90413] by eric@webkit.org
  • 7 edits in trunk

2011-07-05 Eric Seidel <eric@webkit.org>

buildbot needs to understand whether NRWT exited early after having too many failures
https://bugs.webkit.org/show_bug.cgi?id=63839

Reviewed by Adam Barth.

Teach the new results.html how to display a warning when testing exited early.
The warning isn't quite as nice as ORWT since I couldn't figure out how to
find the total tests run, or total unexpected crashes. I figure
this is enough to get us going and we can refine it further.

  • fast/harness/results.html:

2011-07-05 Eric Seidel <eric@webkit.org>

buildbot needs to understand whether NRWT exited early after having too many failures
https://bugs.webkit.org/show_bug.cgi?id=63839

Reviewed by Adam Barth.

Fix-up the exited early messages printed by NRWT so that
the buildbot can parse them as expected.
It looks for lines using "if line.find('Exiting early') >= 0:"

I also plumbed the "early exit" status through to results.json
in the form of an "interrupted" bool. It was unclear to me
if results.json already had enough information to compute this bool
itself. It's possible Ojan could come up with a better fix.

  • Scripts/webkitpy/layout_tests/layout_package/manager.py:
  • Scripts/webkitpy/layout_tests/layout_package/manager_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/result_summary.py:
3:59 PM Changeset in webkit [90412] by eric@webkit.org
  • 3 edits in trunk/Tools

2011-07-05 Eric Seidel <eric@webkit.org>

new-run-webkit-tests fails to start http server if one is already running
https://bugs.webkit.org/show_bug.cgi?id=63956

Reviewed by Adam Barth.

  • BuildSlaveSupport/kill-old-processes:
    • Removed M line endings and sorted the process names.
    • There are no functional changes to this file. (except that now that its using unix line endings it can be executed directly!)
  • Scripts/webkitpy/layout_tests/servers/apache_http_server.py:
    • Change the name to 'httpd' to match old-run-webkit-test paths.
3:54 PM Changeset in webkit [90411] by abarth@webkit.org
  • 4 edits
    1 add in trunk/Tools

2011-07-05 Adam Barth <abarth@webkit.org>

Add basic ajax support to garden-o-matic
https://bugs.webkit.org/show_bug.cgi?id=63874

Reviewed by Eric Seidel.

This patch adds some basic infrastructure to garden-o-matic.
Currently, the infrastructure is just used to make the "quit" command
use Ajax, but in the future, this infrastructure will be used to do
more sophistocated remote proceedure calls.

  • Scripts/webkitpy/tool/servers/data/gardeningserver/index.html:
  • Scripts/webkitpy/tool/servers/data/gardeningserver/main.js: Added.
  • Scripts/webkitpy/tool/servers/gardeningserver.py:
  • Scripts/webkitpy/tool/servers/reflectionhandler.py:
3:52 PM Changeset in webkit [90410] by abarth@webkit.org
  • 4 edits
    5 adds in trunk/Tools

2011-07-05 Adam Barth <abarth@webkit.org>

Add trivial garden-o-matic command (with server)
https://bugs.webkit.org/show_bug.cgi?id=63872

Reviewed by Eric Seidel.

This patch adds a trivial "Hello, world" garden-o-matic command,
complete with an HTTP server. This command re-uses a bunch of code
from the existing rebaseline-server command. Over time, this command
will grow to be a tool useful for gardening the WebKit tree.

  • Scripts/webkitpy/tool/commands/init.py:
  • Scripts/webkitpy/tool/commands/abstractservercommand.py: Added.
  • Scripts/webkitpy/tool/commands/gardenomatic.py: Added.
  • Scripts/webkitpy/tool/commands/rebaselineserver.py:
  • Scripts/webkitpy/tool/servers/data/gardeningserver/index.html: Added.
  • Scripts/webkitpy/tool/servers/gardeningserver.py: Added.
  • Scripts/webkitpy/tool/servers/rebaselineserver.py:
3:20 PM Changeset in webkit [90409] by Adam Roben
  • 2 edits in trunk/Tools

Make prepare-ChangeLog include modified Perl functions in its ChangeLog template

This is a very simple first cut. Functions must start with a line that starts with "sub "
and end with a line that starts with a closing brace. No leading whitespace is allowed.
Package names aren't parsed at all.

Fixes <http://webkit.org/b/21591> prepare-ChangeLog should know how to find functions in
Perl files

Reviewed by David Kilzer.

  • Scripts/prepare-ChangeLog: Removed redundant code that ignored certain files when

searching for function line ranges. This is already done inside the get_function_line_ranges
function.
(get_function_line_ranges): Cleaned up coding style a little bit. Call
get_function_line_ranges_for_perl for files with .pl and .pm extensions. For files with an
unknown extension or no extension, read the shebang line to try to determine the script
interpreter. Call get_function_line_ranges_for_perl if the interpreter seems to be Perl.
(get_function_line_ranges_for_perl): Added. Does extremely basic parsing of the file to find
lines starting with "sub " or "}".

3:20 PM Changeset in webkit [90408] by Adam Roben
  • 2 edits in trunk/Tools

Clean up run-api-tests output on Windows

We were mixing run-api-tests output with gtest output, and the result was a mess.

Fixes <http://webkit.org/b/63954> run-api-tests output is very confusing on Windows

Reviewed by Dan Bates.

  • Scripts/run-api-tests:

(runTest):
(populateTests):
Made Windows use the formerly-Mac-only codepaths which correctly suppress gtest output
except in verbose mode. Most changes are due to indentation. Use of the arch utility is now
the only Mac-specific thing in these functions. Some other platform-specific code was moved
from here...

(prepareEnvironmentForRunningTestTool):
(testToolPath):
...to here.

3:13 PM Changeset in webkit [90407] by hans@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-07-05 Hans Wennborg <hans@chromium.org>

[V8][Chromium] Remove use of OwnHandle from V8LocalContext
https://bugs.webkit.org/show_bug.cgi?id=63925

Reviewed by Adam Barth.

Replace the OwnHandle<> member in V8LocalContext with
v8::Persistent<v8::Context>.
The code that used OwnHandle<> would not correctly Dispose
of the Context, which would cause the memory to fill up when creating
and destructing many V8LocalContext objects, such as in the case
of IndexedDB.

  • bindings/v8/V8Utilities.cpp: (WebCore::V8LocalContext::V8LocalContext): (WebCore::V8LocalContext::~V8LocalContext):
  • bindings/v8/V8Utilities.h:
2:29 PM Changeset in webkit [90406] by commit-queue@webkit.org
  • 10 edits
    1 delete in trunk/Source/WebCore

2011-07-05 Igor Oliveira <igor.oliveira@openbossa.org>

[Qt] Add ShadowBlur support
https://bugs.webkit.org/show_bug.cgi?id=62997

Use ShadowBlur instead of ContextShadow to handle canvas and css shadows. ShadowBlur is
a newer and platform independent shadow implementation.
Qt QPainter does not have support for shadows so is necessary to create a layer support
to draw complex shapes and fonts. The beginShadowLayer creates a temporary shadow buffer
where the canvas can draw onto and endShadowLayer applies the blur filter, colors the
shadow buffer and draw the final shadow to the graphics context.

Reviewed by Andreas Kling.

  • WebCore.pro:
  • platform/graphics/GraphicsContext.cpp:
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/ShadowBlur.cpp: (WebCore::ShadowBlur::ShadowBlur): (WebCore::ShadowBlur::setShadowValues): (WebCore::ShadowBlur::updateShadowBlurValues): (WebCore::ShadowBlur::clear): (WebCore::ShadowBlur::blurAndColorShadowBuffer): (WebCore::ShadowBlur::beginShadowLayer): (WebCore::ShadowBlur::endShadowLayer):
  • platform/graphics/ShadowBlur.h: (WebCore::ShadowBlur::type):
  • platform/graphics/qt/ContextShadowQt.cpp: Removed.
  • platform/graphics/qt/FontQt.cpp: (WebCore::drawTextCommon): (WebCore::Font::drawGlyphs):
  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContextPlatformPrivate::mustUseShadowBlur): (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): (WebCore::GraphicsContextPlatformPrivate::~GraphicsContextPlatformPrivate): (WebCore::GraphicsContext::savePlatformState): (WebCore::GraphicsContext::restorePlatformState): (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::strokePath): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::fillRoundedRect): (WebCore::GraphicsContext::shadowBlur): (WebCore::GraphicsContext::clipBounds): (WebCore::GraphicsContext::setPlatformShadow): (WebCore::GraphicsContext::clearPlatformShadow):
  • platform/graphics/qt/ImageQt.cpp: (WebCore::BitmapImage::draw):
  • platform/graphics/qt/StillImageQt.cpp: (WebCore::StillImage::draw):
1:51 PM Changeset in webkit [90405] by weinig@apple.com
  • 3 edits
    2 adds in trunk

2011-07-05 Sam Weinig <sam@webkit.org>

Null deref accessing CustomEvent.detail
<rdar://problem/9724577>
https://bugs.webkit.org/show_bug.cgi?id=63885

Reviewed by Anders Carlsson.

Test: fast/events/custom-event-uninitialized.html

  • bindings/scripts/CodeGeneratorJS.pm: Guard against uninitialized script value.

2011-07-05 Sam Weinig <sam@webkit.org>

Null deref accessing CustomEvent.detail
<rdar://problem/9724577>
https://bugs.webkit.org/show_bug.cgi?id=63885

Reviewed by Anders Carlsson.

Add test which exercises an initialized details property on
a CustomEvent instance.

  • fast/events/custom-event-uninitialized-expected.txt: Added.
  • fast/events/custom-event-uninitialized.html: Added.
1:09 PM Changeset in webkit [90404] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Build fix following last patch.

  • runtime/JSFunction.cpp:

(JSC::createPrototypeProperty):

12:45 PM Changeset in webkit [90403] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/qt

2011-07-05 Rafael Brandao <rafael.lobo@openbossa.org>

[Qt] Fix tst_QWebFrame::setHtmlWithResource() API test
https://bugs.webkit.org/show_bug.cgi?id=63235

Modified baseUrl to be a local file in order to get a security origin
that is allowed to request local resources. Removed QSignalSpy from it
as loadFinished always happens, and the original test was split into two.

Reviewed by Benjamin Poulain.

  • tests/qwebframe/tst_qwebframe.cpp: (tst_QWebFrame::setHtmlWithImageResource): (tst_QWebFrame::setHtmlWithStylesheetResource): (tst_QWebFrame::setHtmlWithBaseURL):
12:02 PM Changeset in webkit [90402] by barraclough@apple.com
  • 5 edits in trunk

https://bugs.webkit.org/show_bug.cgi?id=63947
ASSERT running Object.preventExtensions(Math.sin)

Reviewed by Oliver Hunt.

This is due to calling scope() on a hostFunction as a part of
calling createPrototypeProperty to reify the prototype property.
But host functions don't have a prototype property anyway!

Source/JavaScriptCore:

Prevent callling createPrototypeProperty on a host function.

  • runtime/JSFunction.cpp:

(JSC::JSFunction::createPrototypeProperty):
(JSC::JSFunction::preventExtensions):

LayoutTests:

Add test case for calling preventExtensions on a host function.

  • fast/js/preventExtensions-expected.txt:
  • fast/js/script-tests/preventExtensions.js:
12:01 PM Changeset in webkit [90401] by barraclough@apple.com
  • 9 edits in trunk

https://bugs.webkit.org/show_bug.cgi?id=63880
Evaluation order of conversions of operands to >, >= incorrect.

Reviewed by Sam Weinig.

Source/JavaScriptCore:

Add 'leftFirst' parameter to jsLess, jsLessEq matching that described in the ES5
spec. This allows these methods to be reused to perform >, >= relational compares
with correct ordering of type conversions.

  • dfg/DFGOperations.cpp:
  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • runtime/Operations.h:

(JSC::jsLess):
(JSC::jsLessEq):

LayoutTests:

Updated layout test results. Two of these tests now pass, however for
the third we now need to check in failing results, since the test is
incorrect!

The problem if that the test author has made the mistake of thinking
that the evaluation order for the operands to '>' is RHS then LHS.
This is due to a quirk in the way the spec is written. The greater
than opeator is defined to call the abstract relational comparison
algorithm with 'leftFirst' set to false, and as such conversion is
performed on the second operand ('y') first (see 11.8.5). However
the abstract relational comparison algorith is performing a less
than comaprison, and the greater than operator calls this algorithm
with the operands to the greater than operator reversed (see 11.8.2).
As such, the second operand to the abstract comaparison is the LHS
of the greater than. This bug also affects the corresponding less
than or equals test, where we already we have failing results checked
in, and again it is the test that is wrong (for the same reason).

  • fast/js/exception-sequencing-binops2-expected.txt:
  • sputnik/Conformance/11_Expressions/11.8_Relational_Operators/11.8.2_The_Greater_than_Operator/S11.8.2_A2.3_T1-expected.txt:
  • sputnik/Conformance/11_Expressions/11.8_Relational_Operators/11.8.4_The_Grater_than_or_equal_Operator/S11.8.4_A2.3_T1-expected.txt:
10:34 AM Changeset in webkit [90400] by psolanki@apple.com
  • 2 edits in trunk/Source/WebCore

2011-07-05 Pratik Solanki <psolanki@apple.com>

Reviewed by Dan Bernstein.

Coalesce data array into one NSData before calling didReceiveData
https://bugs.webkit.org/show_bug.cgi?id=63916
<rdar://problem/9715181>

Instead of calling didReceiveData multiple times with smaller chunks of data, we merge the
data buffers into one and call it once.

No new tests because the flag isn't enabled yet.

  • platform/network/mac/ResourceHandleMac.mm: (-[WebCoreResourceHandleAsDelegate connection:didReceiveDataArray:]):
9:07 AM Changeset in webkit [90399] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

2011-07-05 Tamas Czene <Czene.Tamas@stud.u-szeged.hu>

Reviewed by Simon Fraser.

Remove virtual functions from StyleImage.
https://bugs.webkit.org/show_bug.cgi?id=63909

We can remove the virtual "isChachedImage, isPendingImage, isGeneratedImage" functions to speed up the StyleImage.
Some profile data shows this part is significant, especially in small CSS-based sites.
Removing the virtual functions - removing the calls - makes a small performance progression on this part.

  • rendering/style/StyleCachedImage.h: (WebCore::StyleCachedImage::StyleCachedImage):
  • rendering/style/StyleGeneratedImage.h: (WebCore::StyleGeneratedImage::StyleGeneratedImage):
  • rendering/style/StyleImage.h: (WebCore::StyleImage::isCachedImage): (WebCore::StyleImage::isPendingImage): (WebCore::StyleImage::isGeneratedImage): (WebCore::StyleImage::StyleImage):
  • rendering/style/StylePendingImage.h: (WebCore::StylePendingImage::StylePendingImage):
9:01 AM Changeset in webkit [90398] by yurys@chromium.org
  • 7 edits in trunk/Source/WebKit/chromium

2011-07-05 Yury Semikhatsky <yurys@chromium.org>

[Chromium] Web Inspector: extend WebSharedWorker with inspector transport methods
https://bugs.webkit.org/show_bug.cgi?id=63942

Added methods for connecting inspector to shared worker and for passing
messages between shared worker inspector and corresponding inspector frontend.

Reviewed by Pavel Feldman.

  • public/WebCommonWorkerClient.h: (WebKit::WebCommonWorkerClient::dispatchDevToolsMessage):
  • public/WebSharedWorker.h: (WebKit::WebSharedWorker::attachDevTools): (WebKit::WebSharedWorker::detachDevTools): (WebKit::WebSharedWorker::dispatchDevToolsMessage):
  • public/WebWorkerClient.h:
  • src/WebSharedWorkerImpl.cpp: (WebKit::connectToWorkerContextInspectorTask): (WebKit::WebSharedWorkerImpl::attachDevTools): (WebKit::disconnectFromWorkerContextInspectorTask): (WebKit::WebSharedWorkerImpl::detachDevTools): (WebKit::dispatchOnInspectorBackendTask): (WebKit::WebSharedWorkerImpl::dispatchDevToolsMessage):
  • src/WebSharedWorkerImpl.h:
  • src/WebWorkerBase.cpp: (WebKit::WebWorkerBase::postMessageToPageInspectorTask):
8:35 AM Changeset in webkit [90397] by pfeldman@chromium.org
  • 6 edits
    1 add in trunk/Source/WebCore

2011-07-05 Pavel Feldman <pfeldman@google.com>

Web Inspector: add "element state" setting to the styles section title.
https://bugs.webkit.org/show_bug.cgi?id=63932

Reviewed by Yury Semikhatsky.

  • English.lproj/localizedStrings.js:
  • WebCore.gypi:
  • inspector/front-end/Images/paneElementStateButtons.png: Added.
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.muteEventListener): (WebInspector.StylesSidebarPane): (WebInspector.StylesSidebarPane.prototype.update): (WebInspector.StylesSidebarPane.prototype._rebuildUpdate): (WebInspector.StylesSidebarPane.prototype._rebuildSectionsForStyleRules): (WebInspector.StylesSidebarPane.prototype.addBlankSection): (WebInspector.StylesSidebarPane.prototype.registerShortcuts): (WebInspector.StylesSidebarPane.prototype._toggleElementStatePane): (WebInspector.StylesSidebarPane.prototype._createElementStatePane.clickListener): (WebInspector.StylesSidebarPane.prototype._createElementStatePane.createCheckbox): (WebInspector.StylesSidebarPane.prototype._createElementStatePane):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css: (.pane > .title > .pane-title-button:active, .pane > .title > .pane-title-button.toggled): (.pane > .title > .pane-title-button.element-state): (.styles-element-state-pane): (.styles-element-state-pane.expanded): (.styles-element-state-pane > table): (.styles-element-state-pane input):
7:03 AM clutter edited by kov@webkit.org
(diff)
6:59 AM WebKit Team edited by siddharth.mathur@nokia.com
added Siddharth Mathur to committer list (diff)
6:41 AM Changeset in webkit [90396] by caseq@chromium.org
  • 2 edits in trunk/LayoutTests

2011-07-05 Andrey Kosyakov <caseq@chromium.org>

Unreviewed. Marked css2.1/20110323/replaced-intrinsic-ratio-001.htm as flaky on mac.

  • platform/chromium/test_expectations.txt:
6:31 AM Changeset in webkit [90395] by podivilov@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-06-22 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: extract duplicated code that deals with source mappings.
https://bugs.webkit.org/show_bug.cgi?id=63139

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleMessage.prototype.isEqual): (WebInspector.ConsoleMessage.prototype.get stackTrace):
  • inspector/front-end/DebuggerPresentationModel.js: (WebInspector.DebuggerPresentationModel.prototype._uiLocationToScriptLocation): (WebInspector.DebuggerPresentationModel.prototype._updateAnchor): (WebInspector.DebuggerPresentationModel.prototype._addConsoleMessage.didGetUILocation): (WebInspector.DebuggerPresentationModel.prototype._addConsoleMessage): (WebInspector.DebuggerPresentationModel.prototype.continueToLine): (WebInspector.DebuggerPresentationModel.prototype._setBreakpointInDebugger.didGetScriptLocation): (WebInspector.DebuggerPresentationModel.prototype._setBreakpointInDebugger): (WebInspector.DebuggerPresentationModel.prototype._breakpointAdded.updateSourceFileBreakpointsAndDispatchEvent): (WebInspector.DebuggerPresentationModel.prototype._breakpointAdded.didGetUILocation): (WebInspector.DebuggerPresentationModel.prototype._breakpointAdded): (WebInspector.DebuggerPresentationModel.prototype._debuggerPaused): (WebInspector.DebuggerPresentationModel.prototype._sourceFileForScript): (WebInspector.PresenationCallFrame): (WebInspector.PresenationCallFrame.prototype.sourceLine):
6:10 AM Changeset in webkit [90394] by loislo@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-07-05 Ilya Tikhonovsky <loislo@chromium.org>

Web Inspector: WebInspector frontend reports a protocol error in remote debugging mode.
https://bugs.webkit.org/show_bug.cgi?id=63935

it was error generated for obsolete message "loaded" which was sent from frontend to backend.

Reviewed by Yury Semikhatsky.

5:50 AM Changeset in webkit [90393] by caseq@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-07-05 Andrey Kosyakov <caseq@chromium.org>

Web Inspector: [refactoring] remove network-sidebar style, add network-log-grid
https://bugs.webkit.org/show_bug.cgi?id=63913

Reviewed by Yury Semikhatsky.

  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel): (WebInspector.NetworkPanel.prototype._createTable):
  • inspector/front-end/networkPanel.css: (.network-log-grid): (.network-log-grid table.data): (.network-log-grid.small table.data): (.network-log-grid td): (.network-log-grid.small td): (.network-log-grid th): (.network-log-grid.small th): (.network-log-grid th, .network.panel .data-grid th.sort-descending, .network.panel .data-grid th.sort-ascending): (.network-log-grid .data-container): (.network-log-grid.small .data-container): (.network-log-grid select): (.network-log-grid.small tr.offscreen): (.network-log-grid tr.offscreen): (.network-log-grid tr.offscreen td): (.network-log-grid tr.filler): (.network-log-grid tr:not(.filler) td.name-column): (.network.panel:not(.viewing-resource) .network-log-grid td.name-column:hover): (.network-log-grid td.time-column): (.network-log-grid.small .network-graph-side): (.network-log-grid th.sortable:active): (.network-log-grid tr.selected .network-cell-subtitle): (.network-log-grid .icon): (.network-log-grid.small .icon): (.network-log-grid .network-category-scripts .icon): (.network-log-grid.small .network-category-scripts .icon): (.network-log-grid .network-category-documents .icon): (.network-log-grid.small .network-category-documents .icon): (.network-log-grid .network-category-stylesheets .icon): (.network-log-grid.small .network-category-stylesheets .icon): (.network-log-grid .network-category-images .icon): (.network-log-grid.small .network-category-images .icon): (.network-log-grid .image-network-icon-preview): (.network-log-grid.small .image-network-icon-preview): (.network-timeline-grid.small .network-event-divider): (.network-log-grid .resources-dividers): (.network-log-grid .resources-dividers-label-bar): (.network-timeline-grid.small .resources-dividers-label-bar): (.network-timeline-grid .resources-divider-label): (.network-timeline-grid .resources-dividers-label-bar .resources-divider): (.network-timeline-grid.small .resources-dividers-label-bar .resources-divider): (.network-timeline-grid .resources-divider.first .resources-divider-label): (.network-timeline-grid .resources-dividers-label-bar .resources-divider.first): (.network-timeline-grid tr.filler td): (.network-summary-bar td): (.network.panel.viewing-resource .network-log-grid th): (.network-log-grid.full-grid-mode .viewer-column): (.network-log-grid.brief-grid-mode .time-column): (.network-log-grid.viewing-resource-mode .timeline-column): (.network.panel .sidebar): (.network.panel.viewing-resource .network-log-grid .data-grid-resizer):
5:49 AM Changeset in webkit [90392] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-07-05 Alexandru Chiculita <Alexandru Chiculita>

Reviewed by Simon Hausmann.

[Qt] "cat" is not available when building.
https://bugs.webkit.org/show_bug.cgi?id=63924

Changed 'cat' to 'perl -ne \"print $1\"' because 'cat' is not available on Windows with
MSVC toolchain. It passed the Qt buildbots, because they run with cygwin.

Project file change. No tests needed.

4:12 AM Changeset in webkit [90391] by caseq@chromium.org
  • 1 edit
    2 deletes in trunk/LayoutTests

2011-07-05 Andrey Kosyakov <caseq@chromium.org>

Unreviewed. Removed custom chromium-linux baselines for
navigator-detached-no-crash.html & register-protocol-handler.html

  • platform/chromium-linux/fast/dom/navigator-detached-no-crash-expected.txt: Removed.
  • platform/chromium-linux/fast/dom/register-protocol-handler-expected.txt: Removed.
3:36 AM Changeset in webkit [90390] by vsevik@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-07-05 Vsevolod Vlasov <vsevik@chromium.org>

Unreviewed; Qt Linux Release minimal compile fix.

  • testing/Internals.h: (WebCore::Internals::setInspectorResourcesDataSizeLimits):
  • testing/Internals.idl:
3:16 AM Changeset in webkit [90389] by vsevik@chromium.org
  • 33 edits
    9 adds in trunk

2011-07-05 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Show content for plugin requests in network panel.
https://bugs.webkit.org/show_bug.cgi?id=30080

Added GTK symbols for new window.internals methods.

Reviewed by Pavel Feldman.

  • Source/autotools/symbols.filter:

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Show content for plugin requests in network panel.
https://bugs.webkit.org/show_bug.cgi?id=30080

Added tests for plugin content saving and replacing in NetworkResourcesData.
Adjusted older tests to use new window.internals method setInspectorResourcesDataSizeLimits
to ensure NetworkResourcesData is in the correct state before test.

Reviewed by Pavel Feldman.

  • http/tests/inspector/network/network-content-replacement-embed-expected.txt: Added.
  • http/tests/inspector/network/network-content-replacement-embed.html: Added.
  • http/tests/inspector/network/network-content-replacement-xhr-expected.txt: Added.
  • http/tests/inspector/network/network-content-replacement-xhr.html: Added.
  • http/tests/inspector/network/network-embed-expected.txt: Added.
  • http/tests/inspector/network/network-embed.html: Added.
  • http/tests/inspector/network/network-xhr-async-expected.txt:
  • http/tests/inspector/network/network-xhr-async.html:
  • http/tests/inspector/network/network-xhr-sync-expected.txt:
  • http/tests/inspector/network/network-xhr-sync.html:
  • http/tests/inspector/network/resources/cp1251.xml: Added.
  • http/tests/inspector/network/resources/plugin-data.php: Added.
  • http/tests/inspector/network/resources/resource.php:
  • http/tests/inspector/network/resources/utf8.xml: Added.
  • platform/chromium/test_expectations.txt:

2011-07-05 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Show content for plugin requests in network panel.
https://bugs.webkit.org/show_bug.cgi?id=30080

Now the data from didReceiveData callback is passed to inspector.
It is then saved in NetworkResourcesData structure for plugin requests.
Added window.internals.setInspectorResourcesDataSizeLimits() method
for testing replacement logic.

Reviewed by Pavel Feldman.

Tests: http/tests/inspector/network/network-content-replacement-embed.html

http/tests/inspector/network/network-content-replacement-xhr.html
http/tests/inspector/network/network-embed.html

  • WebCore.exp.in:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::setResourcesDataSizeLimitsFromInternals):
  • inspector/InspectorController.h:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::didReceiveDataImpl):
  • inspector/InspectorInstrumentation.h: (WebCore::InspectorInstrumentation::didReceiveData):
  • inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::didReceiveResponse): (WebCore::InspectorResourceAgent::didReceiveData): (WebCore::InspectorResourceAgent::didFinishLoading): (WebCore::InspectorResourceAgent::setInitialXHRContent): (WebCore::InspectorResourceAgent::setResourcesDataSizeLimitsFromInternals): (WebCore::InspectorResourceAgent::getResourceContent):
  • inspector/InspectorResourceAgent.h:
  • inspector/NetworkResourcesData.cpp: (WebCore::NetworkResourcesData::ResourceData::ResourceData): (WebCore::NetworkResourcesData::ResourceData::setContent): (WebCore::NetworkResourcesData::ResourceData::purgeContent): (WebCore::NetworkResourcesData::ResourceData::createDecoder): (WebCore::NetworkResourcesData::ResourceData::dataLength): (WebCore::NetworkResourcesData::ResourceData::appendData): (WebCore::NetworkResourcesData::ResourceData::decodeDataToContent): (WebCore::NetworkResourcesData::NetworkResourcesData): (WebCore::NetworkResourcesData::responseReceived): (WebCore::NetworkResourcesData::setResourceContent): (WebCore::NetworkResourcesData::maybeAddResourceData): (WebCore::NetworkResourcesData::maybeDecodeDataToContent): (WebCore::NetworkResourcesData::data): (WebCore::NetworkResourcesData::clear): (WebCore::NetworkResourcesData::setResourcesDataSizeLimits): (WebCore::NetworkResourcesData::ensureNoDataForIdentifier): (WebCore::NetworkResourcesData::ensureFreeSpace):
  • inspector/NetworkResourcesData.h: (WebCore::NetworkResourcesData::ResourceData::hasContent): (WebCore::NetworkResourcesData::ResourceData::content): (WebCore::NetworkResourcesData::ResourceData::decoder): (WebCore::NetworkResourcesData::ResourceData::buffer): (WebCore::NetworkResourcesData::ResourceData::setBuffer): (WebCore::NetworkResourcesData::ResourceData::hasData):
  • inspector/front-end/Resource.js: (WebInspector.Resource.prototype.get contentEncoded):
  • inspector/front-end/ResourcePreviewView.js: (WebInspector.ResourcePreviewView.prototype.contentLoaded): (WebInspector.ResourcePreviewView.prototype._createEmptyView): (WebInspector.ResourcePreviewView.prototype._createPreviewView):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.hasTextContent):
  • loader/DocumentThreadableLoader.cpp: (WebCore::DocumentThreadableLoader::didReceiveData):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::willLoadMediaElementURL): (WebCore::FrameLoader::commitProvisionalLoad): (WebCore::FrameLoader::loadResourceSynchronously): (WebCore::FrameLoader::loadedResourceFromMemoryCache):
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::didReceiveData): (WebCore::ResourceLoadNotifier::dispatchDidReceiveData): (WebCore::ResourceLoadNotifier::sendRemainingDelegateMessages):
  • loader/ResourceLoadNotifier.h:
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didReceiveData):
  • testing/Internals.cpp: (WebCore::Internals::setInspectorResourcesDataSizeLimits):
  • testing/Internals.h:
  • testing/Internals.idl:

2011-07-05 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Show content for plugin requests in network panel.
https://bugs.webkit.org/show_bug.cgi?id=30080

Added win symbols for new window.internals methods.

Reviewed by Pavel Feldman.

  • win/WebKit2.def:
  • win/WebKit2CFLite.def:
2:31 AM Changeset in webkit [90388] by tkent@chromium.org
  • 4 edits in trunk/Source/WebCore

2011-07-05 Kent Tamura <tkent@chromium.org>

Add an assertion that percent height box is unregistered correctly.
https://bugs.webkit.org/show_bug.cgi?id=62792

Reviewed by Hajime Morita.

No new tests. This won't change behavior.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::hasPercentHeightDescendant): Added.
  • rendering/RenderBlock.h: Declare hasPercentHeightDescendant().
  • rendering/RenderBox.cpp: (WebCore::RenderBox::destroy): Assert !hasPercentHeightDescendant().
1:44 AM Changeset in webkit [90387] by pfeldman@chromium.org
  • 9 edits in trunk/Source/WebCore

2011-07-04 Pavel Feldman <pfeldman@google.com>

Web Inspector: Enable settings screen with settings for console and elements panels.
https://bugs.webkit.org/show_bug.cgi?id=62853

Reviewed by Yury Semikhatsky.

  • English.lproj/localizedStrings.js:
  • inspector/front-end/HelpScreen.js: (WebInspector.HelpScreen.prototype.hide):
  • inspector/front-end/SettingsScreen.js: (WebInspector.SettingsScreen): (WebInspector.SettingsScreen.prototype._createRadioSetting.clickListener): (WebInspector.SettingsScreen.prototype._createRadioSetting.get for):
  • inspector/front-end/ShortcutsScreen.js: (WebInspector.ShortcutsScreen.prototype.hide):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype._colorFormatSettingChanged): (WebInspector.StylesSidebarPane.prototype._updateColorFormatFilter.get var): (WebInspector.StylesSidebarPane.prototype._updateColorFormatFilter): (WebInspector.StylesSidebarPane.prototype._changeSetting): (WebInspector.StylePropertyTreeElement.prototype.updateTitle.):
  • inspector/front-end/inspector.css: (#counters):
  • inspector/front-end/inspector.js: (WebInspector._createGlobalStatusBarItems): (WebInspector._toggleSettings): (WebInspector._showShortcutsScreen): (WebInspector._hideShortcutsScreen): (WebInspector._showSettingsScreen.onhide): (WebInspector._showSettingsScreen): (WebInspector._hideSettingsScreen): (WebInspector.documentKeyDown):
1:35 AM Changeset in webkit [90386] by kinuko@chromium.org
  • 2 edits in trunk/LayoutTests

2011-07-04 Kinuko Yasuda <kinuko@chromium.org>

[Chromium] LayoutTests/fast/filesystem/workers/simple-persistent-sync.html is failing on chromium worker tests
https://bugs.webkit.org/show_bug.cgi?id=63906

The expected text file needs to be updated to reflect the recent API change for adding the prefix vendor (i.e. requestFileSystem -> webkitRequestFileSystem).

Unreviewed; chromium test expectations change.

  • fast/filesystem/workers/simple-persistent-sync-expected.txt:
12:18 AM Changeset in webkit [90385] by commit-queue@webkit.org
  • 6 edits in trunk

2011-07-05 Shinya Kawanaka <shinyak@google.com>

Reviewed by Kent Tamura.

The default step is used when the step attribute of input[type='number'] is "any".
https://bugs.webkit.org/show_bug.cgi?id=57723

Changed Layout tests to use the default step when the step attribute of
input[type='number'] is "any".

  • fast/forms/input-stepup-stepdown-from-renderer-expected.txt: Regenerated.
  • fast/forms/script-tests/input-stepup-stepdown-from-renderer.js:

2011-07-05 Shinya Kawanaka <shinyak@google.com>

Reviewed by Kent Tamura.

The default step is used when attribute step of input[type='number'] is "any".
https://bugs.webkit.org/show_bug.cgi?id=57723

When step is "any", 0 was chosen as step, but this is not intuitive.
So changed to use the default step.

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::getAllowedValueStep): Calls getAllowedValueStepWithDecimalPlaces with RejectAny. (WebCore::HTMLInputElement::getAllowedValueStepWithDecimalPlaces): Takes a new arugument AnyStepHandling. If it is AnyIsDefaultStep, this method allows "step" attribute to take a value "any" to use the default step as step value. Otherwise, INVALID_STATE_ERR will be returned if "step" is "any". (WebCore::HTMLInputElement::applyStep): ditto. (WebCore::HTMLInputElement::stepUp): Calls applyStep with RejectAny. (WebCore::HTMLInputElement::stepDown): ditto. (WebCore::HTMLInputElement::stepUpFromRenderer): Removes a check that step is "any". It is checked in getAllowedValueStepWithDecimalPlaces.
  • html/HTMLInputElement.h:

Jul 4, 2011:

9:21 PM Changeset in webkit [90384] by dominicc@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-07-04 Dominic Cooney <dominicc@chromium.org>

Simplify <progress> element's isDeterminate logic.
https://bugs.webkit.org/show_bug.cgi?id=63889

Reviewed by Hajime Morita.

Covered by existing tests
fast/dom/HTMLProgressElement/indeterminate-progress-*.html

  • html/HTMLProgressElement.cpp: (WebCore::HTMLProgressElement::position): don't duplicate isDeterminate (WebCore::HTMLProgressElement::isDeterminate): remove redundant tests (WebCore::HTMLProgressElement::didElementStateChange): fix formatting
  • html/HTMLProgressElement.h:
7:59 PM Changeset in webkit [90383] by barraclough@apple.com
  • 7 edits
    3 adds in trunk

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=16652
Firefox and JavaScriptCore differ in Number.toString(integer)

Source/JavaScriptCore:

Our arbitrary radix (2..36) toString conversion is inaccurate.
This is partly because it uses doubles to perform math that requires
higher accuracy, and partly becasue it does not attempt to correctly
detect where to terminate, instead relying on a simple 'epsilon'.

  • runtime/NumberPrototype.cpp:

(JSC::decomposeDouble):

  • helper function to extract sign, exponent, mantissa from IEEE doubles.

(JSC::Uint16WithFraction::Uint16WithFraction):

  • helper class, u16int with infinite precision fraction, used to convert the fractional part of the number to a string.

(JSC::Uint16WithFraction::operator*=):

  • Multiply by a uint16.

(JSC::Uint16WithFraction::operator<):

  • Compare two Uint16WithFractions.

(JSC::Uint16WithFraction::floorAndSubtract):

  • Extract the integer portion of the number, and subtract it (clears the integer portion).

(JSC::Uint16WithFraction::comparePoint5):

  • Compare to 0.5.

(JSC::Uint16WithFraction::sumGreaterThanOne):

  • Passed a second Uint16WithFraction, returns true if the result of adding the two values would be greater than one.

(JSC::Uint16WithFraction::isNormalized):

  • Used by ASSERTs to consistency check internal representation.

(JSC::BigInteger::BigInteger):

  • helper class, unbounded integer value, used to convert the integer part of the number to a string.

(JSC::BigInteger::divide):

  • Divide this value through by a uint32.

(JSC::BigInteger::operator!):

  • test for zero.

(JSC::toStringWithRadix):

  • Performs number to string conversion, with the given radix (2..36).

(JSC::numberProtoFuncToString):

  • Changed to use toStringWithRadix.

Tools:

Added forwarding header.

  • DumpRenderTree/ForwardingHeaders/wtf/StdLibExtras.h: Added.

LayoutTests:

Our arbitrary radix (2..36) toString conversion is inaccurate.
This is partly because it uses doubles to perform math that requires
higher accuracy, and partly becasue it does not attempt to correctly
detect where to terminate, instead relying on a simple 'epsilon'.

  • fast/js/number-toString-expected.txt:
    • Update expected results from FAIL to PASS.
7:19 PM Changeset in webkit [90382] by gyuyoung.kim@samsung.com
  • 2 edits in trunk/Source/WebCore

2011-07-04 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Reviewed by Kent Tamura.

[EFL] Add hasOwnDisabledStateHandlingFor() to RenderThemeEfl
https://bugs.webkit.org/show_bug.cgi?id=63884

As MediaControlRootElement decides whether it will display timeline
according to the return value of RenderTheme's hasOwnDisabledStateHandlingFor(),
RenderThemeEfl has to implement this function.

Existing tests cover the visibility of the controls.

  • platform/efl/RenderThemeEfl.h: (WebCore::RenderThemeEfl::hasOwnDisabledStateHandlingFor):
7:18 PM Changeset in webkit [90381] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2011-07-04 Mihai Parparita <mihaip@chromium.org>

As of r90372 there are WebKit2 Mac-specific tests; don't expect those to
pass in Chromium, just like other platform-specific tests.

  • platform/chromium/test_expectations.txt:
6:54 PM Changeset in webkit [90380] by mitz@apple.com
  • 3 edits in trunk/Source/WebKit/mac

Moved handling of frame size changes into its own method.

Reviewed by Oliver Hunt.

  • WebView/WebFrameView.mm:

(-[WebFrameView _frameSizeChanged]): Added. Moved code here...
(-[WebFrameView setFrameSize:]): ...from here.

  • WebView/WebFrameViewInternal.h:
6:27 PM Changeset in webkit [90379] by tkent@chromium.org
  • 5 edits
    3 adds in trunk

2011-07-04 Kent Tamura <tkent@chromium.org>

REGRESSION (r87067): Text overflows from short height text field.
https://bugs.webkit.org/show_bug.cgi?id=61768

Reviewed by Dimitri Glazkov.

  • fast/forms/textfield-overflow-by-value-update.html: Added.
  • platform/chromium/test_expectations.txt: Add textfield-overflow-by-value-update.html.
  • platform/mac/fast/forms/textfield-overflow-by-value-update-expected.png: Added.
  • platform/mac/fast/forms/textfield-overflow-by-value-update-expected.txt: Added.

2011-07-04 Kent Tamura <tkent@chromium.org>

REGRESSION (r87067): Text overflows from short height text field.
https://bugs.webkit.org/show_bug.cgi?id=61768

Reviewed by Dimitri Glazkov.

Since r87067, the RenderStyle for the inner text element is reset
by the value update, but the renderer for the inner text element
keeps to have metrics calculated by the previous style. So
RenderTextControlSingleLine::layout() didn't set a desired height
to the renderer for the inner text element.

This patch changes the style creation code so that it has the
desired height initially.

Test: fast/forms/textfield-overflow-by-value-update.html

  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::RenderTextControlSingleLine): Initialize m_desiredInnerTextHeight. (WebCore::RenderTextControlSingleLine::layout): Update m_desiredInnerTextHeight. (WebCore::RenderTextControlSingleLine::styleDidChange): Reset m_desiredInnerTextHeight. (WebCore::RenderTextControlSingleLine::createInnerTextStyle): Sets the height if m_desiredInnerTextHeight has a concrete value.
  • rendering/RenderTextControlSingleLine.h: Declare m_desiredInnerTextHeight.
6:11 PM Changeset in webkit [90378] by morrita@google.com
  • 5 edits
    2 adds in trunk

2011-07-04 MORITA Hajime <morrita@google.com>

Accessing ShadowRoot.nodeList causes an assertion failure.
http://webkit.org/b/63798

Reviewed by Dimitri Glazkov.

  • fast/dom/shadow/shadow-root-node-list-expected.txt: Added.
  • fast/dom/shadow/shadow-root-node-list.html: Added.

2011-07-04 MORITA Hajime <morrita@google.com>

Accessing ShadowRoot.nodeList causes an assertion failure.
http://webkit.org/b/63798

The TreeScope destructor did clear the reference to TreeScope,
then the Node destructor accessing treeScope(), which results
an inconsistent state during the NodeList cache cleanup.

This change clears NodeList cache during the TreeScope destructor.
The Node destructor no longer does problematic NodeList cache
manipulation.

Test: fast/dom/shadow/shadow-root-node-list.html

Reviewed by Dimitri Glazkov.

  • dom/Node.cpp: (WebCore::Node::~Node): (WebCore::Node::clearRareData):
  • dom/Node.h:
  • dom/TreeScope.cpp: (WebCore::TreeScope::~TreeScope):
2:30 PM QtWebKitReleases edited by Ademar Reis
(diff)
2:22 PM QtWebKitRelease22 edited by Ademar Reis
(diff)
1:29 PM Changeset in webkit [90377] by senorblanco@chromium.org
  • 2 edits in trunk/LayoutTests

2011-07-04 Stephen White <senorblanco@chromium.org>

Unreviewed; chromium test expectations change.

Mark four tests as passing on the GPU step.

  • platform/chromium/test_expectations.txt:
1:28 PM Changeset in webkit [90376] by vsevik@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

ResourceLoadNotifier::dispatchWillSendRequest should not compare StringImpl directly
https://bugs.webkit.org/show_bug.cgi?id=63873

Reviewed by Alexey Proskuryakov.

  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::dispatchWillSendRequest):
1:26 PM Changeset in webkit [90375] by commit-queue@webkit.org
  • 33 edits
    9 deletes in trunk

2011-07-04 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r90373.
http://trac.webkit.org/changeset/90373
https://bugs.webkit.org/show_bug.cgi?id=63917

Breaks Qt Linux Release minimal (Requested by vsevik on
#webkit).

  • Source/autotools/symbols.filter:

2011-07-04 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r90373.
http://trac.webkit.org/changeset/90373
https://bugs.webkit.org/show_bug.cgi?id=63917

Breaks Qt Linux Release minimal (Requested by vsevik on
#webkit).

  • http/tests/inspector/network/network-content-replacement-embed-expected.txt: Removed.
  • http/tests/inspector/network/network-content-replacement-embed.html: Removed.
  • http/tests/inspector/network/network-content-replacement-xhr-expected.txt: Removed.
  • http/tests/inspector/network/network-content-replacement-xhr.html: Removed.
  • http/tests/inspector/network/network-embed-expected.txt: Removed.
  • http/tests/inspector/network/network-embed.html: Removed.
  • http/tests/inspector/network/network-xhr-async-expected.txt:
  • http/tests/inspector/network/network-xhr-async.html:
  • http/tests/inspector/network/network-xhr-sync-expected.txt:
  • http/tests/inspector/network/network-xhr-sync.html:
  • http/tests/inspector/network/resources/cp1251.xml: Removed.
  • http/tests/inspector/network/resources/plugin-data.php: Removed.
  • http/tests/inspector/network/resources/resource.php:
  • http/tests/inspector/network/resources/utf8.xml: Removed.
  • platform/chromium/test_expectations.txt:

2011-07-04 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r90373.
http://trac.webkit.org/changeset/90373
https://bugs.webkit.org/show_bug.cgi?id=63917

Breaks Qt Linux Release minimal (Requested by vsevik on
#webkit).

  • WebCore.exp.in:
  • inspector/InspectorController.cpp:
  • inspector/InspectorController.h:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::didReceiveContentLengthImpl):
  • inspector/InspectorInstrumentation.h: (WebCore::InspectorInstrumentation::didReceiveContentLength):
  • inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::didReceiveResponse): (WebCore::InspectorResourceAgent::didReceiveContentLength): (WebCore::InspectorResourceAgent::didFinishLoading): (WebCore::InspectorResourceAgent::setInitialXHRContent): (WebCore::InspectorResourceAgent::getResourceContent):
  • inspector/InspectorResourceAgent.h:
  • inspector/NetworkResourcesData.cpp: (WebCore::NetworkResourcesData::ResourceData::ResourceData): (WebCore::NetworkResourcesData::ResourceData::content): (WebCore::NetworkResourcesData::ResourceData::appendContent): (WebCore::NetworkResourcesData::ResourceData::purgeContent): (WebCore::NetworkResourcesData::NetworkResourcesData): (WebCore::NetworkResourcesData::responseReceived): (WebCore::NetworkResourcesData::addResourceContent): (WebCore::NetworkResourcesData::data): (WebCore::NetworkResourcesData::clear): (WebCore::NetworkResourcesData::ensureNoDataForIdentifier): (WebCore::NetworkResourcesData::ensureFreeSpace):
  • inspector/NetworkResourcesData.h: (WebCore::NetworkResourcesData::ResourceData::hasContent): (WebCore::NetworkResourcesData::ResourceData::setIsContentPurged): (WebCore::NetworkResourcesData::ResourceData::textEncodingName): (WebCore::NetworkResourcesData::ResourceData::setTextEncodingName):
  • inspector/front-end/Resource.js:
  • inspector/front-end/ResourcePreviewView.js: (WebInspector.ResourcePreviewView.prototype.contentLoaded): (WebInspector.ResourcePreviewView.prototype._createInnerView):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.hasTextContent):
  • loader/DocumentThreadableLoader.cpp: (WebCore::DocumentThreadableLoader::didReceiveData):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::willLoadMediaElementURL): (WebCore::FrameLoader::commitProvisionalLoad): (WebCore::FrameLoader::loadResourceSynchronously): (WebCore::FrameLoader::loadedResourceFromMemoryCache):
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::didReceiveData): (WebCore::ResourceLoadNotifier::dispatchDidReceiveContentLength): (WebCore::ResourceLoadNotifier::sendRemainingDelegateMessages):
  • loader/ResourceLoadNotifier.h:
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didReceiveData):
  • testing/Internals.cpp:
  • testing/Internals.h:
  • testing/Internals.idl:

2011-07-04 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r90373.
http://trac.webkit.org/changeset/90373
https://bugs.webkit.org/show_bug.cgi?id=63917

Breaks Qt Linux Release minimal (Requested by vsevik on
#webkit).

  • win/WebKit2.def:
  • win/WebKit2CFLite.def:
1:17 PM Changeset in webkit [90374] by andersca@apple.com
  • 2 edits in trunk/Tools

Try to fix Qt build.

  • DumpRenderTree/TestNetscapePlugIn/Tests/NPRuntimeRemoveProperty.cpp:
1:07 PM Changeset in webkit [90373] by vsevik@chromium.org
  • 33 edits
    9 adds in trunk

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Show content for plugin requests in network panel.
https://bugs.webkit.org/show_bug.cgi?id=30080

Reviewed by Pavel Feldman.

  • Source/autotools/symbols.filter:

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Show content for plugin requests in network panel.
https://bugs.webkit.org/show_bug.cgi?id=30080

Reviewed by Pavel Feldman.

  • http/tests/inspector/network/network-content-replacement-embed-expected.txt: Added.
  • http/tests/inspector/network/network-content-replacement-embed.html: Added.
  • http/tests/inspector/network/network-content-replacement-xhr-expected.txt: Added.
  • http/tests/inspector/network/network-content-replacement-xhr.html: Added.
  • http/tests/inspector/network/network-embed-expected.txt: Added.
  • http/tests/inspector/network/network-embed.html: Added.
  • http/tests/inspector/network/network-xhr-async-expected.txt:
  • http/tests/inspector/network/network-xhr-async.html:
  • http/tests/inspector/network/network-xhr-sync-expected.txt:
  • http/tests/inspector/network/network-xhr-sync.html:
  • http/tests/inspector/network/resources/cp1251.xml: Added.
  • http/tests/inspector/network/resources/plugin-data.php: Added.
  • http/tests/inspector/network/resources/resource.php:
  • http/tests/inspector/network/resources/utf8.xml: Added.
  • platform/chromium/test_expectations.txt:

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Show content for plugin requests in network panel.
https://bugs.webkit.org/show_bug.cgi?id=30080

Reviewed by Pavel Feldman.

Tests: http/tests/inspector/network/network-content-replacement-embed.html

http/tests/inspector/network/network-content-replacement-xhr.html
http/tests/inspector/network/network-embed.html

  • WebCore.exp.in:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::setResourcesDataSizeLimitsFromInternals):
  • inspector/InspectorController.h:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::didReceiveDataImpl):
  • inspector/InspectorInstrumentation.h: (WebCore::InspectorInstrumentation::didReceiveData):
  • inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::didReceiveResponse): (WebCore::InspectorResourceAgent::didReceiveData): (WebCore::InspectorResourceAgent::didFinishLoading): (WebCore::InspectorResourceAgent::setInitialXHRContent): (WebCore::InspectorResourceAgent::setResourcesDataSizeLimitsFromInternals): (WebCore::InspectorResourceAgent::getResourceContent):
  • inspector/InspectorResourceAgent.h:
  • inspector/NetworkResourcesData.cpp: (WebCore::NetworkResourcesData::ResourceData::ResourceData): (WebCore::NetworkResourcesData::ResourceData::setContent): (WebCore::NetworkResourcesData::ResourceData::purgeContent): (WebCore::NetworkResourcesData::ResourceData::createDecoder): (WebCore::NetworkResourcesData::ResourceData::dataLength): (WebCore::NetworkResourcesData::ResourceData::appendData): (WebCore::NetworkResourcesData::ResourceData::decodeDataToContent): (WebCore::NetworkResourcesData::NetworkResourcesData): (WebCore::NetworkResourcesData::responseReceived): (WebCore::NetworkResourcesData::setResourceContent): (WebCore::NetworkResourcesData::maybeAddResourceData): (WebCore::NetworkResourcesData::maybeDecodeDataToContent): (WebCore::NetworkResourcesData::data): (WebCore::NetworkResourcesData::clear): (WebCore::NetworkResourcesData::setResourcesDataSizeLimits): (WebCore::NetworkResourcesData::ensureNoDataForIdentifier): (WebCore::NetworkResourcesData::ensureFreeSpace):
  • inspector/NetworkResourcesData.h: (WebCore::NetworkResourcesData::ResourceData::hasContent): (WebCore::NetworkResourcesData::ResourceData::content): (WebCore::NetworkResourcesData::ResourceData::decoder): (WebCore::NetworkResourcesData::ResourceData::buffer): (WebCore::NetworkResourcesData::ResourceData::setBuffer): (WebCore::NetworkResourcesData::ResourceData::hasData):
  • inspector/front-end/Resource.js: (WebInspector.Resource.prototype.get contentEncoded):
  • inspector/front-end/ResourcePreviewView.js: (WebInspector.ResourcePreviewView.prototype.contentLoaded): (WebInspector.ResourcePreviewView.prototype._createEmptyView): (WebInspector.ResourcePreviewView.prototype._createPreviewView):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.hasTextContent):
  • loader/DocumentThreadableLoader.cpp: (WebCore::DocumentThreadableLoader::didReceiveData):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::willLoadMediaElementURL): (WebCore::FrameLoader::commitProvisionalLoad): (WebCore::FrameLoader::loadResourceSynchronously): (WebCore::FrameLoader::loadedResourceFromMemoryCache):
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::didReceiveData): (WebCore::ResourceLoadNotifier::dispatchDidReceiveData): (WebCore::ResourceLoadNotifier::sendRemainingDelegateMessages):
  • loader/ResourceLoadNotifier.h:
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didReceiveData):
  • testing/Internals.cpp: (WebCore::Internals::setInspectorResourcesDataSizeLimits):
  • testing/Internals.h:
  • testing/Internals.idl:

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Show content for plugin requests in network panel.
https://bugs.webkit.org/show_bug.cgi?id=30080

Reviewed by Pavel Feldman.

  • win/WebKit2.def:
  • win/WebKit2CFLite.def:
12:58 PM Changeset in webkit [90372] by andersca@apple.com
  • 8 edits
    2 adds in trunk

2011-07-04 Anders Carlsson <andersca@apple.com>

NP_RemoveProperty is not called back by Safari when delete npObject.prop is encountered in JavaScript
https://bugs.webkit.org/show_bug.cgi?id=63915
<rdar://problem/7124300>

Reviewed by Sam Weinig.

  • WebProcess/Plugins/Netscape/JSNPObject.cpp: (WebKit::JSNPObject::deleteProperty): Call NP_RemoveProperty on the NPObject.
  • WebProcess/Plugins/Netscape/JSNPObject.h: Add deleteProperty.

2011-07-04 Anders Carlsson <andersca@apple.com>

NP_RemoveProperty is not called back by Safari when delete npObject.prop is encountered in JavaScript
https://bugs.webkit.org/show_bug.cgi?id=63915
<rdar://problem/7124300>

Reviewed by Sam Weinig.

Add a test. I made this Mac-WebKit2 specific for now since no other plug-in implementations support this,
and there doesn't seem to be a way to make a WebKit2 specific test.

  • platform/mac-wk2/plugins/npruntime/remove-property-from-javascript-expected.txt: Added.
  • platform/mac-wk2/plugins/npruntime/remove-property-from-javascript.html: Added.

2011-07-04 Anders Carlsson <andersca@apple.com>

NP_RemoveProperty is not called back by Safari when delete npObject.prop is encountered in JavaScript
https://bugs.webkit.org/show_bug.cgi?id=63915
<rdar://problem/7124300>

Reviewed by Sam Weinig.

Extend the NPRuntimeRemoveProperty to handle delete object.property from JavaScript.

  • DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp: (PluginTest::NPN_IdentifierIsString): (PluginTest::NPN_UTF8FromIdentifier): (PluginTest::NPN_IntFromIdentifier): (PluginTest::NPN_RetainObject): (PluginTest::NPN_ReleaseObject):
  • DumpRenderTree/TestNetscapePlugIn/PluginTest.h: (PluginTest::Object::removeProperty): (PluginTest::Object::identifierIs): (PluginTest::Object::NP_RemoveProperty): (PluginTest::Object::npClass):
  • DumpRenderTree/TestNetscapePlugIn/Tests/NPRuntimeRemoveProperty.cpp: (NPRuntimeRemoveProperty::TestObject::TestObject): (NPRuntimeRemoveProperty::TestObject::hasProperty): (NPRuntimeRemoveProperty::TestObject::getProperty): (NPRuntimeRemoveProperty::TestObject::removeProperty): (NPRuntimeRemoveProperty::PluginObject::PluginObject): (NPRuntimeRemoveProperty::PluginObject::~PluginObject): (NPRuntimeRemoveProperty::PluginObject::hasMethod): (NPRuntimeRemoveProperty::PluginObject::invoke): (NPRuntimeRemoveProperty::PluginObject::hasProperty): (NPRuntimeRemoveProperty::PluginObject::getProperty): (NPRuntimeRemoveProperty::NPP_GetValue):
12:26 PM Changeset in webkit [90371] by barraclough@apple.com
  • 22 edits in trunk/Source/JavaScriptCore

https://bugs.webkit.org/show_bug.cgi?id=63881
Need separate bytecodes for handling >, >= comparisons.

Reviewed by Oliver Hunt.

This clears the way to fix Bug#63880. We currently handle greater-than comparisons
as being using the corresponding op_less, etc opcodes. This is incorrect with
respect to evaluation ordering of the implicit conversions performed on operands -
we should be calling ToPrimitive on the LHS and RHS operands to the greater than,
but instead convert RHS then LHS.

This patch adds opcodes for greater-than comparisons mirroring existing ones used
for less-than.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::dump):

  • bytecode/Opcode.h:
  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitJumpIfTrue):
(JSC::BytecodeGenerator::emitJumpIfFalse):

  • bytecompiler/NodesCodegen.cpp:
  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGNode.h:
  • dfg/DFGNonSpeculativeJIT.cpp:

(JSC::DFG::NonSpeculativeJIT::compare):
(JSC::DFG::NonSpeculativeJIT::compile):

  • dfg/DFGNonSpeculativeJIT.h:
  • dfg/DFGOperations.cpp:
  • dfg/DFGOperations.h:
  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compare):
(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT.h:
  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

  • jit/JIT.cpp:

(JSC::JIT::privateCompileMainPass):
(JSC::JIT::privateCompileSlowCases):

  • jit/JIT.h:

(JSC::JIT::emit_op_loop_if_greater):
(JSC::JIT::emitSlow_op_loop_if_greater):
(JSC::JIT::emit_op_loop_if_greatereq):
(JSC::JIT::emitSlow_op_loop_if_greatereq):

  • jit/JITArithmetic.cpp:

(JSC::JIT::emit_op_jgreater):
(JSC::JIT::emit_op_jgreatereq):
(JSC::JIT::emit_op_jngreater):
(JSC::JIT::emit_op_jngreatereq):
(JSC::JIT::emitSlow_op_jgreater):
(JSC::JIT::emitSlow_op_jgreatereq):
(JSC::JIT::emitSlow_op_jngreater):
(JSC::JIT::emitSlow_op_jngreatereq):
(JSC::JIT::emit_compareAndJumpSlow):

  • jit/JITArithmetic32_64.cpp:

(JSC::JIT::emitBinaryDoubleOp):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • jit/JITStubs.h:
  • parser/NodeConstructors.h:

(JSC::GreaterNode::GreaterNode):
(JSC::GreaterEqNode::GreaterEqNode):

  • parser/Nodes.h:
11:25 AM Changeset in webkit [90370] by senorblanco@chromium.org
  • 2 edits in trunk/LayoutTests

2011-07-04 Stephen White <senorblanco@chromium.org>

Unreviewed; chromium test expectations change.


Mark two tests as failing on the GPU step.

  • platform/chromium/test_expectations.txt:
11:16 AM Changeset in webkit [90369] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-07-04 Jeff Timanus <twiz@chromium.org>

Reviewed by Stephen White.

Prevent calls to GrapicsContext3D::deleteTexture with id 0 in DrawingBuffer::clear().
https://bugs.webkit.org/show_bug.cgi?id=63756

  • platform/graphics/gpu/DrawingBuffer.cpp: (WebCore::DrawingBuffer::clear):
11:03 AM Changeset in webkit [90368] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-07-04 Martin Robinson <mrobinson@igalia.com>

Reviewed by Andreas Kling.

[GTK] r90148 did not apply to X11 platforms in WebKit1
https://bugs.webkit.org/show_bug.cgi?id=63862

Correct the guard for an X11 specific piece of code. TARGET(X11) not
defined for the GTK+ port.

No new tests. This is covered by an existing test.

  • plugins/npapi.cpp: Correct the X11 guard. (NPN_InvalidateRect):
8:53 AM Changeset in webkit [90367] by vsevik@chromium.org
  • 4 edits in trunk/LayoutTests

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: network-preflight-options.html test is flaky.
https://bugs.webkit.org/show_bug.cgi?id=63904

Unreviewed test fix.

  • http/tests/inspector/network-preflight-options-expected.txt:
  • http/tests/inspector/network-preflight-options.html:
  • platform/qt/Skipped:
8:31 AM Changeset in webkit [90366] by abecsi@webkit.org
  • 2 edits in trunk/LayoutTests

2011-07-04 Andras Becsi <abecsi@webkit.org>

[Qt] http/tests/inspector/network-preflight-options.html fails frequently
https://bugs.webkit.org/show_bug.cgi?id=63910

Unreviewed.

  • platform/qt/Skipped: Skip http/tests/inspector/network-preflight-options.html.
7:48 AM Changeset in webkit [90365] by pfeldman@chromium.org
  • 26 edits
    1 move
    1 add in trunk

2011-07-04 Pavel Feldman <pfeldman@google.com>

Web Inspector: implement settings panel.
https://bugs.webkit.org/show_bug.cgi?id=63907

Reviewed by Yury Semikhatsky.

  • English.lproj/localizedStrings.js:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.XHRBreakpointsSidebarPane.prototype._saveBreakpoints): (WebInspector.EventListenerBreakpointsSidebarPane.prototype._saveBreakpoints):
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView): (WebInspector.ConsoleView.prototype._handleContextMenuEvent.monitoringXHRItemAction): (WebInspector.ConsoleView.prototype._handleContextMenuEvent.get preserveLogItemAction): (WebInspector.ConsoleView.prototype._monitoringXHREnabledSettingChanged): (WebInspector.ConsoleView.prototype._registerShortcuts):
  • inspector/front-end/DOMBreakpointsSidebarPane.js: (WebInspector.DOMBreakpointsSidebarPane.prototype._saveBreakpoints):
  • inspector/front-end/DebuggerPresentationModel.js: (WebInspector.DebuggerPresentationModel.prototype._saveBreakpoints):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype._contextMenuEventFired): (WebInspector.ElementsPanel.prototype._domWordWrapSettingChanged): (WebInspector.ElementsPanel.prototype._registerShortcuts):
  • inspector/front-end/EventListenersSidebarPane.js: (WebInspector.EventListenersSidebarPane.prototype): ():
  • inspector/front-end/HelpScreen.js: (WebInspector.HelpScreen): (WebInspector.HelpScreen.prototype.show): (WebInspector.HelpScreen.prototype.hide): (WebInspector.HelpScreen.prototype._onKeyDown):
  • inspector/front-end/NetworkItemView.js:
  • inspector/front-end/NetworkPanel.js:
  • inspector/front-end/Panel.js: (WebInspector.Panel.prototype.saveSidebarWidth):
  • inspector/front-end/ProfilesPanel.js:
  • inspector/front-end/ResourcesPanel.js: (WebInspector.BaseStorageTreeElement.prototype.onselect): (WebInspector.StorageCategoryTreeElement.prototype.onexpand): (WebInspector.StorageCategoryTreeElement.prototype.set oncollapse):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._sourceFileAdded.get if): (WebInspector.ScriptsPanel.prototype._setPauseOnExceptions.callback): (WebInspector.ScriptsPanel.prototype._createSidebarButtonAndRegisterShortcuts):
  • inspector/front-end/Settings.js: (WebInspector.Settings): (WebInspector.Settings.prototype.installApplicationSetting): (WebInspector.Setting): (WebInspector.Setting.prototype.addChangeListener): (WebInspector.Setting.prototype.removeChangeListener): (WebInspector.Setting.prototype.get var): (WebInspector.Setting.prototype): (WebInspector.Setting.prototype.set if):
  • inspector/front-end/SettingsScreen.js: Added. (WebInspector.SettingsScreen): (WebInspector.SettingsScreen.prototype._appendSection): (WebInspector.SettingsScreen.prototype._columnElement): (WebInspector.SettingsScreen.prototype._createCheckboxSetting.get listener): (WebInspector.SettingsScreen.prototype._createRadioSetting):
  • inspector/front-end/ShortcutsScreen.js: Renamed from Source/WebCore/inspector/front-end/ShortcutsHelp.js. (WebInspector.ShortcutsScreen):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype.registerShortcuts):
  • inspector/front-end/WatchExpressionsSidebarPane.js:
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/helpScreen.css: (.help-window-main): (body.attached .help-window-main): (.help-window-caption): (body.attached .help-window-caption): (.help-content): (.help-close-button): (body.attached .help-close-button): (body.attached .help-content): (.help-table): (body.platform-mac .help-key): (.help-combine-keys): (.help-content p): (.help-content fieldset): (.help-content fieldset p): (.help-content p.help-section): (.help-section-title): (.help-content input[type=checkbox]):
  • inspector/front-end/inspector.css: (button.settings-status-bar-item): (.settings-status-bar-item .glyph):
  • inspector/front-end/inspector.html:
  • inspector/front-end/inspector.js: (WebInspector._createGlobalStatusBarItems): (WebInspector._toggleAttach): (WebInspector._toggleSettings): (WebInspector._registerShortcuts): (WebInspector.documentKeyDown):
7:09 AM Changeset in webkit [90364] by vsevik@chromium.org
  • 5 edits
    3 adds in trunk

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Resources loaded to Resources Panel from InspectorPageAgent do not have mime type set.
https://bugs.webkit.org/show_bug.cgi?id=63701

Added a test that checks mime type when resources are loaded from cache by InspectorPageAgent.

Reviewed by Yury Semikhatsky.

  • http/tests/inspector/resource-tree/resource-tree-mimetype-expected.txt: Added.
  • http/tests/inspector/resource-tree/resource-tree-mimetype.html: Added.
  • http/tests/inspector/resource-tree/resources/empty.png: Added.

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

Web Inspector: Resources loaded to Resources Panel from InspectorPageAgent do not have mime type set.
https://bugs.webkit.org/show_bug.cgi?id=63701

Pass cachedResource/documentLoader mimeType from InspectorPageAgent to frontend.

Reviewed by Yury Semikhatsky.

Test: http/tests/inspector/resource-tree/resource-tree-mimetype.html

  • inspector/Inspector.json:
  • inspector/InspectorPageAgent.cpp: (WebCore::InspectorPageAgent::buildObjectForFrame): (WebCore::InspectorPageAgent::buildObjectForFrameTree):
  • inspector/front-end/ResourceTreeModel.js: (WebInspector.ResourceTreeModel.prototype._addFramesRecursively): (WebInspector.ResourceTreeModel.prototype._createResource):
6:29 AM Changeset in webkit [90363] by bulach@chromium.org
  • 2 edits in trunk/LayoutTests

2011-07-04 Marcus Bulach <bulach@chromium.org>

[chromium] Updates expectations for failures on svg/animations/svgtransform-animation-1.html
https://bugs.webkit.org/show_bug.cgi?id=63902

Unreviewed expectations fix.

  • platform/chromium/test_expectations.txt:
5:50 AM Changeset in webkit [90362] by yurys@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-07-04 Yury Semikhatsky <yurys@chromium.org>

Web Inspector: repeated message counter should be aligned with other message icons
https://bugs.webkit.org/show_bug.cgi?id=63901

Reviewed by Pavel Feldman.

  • inspector/front-end/inspector.css: (.console-message .bubble):
5:18 AM Changeset in webkit [90361] by commit-queue@webkit.org
  • 1 edit
    1 add in trunk/Tools

2011-07-04 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Kent Tamura.

[EFL] Add LayoutTestControllerEfl.
https://bugs.webkit.org/show_bug.cgi?id=61974

Only the core features are implemented. Most functions are actually
stubs and will be implemented as soon as its features are needed.
Build system changes will be made as soon as DRT is completely
upstreamed.

  • DumpRenderTree/efl/LayoutTestControllerEfl.cpp: Added. (LayoutTestController::~LayoutTestController): (LayoutTestController::addDisallowedURL): (LayoutTestController::clearBackForwardList): (LayoutTestController::copyDecodedHostName): (LayoutTestController::copyEncodedHostName): (LayoutTestController::dispatchPendingLoadRequests): (LayoutTestController::display): (LayoutTestController::counterValueForElementById): (LayoutTestController::keepWebHistory): (LayoutTestController::computedStyleIncludingVisitedInfo): (LayoutTestController::nodesFromRect): (LayoutTestController::layerTreeAsText): (LayoutTestController::pageNumberForElementById): (LayoutTestController::numberOfPages): (LayoutTestController::pageProperty): (LayoutTestController::isPageBoxVisible): (LayoutTestController::pageSizeAndMarginsInPixels): (LayoutTestController::webHistoryItemCount): (LayoutTestController::workerThreadCount): (LayoutTestController::notifyDone): (LayoutTestController::pathToLocalResource): (LayoutTestController::queueLoad): (LayoutTestController::setAcceptsEditing): (LayoutTestController::setAlwaysAcceptCookies): (LayoutTestController::setCustomPolicyDelegate): (LayoutTestController::waitForPolicyDelegate): (LayoutTestController::setScrollbarPolicy): (LayoutTestController::addOriginAccessWhitelistEntry): (LayoutTestController::removeOriginAccessWhitelistEntry): (LayoutTestController::setMainFrameIsFirstResponder): (LayoutTestController::setTabKeyCyclesThroughElements): (LayoutTestController::setUseDashboardCompatibilityMode): (LayoutTestController::setUserStyleSheetEnabled): (LayoutTestController::setUserStyleSheetLocation): (LayoutTestController::setValueForUser): (LayoutTestController::setViewModeMediaFeature): (LayoutTestController::setWindowIsKey): (LayoutTestController::setSmartInsertDeleteEnabled): (waitToDumpWatchdogFired): (LayoutTestController::setWaitToDump): (LayoutTestController::windowCount): (LayoutTestController::setPrivateBrowsingEnabled): (LayoutTestController::setJavaScriptCanAccessClipboard): (LayoutTestController::setXSSAuditorEnabled): (LayoutTestController::setFrameFlatteningEnabled): (LayoutTestController::setSpatialNavigationEnabled): (LayoutTestController::setAllowUniversalAccessFromFileURLs): (LayoutTestController::setAllowFileAccessFromFileURLs): (LayoutTestController::setAuthorAndUserStylesEnabled): (LayoutTestController::setAutofilled): (LayoutTestController::disableImageLoading): (LayoutTestController::setMockDeviceOrientation): (LayoutTestController::setMockGeolocationPosition): (LayoutTestController::setMockGeolocationError): (LayoutTestController::setGeolocationPermission): (LayoutTestController::numberOfPendingGeolocationPermissionRequests): (LayoutTestController::addMockSpeechInputResult): (LayoutTestController::setIconDatabaseEnabled): (LayoutTestController::setJavaScriptProfilingEnabled): (LayoutTestController::setSelectTrailingWhitespaceEnabled): (LayoutTestController::setPopupBlockingEnabled): (LayoutTestController::setPluginsEnabled): (LayoutTestController::elementDoesAutoCompleteForElementWithId): (LayoutTestController::execCommand): (LayoutTestController::findString): (LayoutTestController::isCommandEnabled): (LayoutTestController::setCacheModel): (LayoutTestController::setPersistentUserStyleSheetLocation): (LayoutTestController::clearPersistentUserStyleSheet): (LayoutTestController::clearAllApplicationCaches): (LayoutTestController::setApplicationCacheOriginQuota): (LayoutTestController::clearApplicationCacheForOrigin): (LayoutTestController::localStorageDiskUsageForOrigin): (LayoutTestController::originsWithApplicationCache): (LayoutTestController::applicationCacheDiskUsageForOrigin): (LayoutTestController::clearAllDatabases): (LayoutTestController::setDatabaseQuota): (LayoutTestController::originsWithLocalStorage): (LayoutTestController::deleteAllLocalStorage): (LayoutTestController::deleteLocalStorageForOrigin): (LayoutTestController::observeStorageTrackerNotifications): (LayoutTestController::syncLocalStorage): (LayoutTestController::setDomainRelaxationForbiddenForURLScheme): (LayoutTestController::goBack): (LayoutTestController::setDefersLoading): (LayoutTestController::setAppCacheMaximumSize): (LayoutTestController::pauseAnimationAtTimeOnElementWithId): (LayoutTestController::pauseTransitionAtTimeOnElementWithId): (LayoutTestController::sampleSVGAnimationForElementAtTime): (LayoutTestController::numberOfActiveAnimations): (LayoutTestController::suspendAnimations): (LayoutTestController::resumeAnimations): (LayoutTestController::overridePreference): (LayoutTestController::addUserScript): (LayoutTestController::addUserStyleSheet): (LayoutTestController::setDeveloperExtrasEnabled): (LayoutTestController::setAsynchronousSpellCheckingEnabled): (LayoutTestController::showWebInspector): (LayoutTestController::closeWebInspector): (LayoutTestController::evaluateInWebInspector): (LayoutTestController::evaluateScriptInIsolatedWorld): (LayoutTestController::removeAllVisitedLinks): (LayoutTestController::callShouldCloseOnWebView): (LayoutTestController::apiTestNewWindowDataLoadBaseURL): (LayoutTestController::apiTestGoToCurrentBackForwardItem): (LayoutTestController::setWebViewEditable): (LayoutTestController::markerTextForListItem): (LayoutTestController::authenticateSession): (LayoutTestController::setEditingBehavior): (LayoutTestController::abortModal): (LayoutTestController::hasSpellingMarker): (LayoutTestController::hasGrammarMarker): (LayoutTestController::dumpConfigurationForViewport): (LayoutTestController::setSerializeHTTPLoads): (LayoutTestController::setMinimumTimerInterval): (LayoutTestController::setTextDirection): (LayoutTestController::allowRoundingHacks):
4:55 AM Changeset in webkit [90360] by apavlov@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-07-04 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Adding new CSS rules should be more discoverable in the UI
https://bugs.webkit.org/show_bug.cgi?id=63803

  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype._createNewRule):
3:56 AM Changeset in webkit [90359] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-07-04 Danilo Cesar Lemes de Paula <danilo.cesar@collabora.co.uk>

[Qt] Fixing build with --media-stream flag enabled
https://bugs.webkit.org/show_bug.cgi?id=63831

Reviewed by Benjamin Poulain.

  • features.pri: Adding support to ENABLE_MEDIA_STREAM flag.
3:07 AM Changeset in webkit [90358] by vsevik@chromium.org
  • 2 edits in trunk/Tools

2011-06-24 Vsevolod Vlasov <vsevik@chromium.org>

Unreviewed, add myself as committer.

  • Scripts/webkitpy/common/config/committers.py:
2:51 AM Changeset in webkit [90357] by bulach@chromium.org
  • 1 edit
    5 adds in trunk/LayoutTests

2011-07-04 Marcus Bulach <bulach@chromium.org>

[chromium] Rebaseline following r90338
https://bugs.webkit.org/show_bug.cgi?id=63896

Unreviewed rebaseline.

  • platform/chromium-linux/plugins/mouse-click-plugin-clears-selection-expected.png: Added.
  • platform/chromium-mac/plugins/mouse-click-plugin-clears-selection-expected.png: Added.
  • platform/chromium-mac/plugins/mouse-click-plugin-clears-selection-expected.txt: Added.
  • platform/chromium-win-vista/plugins/mouse-click-plugin-clears-selection-expected.png: Added.
2:36 AM Changeset in webkit [90356] by vsevik@chromium.org
  • 4 edits in trunk/Source/WebCore

2011-07-04 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Update network resource url from response data.
https://bugs.webkit.org/show_bug.cgi?id=63736

  • inspector/Inspector.json:
  • inspector/InspectorResourceAgent.cpp: (WebCore::buildObjectForResourceResponse):
  • inspector/front-end/NetworkManager.js: (WebInspector.NetworkDispatcher.prototype._updateResourceWithResponse):
2:20 AM Changeset in webkit [90355] by caseq@chromium.org
  • 5 edits in trunk

2011-07-04 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: [Extensions API] audit formatter names should begin with create
https://bugs.webkit.org/show_bug.cgi?id=63802

  • inspector/extensions/extensions-audits-api-expected.txt:
  • inspector/extensions/extensions-audits.html:

2011-07-04 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: [Extensions API] audit formatter names should begin with create
https://bugs.webkit.org/show_bug.cgi?id=63802

  • inspector/front-end/ExtensionAPI.js: (WebInspector.injectedExtensionAPI.AuditResultImpl):
2:18 AM Changeset in webkit [90354] by yutak@chromium.org
  • 45 edits in trunk/LayoutTests

2011-07-04 Yuta Kitamura <yutak@chromium.org>

WebSocket: Use jsTestIsAsync and finishJSTest() wherever possible in WebSocket tests
https://bugs.webkit.org/show_bug.cgi?id=63892

Reviewed by Kent Tamura.

Use jsTestIsAsync and finishJSTest() in WebSocket tests which are run asynchronously.
Script tags loading js-test-post-function.js must be removed, and instead js-test-post.js
must be loaded after each test.

  • http/tests/websocket/tests/alert-in-event-handler.html:
  • http/tests/websocket/tests/bad-handshake-crash.html:
  • http/tests/websocket/tests/bad-sub-protocol-control-chars.html: This test is done synchronously, thus there is no need to use jsTestIsAsync and finishJSTest(). As we are moving away from js-test-post-function.js, isSuccessfullyParsed() should not be used anymore.
  • http/tests/websocket/tests/bad-sub-protocol-empty.html:
  • http/tests/websocket/tests/bad-sub-protocol-non-ascii.html:
  • http/tests/websocket/tests/bufferedAmount-after-close.html:
  • http/tests/websocket/tests/close-before-open.html:
  • http/tests/websocket/tests/close-on-navigate-new-location.html:
  • http/tests/websocket/tests/close-on-unload-and-force-gc.html:
  • http/tests/websocket/tests/close-on-unload-reference-in-parent.html:
  • http/tests/websocket/tests/close-on-unload.html:
  • http/tests/websocket/tests/close-unref-websocket.html:
  • http/tests/websocket/tests/cross-origin.html:
  • http/tests/websocket/tests/error-detect.html:
  • http/tests/websocket/tests/frame-length-longer-than-buffer.html:
  • http/tests/websocket/tests/frame-length-overflow.html: Add close tags for <body> and <html> elements which were not present for some reason.
  • http/tests/websocket/tests/frame-length-skip.html:
  • http/tests/websocket/tests/handshake-challenge-randomness.html:
  • http/tests/websocket/tests/handshake-error.html:
  • http/tests/websocket/tests/handshake-fail-by-cross-origin.html:
  • http/tests/websocket/tests/handshake-fail-by-maxlength.html:
  • http/tests/websocket/tests/handshake-fail-by-no-connection-header.html:
  • http/tests/websocket/tests/handshake-fail-by-no-cr.html:
  • http/tests/websocket/tests/handshake-fail-by-no-upgrade-header.html:
  • http/tests/websocket/tests/handshake-fail-by-prepended-null.html:
  • http/tests/websocket/tests/handshake-fail-by-sub-protocol-mismatch.html:
  • http/tests/websocket/tests/long-invalid-header.html:
  • http/tests/websocket/tests/null-character.html:
  • http/tests/websocket/tests/reload-crash.html:
  • http/tests/websocket/tests/send-after-close-on-unload.html:
  • http/tests/websocket/tests/send-throw.html:
  • http/tests/websocket/tests/send.html:
  • http/tests/websocket/tests/simple.html:
  • http/tests/websocket/tests/sub-protocol-with-space.html:
  • http/tests/websocket/tests/sub-protocol.html:
  • http/tests/websocket/tests/unicode.html:
  • http/tests/websocket/tests/url-no-trailing-slash.html:
  • http/tests/websocket/tests/url-parsing.html:
  • http/tests/websocket/tests/url-with-credential.html:
  • http/tests/websocket/tests/url-with-empty-query.html:
  • http/tests/websocket/tests/url-with-query-for-no-query.html:
  • http/tests/websocket/tests/url-with-query.html:
  • http/tests/websocket/tests/websocket-event-target.html:
  • http/tests/websocket/tests/websocket-protocol-ignored.html:
12:49 AM Changeset in webkit [90353] by kbalazs@webkit.org
  • 2 edits in trunk/LayoutTests

2011-07-03 Balazs Kelemen <kbalazs@webkit.org>

[Qt][WK2] Remove passing tests from the qt-wk2 list
https://bugs.webkit.org/show_bug.cgi?id=63877

Reviewed by Kenneth Rohde Christiansen.

  • platform/qt-wk2/Skipped:
Note: See TracTimeline for information about the timeline view.