Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(213)

Side by Side Diff: Source/core/css/DragUpdateTest.cpp

Issue 140053005: Use LocalStyleChange for updating :-webkit-drag. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « Source/core/core.gypi ('k') | Source/core/rendering/RenderObject.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "config.h"
6 #include "core/dom/Element.h"
7 #include "core/frame/FrameView.h"
8 #include "core/html/HTMLDocument.h"
9 #include "core/html/HTMLElement.h"
10 #include "core/testing/DummyPageHolder.h"
11 #include <gtest/gtest.h>
12
13 using namespace WebCore;
14
15 namespace {
16
17 TEST(DragUpdateTest, AffectedByDragUpdate)
18 {
19 // Check that when dragging the div in the document below, you only get a
20 // single element style recalc.
21
22 OwnPtr<DummyPageHolder> dummyPageHolder = DummyPageHolder::create(IntSize(80 0, 600));
23 HTMLDocument* document = toHTMLDocument(&dummyPageHolder->document());
24 document->documentElement()->setInnerHTML("<style>div {width:100px;height:10 0px} div:-webkit-drag { background-color: green }</style>"
25 "<div>"
26 "<span></span>"
27 "<span></span>"
28 "<span></span>"
29 "<span></span>"
30 "</div>", ASSERT_NO_EXCEPTION);
31
32 document->view()->updateLayoutAndStyleIfNeededRecursive();
ojan 2014/02/07 20:52:46 Not sure it really matters, but I think you could
rune 2014/02/07 21:53:17 Done.
33 unsigned startCount = document->styleEngine()->resolverAccessCount();
34
35 document->documentElement()->renderer()->updateDragState(true);
36 document->view()->updateLayoutAndStyleIfNeededRecursive();
37
38 unsigned accessCount = document->styleEngine()->resolverAccessCount() - star tCount;
39
40 ASSERT_EQ(1U, accessCount);
41 }
42
43 TEST(DragUpdateTest, ChildrenAffectedByDragUpdate)
44 {
45 // Check that when dragging the div in the document below, you get a
46 // full subtree style recalc.
47
48 OwnPtr<DummyPageHolder> dummyPageHolder = DummyPageHolder::create(IntSize(80 0, 600));
49 HTMLDocument* document = toHTMLDocument(&dummyPageHolder->document());
50 document->documentElement()->setInnerHTML("<style>div {width:100px;height:10 0px} div:-webkit-drag span { background-color: green }</style>"
51 "<div>"
52 "<span></span>"
53 "<span></span>"
54 "<span></span>"
55 "<span></span>"
56 "</div>", ASSERT_NO_EXCEPTION);
57
58 document->view()->updateLayoutAndStyleIfNeededRecursive();
59 unsigned startCount = document->styleEngine()->resolverAccessCount();
60
61 document->documentElement()->renderer()->updateDragState(true);
62 document->view()->updateLayoutAndStyleIfNeededRecursive();
63
64 unsigned accessCount = document->styleEngine()->resolverAccessCount() - star tCount;
65
66 ASSERT_EQ(5U, accessCount);
67 }
68
69 } // namespace
OLDNEW
« no previous file with comments | « Source/core/core.gypi ('k') | Source/core/rendering/RenderObject.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698