Right clicking outside of a selected area should deselect text and bring up the correct context men…

pdf/pdfium/pdfium_engine.h

Index: pdf/pdfium/pdfium_engine.h
diff --git a/pdf/pdfium/pdfium_engine.h b/pdf/pdfium/pdfium_engine.h
index 11d304a8379bcdd9ed2fa89cb9216b053ac1d4f8..a752b8f9ffe60483d7359ab4a61a2d671bfc89fd 100644
--- a/pdf/pdfium/pdfium_engine.h
+++ b/pdf/pdfium/pdfium_engine.h
@@ -246,6 +246,10 @@ class PDFiumEngine : public PDFEngine,
   // PDFiumPage because we might not have that structure when we need this.
   pp::Size GetPageSize(int index);
 
+  void GetAllScreenRectsUnion(std::vector<PDFiumRange> rectRange,

[gene, 2015/01/06 18:40:04]

nit: variable names are not camel case (like offset_point below)
nit2: please use const reference here instead of pass by value

[Deepak, 2015/01/07 04:59:01]

changed to reference, but can not use const as GetScreenRect() is not marked as
const.

+                              const pp::Point offset_point,

[gene, 2015/01/06 18:40:04]

nit: const reference?

[Deepak, 2015/01/07 04:59:01]

Done.

+                              std::vector<pp::Rect>& rectVector);

[gene, 2015/01/06 18:40:04]

nit: variable names are not camel case (like offset_point above)
nit2: if you need to return vector, please use pointer instead of reference (see
CheckPageAvailable above)

[Deepak, 2015/01/07 04:59:01]

Done.

+
   void UpdateTickMarks();
 
   // Called to continue searching so we don't block the main thread.