On Dec 2, 6:09 pm, Marc-Antoine Ruel <[EMAIL PROTECTED]> wrote:
> I like the idea of fixing test_shell and not trying to convert
> everything at first.
>
> But don't work on 5025, I think it's wasted time. It's a non-essential plugin.
>
..
> There's zero reference to WMIUtil in the code so to fix 5021, it's
> probably faster to just delete the source file.
mmmm .. I see: src\chrome\installer\util\google_chrome_distribution.cc
(71): WMIProcessUtil::Launch(command, &pid);
>
> M-A
>
> On Tue, Dec 2, 2008 at 9:04 PM, Ben Goodger (Google) <[EMAIL PROTECTED]>
> wrote:
>
>
>
> > BTW I changed all these bugs to "Type-Cleanup" which is a good way for
> > people to find the set of work required to move towards this
> > objective.
>
> > -Ben
>
> > On Tue, Dec 2, 2008 at 5:54 PM, Erik Kay <[EMAIL PROTECTED]> wrote:
>
> >> It would be nice to remove ATL/WTL dependencies from Chromium so that
> >> people can build it with Visual C++ Express.
>
> >> While rewriting our UI code is unlikely to happen overnight, I was
> >> thinking that it should be possible to at least build test_shell
> >> without ATL. Unfortunately, we have a handful of dependencies in base
> >> and in our webkit port. Some of these are trivial to fix, others not
> >> so much. I filed a handful of bugs against the ones that I found, so
> >> if you've got some downtime or are looking for a small focused bit of
> >> work to bite off, take a look at one of these:
>
> >> base/wmi_util.cc - bug 5021 - lots of COM
> >> webkit/tools/test_shell - bug 5022 - a few small issues
> >> webkit/default_plugin - bug 5023 - UI for handling plugin installation
> >> webkit/activex_shim - bug 5025 - more COM
> >> tools/grit - string table .res files generated with ATL dependencies
>
> >> Erik
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Chromium-dev" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/chromium-dev?hl=en
-~----------~----~----~----~------~----~------~--~---
