Returns empty dict for GetIOStats on CrOS platforms.

Returns empty dict for GetIOStats on CrOS platforms.

BUG=252912
TEST=Manully test the code on link
NOTRY=True

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

[Owen Lin, 2013-06-24 07:05:54]

PTAL. Thanks.

[tonyg, 2013-06-24 15:39:19]

On 2013/06/24 07:05:54, Owen Lin wrote:
> PTAL. Thanks.

lgtm, thanks!

[nduca, 2013-06-24 17:09:10]

Do we have a bug filed about not having IO stats on CrOS? That seems like
somethign worth having.

[Owen Lin, 2013-06-25 02:39:58]

On 2013/06/24 17:09:10, nduca wrote:
> Do we have a bug filed about not having IO stats on CrOS? That seems like
> somethign worth having.

I just created one [http://crbug.com/253786] and assigned to Tony.

[commit-bot: I haz the power, 2013-06-25 02:41:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/owenlin@chromium.org/17591003/1

[commit-bot: I haz the power, 2013-06-25 02:56:42]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2013-06-25 07:20:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/owenlin@chromium.org/17591003/1

[commit-bot: I haz the power, 2013-06-25 07:38:32]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2013-06-28 02:45:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/owenlin@chromium.org/17591003/1

[commit-bot: I haz the power, 2013-06-28 02:45:37]

Failed to apply patch for
tools/telemetry/docs/telemetry.page.page_test_runner.html:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file tools/telemetry/docs/telemetry.page.page_test_runner.html
  Hunk #1 FAILED at 100.
  1 out of 1 hunk FAILED -- saving rejects to file
tools/telemetry/docs/telemetry.page.page_test_runner.html.rej

Patch:       tools/telemetry/docs/telemetry.page.page_test_runner.html
Index: tools/telemetry/docs/telemetry.page.page_test_runner.html
diff --git a/tools/telemetry/docs/telemetry.page.page_test_runner.html
b/tools/telemetry/docs/telemetry.page.page_test_runner.html
index
4acbb00af4b0883fe109c1f08198280b12e4ccb1..6856c9bf6a1a2155965c955db8ef9ac7902565b3
100644
--- a/tools/telemetry/docs/telemetry.page.page_test_runner.html
+++ b/tools/telemetry/docs/telemetry.page.page_test_runner.html
@@ -100,12 +100,5 @@ Data descriptors defined here:<br>
 &nbsp;<br>
 Args:<br>

&nbsp;&nbsp;base_dir:&nbsp;Path&nbsp;to&nbsp;directory&nbsp;containing&nbsp;tests&nbsp;and&nbsp;ProfileCreators.</tt></dd></dl>
-</td></tr></table><p>
-<table width="100%" cellspacing=0 cellpadding=2 border=0 summary="section">
-<tr bgcolor="#55aa55">
-<td colspan=3 valign=bottom>&nbsp;<br>
-<font color="#ffffff" face="helvetica,
arial"><big><strong>Data</strong></big></font></td></tr>
-    
-<tr><td
bgcolor="#55aa55"><tt>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</tt></td><td>&nbsp;</td>
-<td width="100%"><strong>__warningregistry__</strong> = {("Not importing
directory '/usr/local/google/code/...ls/telemetry/telemetry/test': missing
__init__.py", &lt;type 'exceptions.ImportWarning'&gt;, 7):
True}</td></tr></table>
+</td></tr></table>
 </body></html>
\ No newline at end of file

[commit-bot: I haz the power, 2013-06-28 03:01:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/owenlin@chromium.org/17591003/17001

[commit-bot: I haz the power, 2013-06-28 03:15:08]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2013-06-28 03:44:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/owenlin@chromium.org/17591003/24001

[commit-bot: I haz the power, 2013-06-28 03:54:20]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2013-06-28 04:19:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/owenlin@chromium.org/17591003/19002

[commit-bot: I haz the power, 2013-06-28 04:38:07]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[tonyg, 2013-06-28 04:39:17]

On 2013/06/28 04:38:07, I haz the power (commit-bot) wrote:
> Retried try job too often on chromium_presubmit for step(s) presubmit
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

You are going to have to NOTRY=True this patch. It looks like there is a bug in
the presubmit. I'll file it.

[owenlin_google, 2013-06-28 06:00:19]

Thank you so much. :)

Actually, I did try to add the NOTRY entry in the commit.
But I did that in a local git commit (and then upload it).

On Fri, Jun 28, 2013 at 12:39 PM,  <tonyg@chromium.org> wrote:
> On 2013/06/28 04:38:07, I haz the power (commit-bot) wrote:
>>
>> Retried try job too often on chromium_presubmit for step(s) presubmit
>
>
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...
>
> You are going to have to NOTRY=True this patch. It looks like there is a bug
> in
> the presubmit. I'll file it.
>
> https://chromiumcodereview.appspot.com/17591003/



-- 
Owen Cheng-Ru Lin
Google Taipei Software Engineer

[commit-bot: I haz the power, 2013-06-28 06:00:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/owenlin@chromium.org/17591003/19002

[commit-bot: I haz the power, 2013-06-28 17:46:34]

Change committed as 209177