Remove PlatformFile from fileapi::FileSystemOperation

Remove PlatformFile from fileapi::FileSystemOperation

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

[rvargas (doing something else), 2014-05-22 00:36:59]

PTAL

bbudge -> pepper
kinuko -> everything else.

[kinuko, 2014-05-22 04:13:32]

On 2014/05/22 00:36:59, rvargas wrote:
> PTAL
> 
> bbudge -> pepper
> kinuko -> everything else.

lgtm for non-pepper files

[bbudge, 2014-05-22 17:12:20]

Pepper lgtm

https://codereview.chromium.org/291513003/diff/90001/content/browser/renderer...
File content/browser/renderer_host/pepper/pepper_file_io_host.cc (right):

https://codereview.chromium.org/291513003/diff/90001/content/browser/renderer...
content/browser/renderer_host/pepper/pepper_file_io_host.cc:253:
Passed(&file)));
nit: could you fully qualify this (base::Passed) since we're not in namespace
base, just to make it a little clearer what Passed is?

[rvargas (doing something else), 2014-05-22 20:52:53]

Thanks

https://codereview.chromium.org/291513003/diff/90001/content/browser/renderer...
File content/browser/renderer_host/pepper/pepper_file_io_host.cc (right):

https://codereview.chromium.org/291513003/diff/90001/content/browser/renderer...
content/browser/renderer_host/pepper/pepper_file_io_host.cc:253:
Passed(&file)));
On 2014/05/22 17:12:20, bbudge wrote:
> nit: could you fully qualify this (base::Passed) since we're not in namespace
> base, just to make it a little clearer what Passed is?

Done.

[rvargas (doing something else), 2014-05-22 20:52:56]

The CQ bit was checked by rvargas@chromium.org

[commit-bot: I haz the power, 2014-05-22 20:53:46]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rvargas@chromium.org/291513003/110001

[commit-bot: I haz the power, 2014-05-23 01:54:40]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/18...)
  linux_chromium_chromeos_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_clang_...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)
  mac_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_compile_...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_compile_...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/8353)

[commit-bot: I haz the power, 2014-05-23 02:08:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-23 02:08:52]

Try jobs failed on following builders:
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/18...)

[rvargas (doing something else), 2014-05-23 19:33:12]

The CQ bit was checked by rvargas@chromium.org

[commit-bot: I haz the power, 2014-05-23 19:34:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rvargas@chromium.org/291513003/130001

[commit-bot: I haz the power, 2014-05-24 01:05:24]

Change committed as 272661