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

Side by Side Diff: third_party/WebKit/Source/core/page/SpatialNavigation.cpp

Issue 2465983002: Rename "paint invalidation rect" etc. to "visual rect". (Closed)
Patch Set: - Created 4 years, 1 month 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009 Nokia Corporation and/or its subsidiary(-ies) 2 * Copyright (C) 2009 Nokia Corporation and/or its subsidiary(-ies)
3 * Copyright (C) 2009 Antonio Gomes <tonikitoo@webkit.org> 3 * Copyright (C) 2009 Antonio Gomes <tonikitoo@webkit.org>
4 * 4 *
5 * All rights reserved. 5 * All rights reserved.
6 * 6 *
7 * Redistribution and use in source and binary forms, with or without 7 * Redistribution and use in source and binary forms, with or without
8 * modification, are permitted provided that the following conditions 8 * modification, are permitted provided that the following conditions
9 * are met: 9 * are met:
10 * 1. Redistributions of source code must retain the above copyright 10 * 1. Redistributions of source code must retain the above copyright
(...skipping 170 matching lines...) Expand 10 before | Expand all | Expand 10 after
181 pixelsPerLineStep); 181 pixelsPerLineStep);
182 break; 182 break;
183 default: 183 default:
184 break; 184 break;
185 } 185 }
186 186
187 LayoutObject* layoutObject = node->layoutObject(); 187 LayoutObject* layoutObject = node->layoutObject();
188 if (!layoutObject) 188 if (!layoutObject)
189 return true; 189 return true;
190 190
191 LayoutRect rect(layoutObject->absoluteClippedOverflowRect()); 191 LayoutRect rect(layoutObject->absoluteVisualRect());
192 if (rect.isEmpty()) 192 if (rect.isEmpty())
193 return true; 193 return true;
194 194
195 return !containerViewportRect.intersects(rect); 195 return !containerViewportRect.intersects(rect);
196 } 196 }
197 197
198 bool scrollInDirection(LocalFrame* frame, WebFocusType type) { 198 bool scrollInDirection(LocalFrame* frame, WebFocusType type) {
199 ASSERT(frame); 199 ASSERT(frame);
200 200
201 if (frame && canScrollInDirection(frame->document(), type)) { 201 if (frame && canScrollInDirection(frame->document(), type)) {
(...skipping 479 matching lines...) Expand 10 before | Expand all | Expand 10 after
681 return rect; 681 return rect;
682 } 682 }
683 683
684 HTMLFrameOwnerElement* frameOwnerElement(FocusCandidate& candidate) { 684 HTMLFrameOwnerElement* frameOwnerElement(FocusCandidate& candidate) {
685 return candidate.isFrameOwnerElement() 685 return candidate.isFrameOwnerElement()
686 ? toHTMLFrameOwnerElement(candidate.visibleNode) 686 ? toHTMLFrameOwnerElement(candidate.visibleNode)
687 : nullptr; 687 : nullptr;
688 }; 688 };
689 689
690 } // namespace blink 690 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698