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

Side by Side Diff: sky/engine/core/page/Chrome.cpp

Issue 855803002: Remove attrs sky doesn't support. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 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
« no previous file with comments | « sky/engine/core/loader/ImageLoader.cpp ('k') | sky/engine/core/page/EventHandler.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2009, 2011 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2009, 2011 Apple Inc. All rights reserved.
3 * Copyright (C) 2008, 2010 Nokia Corporation and/or its subsidiary(-ies) 3 * Copyright (C) 2008, 2010 Nokia Corporation and/or its subsidiary(-ies)
4 * Copyright (C) 2012, Samsung Electronics. All rights reserved. 4 * Copyright (C) 2012, Samsung Electronics. 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 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 void Chrome::show(NavigationPolicy policy) const 99 void Chrome::show(NavigationPolicy policy) const
100 { 100 {
101 m_client->show(policy); 101 m_client->show(policy);
102 } 102 }
103 103
104 void Chrome::mouseDidMoveOverElement(const HitTestResult& result, unsigned modif ierFlags) 104 void Chrome::mouseDidMoveOverElement(const HitTestResult& result, unsigned modif ierFlags)
105 { 105 {
106 m_client->mouseDidMoveOverElement(result, modifierFlags); 106 m_client->mouseDidMoveOverElement(result, modifierFlags);
107 } 107 }
108 108
109 // TODO(esprehn): Remove this in sky.
109 void Chrome::setToolTip(const HitTestResult& result) 110 void Chrome::setToolTip(const HitTestResult& result)
110 { 111 {
111 // First priority is a potential toolTip representing a spelling or grammar error 112 // First priority is a potential toolTip representing a spelling or grammar error
112 TextDirection toolTipDirection; 113 TextDirection toolTipDirection;
113 String toolTip = result.spellingToolTip(toolTipDirection); 114 String toolTip = result.spellingToolTip(toolTipDirection);
114 115
115 // Next we'll consider a tooltip for element with "title" attribute
116 if (toolTip.isEmpty())
117 toolTip = result.title(toolTipDirection);
118
119 m_client->setToolTip(toolTip, toolTipDirection); 116 m_client->setToolTip(toolTip, toolTipDirection);
120 } 117 }
121 118
122 void Chrome::setCursor(const Cursor& cursor) 119 void Chrome::setCursor(const Cursor& cursor)
123 { 120 {
124 m_client->setCursor(cursor); 121 m_client->setCursor(cursor);
125 } 122 }
126 123
127 void Chrome::scheduleAnimation() 124 void Chrome::scheduleAnimation()
128 { 125 {
129 WTF_LOG(ScriptedAnimationController, "Chrome::scheduleAnimation"); 126 WTF_LOG(ScriptedAnimationController, "Chrome::scheduleAnimation");
130 m_page->animator().setAnimationFramePending(); 127 m_page->animator().setAnimationFramePending();
131 m_client->scheduleAnimation(); 128 m_client->scheduleAnimation();
132 } 129 }
133 130
134 void Chrome::willBeDestroyed() 131 void Chrome::willBeDestroyed()
135 { 132 {
136 m_client->chromeDestroyed(); 133 m_client->chromeDestroyed();
137 } 134 }
138 135
139 } // namespace blink 136 } // namespace blink
OLDNEW
« no previous file with comments | « sky/engine/core/loader/ImageLoader.cpp ('k') | sky/engine/core/page/EventHandler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698