- 12 Sep, 2014 40 commits
-
-
loislo@chromium.org authored
imported/web-platform-tests/shadow-dom/elements-and-dom-objects/shadowroot-object/shadowroot-methods/test-004.html imported/web-platform-tests/shadow-dom/user-interaction/ranges-and-selections/test-001.html imported/web-platform-tests/shadow-dom/user-interaction/ranges-and-selections/test-002.html NOTRY=true BUG=413731 TBR= dpranke@chromium.org, jianli@chromium.org Review URL: https://codereview.chromium.org/569673002 git-svn-id: svn://svn.chromium.org/blink/trunk@181914 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
vivek.vg@samsung.com authored
Remove all the embedded resources from blink and migrate to using grd generated pak files BUG=312586 R=haraken@chromium.org, jochen@chromium.org Part 1: https://codereview.chromium.org/558933002 Part 2: https://codereview.chromium.org/561653002 Review URL: https://codereview.chromium.org/560763002 git-svn-id: svn://svn.chromium.org/blink/trunk@181913 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
sunil.ratnu@samsung.com authored
The spec says that 'vertical-align' has no effect on a grid item. Currently there are no tests for this, so added a new test case for verifying the same. BUG=395788 Review URL: https://codereview.chromium.org/543493004 git-svn-id: svn://svn.chromium.org/blink/trunk@181912 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
mvanouwerkerk@chromium.org authored
Revert of HTMLSelectElement does not include selected index/indices while saving state (patchset #8 id:160001 of https://codereview.chromium.org/541693003/) Reason for revert: Looks like this test crashes consistently on Android Tests (dbg) starting at this build: http://build.chromium.org/p/chromium.webkit/builders/Android%20Tests%20%28dbg%29/builds/21459 Original issue's description: > HTMLSelectElement does not include selected index/indices while saving state > > Since index is not part of the FormControl state for select, there is no > way to restore it to proper index if there are duplicate values. This > patch includes the index in the FormControl state for select. > > BUG=401506 > > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181883 TBR=tkent@chromium.org,keishi@chromium.org,sudarshan.p@samsung.com NOTREECHECKS=true NOTRY=true BUG=401506 Review URL: https://codereview.chromium.org/566913002 git-svn-id: svn://svn.chromium.org/blink/trunk@181911 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
mkwst@chromium.org authored
We don't have an ExecutionContext when dealing with 'frame-ancestors' violations. This patch uses the LocalFrame which we pass into allowAncestors in order to provide hooks for both PingLoader and ConsoleMessages. It also fiddles a bit with the reporting data, as we'd otherwise suggest that 'about:blank' failed to load (as that's the initial contents of a frame before we navigate to a document). BUG=412725 Review URL: https://codereview.chromium.org/553423002 git-svn-id: svn://svn.chromium.org/blink/trunk@181910 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
fs@opera.com authored
Review URL: https://codereview.chromium.org/570583002 git-svn-id: svn://svn.chromium.org/blink/trunk@181909 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
picksi@chromium.org authored
BUG=391005 Review URL: https://codereview.chromium.org/490913002 git-svn-id: svn://svn.chromium.org/blink/trunk@181908 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
fs@opera.com authored
Factor the fill/stroke code out of the various paint-server RenderSVGResource subclasses and move it to SVGRenderSupport. Review URL: https://codereview.chromium.org/571463002 git-svn-id: svn://svn.chromium.org/blink/trunk@181907 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
fs@opera.com authored
This method is large, and the gain that propagating the constant mode from its caller is relatively small. Make {fill,stroke}PaintingResource default inline instead, and un-inline requestPaintingResource and move it into the class. Review URL: https://codereview.chromium.org/563973003 git-svn-id: svn://svn.chromium.org/blink/trunk@181906 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
loislo@chromium.org authored
On windows we receive longer script because \r\n BUG=30536 TBR= vsevik@chromium.org Review URL: https://codereview.chromium.org/569543002 git-svn-id: svn://svn.chromium.org/blink/trunk@181905 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
fs@opera.com authored
The SVGInlineTextBox member variable |m_paintingResource| is only set temporarily within paintTextWithShadows and paintDecorationWithStyle, and is only accessed from the methods that do setup and teardown of it. This makes it possible to remove it from SVGInlineTextBox and instead manage it in through a variable on the stack. The lifetime management is kept manual/the same as previously. The interaction with TextRun::RenderingContext is changed to be passed as a parameter to the added "scope" object in order to simplify the interface a bit. Review URL: https://codereview.chromium.org/568723002 git-svn-id: svn://svn.chromium.org/blink/trunk@181904 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
semeny@google.com authored
BUG=391593 Review URL: https://codereview.chromium.org/559423004 git-svn-id: svn://svn.chromium.org/blink/trunk@181903 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
rsesek@chromium.org authored
BUG=413067 R=ojan@chromium.org Review URL: https://codereview.chromium.org/559843006 git-svn-id: svn://svn.chromium.org/blink/trunk@181902 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
iliia@google.com authored
BUG=391593 Review URL: https://codereview.chromium.org/539353004 git-svn-id: svn://svn.chromium.org/blink/trunk@181901 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
loislo@chromium.org authored
Gardening: mark large-inlined-script.html as Skip because it is really long, may fail, pass and timeout. TBR= eustas@chromium.org, jianli@chromium.org BUG=413249 NOTRY=true Review URL: https://codereview.chromium.org/565923002 git-svn-id: svn://svn.chromium.org/blink/trunk@181900 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
noel@chromium.org authored
Add setDeviceColorProfile() to allow a render widget to set the screen color profile of its blink::Page. Also add a resetDeviceColorProfile() method used to restore the blink::Page to default color profile state. Implementation of each in Page are FIXME. BUG=368663 Review URL: https://codereview.chromium.org/261633002 git-svn-id: svn://svn.chromium.org/blink/trunk@181899 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
jl@opera.com authored
A trivial conversion is defined as one that cannot throw an exception, and thus need no v8::TryCatch or ExceptionState, and no "did it succeed?" check after the conversion. For such conversions, generate the code Type variable = convertedValue() instead of TONATIVE_VOID(Type, variable, convertedValue()) Also include all such conversions in the set of argument conversions that don't require a v8::TryCatch, since they then no longer do. BUG=413257 Review URL: https://codereview.chromium.org/564063002 git-svn-id: svn://svn.chromium.org/blink/trunk@181897 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
bratell@opera.com authored
String concatenation is used a lot to build error and exception messages and it was larger than intended because of over eager inlining. Outlining template code saves 90 KB with clang and 65 KB with gcc (lin64 content_shell). After this there is roughly 60 KB of concatenation code left (clang). clang: Total change: -91866 bytes ========================== 47 added, totalling +6264 bytes across 3 sources 38 removed, totalling -18354 bytes across 2 sources 79 grown, for a net change of +4604 bytes (98534 bytes before, 103138 bytes after) across 50 sources 405 shrunk, for a net change of -84380 bytes (370130 bytes before, 285750 bytes after) across 134 sources gcc: Total change: -66849 bytes ========================== 333 added, totalling +81561 bytes across 69 sources 203 removed, totalling -65785 bytes across 76 sources 211 grown, for a net change of +18666 bytes (228381 bytes before, 247047 bytes after) across 85 sources 406 shrunk, for a net change of -101291 bytes (568083 bytes before, 466792 bytes after) across 160 sources BUG= Review URL: https://codereview.chromium.org/559133002 git-svn-id: svn://svn.chromium.org/blink/trunk@181896 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
marja@chromium.org authored
Resources which were already loaded and then revalidated were skewing the data: load time was coming from the already loaded resource, compile time was the new compilation time. R=jochen@chromium.org BUG= Review URL: https://codereview.chromium.org/562653005 git-svn-id: svn://svn.chromium.org/blink/trunk@181895 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
pgal.u-szeged@partner.samsung.com authored
After removing the temporary SCM directory change back to the original cwd. This will avoid errors on future os.cwd() calls. BUG=412774 TEST=test-webkitpy --all Review URL: https://codereview.chromium.org/558103003 git-svn-id: svn://svn.chromium.org/blink/trunk@181894 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
pgal.u-szeged@partner.samsung.com authored
Remove Host() call from RebaselineServer. The host object is passed to the RebaselineServer from outside. This could help with the Mocking as the FIXME mentioned. BUG=None TEST=test-webkitpy Review URL: https://codereview.chromium.org/561733003 git-svn-id: svn://svn.chromium.org/blink/trunk@181893 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
loislo@chromium.org authored
BUG=413604 TBR= pmeenan@chromium.org, jianli@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/551543003 git-svn-id: svn://svn.chromium.org/blink/trunk@181892 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
bashi@chromium.org authored
To follow the spec[1]. We should throw an type error when a V8 value for IDL dictionary is neither an object, null, nor undefined. The current implementation returns an impl instance with default values. [1] http://heycam.github.io/webidl/#es-dictionary BUG=321462 Review URL: https://codereview.chromium.org/568703002 git-svn-id: svn://svn.chromium.org/blink/trunk@181891 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
bashi@chromium.org authored
Replacing generic Dictionary. No changes in behavior. Tests under fast/encoding cover this change. BUG=403150 Review URL: https://codereview.chromium.org/564463002 git-svn-id: svn://svn.chromium.org/blink/trunk@181890 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
ager@chromium.org authored
Document for no reason. This is important for Oilpan because the Ranges do not die until the next GC. This gives a 10% improvement on DOM/textarea-edit in the oilpan build. R=yosin@chromium.org BUG=388681 Review URL: https://codereview.chromium.org/565613002 git-svn-id: svn://svn.chromium.org/blink/trunk@181889 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
loislo@chromium.org authored
delete LayoutTests/platform/win-xp/svg/filters/feMerge-wrong-input-expected.txt BUG= TBR= jianli@chromium.org Review URL: https://codereview.chromium.org/567983002 git-svn-id: svn://svn.chromium.org/blink/trunk@181888 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
tkent@chromium.org authored
Month buttons should have role=gridcell, and YearListCell should not. So this CL moves role=gridcell from ListCell to DayCell. Month buttons should have their own IDs, and readable month label. BUG=123896 Review URL: https://codereview.chromium.org/564053002 git-svn-id: svn://svn.chromium.org/blink/trunk@181887 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
iliia@google.com authored
Code border became light gray; indents appeared. BUG=391593 Review URL: https://codereview.chromium.org/551463003 git-svn-id: svn://svn.chromium.org/blink/trunk@181886 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
mkwst@chromium.org authored
This patch completes the transition away from MixedContentChecker as an object with state, into a world where it just provides static methods. This should clear up the interface, and make migration to RemoteFrame a bit simpler. After this patch, we no longer hang a MixedContentChecker object off of FrameLoader, nor does MixedContentChecker need to hold a reference to a frame. Everything we need to do the checks can be done by passing values into the checker methods. Note also that this patch deletes a number of expectation files: these are testharness.js tests, which shouldn't have expectations committed into the repository. Rather than modifying them, this patch removes them. As a drive-by, this also corrects the FormsSubmitted counter. It ought to count all form submissions as "FormsSubmitted", and mixed submissions as a subset thereof (e.g. 5 form submissions, with 2 of them mixed would result in FormsSubmitted: 5, MixedContentFormsSubmitted: 2, as opposed to the current 3 and 2). BUG=411184 Review URL: https://codereview.chromium.org/566533003 git-svn-id: svn://svn.chromium.org/blink/trunk@181885 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
bashi@chromium.org authored
- bindings_core_dictionary_impl_generated target should have core_dictionary_idl_files as inputs, otherwise IDL dictionaries impl classes won't get re-generated when they are edited. - impl_includes_for_type() should check idl_type.base_type (not idl_type.name), otherwise the CG won't include necessary header files for nullable interface members. BUG=321462 Review URL: https://codereview.chromium.org/563163002 git-svn-id: svn://svn.chromium.org/blink/trunk@181884 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
sudarshan.p@samsung.com authored
Since index is not part of the FormControl state for select, there is no way to restore it to proper index if there are duplicate values. This patch includes the index in the FormControl state for select. BUG=401506 Review URL: https://codereview.chromium.org/541693003 git-svn-id: svn://svn.chromium.org/blink/trunk@181883 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
jl@opera.com authored
There was custom generated code specifically for handling arguments with [Clamp], which is quite unnecessary since the toUInt*() have an integer conversion configuration parameter that among other things can be used to request clamping. Related: Fix the somewhat broken handling the |Clamp| integer conversion configuration in the toUInt*() helpers. An optimization code path for the case that the V8 value is an int32 forgot to do clamping, there was no clamping at all in toUInt64(), and +/- Infinity was converted to zero instead of being clamped. Clamping support is not added to toInt64(), since the clampTo<> helper in Source/wtf/MathExtras.h explicitly doesn't support |long long int|, which is int64_t on some platforms. BUG=413257 Review URL: https://codereview.chromium.org/563793002 git-svn-id: svn://svn.chromium.org/blink/trunk@181882 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
bashi@chromium.org authored
To remove generic Dictionary from MutationObserver. Added Vector<String> -> HashSet<AtomicString> conversion because attributeFilter member in MutationObserverInit is sequence<DOMString>. Tests under fast/dom/MutationObserver cover this change. Spec: http://www.w3.org/TR/dom/#interface-mutationobserver BUG=403150 Review URL: https://codereview.chromium.org/560003002 git-svn-id: svn://svn.chromium.org/blink/trunk@181881 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
chrishtr@chromium.org authored
BUG=412088 Review URL: https://codereview.chromium.org/559733005 git-svn-id: svn://svn.chromium.org/blink/trunk@181880 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
bashi@chromium.org authored
Use IDL dictionary (NotificationOptions) instead of generic Dictionary. Tests under fast/notifications are passed. BUG=403150 Review URL: https://codereview.chromium.org/556193002 git-svn-id: svn://svn.chromium.org/blink/trunk@181879 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
horo@chromium.org authored
Revert of Revert of Service Worker: Add tests for same-scope new script registration (patchset #1 id:1 of https://codereview.chromium.org/512703002/) Reason for revert: This crash will be fixed by https://codereview.chromium.org/556003006. Original issue's description: > Revert of Service Worker: Add tests for same-scope new script registration (patchset #6 of https://codereview.chromium.org/480943002/) > > Reason for revert: > This is crashing on Mac. > > Original issue's description: > > Service Worker: Add tests for same-scope, new script registration > > > > Tests for Chromium patch: > > https://codereview.chromium.org/506043002/ > > > > BUG=398355 > > > > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180948 > > TBR=michaeln@chromium.org,nhiroki@chromium.org,falken@chromium.org > NOTREECHECKS=true > NOTRY=true > BUG=398355 > > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180956 TBR=michaeln@chromium.org,nhiroki@chromium.org,falken@chromium.org,mlamouri@chromium.org NOTREECHECKS=true NOTRY=true BUG=398355 Review URL: https://codereview.chromium.org/547743003 git-svn-id: svn://svn.chromium.org/blink/trunk@181878 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
bashi@chromium.org authored
To remove generic Dictionary from a DOMPoint constructor. No changes in behavior; fast/dom/geometry-interfaces-dom-point.html should cover this change. Spec: http://dev.w3.org/fxtf/geometry/#DOMPoint BUG=403150 Review URL: https://codereview.chromium.org/558163002 git-svn-id: svn://svn.chromium.org/blink/trunk@181877 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
jamesr@chromium.org authored
public/web/WebSettings.h does: and uses some types from ICU in its definition. Thus, anything that depends on the blink public API and includes this header must have the icu include dirs on their include path. This forwards icu_config from the blink_headers target so dependents do not have to do this themselves (unless they are linking against ICU directly) R=brettw@chromium.org Review URL: https://codereview.chromium.org/563963003 git-svn-id: svn://svn.chromium.org/blink/trunk@181876 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
tasak@google.com authored
Before applying this patch, run-bindings-tests uses 'tests' as component_dir. So tests results had '#include "bindings/tests/.../V8Test....h". However, for testing binding modularization, need to emluate 'Source/core' and 'Source/modules'. So Source/bindings/tests/idls/core should be treated as 'core' (i.e. the tests should be treated as if they are placed under Source/core). Souce/bindings/tests/idsl/modules should be treated as 'modules'. BUG=358074 Review URL: https://codereview.chromium.org/563453002 git-svn-id: svn://svn.chromium.org/blink/trunk@181875 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-
dpranke@chromium.org authored
The test suites are moving from being pulled in via src/DEPS to LayoutTests/w3c to being checked in directly to LayoutTests/imported. This import uses update-w3c-deps run manually and imported csswg-test@6ed3bb6642b0d382ce8c3f7912172f2420eb7756 and imported web-platform-tests@6ed3bb6642b0d382ce8c3f7912172f2420eb7756 In addition, we modify run-webkit-tests to manually skip the old LayoutTests/w3c directory if it exists, and to start printing warnings that the repos are stale and should be deleted. This is patchset 3 of the 4 needed to complete the cutover. TBR=eseidel@chromium.org BUG=413454 Review URL: https://codereview.chromium.org/560893005 git-svn-id: svn://svn.chromium.org/blink/trunk@181874 bbb929c8-8fbe-4397-9dbb-9b2b20218538
-