[Webkit-unassigned] [Bug 57548] Web Inspector: Separate sendMessageToFrontend function from InspectorClient to individual interface

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Apr 1 06:38:21 PDT 2011


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


Yury Semikhatsky <yurys at chromium.org> changed:

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




--- Comment #2 from Yury Semikhatsky <yurys at chromium.org>  2011-04-01 06:38:21 PST ---
(From update of attachment 87846)
View in context: https://bugs.webkit.org/attachment.cgi?id=87846&action=review

> Source/WebCore/WebCore.gypi:3224
> +            'inspector/InspectorFrontendChannel.h',

Mind alphabetic order.

> Source/WebCore/WebCore.xcodeproj/project.pbxproj:22772
> +				227777601345DEA9008EA455 /* InspectorFrontendChannel.h in Headers */,

Other headers in this section seem to be sorted, please keep the order.

> Source/WebCore/inspector/CodeGeneratorInspector.pm:100
> +$typeTransform{"InspectorFrontendChannel"} = {

I think may not need "InspectorClient" forward declaration and header import anymore, please check and remove corresponding code from the generator if it is so.

> Source/WebCore/inspector/InspectorClient.h:29
> +#include "InspectorFrontendChannel.h"

Please add  Copyright (C) 2011 Google Inc.... header to the license above since you're modifying this file.

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


More information about the webkit-unassigned mailing list