| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Foundation.h is about 100k lines of preprocessed source.
ApplicationServices is one of the many things it includes, and it is
also large (80k lines) and appears in ui/gfx/geometry headers that are
used in many places that don't care specifically about mac.
This CL avoids including Foundation.h or ApplicationServices.h from
commonly-used header files. The particular focus is on header files
typically included from other header files (versus headers most commonly
included only from .cc or .mm files).
The geometry headers get a forward-declare and fewer inline functions.
rect.h on Mac was already de-inlined, so this makes size and point
consistent with that. It also makes Mac more consistent with the
approach on Windows for native geometry types.
The main Foundation.h contributor was scoped_nsobject.h. It switches
from <Foundation/Foundation.h> (100k lines) to <Foundation/NSObject.h>
(1.5k lines). Note that even NSObject.h is not strictly needed:
<objc/objc.h> for the declaration of "nil" is sufficient, but forces
users of scoped_nsobject to make their own decision of the header to
include.
This results in a 36m20s [+/- 1s] CPU time improvement to compile times
on Mac (or a measured 1m33s [+/- 2s] real/wall time improvement on a
recent macpro). Error range gives the interval wherein lies the true
mean with a 90% confidence (6 measurements each).
There's a also a reduction in name collisions. E.g. kSmallIconSize which
is a global symbol in both Chrome and CoreServices, and is what actually
set me down this path in the first place.
BUG=None
TBR=sdefresne@chromium.org,stuartmorgan@chromium.org,thestig@chromium.org
Review URL: https://codereview.chromium.org/1092033006
Cr-Commit-Position: refs/heads/master@{#328102}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This removes the ability to pass in a custom CookieStore to a URLRequest. It's
a small change to net/url_request which then balloons to everything in the
project that ever makes a URLRequest.
This reverts the rest of https://codereview.chromium.org/188693003 (and then
does a whole lot more because URLRequest constructors were unified recently to
always require passing in the fourth argument).
BUG=457344
Review URL: https://codereview.chromium.org/1003953008
Cr-Commit-Position: refs/heads/master@{#321820}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1022813002
Cr-Commit-Position: refs/heads/master@{#321578}
|
|
This CL moves WebP decoding out of the web layer to a component that
does not depend on web.
This will allow to use the WebP decoding without pulling a dependency on
the web layer.
BUG=430110
TBR=cbentzel
Review URL: https://codereview.chromium.org/814013003
Cr-Commit-Position: refs/heads/master@{#309181}
|