Skip to content

Commit

Permalink
FBTF: New IPC definitions, only applied to async ROUTED and CONTROL m…
Browse files Browse the repository at this point in the history 
…essages.

The slowest cc files in chrome include render_messages.h and other IPC message
definitions. Including one of these files will bring in half of chrome because
in the IPC system previously required full class definitions due to
implementation details.

The new system allows forward declarations and places the implementations of
functions that need the full class definitions (ctor/dtor()/Log() and
superclass ctor/Read() methods) into a separate xxx_messages.cc file using a
parallel set of macros to ipc_message_macros.h. This has the added benefit
of moving most of the template instantiation junk into a small number of
files.

Pros:
- Will speed up compiling by a lot once everything is forward declared.
- Already, intermediary .o/.a files are smaller.

Cons:
- Adds a 4th pass to the messages system, this time in a different header.

BUG=51411
TEST=none

Review URL: http://codereview.chromium.org/2873090

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55259 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
erg@google.com committed Aug 6, 2010
1 parent b714b8c commit c5406e5
Show file tree
Hide file tree
Showing 27 changed files with 867 additions and 234 deletions.
264 changes: 156 additions & 108 deletions base/tuple.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,9 @@ struct TupleTraits<P&> {
typedef P& ParamType;
};

template <class P>
struct TupleTypes { };

// Tuple -----------------------------------------------------------------------
//
// This set of classes is useful for bundling 0 or more heterogeneous data types
Expand All @@ -76,9 +79,6 @@ template <class A>
struct Tuple1 {
public:
typedef A TypeA;
typedef Tuple1<typename TupleTraits<A>::ValueType> ValueTuple;
typedef Tuple1<typename TupleTraits<A>::RefType> RefTuple;
typedef Tuple1<typename TupleTraits<A>::ParamType> ParamTuple;

Tuple1() {}
explicit Tuple1(typename TupleTraits<A>::ParamType a) : a(a) {}
Expand All @@ -91,12 +91,6 @@ struct Tuple2 {
public:
typedef A TypeA;
typedef B TypeB;
typedef Tuple2<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType> ValueTuple;
typedef Tuple2<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType> RefTuple;
typedef Tuple2<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType> ParamTuple;

Tuple2() {}
Tuple2(typename TupleTraits<A>::ParamType a,
Expand All @@ -114,15 +108,6 @@ struct Tuple3 {
typedef A TypeA;
typedef B TypeB;
typedef C TypeC;
typedef Tuple3<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType> ValueTuple;
typedef Tuple3<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType> RefTuple;
typedef Tuple3<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType> ParamTuple;

Tuple3() {}
Tuple3(typename TupleTraits<A>::ParamType a,
Expand All @@ -143,18 +128,6 @@ struct Tuple4 {
typedef B TypeB;
typedef C TypeC;
typedef D TypeD;
typedef Tuple4<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType> ValueTuple;
typedef Tuple4<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType> RefTuple;
typedef Tuple4<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType> ParamTuple;

Tuple4() {}
Tuple4(typename TupleTraits<A>::ParamType a,
Expand All @@ -178,21 +151,6 @@ struct Tuple5 {
typedef C TypeC;
typedef D TypeD;
typedef E TypeE;
typedef Tuple5<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType,
typename TupleTraits<E>::ValueType> ValueTuple;
typedef Tuple5<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType,
typename TupleTraits<E>::RefType> RefTuple;
typedef Tuple5<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType,
typename TupleTraits<E>::ParamType> ParamTuple;

Tuple5() {}
Tuple5(typename TupleTraits<A>::ParamType a,
Expand All @@ -219,24 +177,6 @@ struct Tuple6 {
typedef D TypeD;
typedef E TypeE;
typedef F TypeF;
typedef Tuple6<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType,
typename TupleTraits<E>::ValueType,
typename TupleTraits<F>::ValueType> ValueTuple;
typedef Tuple6<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType,
typename TupleTraits<E>::RefType,
typename TupleTraits<F>::RefType> RefTuple;
typedef Tuple6<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType,
typename TupleTraits<E>::ParamType,
typename TupleTraits<F>::ParamType> ParamTuple;

Tuple6() {}
Tuple6(typename TupleTraits<A>::ParamType a,
Expand Down Expand Up @@ -266,27 +206,6 @@ struct Tuple7 {
typedef E TypeE;
typedef F TypeF;
typedef G TypeG;
typedef Tuple7<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType,
typename TupleTraits<E>::ValueType,
typename TupleTraits<F>::ValueType,
typename TupleTraits<G>::ValueType> ValueTuple;
typedef Tuple7<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType,
typename TupleTraits<E>::RefType,
typename TupleTraits<F>::RefType,
typename TupleTraits<G>::RefType> RefTuple;
typedef Tuple7<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType,
typename TupleTraits<E>::ParamType,
typename TupleTraits<F>::ParamType,
typename TupleTraits<G>::ParamType> ParamTuple;

Tuple7() {}
Tuple7(typename TupleTraits<A>::ParamType a,
Expand Down Expand Up @@ -320,30 +239,6 @@ struct Tuple8 {
typedef F TypeF;
typedef G TypeG;
typedef H TypeH;
typedef Tuple8<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType,
typename TupleTraits<E>::ValueType,
typename TupleTraits<F>::ValueType,
typename TupleTraits<G>::ValueType,
typename TupleTraits<H>::ValueType> ValueTuple;
typedef Tuple8<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType,
typename TupleTraits<E>::RefType,
typename TupleTraits<F>::RefType,
typename TupleTraits<G>::RefType,
typename TupleTraits<H>::RefType> RefTuple;
typedef Tuple8<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType,
typename TupleTraits<E>::ParamType,
typename TupleTraits<F>::ParamType,
typename TupleTraits<G>::ParamType,
typename TupleTraits<H>::ParamType> ParamTuple;

Tuple8() {}
Tuple8(typename TupleTraits<A>::ParamType a,
Expand All @@ -367,6 +262,159 @@ struct Tuple8 {
H h;
};

// Tuple types ----------------------------------------------------------------
//
// Allows for selection of ValueTuple/RefTuple/ParamTuple without needing the
// definitions of class types the tuple takes as parameters.

template <>
struct TupleTypes< Tuple0 > {
typedef Tuple0 ValueTuple;
typedef Tuple0 RefTuple;
typedef Tuple0 ParamTuple;
};

template <class A>
struct TupleTypes< Tuple1<A> > {
typedef Tuple1<typename TupleTraits<A>::ValueType> ValueTuple;
typedef Tuple1<typename TupleTraits<A>::RefType> RefTuple;
typedef Tuple1<typename TupleTraits<A>::ParamType> ParamTuple;
};

template <class A, class B>
struct TupleTypes< Tuple2<A, B> > {
typedef Tuple2<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType> ValueTuple;
typedef Tuple2<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType> RefTuple;
typedef Tuple2<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType> ParamTuple;
};

template <class A, class B, class C>
struct TupleTypes< Tuple3<A, B, C> > {
typedef Tuple3<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType> ValueTuple;
typedef Tuple3<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType> RefTuple;
typedef Tuple3<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType> ParamTuple;
};

template <class A, class B, class C, class D>
struct TupleTypes< Tuple4<A, B, C, D> > {
typedef Tuple4<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType> ValueTuple;
typedef Tuple4<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType> RefTuple;
typedef Tuple4<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType> ParamTuple;
};

template <class A, class B, class C, class D, class E>
struct TupleTypes< Tuple5<A, B, C, D, E> > {
typedef Tuple5<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType,
typename TupleTraits<E>::ValueType> ValueTuple;
typedef Tuple5<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType,
typename TupleTraits<E>::RefType> RefTuple;
typedef Tuple5<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType,
typename TupleTraits<E>::ParamType> ParamTuple;
};

template <class A, class B, class C, class D, class E, class F>
struct TupleTypes< Tuple6<A, B, C, D, E, F> > {
typedef Tuple6<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType,
typename TupleTraits<E>::ValueType,
typename TupleTraits<F>::ValueType> ValueTuple;
typedef Tuple6<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType,
typename TupleTraits<E>::RefType,
typename TupleTraits<F>::RefType> RefTuple;
typedef Tuple6<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType,
typename TupleTraits<E>::ParamType,
typename TupleTraits<F>::ParamType> ParamTuple;
};

template <class A, class B, class C, class D, class E, class F, class G>
struct TupleTypes< Tuple7<A, B, C, D, E, F, G> > {
typedef Tuple7<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType,
typename TupleTraits<E>::ValueType,
typename TupleTraits<F>::ValueType,
typename TupleTraits<G>::ValueType> ValueTuple;
typedef Tuple7<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType,
typename TupleTraits<E>::RefType,
typename TupleTraits<F>::RefType,
typename TupleTraits<G>::RefType> RefTuple;
typedef Tuple7<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType,
typename TupleTraits<E>::ParamType,
typename TupleTraits<F>::ParamType,
typename TupleTraits<G>::ParamType> ParamTuple;
};

template <class A, class B, class C, class D, class E, class F, class G,
class H>
struct TupleTypes< Tuple8<A, B, C, D, E, F, G, H> > {
typedef Tuple8<typename TupleTraits<A>::ValueType,
typename TupleTraits<B>::ValueType,
typename TupleTraits<C>::ValueType,
typename TupleTraits<D>::ValueType,
typename TupleTraits<E>::ValueType,
typename TupleTraits<F>::ValueType,
typename TupleTraits<G>::ValueType,
typename TupleTraits<H>::ValueType> ValueTuple;
typedef Tuple8<typename TupleTraits<A>::RefType,
typename TupleTraits<B>::RefType,
typename TupleTraits<C>::RefType,
typename TupleTraits<D>::RefType,
typename TupleTraits<E>::RefType,
typename TupleTraits<F>::RefType,
typename TupleTraits<G>::RefType,
typename TupleTraits<H>::RefType> RefTuple;
typedef Tuple8<typename TupleTraits<A>::ParamType,
typename TupleTraits<B>::ParamType,
typename TupleTraits<C>::ParamType,
typename TupleTraits<D>::ParamType,
typename TupleTraits<E>::ParamType,
typename TupleTraits<F>::ParamType,
typename TupleTraits<G>::ParamType,
typename TupleTraits<H>::ParamType> ParamTuple;
};

// Tuple creators -------------------------------------------------------------
//
// Helper functions for constructing tuples while inferring the template
Expand Down
7 changes: 6 additions & 1 deletion chrome/browser/importer/firefox_importer_unittest_utils_mac.cc
View file
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,16 @@
#include "ipc/ipc_switches.h"
#include "testing/multiprocess_func_list.h"

// Definition of IPC Messages used for this test.
// Declaration of IPC Messages used for this test.
#define MESSAGES_INTERNAL_FILE \
"chrome/browser/importer/firefox_importer_unittest_messages_internal.h"
#include "ipc/ipc_message_macros.h"

// Definition of IPC Messages used for this test.
#define MESSAGES_INTERNAL_IMPL_FILE \
"chrome/browser/importer/firefox_importer_unittest_messages_internal.h"
#include "ipc/ipc_message_impl_macros.h"

namespace {

// Name of IPC Channel to use for Server<-> Child Communications.
Expand Down
11 changes: 11 additions & 0 deletions chrome/browser/importer/importer_messages.cc
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
// Copyright (c) 2010 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#include "chrome/browser/importer/importer_messages.h"

#include "base/values.h"

#define MESSAGES_INTERNAL_IMPL_FILE \
"chrome/browser/importer/importer_messages_internal.h"
#include "ipc/ipc_message_impl_macros.h"
1 change: 1 addition & 0 deletions chrome/chrome.gyp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -1704,6 +1704,7 @@
'test/automation/automation_constants.h',
'test/automation/automation_handle_tracker.cc',
'test/automation/automation_handle_tracker.h',
'test/automation/automation_messages.cc',
'test/automation/automation_messages.h',
'test/automation/automation_messages_internal.h',
'test/automation/automation_proxy.cc',
Expand Down
7 changes: 7 additions & 0 deletions chrome/chrome_browser.gypi
View file
Original file line number Diff line number Diff line change
Expand Up @@ -1825,6 +1825,7 @@
'browser/importer/importer_data_types.h',
'browser/importer/importer_list.cc',
'browser/importer/importer_list.h',
'browser/importer/importer_messages.cc',
'browser/importer/importer_messages.h',
'browser/importer/importer_messages_internal.h',
'browser/importer/mork_reader.cc',
Expand Down Expand Up @@ -2912,6 +2913,12 @@
'browser/zygote_host_linux.cc',
'browser/zygote_main_linux.cc',

# TODO(erg): http://crbug.com/51409 These files are also specified as
# part of the automation project, but we need these definitions here,
# too, due to usage in our automation provider.
'test/automation/automation_messages.cc',
'test/automation/automation_messages.h',

# These files are generated by GRIT.
'<(grit_out_dir)/grit/bookmark_manager_resources_map.cc',
'<(grit_out_dir)/grit/shared_resources_map.cc',
Expand Down
Loading

0 comments on commit c5406e5

Please sign in to comment.