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

Side by Side Diff: third_party/WebKit/Source/core/html/HTMLAreaElement.cpp

Issue 2181953004: Changed ComputedStyle's EVisibility to be an enum class and fixed naming (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@_move_computedstyle_around
Patch Set: Rebase Created 4 years, 4 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * Copyright (C) 2004, 2005, 2006, 2009, 2011 Apple Inc. All rights reserved. 4 * Copyright (C) 2004, 2005, 2006, 2009, 2011 Apple Inc. All rights reserved.
5 * 5 *
6 * This library is free software; you can redistribute it and/or 6 * This library is free software; you can redistribute it and/or
7 * modify it under the terms of the GNU Library General Public 7 * modify it under the terms of the GNU Library General Public
8 * License as published by the Free Software Foundation; either 8 * License as published by the Free Software Foundation; either
9 * version 2 of the License, or (at your option) any later version. 9 * version 2 of the License, or (at your option) any later version.
10 * 10 *
(...skipping 181 matching lines...) Expand 10 before | Expand all | Expand 10 after
192 } 192 }
193 193
194 bool HTMLAreaElement::isMouseFocusable() const 194 bool HTMLAreaElement::isMouseFocusable() const
195 { 195 {
196 return isFocusable(); 196 return isFocusable();
197 } 197 }
198 198
199 bool HTMLAreaElement::layoutObjectIsFocusable() const 199 bool HTMLAreaElement::layoutObjectIsFocusable() const
200 { 200 {
201 HTMLImageElement* image = imageElement(); 201 HTMLImageElement* image = imageElement();
202 if (!image || !image->layoutObject() || image->layoutObject()->style()->visi bility() != VISIBLE) 202 if (!image || !image->layoutObject() || image->layoutObject()->style()->visi bility() != EVisibility::Visible)
203 return false; 203 return false;
204 204
205 return supportsFocus() && Element::tabIndex() >= 0; 205 return supportsFocus() && Element::tabIndex() >= 0;
206 } 206 }
207 207
208 void HTMLAreaElement::setFocus(bool shouldBeFocused) 208 void HTMLAreaElement::setFocus(bool shouldBeFocused)
209 { 209 {
210 if (focused() == shouldBeFocused) 210 if (focused() == shouldBeFocused)
211 return; 211 return;
212 212
(...skipping 14 matching lines...) Expand all
227 { 227 {
228 document().updateStyleAndLayoutTreeForNode(this); 228 document().updateStyleAndLayoutTreeForNode(this);
229 if (!isFocusable()) 229 if (!isFocusable())
230 return; 230 return;
231 231
232 if (HTMLImageElement* imageElement = this->imageElement()) 232 if (HTMLImageElement* imageElement = this->imageElement())
233 imageElement->updateFocusAppearance(selectionBehavior); 233 imageElement->updateFocusAppearance(selectionBehavior);
234 } 234 }
235 235
236 } // namespace blink 236 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698