[webkit-reviews] review requested: [Bug 96763] Web Inspector: CSS property names autocomplete: Suggest most used rather than alphabeticaly first : [Attachment 165397] Fix nits

[bugzilla-daemon at webkit.org]

Nikita Vasilyev <me at elv1s.ru> has asked  for review:
Bug 96763: Web Inspector: CSS property names autocomplete: Suggest most used
rather than alphabeticaly first
https://bugs.webkit.org/show_bug.cgi?id=96763

Attachment 165397: Fix nits
https://bugs.webkit.org/attachment.cgi?id=165397&action=review

------- Additional Comments from Nikita Vasilyev <me at elv1s.ru>
> >> Source/WebCore/inspector/front-end/TextPrompt.js:1151
> >> +	      var fragment = document.createDocumentFragment();
> > 
> > Is using DocumentFragment any faster than the direct modification of
element's children in this case?
> 
> I should guarantee a single repaint.

I did some profiling: it does a single repaint in both cases and there is no
performance deviation between them (takes 1–4ms on my machine). Therefor I keep
it as it was.