[Webkit-unassigned] [Bug 39618] Allow embedding applications to decide what is or isn't an application specific plugin type

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=39618


Darin Fisher (:fishd, Google) <fishd at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #56925|review?                     |review-
               Flag|                            |




--- Comment #3 from Darin Fisher (:fishd, Google) <fishd at chromium.org>  2010-05-24 15:30:09 PST ---
(From update of attachment 56925)
WebKit/chromium/src/ChromiumBridge.cpp:471
 +  // MimeType -------------------------------------------------------------------
nit: please preserve the new line that followed this comment.

WebKit/chromium/src/ChromiumBridge.cpp:474
 +      return webKitClient()->mimeRegistry()->isApplicationPluginMIMEType(
nit: webkit style is to just put this on one line.  exceeding 80 chars is ok!

r- due to style issues

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.