Fix style issues in NaClDomHandlerProxy.

Fix style issues in NaClDomHandlerProxy.

BUG=

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

[yael.aharon1, 2013-05-17 01:45:41]

Address style issues from https://codereview.chromium.org/14860020/#msg28
Please review, thanks!

[Dan Beam, 2013-05-17 02:05:49]

https://codereview.chromium.org/15243002/diff/1/chrome/browser/ui/webui/nacl_...
File chrome/browser/ui/webui/nacl_ui.cc (right):

https://codereview.chromium.org/15243002/diff/1/chrome/browser/ui/webui/nacl_...
chrome/browser/ui/webui/nacl_ui.cc:91: virtual ~NaClDomHandlerProxy() {}
move these above validatepncalpathcallback()

https://codereview.chromium.org/15243002/diff/1/chrome/browser/ui/webui/nacl_...
chrome/browser/ui/webui/nacl_ui.cc:94: NaClDomHandler* handler_; // weak
;  // weak.

(two spaces, .)

https://codereview.chromium.org/15243002/diff/1/chrome/browser/ui/webui/nacl_...
chrome/browser/ui/webui/nacl_ui.cc:165: got_path && !pnacl_path.empty() &&
file_util::PathExists(pnacl_path) );
nit: s/) )/))/ (remove space)

[yael.aharon1, 2013-05-17 02:20:06]

Updated the patch per your comments. thanks

https://codereview.chromium.org/15243002/diff/1/chrome/browser/ui/webui/nacl_...
File chrome/browser/ui/webui/nacl_ui.cc (right):

https://codereview.chromium.org/15243002/diff/1/chrome/browser/ui/webui/nacl_...
chrome/browser/ui/webui/nacl_ui.cc:91: virtual ~NaClDomHandlerProxy() {}
On 2013/05/17 02:05:50, Dan Beam wrote:
> move these above validatepncalpathcallback()

Done.

https://codereview.chromium.org/15243002/diff/1/chrome/browser/ui/webui/nacl_...
chrome/browser/ui/webui/nacl_ui.cc:94: NaClDomHandler* handler_; // weak
On 2013/05/17 02:05:50, Dan Beam wrote:
> ;  // weak.
> 
> (two spaces, .)

Done.

https://codereview.chromium.org/15243002/diff/1/chrome/browser/ui/webui/nacl_...
chrome/browser/ui/webui/nacl_ui.cc:165: got_path && !pnacl_path.empty() &&
file_util::PathExists(pnacl_path) );
On 2013/05/17 02:05:50, Dan Beam wrote:
> nit: s/) )/))/ (remove space)

Done.

[Dan Beam, 2013-05-17 02:25:22]

lgtm thanks for the follow up cl! :)

[commit-bot: I haz the power, 2013-05-17 02:27:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yael.aharon@intel.com/15243002/5001

[commit-bot: I haz the power, 2013-05-17 07:01:02]

Change committed as 200738