)]}' { "commit": "7845a57134091b9854f468bcda169a0d12073d8d", "tree": "226ee041bf577044de64544f15df317edf1e4ad3", "parents": [ "d78f3df6bc49e48ca3efb4a292adac7d9bf60709" ], "author": { "name": "Alex Deymo", "email": "deymo@chromium.org", "time": "Mon Nov 10 19:55:35 2014 -0800" }, "committer": { "name": "chrome-internal-fetch", "email": "chrome-internal-fetch@google.com", "time": "Wed Nov 12 19:56:58 2014 +0000" }, "message": "Include the implemented header first in all projects.\n\nThe Google C++ style guide dictates that foo.cc and foo_unittest.cc\nshould include foo.h in the first place, so missing headers in foo.h\nare detected with a compile error of the module implementing them and\nnot when another module uses them.\n\nThis CL sweeps across all the .cc file in platform2 enforcing this.\n\nBUG\u003dNone\nTEST\u003dcbuildbot amd64-generic\n\nChange-Id: I41835835caba13f54c3c844ecf552eb0e47efa9d\nReviewed-on: https://chromium-review.googlesource.com/228894\nTested-by: Alex Deymo \u003cdeymo@chromium.org\u003e\nReviewed-by: Alex Deymo \u003cdeymo@chromium.org\u003e\nCommit-Queue: Alex Vakulenko \u003cavakulenko@chromium.org\u003e\n", "tree_diff": [ { "type": "modify", "old_id": "3e2e261b91a70ed49834ce410c877fca6897b76b", "old_mode": 33188, "old_path": "metrics/c_metrics_library.cc", "new_id": "90a2d59c9b8eb2ad097d9013273642d053689fae", "new_mode": 33188, "new_path": "metrics/c_metrics_library.cc" } ] }