[refactor] Rename some constants in net-internals. Mostly just removes a redundant "Log" prefix.

[refactor] Rename some constants in net-internals. Mostly just removes a redundant "Log" prefix.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=141180

[eroman, 2012-06-08 00:00:53]

I had some OCD while working on the other file, and decided to do some renames.
Feel free to shut this down if you don't like it.

Basically here is how I generated this patch:

sed -i -e 's/\<LogEventType\>/EventType/g' *
sed -i -e 's/\<LogEventTypeNames\>/EventTypeNames/g' *
sed -i -e 's/\<LogEventPhase\>/EventPhase/g' *
sed -i -e 's/\<LogSourceType\>/EventSourceType/g' *
sed -i -e 's/\<LogSourceTypeNames\>/EventSourceTypeNames/g' *
sed -i -e 's/\<logEventTypes\>/eventType/g' *
sed -i -e 's/\<logEventPhase\>/eventPhase/g' *
sed -i -e 's/\<logSourceType\>/sourceType/g' *

[mmenke, 2012-06-08 00:12:51]

https://chromiumcodereview.appspot.com/10534055/diff/2001/chrome/browser/reso...
File chrome/browser/resources/net_internals/main.js (right):

https://chromiumcodereview.appspot.com/10534055/diff/2001/chrome/browser/reso...
chrome/browser/resources/net_internals/main.js:250: EventSourceType =
Constants.sourceType;
This breaks log loading compatibility.

https://chromiumcodereview.appspot.com/10534055/diff/2001/chrome/browser/reso...
chrome/browser/resources/net_internals/main.js:271:
typeof(receivedConstants.sourceType) == 'object' &&
Ditto.

[eroman, 2012-06-08 00:13:29]

https://chromiumcodereview.appspot.com/10534055/diff/2001/chrome/browser/reso...
File chrome/browser/resources/net_internals/main.js (right):

https://chromiumcodereview.appspot.com/10534055/diff/2001/chrome/browser/reso...
chrome/browser/resources/net_internals/main.js:250: EventSourceType =
Constants.sourceType;
On 2012/06/08 00:12:51, Matt Menke wrote:
> This breaks log loading compatibility.

Good point. Let me rework that!

[eroman, 2012-06-08 00:33:50]

PTAL

[mmenke, 2012-06-08 00:35:07]

LGTM

[commit-bot: I haz the power, 2012-06-08 05:05:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eroman@chromium.org/10534055/2004

[commit-bot: I haz the power, 2012-06-08 06:11:57]

Change committed as 141180