[Webkit-unassigned] [Bug 26623] [Chromium] Upstream V8Proxy

[bugzilla-daemon at webkit.org]

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


levin at chromium.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #31893|review?                     |review+
               Flag|                            |




------- Comment #8 from levin at chromium.org  2009-06-25 22:25 PDT -------
(From update of attachment 31893)
I think it has made huge progress and we should land it.

Please fix this *before* landing (just fix it and land it *without* uploading
another patch):

  > WebCore/bindings/v8/V8DOMMap.cpp

   Has unresolved conflict in it at line 495.





Ideally we'd have a *subsequent* patch to fix the following few things: 
(Please *don't* update this patch with these fixes.)

  > WebCore/bindings/v8/V8Proxy.cpp


  1. Finish getting rid of USE_VAR and replace with UNUSED_PARAM.
  2. In all classes, public: protected: private: shouldn't be indented.
  3. In class GrouperItem,
     a. the variable names need to have this m_style
     b. group_id should be groupId (for the method and variable)
     c. For the constructor: "Each member (and superclass) should be indented
on a separate line"
  4. In several of the class the functions have braces at the end of line.
  5. In ObjectGrouperVisitor::visitDOMWrapper
     a. typo "elment"
     b. braces around single line statement
  6. ObjectGrouperVisitor::grouper_ naming.
  7. JavaScriptConsoleMessage::JavaScriptConsoleMessage constructor braces are
too indented.
  8. ConsoleMessageManager::processDelayedMessages braces on single line
statement.
  9. There are several places that do the two spaces before end of line
comments (should be one) -- search for ";  //"
  10. V8Proxy::setContextDebugId has end of line brace.




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