Performance guidelines for decl. WebRequest API

Performance guidelines for decl. WebRequest API

Provide performance guidelines for the declarative WebRequest API.

BUG=227523
NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192825

[battre, 2013-04-08 13:06:33]

Hi Mike,

can you please review this CL?

Thanks,
Dominic

[Mike West, 2013-04-08 13:52:53]

Thanks! One note:

https://codereview.chromium.org/13523005/diff/1/chrome/common/extensions/docs...
File chrome/common/extensions/docs/templates/intros/events.html (right):

https://codereview.chromium.org/13523005/diff/1/chrome/common/extensions/docs...
chrome/common/extensions/docs/templates/intros/events.html:172: mind:
These are excellent tips: I think they'd be even more excellent if you add
example code demonstrating the right way to do things. Developers generally end
up copy/pasting; it'd be good to give them something to start with.

[battre, 2013-04-08 14:42:55]

PTAL

https://codereview.chromium.org/13523005/diff/1/chrome/common/extensions/docs...
File chrome/common/extensions/docs/templates/intros/events.html (right):

https://codereview.chromium.org/13523005/diff/1/chrome/common/extensions/docs...
chrome/common/extensions/docs/templates/intros/events.html:172: mind:
On 2013/04/08 13:52:53, Mike West (chromium) wrote:
> These are excellent tips: I think they'd be even more excellent if you add
> example code demonstrating the right way to do things. Developers generally
end
> up copy/pasting; it'd be good to give them something to start with.

Done.

[Mike West, 2013-04-08 15:26:09]

On 2013/04/08 14:42:55, battre wrote:
> PTAL

LGTM!

Thanks. :)

[commit-bot: I haz the power, 2013-04-08 16:05:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/battre@chromium.org/13523005/4001

[commit-bot: I haz the power, 2013-04-08 16:10:33]

Change committed as 192825