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

Timeline



Nov 20, 2011:

10:28 PM Changeset in webkit [100887] by commit-queue@webkit.org
  • 5 edits in trunk

[Chromium] Layout test does not return correct misspelling positions if there are multiple misspelled words.
​https://bugs.webkit.org/show_bug.cgi?id=72655

Patch by Shinya Kawanaka <​shinyak@google.com> on 2011-11-20
Reviewed by Hajime Morita.

Tools:

Fixed the calculation of offset.

  • DumpRenderTree/chromium/WebViewHost.cpp:

(WebViewHost::finishLastTextCheck):

LayoutTests:

Fixed the calculation of offset, and added test cases having multiple misspelled words.

  • editing/spelling/script-tests/spellcheck-paste.js: Added test cases having multiple misspelled words.

(verifyMarker):
(tests):

  • editing/spelling/spellcheck-paste-expected.txt: Recreated.
10:10 PM Changeset in webkit [100886] by bashi@chromium.org
  • 10 edits in trunk

[Chromium] Remove old getFontFamilyForCharacters() and familyForChars() APIs.
​https://bugs.webkit.org/show_bug.cgi?id=72844

Source/WebCore:

Respects bold and italic properties that is given by fontconfig.

Reviewed by Darin Fisher.

  • platform/graphics/chromium/FontCacheLinux.cpp:

(WebCore::FontCache::getFontDataForCharacters): Removed #if and old logic.

Source/WebKit/chromium:

Removes old APIs for getting font family for given characters.

Reviewed by Darin Fisher.

  • public/linux/WebFontInfo.h: Removed old familyForChars().
  • public/linux/WebSandboxSupport.h: Removed old getFontFamilyForCharacters() and changed new getFontFamilyForCharacters() to be a pure virtual function.
  • src/linux/WebFontInfo.cpp: Removes old familyForChars().

LayoutTests:

Font selection mechanism of Chromium Linux port has been improved. Rebaselines affected tests.

Reviewed by Darin Fisher.

  • platform/chromium-linux/fast/text/cg-fallback-bolding-expected.png:
  • platform/chromium-linux/fast/text/cg-fallback-bolding-expected.txt:
  • platform/chromium-linux/fast/text/international/khmer-selection-expected.png:
9:37 PM Changeset in webkit [100885] by commit-queue@webkit.org
  • 3 edits
    4 adds
    1 delete in trunk/LayoutTests

Rebaseline for border change in r100528
​https://bugs.webkit.org/show_bug.cgi?id=72689

Patch by Ben Wells <​benwells@chromium.org> on 2011-11-20
Reviewed by Stephen White.

  • platform/chromium-cg-mac-snowleopard/fast/borders/mixed-border-styles-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/borders/border-mixed-alpha-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/borders/mixed-border-styles-expected.png: Added.
  • platform/chromium-mac/fast/borders/mixed-border-styles-expected.png: Removed.
  • platform/chromium-win/fast/borders/border-mixed-alpha-expected.png: Added.
  • platform/chromium-win/fast/borders/mixed-border-styles-expected.png:
  • platform/chromium/test_expectations.txt:
9:18 PM Changeset in webkit [100884] by noel.gordon@gmail.com
  • 2 edits in trunk/Source/WebCore

[chromium] Remove qt/QtMobileWebStyle from the gyp projects
​https://bugs.webkit.org/show_bug.cgi?id=72843

Reviewed by Antonio Gomes.

platform/qt/QtMobileWebStyle.{h,cpp} were removed in r100123

  • WebCore.gypi: remove platform/qt/QtMobileWebStyle.{h,cpp}
9:03 PM Changeset in webkit [100883] by fpizlo@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Unreviewed interpreter build fix.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::finalizeUnconditionally):

  • dfg/DFGRepatch.h:
9:01 PM Changeset in webkit [100882] by haraken@chromium.org
  • 3 edits in trunk/Source/WebCore

Unreviewed. Rebaselined run-bindings-tests results.

  • bindings/scripts/test/CPP/WebDOMFloat64Array.cpp:

(WebDOMFloat64Array::WebDOMFloat64Array):
(WebDOMFloat64Array::impl):
(toWebCore):
(toWebKit):

  • bindings/scripts/test/CPP/WebDOMFloat64Array.h:
8:47 PM Changeset in webkit [100881] by yuqiang.xian@intel.com
  • 6 edits in trunk/Source/JavaScriptCore

Improve modulo operation on 32bit platforms
​https://bugs.webkit.org/show_bug.cgi?id=72501

Reviewed by Filip Pizlo.

Extend softModulo to support X86 and MIPS in baseline JIT.
Apply the same optimization to 32bit DFG JIT.
1% gain on Kraken, tested on Linux Core i7 Nehalem 32bit.

  • dfg/DFGSpeculativeJIT.h:
  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compileSoftModulo):
(JSC::DFG::SpeculativeJIT::compile):

  • jit/JITArithmetic32_64.cpp:

(JSC::JIT::emit_op_mod):
(JSC::JIT::emitSlow_op_mod):

  • jit/JITOpcodes32_64.cpp:

(JSC::JIT::softModulo):

  • wtf/Platform.h:
8:45 PM Changeset in webkit [100880] by fpizlo@apple.com
  • 19 edits in trunk/Source/JavaScriptCore

Inline caches that refer to otherwise dead objects should be cleared
​https://bugs.webkit.org/show_bug.cgi?id=72311

Reviewed by Geoff Garen.

DFG code blocks now participate in the weak reference harvester fixpoint
so that they only consider themselves to be live if either they are
currently executing, or their owner is live and all of their weak references
are live. If not, the relevant code blocks are jettisoned.

Inline caches in both the old JIT and the DFG are now cleared if any of
their references are not marked at the end of a GC.

This is performance-neutral on SunSpider, V8, and Kraken. With the clear-
all-code-on-GC policy that we currently have, it shows a slight reduction
in memory usage. If we turn that policy off, it's pretty easy to come up
with an example program that will cause ToT to experience linear heap
growth, while with this patch, the heap stays small and remains at a
constant size.

  • assembler/ARMv7Assembler.h:

(JSC::ARMv7Assembler::readCallTarget):

  • assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::readCallTarget):

  • assembler/MacroAssemblerX86.h:

(JSC::MacroAssemblerX86::readCallTarget):

  • assembler/MacroAssemblerX86_64.h:

(JSC::MacroAssemblerX86_64::readCallTarget):

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::visitAggregate):
(JSC::CodeBlock::performTracingFixpointIteration):
(JSC::CodeBlock::visitWeakReferences):
(JSC::CodeBlock::finalizeUnconditionally):
(JSC::CodeBlock::stronglyVisitStrongReferences):
(JSC::MethodCallLinkInfo::reset):
(JSC::ProgramCodeBlock::jettison):
(JSC::EvalCodeBlock::jettison):
(JSC::FunctionCodeBlock::jettison):

  • bytecode/CodeBlock.h:

(JSC::CodeBlock::reoptimize):
(JSC::CodeBlock::shouldImmediatelyAssumeLivenessDuringScan):

  • bytecode/Instruction.h:

(JSC::PolymorphicAccessStructureList::visitWeak):

  • bytecode/StructureStubInfo.cpp:

(JSC::StructureStubInfo::visitWeakReferences):

  • bytecode/StructureStubInfo.h:

(JSC::isGetByIdAccess):
(JSC::isPutByIdAccess):
(JSC::StructureStubInfo::reset):

  • dfg/DFGJITCompiler.cpp:

(JSC::DFG::JITCompiler::link):

  • dfg/DFGOperations.cpp:
  • dfg/DFGRepatch.cpp:

(JSC::DFG::dfgRepatchByIdSelfAccess):
(JSC::DFG::dfgResetGetByID):
(JSC::DFG::dfgResetPutByID):

  • dfg/DFGRepatch.h:

(JSC::DFG::dfgResetGetByID):
(JSC::DFG::dfgResetPutByID):

  • jit/JIT.h:
  • jit/JITPropertyAccess.cpp:

(JSC::JIT::resetPatchGetById):
(JSC::JIT::resetPatchPutById):

  • jit/JITPropertyAccess32_64.cpp:

(JSC::JIT::resetPatchGetById):
(JSC::JIT::resetPatchPutById):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • jit/JITWriteBarrier.h:

(JSC::JITWriteBarrierBase::clearToMaxUnsigned):

8:35 PM Changeset in webkit [100879] by fpizlo@apple.com
  • 6 edits
    3 adds in trunk

Showing the data overlay in OpenStreetMap doesn't work, zooming partially broken
​https://bugs.webkit.org/show_bug.cgi?id=71505

Source/JavaScriptCore:

Reviewed by Oliver Hunt.

The bytecode generator was assuming that call_varargs never reuses the base register
(i.e. the function being called) for the result. This is no longer true.

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitCallVarargs):

  • bytecompiler/BytecodeGenerator.h:
  • bytecompiler/NodesCodegen.cpp:

(JSC::ApplyFunctionCallDotNode::emitBytecode):

LayoutTests:

Reviewed by Oliver Hunt.

  • fast/js/function-dot-apply-replace-base-expected.txt: Added.
  • fast/js/function-dot-apply-replace-base.html: Added.
  • fast/js/script-tests/cross-global-object-inline-global-var.js:

(done):

  • fast/js/script-tests/function-dot-apply-replace-base.js: Added.

(foo):
(bar):

7:32 PM Changeset in webkit [100878] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

DFG 32_64 should directly store double virtual registers on SetLocal
​https://bugs.webkit.org/show_bug.cgi?id=72845

Reviewed by Oliver Hunt.

2% win on Kraken.

  • dfg/DFGSpeculativeJIT32_64.cpp:

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

6:26 PM Changeset in webkit [100877] by abarth@webkit.org
  • 5 edits
    1 delete in trunk

REGRESSION(r100691): Safari error pages and Growl notifications fail to load stylesheets
​https://bugs.webkit.org/show_bug.cgi?id=72836

Reviewed by Sam Weinig.

Source/WebCore:

This patch removes a (minor) security mitigation. Previously, we tried
sequester "directory listings" into unique origins to make it more
difficult for an attacker to crawl the user's local file system.
Unfortunately, this mitigation doesn't really buy us much security
because if the attacker has access to local files, we've probably lost
anyway.

The larger problem, however, is that this condition is overly
complicated and has broken in sublte ways several times in its
(relatively short) lifetime. In the cases reported in this bug, we see
that this check affects error pages in Safari and Growl notifications,
even those have nothing to do with directory listings.

If we have our heart set on this directory listing mitigation, we'll
need a more robust way of triggering the behavior than examining URLs
and guess whether they contain directory listings. For example, if we
implement Allow-From or Access-Control-Deny-Origin, then the embedder
can supply those policies along with the directory listings. Those
seem like much better solutions than the in-engine hack this patch
removes.

  • page/SecurityOrigin.cpp:

(WebCore::shouldTreatAsUniqueOrigin):

LayoutTests:

Update test results to show that XMLHttpRequets for directory listings
aren't blocked.

  • fast/xmlhttprequest/resources/xmlhttprequest-nonexistent-file-real.html:
  • fast/xmlhttprequest/xmlhttprequest-nonexistent-file-expected.txt:
6:03 PM Changeset in webkit [100876] by noel.gordon@gmail.com
  • 2 edits in trunk/Source/JavaScriptCore

[chromium] Remove DFG::JITCodeGenerator from the gyp projects
​https://bugs.webkit.org/show_bug.cgi?id=72842

Reviewed by Filip Pizlo.

dfg/DFGJITCodeGenerator.{h,cpp} were removed in r100244

5:26 PM Changeset in webkit [100875] by ojan@chromium.org
  • 3 edits in trunk/Tools

Change the final place where we use version 3 of the results json output
​https://bugs.webkit.org/show_bug.cgi?id=72838

Reviewed by Adam Barth.

This converts the json from being a flat map of test name --> results
to being hierarchical by directory. This will make the json files
considerably smaller.

Also cleaned up some functions that were returning/checking boolean values
that were always True.

  • TestResultServer/model/jsonresults.py:
  • TestResultServer/model/jsonresults_unittest.py:
3:55 PM Changeset in webkit [100874] by tonikitoo@webkit.org
  • 28 edits in trunk/Source

2011-10-17 Antonio Gomes <​agomes@rim.com>

Pass a Frame* parameter in EditorClient::respondToChangedSelection
​https://bugs.webkit.org/show_bug.cgi?id=70248

Reviewed by Ryosuke Niwa.

Most of the port specific implementations of EditorClient::respondToChangedSelection
(like EditorClient{Qt,Gtk,etc}) are wrongly relying on FocusController::focusedOrMainFrame
method to get the Frame where the selection is happening on.

It is not right, since a selection can be easily triggered in an inner
frame that is not focused.

No new tests since it is a hard thing to test without
to hook into editor client in layout tests. We could
change the "Dumped Delegate" messages, but it would
require a rebasile of +1200 for a minor change.

No new tests since it is a hard thing to test without
to hook into editor client in layout tests. We could
change the "Dumped Delegate" messages, but it would
require a rebasile of +1200 for a minor change.

Source/WebCore:

  • editing/Editor.cpp: Pass the Frame on where the selection is changing to the client. (WebCore::Editor::respondToChangedSelection): Ditto. (WebCore::Editor::changeSelectionAfterCommand): Ditto. (WebCore::EmptyEditorClient::respondToChangedSelection):
  • page/EditorClient.h: Pass Frame* to indicate where the selection is happening.
  • loader/EmptyClients.h: Changed signature as per base class change.

Source/WebKit/chromium:

  • src/EditorClientImpl.cpp: Make use of the newly added Frame* parameter. (WebKit::EditorClientImpl::respondToChangedSelection):
  • src/EditorClientImpl.h:

Source/WebKit/efl:

  • WebCoreSupport/EditorClientEfl.cpp: Make use of the newly added Frame* parameter. (WebCore::EditorClientEfl::respondToChangedSelection):
  • WebCoreSupport/EditorClientEfl.h:

Source/WebKit/gtk:

  • WebCoreSupport/EditorClientGtk.cpp: Make use of the newly added Frame* parameter. (WebKit::EditorClient::respondToChangedSelection):
  • WebCoreSupport/EditorClientGtk.h:

Source/WebKit/mac:

  • WebCoreSupport/WebEditorClient.h: Make use of the newly added Frame* parameter.
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::respondToChangedSelection):

Source/WebKit/qt:

  • WebCoreSupport/EditorClientQt.cpp: Make use of the newly added Frame* parameter. (WebCore::EditorClientQt::respondToChangedSelection):
  • WebCoreSupport/EditorClientQt.h:

Source/WebKit/win:

  • WebCoreSupport/WebEditorClient.cpp: Make use of the newly added Frame* parameter. (WebEditorClient::respondToChangedSelection):
  • WebCoreSupport/WebEditorClient.h:

Source/WebKit/wx:

  • WebKitSupport/EditorClientWx.cpp: Make use of the newly added Frame* parameter. (WebCore::EditorClientWx::respondToChangedSelection):
  • WebKitSupport/EditorClientWx.h:

Source/WebKit2:

  • WebProcess/WebCoreSupport/WebEditorClient.cpp: Make use of the newly added Frame* parameter. (WebKit::WebEditorClient::respondToChangedSelection):
  • WebProcess/WebCoreSupport/WebEditorClient.h:
3:40 PM Changeset in webkit [100873] by ojan@chromium.org
  • 2 edits in trunk/Tools

Include the master name when querying the test results server.
This will soon be required since leaving the master name out mean
that multiple files could match the query.

  • Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py:
2:55 PM Changeset in webkit [100872] by mrowe@apple.com
  • 8 edits in trunk/Source

<​http://webkit.org/b/72646> Disable deprecation warnings around code where we cannot easily
switch away from the deprecated APIs.

Reviewed by Sam Weinig.

Source/WebCore:

  • platform/mac/WebCoreNSStringExtras.mm:
  • platform/network/cf/SocketStreamHandleCFNet.cpp:

(WebCore::SocketStreamHandle::reportErrorToClient):

Source/WebKit/mac:

  • Plugins/WebBasePluginPackage.mm:
  • Plugins/WebNetscapePluginPackage.mm:

(-[WebNetscapePluginPackage _tryLoad]):

Source/WebKit2:

  • Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm:
8:40 AM Changeset in webkit [100871] by abarth@webkit.org
  • 3 edits in trunk/Source/WebKit/chromium

Add WebNode::hasEventListeners for Android
​https://bugs.webkit.org/show_bug.cgi?id=72817

Reviewed by Darin Fisher.

Expose this EventTarget interface in the Chromium WebKit API for use by
Android.

  • public/WebNode.h:
  • src/WebNode.cpp:

(WebKit::WebNode::hasEventListeners):

8:38 AM Changeset in webkit [100870] by abarth@webkit.org
  • 4 edits in trunk/Source/WebKit/chromium

[Chromium] Add GestureFlingStart and GestureFlingCancel to WebGestureEvent for Android
​https://bugs.webkit.org/show_bug.cgi?id=72819

Reviewed by Darin Fisher.

These gesture types are used with FlingAnimator for scrolling.

  • public/WebInputEvent.h:

(WebKit::WebInputEvent::isScrollGestureEventType):
(WebKit::WebGestureEvent::WebGestureEvent):

  • src/WebPopupMenuImpl.cpp:

(WebKit::WebPopupMenuImpl::handleInputEvent):

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::handleInputEvent):

8:36 AM Changeset in webkit [100869] by abarth@webkit.org
  • 2 edits
    3 adds in trunk/Source/WebKit/chromium

[Chromium] Add WebInputEventFactory for Android
​https://bugs.webkit.org/show_bug.cgi?id=67890

Reviewed by Darin Fisher.

Like every Chromium OS, Android needs a WebInputFactory.

  • WebKit.gyp:
  • public/android/WebInputEventFactory.h: Added.
  • src/android/WebInputEventFactory.cpp: Added.

Nov 19, 2011:

8:25 PM Changeset in webkit [100868] by kevino@webkit.org
  • 2 edits in trunk

[wx] Build fix after GeneratedImage refactoring.

8:19 PM Changeset in webkit [100867] by kevino@webkit.org
  • 2 edits in trunk/Source/WebCore

[wx] C++ bindings build fix for move of array classes to WTF.

6:12 PM Changeset in webkit [100866] by jamesr@google.com
  • 11 edits
    1 copy
    2 moves
    26 adds
    5 deletes in trunk/LayoutTests

[chromium] Update various compositing baselines and expectations.

  • platform/chromium-cg-mac-leopard/compositing/geometry/fixed-position-composited-page-scale-down-expected.png: Added.
  • platform/chromium-cg-mac-leopard/compositing/geometry/fixed-position-iframe-composited-page-scale-down-expected.png: Added.
  • platform/chromium-cg-mac-leopard/compositing/geometry/fixed-position-transform-composited-page-scale-down-expected.png: Added.
  • platform/chromium-cg-mac-snowleopard/compositing/geometry/clipped-video-controller-expected.png:
  • platform/chromium-cg-mac-snowleopard/compositing/geometry/fixed-position-iframe-composited-page-scale-down-expected.png: Added.
  • platform/chromium-cg-mac-snowleopard/compositing/geometry/fixed-position-iframe-composited-page-scale-expected.png: Added.
  • platform/chromium-cg-mac-snowleopard/compositing/geometry/fixed-position-transform-composited-page-scale-down-expected.png: Added.
  • platform/chromium-cg-mac-snowleopard/compositing/geometry/fixed-position-transform-composited-page-scale-expected.png: Added.
  • platform/chromium-cg-mac-snowleopard/compositing/geometry/video-fixed-scrolling-expected.png: Removed.
  • platform/chromium-cg-mac-snowleopard/compositing/geometry/video-opacity-overlay-expected.png: Removed.
  • platform/chromium-cg-mac-snowleopard/compositing/layer-creation/spanOverlapsCanvas-expected.txt: Added.
  • platform/chromium-cg-mac-snowleopard/compositing/layers-inside-overflow-scroll-expected.png:
  • platform/chromium-linux/compositing/scaling/tiled-layer-recursion-expected.png:
  • platform/chromium-mac-leopard/compositing/geometry/fixed-position-composited-page-scale-down-expected.png: Added.
  • platform/chromium-mac-leopard/compositing/geometry/fixed-position-iframe-composited-page-scale-down-expected.png: Added.
  • platform/chromium-mac-leopard/compositing/geometry/fixed-position-transform-composited-page-scale-down-expected.png: Added.
  • platform/chromium-mac-snowleopard/compositing/geometry/clipped-video-controller-expected.png:
  • platform/chromium-mac-snowleopard/compositing/geometry/fixed-position-iframe-composited-page-scale-down-expected.png: Added.
  • platform/chromium-mac-snowleopard/compositing/geometry/fixed-position-iframe-composited-page-scale-expected.png: Added.
  • platform/chromium-mac-snowleopard/compositing/geometry/fixed-position-transform-composited-page-scale-down-expected.png: Added.
  • platform/chromium-mac-snowleopard/compositing/geometry/fixed-position-transform-composited-page-scale-expected.png: Added.
  • platform/chromium-mac-snowleopard/compositing/geometry/video-fixed-scrolling-expected.png: Removed.
  • platform/chromium-mac-snowleopard/compositing/geometry/video-opacity-overlay-expected.png: Removed.
  • platform/chromium-mac-snowleopard/compositing/layers-inside-overflow-scroll-expected.png:
  • platform/chromium-mac/compositing/scaling/tiled-layer-recursion-expected.png: Removed.
  • platform/chromium-win/compositing/geometry/fixed-position-composited-page-scale-down-expected.png: Added.
  • platform/chromium-win/compositing/geometry/fixed-position-composited-page-scale-expected.png: Added.
  • platform/chromium-win/compositing/geometry/fixed-position-iframe-composited-page-scale-down-expected.png: Copied from LayoutTests/platform/chromium-linux/compositing/scaling/tiled-layer-recursion-expected.png.
  • platform/chromium-win/compositing/geometry/fixed-position-iframe-composited-page-scale-expected.png: Added.
  • platform/chromium-win/compositing/geometry/fixed-position-transform-composited-page-scale-down-expected.png: Added.
  • platform/chromium-win/compositing/geometry/fixed-position-transform-composited-page-scale-expected.png: Added.
  • platform/chromium/compositing/geometry/fixed-position-composited-page-scale-down-expected.png: Added.
  • platform/chromium/compositing/geometry/fixed-position-composited-page-scale-expected.png: Added.
  • platform/chromium/compositing/geometry/video-fixed-scrolling-expected.png: Added.
  • platform/chromium/compositing/geometry/video-opacity-overlay-expected.png: Added.
  • platform/chromium/compositing/layer-creation/spanOverlapsCanvas-expected.txt: Renamed from LayoutTests/platform/chromium-win/compositing/layer-creation/spanOverlapsCanvas-expected.txt.
  • platform/chromium/compositing/overflow/overflow-compositing-descendant-expected.png:
  • platform/chromium/compositing/overflow/scroll-ancestor-update-expected.png:
  • platform/chromium/compositing/scaling/tiled-layer-recursion-expected.png: Renamed from LayoutTests/platform/chromium-cg-mac/compositing/scaling/tiled-layer-recursion-expected.png.
  • platform/chromium/compositing/self-painting-layers-expected.png:
  • platform/chromium/compositing/video-page-visibility-expected.png:
  • platform/chromium/test_expectations.txt:
6:09 PM Changeset in webkit [100865] by ojan@chromium.org
  • 2 edits in trunk/Tools

Fix jsonp callback code to add the callback if json content
is not jsonp.

  • TestResultServer/handlers/testfilehandler.py:
5:52 PM Changeset in webkit [100864] by ojan@chromium.org
  • 3 edits in trunk/Tools

Modify dashboard_base.js to optionally accept version 4 results json.

Patch by Alice Boxhall <​aboxhall@chromium.org> on 2011-11-19
Reviewed by Ojan Vafai.

  • TestResultServer/static-dashboards/dashboard_base.js:

(ADD_RESULTS):
(flattenTrie):

  • TestResultServer/static-dashboards/flakiness_dashboard_tests.js:

(testFlattenTrie):

5:47 PM Changeset in webkit [100863] by ojan@chromium.org
  • 3 edits in trunk/Tools

Stop storing results files as jsonp in the test results server
​https://bugs.webkit.org/show_bug.cgi?id=72814

Reviewed by Adam Barth.

  • TestResultServer/model/jsonresults.py:
  • TestResultServer/model/jsonresults_unittest.py:
5:13 PM Changeset in webkit [100862] by ojan@chromium.org
  • 6 edits in trunk/Tools

Remove the dependence on jsonp from more of new-run-webkit-tests and the test results server
​https://bugs.webkit.org/show_bug.cgi?id=72813

Reviewed by Adam Barth.

Once this lands, we can start storing pure json in the test results server and then
we can delete the code with all the FIXMEs added here.

  • Scripts/webkitpy/layout_tests/controllers/manager.py:
  • Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py:

Only add jsonp for full_results.json.

  • TestResultServer/model/jsonresults.py:
  • TestResultServer/model/jsonresults_unittest.py:

Accept pure json uploads.

  • TestResultServer/static-dashboards/dashboard_base.js:

(appendJSONScriptElementFor):
Use the callback parameter so that the server can start returning pure json if it's left out.

4:04 PM Changeset in webkit [100861] by ojan@chromium.org
  • 3 edits in trunk/Tools

Allow json NRWT downloads to be pure json and not jsonp
​https://bugs.webkit.org/show_bug.cgi?id=72809

Reviewed by Adam Barth.

I'm moving the server to storing and serving up raw json instead of jsonp.
You can still get the jsonp by passing a "callback" parameter, but there's no
need for run-webkit-tests to get jsonp when all it wants is the raw json.

  • Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py:

Make the wrapper stripping only happen if the json is actually wrapped.

  • Scripts/webkitpy/layout_tests/layout_package/json_results_generator_unittest.py:
12:51 PM Changeset in webkit [100860] by commit-queue@webkit.org
  • 7 edits
    6 moves
    2 adds
    1 delete in trunk

Move gamepad to Modules/ (+ some cleanup)
​https://bugs.webkit.org/show_bug.cgi?id=72785

Patch by Scott Graham <​scottmg@chromium.org> on 2011-11-19
Reviewed by Adam Barth.

Source/WebCore:

Move main files from page/ to Modules/gamepad/. #include guard the
gamepad header inclusions in Navigator.cpp to avoid including for
ports that do not enable GAMEPAD.

  • Modules/gamepad/Gamepad.cpp: Renamed from Source/WebCore/page/Gamepad.cpp.

(WebCore::Gamepad::Gamepad):
(WebCore::Gamepad::axes):
(WebCore::Gamepad::buttons):
(WebCore::Gamepad::~Gamepad):

  • Modules/gamepad/Gamepad.h: Renamed from Source/WebCore/page/Gamepad.h.
  • Modules/gamepad/Gamepad.idl: Renamed from Source/WebCore/page/Gamepad.idl.
  • Modules/gamepad/GamepadList.cpp: Renamed from Source/WebCore/page/GamepadList.cpp.

(WebCore::GamepadList::~GamepadList):
(WebCore::GamepadList::set):
(WebCore::GamepadList::length):
(WebCore::GamepadList::item):

  • Modules/gamepad/GamepadList.h: Renamed from Source/WebCore/page/GamepadList.h.

(WebCore::GamepadList::create):
(WebCore::GamepadList::GamepadList):

  • Modules/gamepad/GamepadList.idl: Renamed from Source/WebCore/page/GamepadList.idl.
  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • page/Navigator.cpp:

LayoutTests:

Use standard pre/post js test harness rather than custom one.

  • gamepad/gamepad-api-expected.txt:
  • gamepad/gamepad-api.html:
  • gamepad/gamepad-test.js: Removed.
12:15 PM Changeset in webkit [100859] by commit-queue@webkit.org
  • 11 edits in trunk/Source

Unreviewed, rolling out r100834.
​http://trac.webkit.org/changeset/100834
​https://bugs.webkit.org/show_bug.cgi?id=72806

this patch makes Qt run-webkit-test exit before finishing
(Requested by igoroliveira on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-11-19

Source/WebCore:

  • platform/graphics/texmap/TextureMapperNode.cpp:
  • platform/graphics/texmap/TextureMapperNode.h:

Source/WebKit2:

  • Shared/WebCoreArgumentCoders.cpp:

(CoreIPC::::encode):
(CoreIPC::::decode):

  • Shared/WebCoreArgumentCoders.h:
  • Shared/WebLayerTreeInfo.cpp:

(WebKit::WebLayerInfo::encode):
(WebKit::WebLayerInfo::decode):

  • Shared/WebLayerTreeInfo.h:
  • UIProcess/LayerTreeHostProxy.h:
  • UIProcess/qt/LayerTreeHostProxyQt.cpp:

(WebKit::LayerTreeHostProxy::LayerTreeHostProxy):
(WebKit::LayerTreeHostProxy::paintToCurrentGLContext):
(WebKit::LayerTreeHostProxy::syncLayerParameters):

  • WebProcess/WebCoreSupport/WebGraphicsLayer.cpp:

(WebCore::WebGraphicsLayer::addAnimation):
(WebCore::WebGraphicsLayer::pauseAnimation):
(WebCore::WebGraphicsLayer::removeAnimation):
(WebCore::WebGraphicsLayer::syncCompositingStateForThisLayerOnly):

12:08 PM Changeset in webkit [100858] by commit-queue@webkit.org
  • 1 edit
    1 delete in trunk/Source/WebCore

Remove WebCore/ForwardingHeaders/runtime/JSObjectWithGlobalObject.h because
JSObjectWithGlobalObject.h has been removed.
​https://bugs.webkit.org/show_bug.cgi?id=72794

r94701 removed JSObjectWithGlobalObject.h.

Patch by Huang Dongsung <​luxtella@company100.net> on 2011-11-19
Reviewed by Oliver Hunt.

  • ForwardingHeaders/runtime/JSObjectWithGlobalObject.h: Removed.
9:32 AM Changeset in webkit [100857] by vsevik@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector:[protocol] actual JSON messages do not conform to Inspector.json in InspectorApplicationCacheAgent.cpp
​https://bugs.webkit.org/show_bug.cgi?id=72734

Reviewed by Timothy Hatcher.

  • inspector/Inspector.json:
7:34 AM Changeset in webkit [100856] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

REGRESSION(r98542): Chromium: CSS text is rendered on page
​https://bugs.webkit.org/show_bug.cgi?id=71703

Patch by David Barr <​davidbarr@chromium.org> on 2011-11-19
Reviewed by Antti Koivisto.

Source/WebCore:

Matched UA declarations uncacheable when using simpleDefaultStyleSheet.

Test: fast/css/style-tag-display-none.html

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::matchUARules):

LayoutTests:

  • fast/css/style-tag-display-none-expected.txt: Added.
  • fast/css/style-tag-display-none.html: Added.
4:52 AM Changeset in webkit [100855] by nduca@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed, rolling out r100854.
​http://trac.webkit.org/changeset/100854

Rollout DEPS roll. Made chromium bots cry.

  • DEPS:
4:24 AM Changeset in webkit [100854] by nduca@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed. Rolled DEPS 109696->110838

  • DEPS:
3:28 AM Changeset in webkit [100853] by Simon Hausmann
  • 2 edits in trunk

[Qt] Add the map file symbols export as a dependency of the final qmake target.
​https://bugs.webkit.org/show_bug.cgi?id=72740

Patch by Alexis Menard <​alexis.menard@openbossa.org> on 2011-11-19
Reviewed by Simon Hausmann.

Add the symbols file part of the target dependencies.

  • Source/api.pri:
12:45 AM Changeset in webkit [100852] by jcivelli@chromium.org
  • 2 edits in trunk/Source/WebCore

When loading a MHTML document, make sure we set the base URL before
we notify the frame was committed (so the document has the right base
URL when the notification happens).
​https://bugs.webkit.org/show_bug.cgi?id=72788

Reviewed by Adam Barth.

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::receivedFirstData):

12:25 AM Changeset in webkit [100851] by abarth@webkit.org
  • 6 edits in trunk/Source

Integrate Source/WTF with the Chromium build system
​https://bugs.webkit.org/show_bug.cgi?id=72790

Reviewed by Eric Seidel.

Source/WebCore:

Add a dependency on the new WTF.

  • WebCore.gyp/WebCore.gyp:

Source/WebKit/chromium:

Add a dependency on the new WTF.

  • WebKit.gyp:

Source/WTF:

Rename the target in this file to "newwtf" to avoid naming conflicts.

  • WTF.gyp/WTF.gyp:
12:09 AM Changeset in webkit [100850] by abarth@webkit.org
  • 1 edit
    30 copies
    5 adds in trunk/Source/WTF

WTF should have an Xcode project
​https://bugs.webkit.org/show_bug.cgi?id=71752

Patch by Mark Rowe <​mrowe@apple.com> on 2011-11-19
Reviewed by Adam Barth.

This adds an Xcode project that includes only Stub.cpp and Stub.h.
They’re built in to a library at the appropriate path for each
configuration (WebKitBuild/{Debug,Release}/libWTF.a and
/usr/local/lib/libWTF.a) and headers are installed in to the
appropriate location (WebKitBuild/{Debug,Release}/usr/local/include/wtf
and /usr/local/include/wtf). I tested building WTF in this project and
everything appears to build except for DateMath.cpp (due to bug 71747).
I have not yet done any work on making JavaScriptCore and other
projects use the built products of this new project.

  • Configurations: Added.
  • Configurations/Base.xcconfig: Copied from Source/JavaScriptCore/Configurations/Base.xcconfig.
  • Configurations/CompilerVersion.xcconfig: Copied from Source/JavaScriptCore/Configurations/CompilerVersion.xcconfig.
  • Configurations/DebugRelease.xcconfig: Copied from Source/JavaScriptCore/Configurations/DebugRelease.xcconfig.
  • Configurations/WTF.xcconfig: Copied from Source/WebKit2/Configurations/Shim.xcconfig.
  • WTF.xcodeproj: Added.
  • WTF.xcodeproj/project.pbxproj: Added.
  • config.h: Copied from Source/JavaScriptCore/config.h.
  • icu: Added.
  • icu/LICENSE: Copied from Source/JavaScriptCore/icu/LICENSE.
  • icu/README: Copied from Source/JavaScriptCore/icu/README.
  • icu/unicode: Added.
  • icu/unicode/parseerr.h: Copied from Source/JavaScriptCore/icu/unicode/parseerr.h.
  • icu/unicode/platform.h: Copied from Source/JavaScriptCore/icu/unicode/platform.h.
  • icu/unicode/putil.h: Copied from Source/JavaScriptCore/icu/unicode/putil.h.
  • icu/unicode/uchar.h: Copied from Source/JavaScriptCore/icu/unicode/uchar.h.
  • icu/unicode/ucnv.h: Copied from Source/JavaScriptCore/icu/unicode/ucnv.h.
  • icu/unicode/ucnv_err.h: Copied from Source/JavaScriptCore/icu/unicode/ucnv_err.h.
  • icu/unicode/ucol.h: Copied from Source/JavaScriptCore/icu/unicode/ucol.h.
  • icu/unicode/uconfig.h: Copied from Source/JavaScriptCore/icu/unicode/uconfig.h.
  • icu/unicode/uenum.h: Copied from Source/JavaScriptCore/icu/unicode/uenum.h.
  • icu/unicode/uiter.h: Copied from Source/JavaScriptCore/icu/unicode/uiter.h.
  • icu/unicode/uloc.h: Copied from Source/JavaScriptCore/icu/unicode/uloc.h.
  • icu/unicode/umachine.h: Copied from Source/JavaScriptCore/icu/unicode/umachine.h.
  • icu/unicode/unorm.h: Copied from Source/JavaScriptCore/icu/unicode/unorm.h.
  • icu/unicode/urename.h: Copied from Source/JavaScriptCore/icu/unicode/urename.h.
  • icu/unicode/uscript.h: Copied from Source/JavaScriptCore/icu/unicode/uscript.h.
  • icu/unicode/uset.h: Copied from Source/JavaScriptCore/icu/unicode/uset.h.
  • icu/unicode/ustring.h: Copied from Source/JavaScriptCore/icu/unicode/ustring.h.
  • icu/unicode/utf.h: Copied from Source/JavaScriptCore/icu/unicode/utf.h.
  • icu/unicode/utf16.h: Copied from Source/JavaScriptCore/icu/unicode/utf16.h.
  • icu/unicode/utf8.h: Copied from Source/JavaScriptCore/icu/unicode/utf8.h.
  • icu/unicode/utf_old.h: Copied from Source/JavaScriptCore/icu/unicode/utf_old.h.
  • icu/unicode/utypes.h: Copied from Source/JavaScriptCore/icu/unicode/utypes.h.
  • icu/unicode/uversion.h: Copied from Source/JavaScriptCore/icu/unicode/uversion.h.
Note: See TracTimeline for information about the timeline view.