[BlackBerry] Reduce the spellcheck checking range
[WebKit-https.git] / Source / WebKit / blackberry / ChangeLog
1 2013-05-06  Nima Ghanavatian  <nghanavatian@blackberry.com>
2
3         [BlackBerry] Reduce the spellcheck checking range
4         https://bugs.webkit.org/show_bug.cgi?id=115479
5
6         Reviewed by Rob Buis.
7
8         Internally reviewed by Mike Fenton.
9
10         PR332773
11         Previously we were spellchecking the entire field on focus. If relayouting
12         occurred we rechecked this region, which is very costly. Switch to check
13         only a small region around the caret in both cases, which should alleviate
14         much of the delays experienced in very large contenteditable fields. This
15         allows for faster key input response and less time processing these requests
16         on the WebKit thread.
17
18         * WebKitSupport/InputHandler.cpp:
19         (BlackBerry::WebKit::InputHandler::requestCheckingOfString):
20         (BlackBerry::WebKit::InputHandler::spellCheckTextBlock):
21         * WebKitSupport/SpellingHandler.cpp:
22         (WebKit):
23         (BlackBerry::WebKit::SpellingHandler::spellCheckTextBlock):
24         * WebKitSupport/SpellingHandler.h:
25         (SpellingHandler):
26
27 2013-05-06  Nima Ghanavatian  <nghanavatian@blackberry.com>
28
29         [BlackBerry] Use a more descriptive timer name
30         https://bugs.webkit.org/show_bug.cgi?id=115481
31
32         Reviewed by Rob Buis.
33
34         Internally reviewed by Mike Fenton.
35
36         Changing m_timer to m_iterationDelayTimer.
37
38         * WebKitSupport/SpellingHandler.cpp:
39         (BlackBerry::WebKit::SpellingHandler::SpellingHandler):
40         (BlackBerry::WebKit::SpellingHandler::spellCheckTextBlock):
41         (BlackBerry::WebKit::SpellingHandler::parseBlockForSpellChecking):
42         * WebKitSupport/SpellingHandler.h:
43         (SpellingHandler):
44
45 2013-05-04  Nima Ghanavatian  <nghanavatian@blackberry.com>
46
47         [BlackBerry] Clean up unused spellcheck code
48         https://bugs.webkit.org/show_bug.cgi?id=115560
49
50         Reviewed by Benjamin Poulain.
51         Internally reviewed by Mike Lattanzio
52
53         * Api/WebPage.cpp:
54         (BlackBerry::WebKit::WebPage::setExtraPluginDirectory):
55         * Api/WebPage.h:
56         * WebKitSupport/InputHandler.cpp:
57         * WebKitSupport/InputHandler.h:
58         (InputHandler):
59
60 2013-05-03  Jacky Jiang  <zhajiang@blackberry.com>
61
62         [BlackBerry] Page rendering scale is changed after go back and forward
63         https://bugs.webkit.org/show_bug.cgi?id=115573
64
65         Reviewed by Rob Buis.
66         Internally reviewed by Jeff Rogers.
67
68         PR: 326886
69         When navigating back from page A with viewport to page B without
70         viewport, we didn't call setViewMode(); therefore, we didn't change
71         the fixed layout size which was set by page A. In that case, WebCore
72         would just pick up page A's fixed layout size to layout page B which
73         caused this issue.
74         Expecting zoomToInitialScaleOnLoad() or other functions to setViewMode()
75         later is not a good way, because zoomToInitialScaleOnLoad() has never
76         been called in this case. So we should always call setViewMode() to set
77         fixed layout size when a new page is committed.
78
79         * Api/WebPage.cpp:
80         (BlackBerry::WebKit::WebPagePrivate::setLoadState):
81
82 2013-05-02  Genevieve Mak  <gmak@rim.com>
83
84         [BlackBerry] Cannot touch scroll readonly text input.
85         https://bugs.webkit.org/show_bug.cgi?id=115378
86
87         Reviewed by Rob Buis.
88
89         PR #332902
90         Reviewed Internally by Mike Fenton.
91         Forgot half the patch.
92         Node::rendererIsEditable() returns false if the input element has the readonly tag set.
93         Check the node type instead.
94
95         * Api/InRegionScroller.cpp:
96         (BlackBerry::WebKit::InRegionScrollerPrivate::canScrollRenderBox):
97
98 2013-05-02  Iris Wu  <shuwu@blackberry.com>
99
100         [BlackBerry] Make scroll position adjustment work with pages with fixed position elements.
101         https://bugs.webkit.org/show_bug.cgi?id=115178
102
103         Reviewed by Rob Buis.
104
105         PR 308796
106         Debug build fix.
107
108         * Api/WebPage.cpp:
109         (BlackBerry::WebKit::WebPage::fixedElementSizeDelta):
110
111 2013-05-01  Iris Wu  <shuwu@blackberry.com>
112
113          [BlackBerry] Upstream smart selection
114          https://bugs.webkit.org/show_bug.cgi?id=111226
115
116         Reviewed by Rob Buis.
117
118         Calling userInterfaceViewportAccessor()->documentViewportRect() on WK thread
119         caused crash.
120         But viewport from webkitThreadViewportAccessor uses unadjusted size which is
121         wrong for email.
122         The solution here is to get actual viewport size on UI thread and then pass it
123         to WebKit::SelectionHandler.
124
125         PR 333763
126         Reviewed Internally By Jakob Petsovits.
127
128         * Api/InRegionScroller.cpp:
129         (BlackBerry::WebKit::InRegionScrollerPrivate::updateSelectionScrollView):
130         * Api/WebPage.cpp:
131         (BlackBerry::WebKit::WebPage::setSelectionDocumentViewportSize):
132         (WebKit):
133         * Api/WebPage.h:
134         * WebKitSupport/SelectionHandler.cpp:
135         (BlackBerry::WebKit::SelectionHandler::selectAtPoint):
136         (BlackBerry::WebKit::SelectionHandler::ensureSelectedTextVisible):
137         (BlackBerry::WebKit::SelectionHandler::selectionViewportRect):
138         * WebKitSupport/SelectionHandler.h:
139         (BlackBerry::WebKit::SelectionHandler::setSelectionViewportSize):
140         (BlackBerry::WebKit::SelectionHandler::setSelectionSubframeViewportRect):
141         (SelectionHandler):
142
143 2013-04-30  Genevieve Mak  <gmak@rim.com>
144
145         [BlackBerry] Cannot touch scroll readonly text input.
146         https://bugs.webkit.org/show_bug.cgi?id=115378
147
148         Reviewed by Rob Buis.
149
150         PR #332902
151         Reviewed Internally by Mike Fenton.
152         Node::rendererIsEditable() returns false if the input element has the readonly tag set.
153         Check the node type instead.
154
155         * WebKitSupport/DOMSupport.cpp:
156         (BlackBerry::WebKit::DOMSupport::isShadowHostTextInputElement):
157         (DOMSupport):
158         * WebKitSupport/DOMSupport.h:
159         * WebKitSupport/InRegionScrollableArea.cpp:
160         (BlackBerry::WebKit::InRegionScrollableArea::InRegionScrollableArea):
161
162 2013-04-29  Jakob Petsovits  <jpetsovits@blackberry.com>
163
164         [BlackBerry] Replace disappearing fillBuffer() API with graphics context drawing
165         https://bugs.webkit.org/show_bug.cgi?id=115360
166         Internal PR 303048.
167
168         Reviewed by Rob Buis.
169
170         Instead of using fillBuffer() to draw directly to the
171         target buffer, we now lock a Drawable on it and fill it
172         with PlatformGraphicsContext::addPredefinedPattern().
173
174         As a bonus, this also includes related clean-ups -
175         simpler checkerboard painting code, removal of
176         fillWindow(), clearWindow() and paintDefaultBackground(),
177         as well as getting rid of the DEBUG_CHECKERBOARD define
178         which has been useless for performance tracing purposes
179         for a while now.
180
181         * Api/BackingStore.cpp:
182         (BlackBerry::WebKit::BackingStorePrivate::blitVisibleContents):
183         * Api/BackingStore_p.h:
184
185 2013-04-26  Nima Ghanavatian  <nghanavatian@blackberry.com>
186
187         [BlackBerry] Check for valid element in elementTouched
188         https://bugs.webkit.org/show_bug.cgi?id=115205
189
190         Reviewed by Rob Buis.
191
192         Internally reviewed by Genevieve Mak.
193
194         PR 331546
195         We might receive a null ptr from nodeAsElementIfApplicable which
196         is passed in here. Check to make sure it's valid before using.
197
198         * WebKitSupport/InputHandler.cpp:
199         (BlackBerry::WebKit::InputHandler::elementTouched):
200
201 2013-04-26  Martin Robinson  <mrobinson@igalia.com>
202
203         Remove the remaining Skia #ifdefs
204         https://bugs.webkit.org/show_bug.cgi?id=114886
205
206         Reviewed by Benjamin Poulain.
207
208         * Api/WebPage.cpp: Remove Skia #ifdef references.
209         * WebCoreSupport/AboutDataUseFeatures.in: Ditto.
210
211 2013-04-26  Mary Wu  <mary.wu@torchmobile.com.cn>
212
213         [BlackBerry] Should check if it's cached resource before download
214         https://bugs.webkit.org/show_bug.cgi?id=115101
215
216         Reviewed by Rob Buis.
217
218         Since main resource maybe cached, if user want to save the resource, we first check
219         if it's cached. If yes, don't need to initiate a fresh load again, but get the
220         cached resource data out to save.
221
222         RIM bug# 324003, internally reviewed by Charles Wei.
223
224         * WebCoreSupport/FrameLoaderClientBlackBerry.cpp:
225         (WebCore::FrameLoaderClientBlackBerry::convertMainResourceLoadToDownload):
226
227 2013-04-26  Mary Wu  <mary.wu@torchmobile.com.cn>
228
229         [BlackBerry] Clean up load interface in WebPage
230         https://bugs.webkit.org/show_bug.cgi?id=113267
231
232         Reviewed by Rob Buis.
233
234         Remove unused loadExtended(), combine load() and download() api in WebPage.
235         RIM Bug# 315535, internally reviewed by Joe Mason.
236
237         * Api/WebPage.cpp:
238         (BlackBerry::WebKit::WebPage::load):
239         * Api/WebPage.h:
240
241 2013-04-25  Andrew Lo  <anlo@rim.com>
242
243         [BlackBerry] Selection overlay on non-composited iframes are incorrectly positioned.
244         https://bugs.webkit.org/show_bug.cgi?id=115197
245
246         Reviewed by Rob Buis.
247
248         When drawing the selection overlay, the rects to
249         paint when selecting text on non-composited sub-frames
250         need to be adjusted by the frame position.
251
252         * WebKitSupport/SelectionOverlay.cpp:
253         (BlackBerry::WebKit::SelectionOverlay::paintContents):
254
255 2013-04-25  Andreas Kling  <akling@apple.com>
256
257         Remove ENABLE(PARSED_STYLE_SHEET_CACHING) and make it always-on.
258
259         Rubber-stamped by Anders Koivisto.
260
261         * WebCoreSupport/AboutDataEnableFeatures.in:
262
263 2013-04-25  Iris Wu  <shuwu@blackberry.com>
264
265         [BlackBerry] Make scroll position adjustment work with pages with fixed position elements.
266         https://bugs.webkit.org/show_bug.cgi?id=115178
267
268         Reviewed by Rob Buis.
269
270         PR 308796
271
272         Currently the position WebPage::adjustDocumentScrollPosition adjusts is the top
273         left point of the viewport.
274         On the page with fixed position elements, we want it to adjust the position beneath
275         the fixed elements so it can be always visible.
276
277         The basic idea is:
278         1.  Detect if there are fixed position elements before going through ProximityDetector.
279         2.  If the fixed element exists, calculate its the size and the actual visible position
280             beneath it.
281         3.  Pass the position to ProximityDetector. Then according to the new position we get,
282             calculate the top left position of the viewport (final scroll position).
283
284         * Api/WebPage.cpp:
285         (BlackBerry::WebKit::WebPage::fixedElementSizeDelta):
286         (WebKit):
287         * Api/WebPage.h:
288         * Api/WebPageCompositor.cpp:
289         (BlackBerry::WebKit::WebPageCompositorPrivate::findFixedElementRect):
290         (WebKit):
291         * Api/WebPageCompositor_p.h:
292         (WebPageCompositorPrivate):
293
294 2013-04-25  Mike Lattanzio  <mlattanzio@blackberry.com>
295
296         [BlackBerry] Enable balanced page group load deferrer behaviour.
297         https://bugs.webkit.org/show_bug.cgi?id=115189
298
299         Reviewed by Rob Buis.
300
301         Prevent a possible deadlock by enabling balanced deferrers.
302         Internally reviewed by: Joe Mason
303         PR 329986
304
305         * Api/WebPage.cpp:
306         (BlackBerry::WebKit::WebPagePrivate::init):
307
308 2013-04-25  Joseph Pecoraro  <pecoraro@apple.com>
309
310         Web Inspector: ConsoleMessage should include line and column number where possible
311         https://bugs.webkit.org/show_bug.cgi?id=114929
312
313         Reviewed by Timothy Hatcher.
314
315         * Api/DumpRenderTreeClient.h:
316         * Api/WebPageClient.h:
317         * WebCoreSupport/ChromeClientBlackBerry.cpp:
318         (WebCore::ChromeClientBlackBerry::addMessageToConsole):
319         * WebCoreSupport/ChromeClientBlackBerry.h:
320         (ChromeClientBlackBerry):
321
322 2013-04-25  Konrad Piascik  <kpiascik@blackberry.com>
323
324         [BlackBerry] Get rid of return in void method
325         https://bugs.webkit.org/show_bug.cgi?id=115186
326
327         Reviewed by Rob Buis.
328
329         * Api/WebPage.cpp:
330         (BlackBerry::WebKit::WebPage::setForcedTextEncoding):
331
332 2013-04-24  Nima Ghanavatian  <nghanavatian@blackberry.com>
333
334         [BlackBerry] Do not clear focus on a node when tapping on form controls
335         https://bugs.webkit.org/show_bug.cgi?id=115055
336
337         Reviewed by Rob Buis.
338
339         Internally reviewed by Genevieve Mak.
340
341         PR316069
342         To allow for rich text editors to apply styles on an input field
343         or highlighted text, we must maintain focus on the current element
344         when tapping on form elements. Moving the code that clears this
345         context to trigger off TouchHold instead of TouchPress.
346
347         * WebKitSupport/TouchEventHandler.cpp:
348         (BlackBerry::WebKit::TouchEventHandler::doFatFingers):
349
350 == Rolled over to ChangeLog-2013-04-24 ==